lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aDe2QbVwsmo83Wu9@archie.me>
Date: Thu, 29 May 2025 08:20:01 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: George Anthony Vernon <contact@...rnon.com>, dmitry.torokhov@...il.com,
	corbet@....net, skhan@...uxfoundation.org
Cc: linux-input@...r.kernel.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-kernel-mentees@...ts.linux.dev
Subject: Re: [PATCH v2 3/4] input: docs: Fix Amiga joysticks grammar

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@...il.com>

-- 
An old man doll... just what I always wanted! - Clara

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ