From: Bagas Sanjaya <bagasdotme@gmail.com>
To: George Anthony Vernon <contact@gvernon.com>,
dmitry.torokhov@gmail.com, corbet@lwn.net,
skhan@linuxfoundation.org
Cc: linux-input@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-kernel-mentees@lists.linux.dev
Subject: Re: [PATCH v2 3/4] input: docs: Fix Amiga joysticks grammar
Date: Thu, 29 May 2025 08:20:01 +0700 [thread overview]
Message-ID: <aDe2QbVwsmo83Wu9@archie.me> (raw)
In-Reply-To: <20250526135957.180254-4-contact@gvernon.com>
[-- Attachment #1: Type: text/plain, Size: 8589 bytes --]
On Mon, May 26, 2025 at 02:59:51PM +0100, George Anthony Vernon wrote:
> diff --git a/Documentation/input/devices/amijoy.rst b/Documentation/input/devices/amijoy.rst
> index ea4de1ac0360..cbc5240f8af3 100644
> --- a/Documentation/input/devices/amijoy.rst
> +++ b/Documentation/input/devices/amijoy.rst
> @@ -1,14 +1,13 @@
> -~~~~~~~~~~~~~~~~~~~~~~~~~
> -Amiga joystick extensions
> -~~~~~~~~~~~~~~~~~~~~~~~~~
> +~~~~~~~~~~~~~~~
> +Amiga joysticks
> +~~~~~~~~~~~~~~~
>
>
> -Amiga 4-joystick parport extension
> -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +Amiga 4-joystick parallel port extension
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Parallel port pins:
>
> -
> ===== ======== ==== ==========
> Pin Meaning Pin Meaning
> ===== ======== ==== ==========
> @@ -90,17 +89,17 @@ Pin Meaning
>
> -------------------------------------------------------------------------------
>
> -======== === ==== ==== ====== ========================================
> +======== === ==== ==== ====== ===========================================
> NAME rev ADDR type chip Description
> -======== === ==== ==== ====== ========================================
> -JOY0DAT 00A R Denise Joystick-mouse 0 data (left vert, horiz)
> -JOY1DAT 00C R Denise Joystick-mouse 1 data (right vert,horiz)
> -======== === ==== ==== ====== ========================================
> +======== === ==== ==== ====== ===========================================
> +JOY0DAT 00A R Denise Joystick-mouse 0 data (left vert., horiz.)
> +JOY1DAT 00C R Denise Joystick-mouse 1 data (right vert., horiz.)
> +======== === ==== ==== ====== ===========================================
>
> These addresses each read a 16 bit register. These in turn
> are loaded from the MDAT serial stream and are clocked in on
> the rising edge of SCLK. MLD output is used to parallel load
> - the external parallel-to-serial converter.This in turn is
> + the external parallel-to-serial converter. This in turn is
> loaded with the 4 quadrature inputs from each of two game
> controller ports (8 total) plus 8 miscellaneous control bits
> which are new for LISA and can be read in upper 8 bits of
> @@ -108,7 +107,7 @@ JOY1DAT 00C R Denise Joystick-mouse 1 data (right vert,horiz)
>
> Register bits are as follows:
>
> - - Mouse counter usage (pins 1,3 =Yclock, pins 2,4 =Xclock)
> + - Mouse counter usage (pins 1,3 =Yclock, pins 2,4 =Xclock)
>
> ======== === === === === === === === === ====== === === === === === === ===
> BIT# 15 14 13 12 11 10 09 08 07 06 05 04 03 02 01 00
> @@ -179,12 +178,12 @@ JOYTEST 036 W Denise Write to all 4 joystick-mouse counters at once.
>
> -------------------------------------------------------------------------------
>
> -======= === ==== ==== ====== ========================================
> +======= === ==== ==== ====== ===========================================
> NAME rev ADDR type chip Description
> -======= === ==== ==== ====== ========================================
> -POT0DAT h 012 R Paula Pot counter data left pair (vert, horiz)
> -POT1DAT h 014 R Paula Pot counter data right pair (vert,horiz)
> -======= === ==== ==== ====== ========================================
> +======= === ==== ==== ====== ===========================================
> +POT0DAT h 012 R Paula Pot counter data left pair (vert., horiz.)
> +POT1DAT h 014 R Paula Pot counter data right pair (vert., horiz.)
> +======= === ==== ==== ====== ===========================================
>
> These addresses each read a pair of 8 bit pot counters.
> (4 counters total). The bit assignment for both
> @@ -213,8 +212,8 @@ POT1DAT h 014 R Paula Pot counter data right pair (vert,horiz)
> +-------+------+-----+-----+-------+
>
> With normal (NTSC or PAL) horiz. line rate, the pots will
> - give a full scale (FF) reading with about 500kohms in one
> - frame time. With proportionally faster horiz line times,
> + give a full scale (FF) reading with about 500k ohm in one
> + frame time. With proportionally faster horiz. line times,
> the counters will count proportionally faster.
> This should be noted when doing variable beam displays.
>
> @@ -238,26 +237,26 @@ POTINP 016 R Paula Pot pin data read
> This register controls a 4 bit bi-direction I/O port
> that shares the same 4 pins as the 4 pot counters above.
>
> - +-------+----------+---------------------------------------------+
> - | BIT# | FUNCTION | DESCRIPTION |
> - +=======+==========+=============================================+
> - | 15 | OUTRY | Output enable for Paula pin 33 |
> - +-------+----------+---------------------------------------------+
> - | 14 | DATRY | I/O data Paula pin 33 |
> - +-------+----------+---------------------------------------------+
> - | 13 | OUTRX | Output enable for Paula pin 32 |
> - +-------+----------+---------------------------------------------+
> - | 12 | DATRX | I/O data Paula pin 32 |
> - +-------+----------+---------------------------------------------+
> - | 11 | OUTLY | Out put enable for Paula pin 36 |
> - +-------+----------+---------------------------------------------+
> - | 10 | DATLY | I/O data Paula pin 36 |
> - +-------+----------+---------------------------------------------+
> - | 09 | OUTLX | Output enable for Paula pin 35 |
> - +-------+----------+---------------------------------------------+
> - | 08 | DATLX | I/O data Paula pin 35 |
> - +-------+----------+---------------------------------------------+
> - | 07-01 | X | Not used |
> - +-------+----------+---------------------------------------------+
> - | 00 | START | Start pots (dump capacitors,start counters) |
> - +-------+----------+---------------------------------------------+
> + +-------+----------+----------------------------------------------+
> + | BIT# | FUNCTION | DESCRIPTION |
> + +=======+==========+==============================================+
> + | 15 | OUTRY | Output enable for Paula pin 33 |
> + +-------+----------+----------------------------------------------+
> + | 14 | DATRY | I/O data Paula pin 33 |
> + +-------+----------+----------------------------------------------+
> + | 13 | OUTRX | Output enable for Paula pin 32 |
> + +-------+----------+----------------------------------------------+
> + | 12 | DATRX | I/O data Paula pin 32 |
> + +-------+----------+----------------------------------------------+
> + | 11 | OUTLY | Out put enable for Paula pin 36 |
> + +-------+----------+----------------------------------------------+
> + | 10 | DATLY | I/O data Paula pin 36 |
> + +-------+----------+----------------------------------------------+
> + | 09 | OUTLX | Output enable for Paula pin 35 |
> + +-------+----------+----------------------------------------------+
> + | 08 | DATLX | I/O data Paula pin 35 |
> + +-------+----------+----------------------------------------------+
> + | 07-01 | X | Not used |
> + +-------+----------+----------------------------------------------+
> + | 00 | START | Start pots (dump capacitors, start counters) |
> + +-------+----------+----------------------------------------------+
LGTM, thanks!
Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
--
An old man doll... just what I always wanted! - Clara
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2025-05-29 1:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-26 13:59 [PATCH v2 0/4] Multiple fixes to Amiga joystick documentation George Anthony Vernon
2025-05-26 13:59 ` [PATCH v2 1/4] input: docs: Fix broken table formatting George Anthony Vernon
2025-05-26 13:59 ` [PATCH v2 2/4] input: docs: Correct Amiga 4-jstick adapter pinout George Anthony Vernon
2025-05-26 13:59 ` [PATCH v2 3/4] input: docs: Fix Amiga joysticks grammar George Anthony Vernon
2025-05-29 1:20 ` Bagas Sanjaya [this message]
2025-05-29 1:23 ` Bagas Sanjaya
2025-05-26 13:59 ` [PATCH v2 4/4] input: docs: Make headings compliant w/ guidelines George Anthony Vernon
2025-05-29 1:20 ` Bagas Sanjaya
2025-05-30 22:35 ` [PATCH v2 0/4] Multiple fixes to Amiga joystick documentation Dmitry Torokhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aDe2QbVwsmo83Wu9@archie.me \
--to=bagasdotme@gmail.com \
--cc=contact@gvernon.com \
--cc=corbet@lwn.net \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel-mentees@lists.linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=skhan@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox