linux-kernel - Re: [PATCH] strcuture: Correct spelling mistakes in comments

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] [day] [month] [year] [list]

Message-ID: <ZFOD/6rT36evHXNz@debian.me>
Date:   Thu, 4 May 2023 17:07:59 +0700
From:   Bagas Sanjaya <bagasdotme@...il.com>
To:     Wang Honghui <honghui.wang@...s.com.cn>,
        Oliver Neukum <oliver@...kum.org>,
        Ali Akcaagac <aliakc@....de>,
        Jamie Lenehan <lenehan@...bble.org>,
        "James E.J. Bottomley" <jejb@...ux.ibm.com>,
        "Martin K. Petersen" <martin.petersen@...cle.com>,
        dc395x@...bble.org, linux-scsi@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] strcuture: Correct spelling mistakes in comments

On Thu, May 04, 2023 at 05:05:51PM +0800, Wang Honghui wrote:
> Subject: [PATCH] strcuture: Correct spelling mistakes in comments

The patch subject should have been "scsi: dc395x: Correct 'structures'
spelling mistakes". For the description, though, I'd like to write
as 'As title', since the subject is self-explanatory.

Care to reroll?

> 
> Signed-off-by: Wang Honghui <honghui.wang@...s.com.cn>
> ---
>  drivers/scsi/dc395x.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
> index c8e86f8a631e..a3f00e6e3296 100644
> --- a/drivers/scsi/dc395x.c
> +++ b/drivers/scsi/dc395x.c
> @@ -3641,7 +3641,7 @@ static void adapter_add_device(struct AdapterCtlBlk *acb,
>   * adapter_remove_device - Removes the device instance from the adaptor
>   * instance. The device instance is not check in any way or freed by this. 
>   * The caller is expected to take care of that. This will simply remove the
> - * device from the adapters data strcutures.
> + * device from the adapters data structures.
>   *
>   * @acb: The adapter device to be updated
>   * @dcb: A device that has previously been added to the adapter.
> @@ -3980,7 +3980,7 @@ static void trms1040_read_all(struct NvRamType *eeprom, unsigned long io_port)
>   * Checks the checksum and if it's not correct it uses a set of default
>   * values.
>   *
> - * @eeprom:	caller allocated strcuture to read the eeprom data into
> + * @eeprom:	caller allocated structure to read the eeprom data into
>   * @io_port:	io port to read from
>   **/
>  static void check_eeprom(struct NvRamType *eeprom, unsigned long io_port)
> @@ -4578,7 +4578,7 @@ static void banner_display(void)
>   * dc395x_init_one - Initialise a single instance of the adapter.
>   *
>   * The PCI layer will call this once for each instance of the adapter
> - * that it finds in the system. The pci_dev strcuture indicates which
> + * that it finds in the system. The pci_dev structure indicates which
>   * instance we are being called from.
>   * 
>   * @dev: The PCI device to initialize.

The diff LGTM, thanks!

For the diff only,

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)