linux-kernel - Re: [PATCH] scsi: Mundane spelling fixes in the file qla1280.c

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: <410164a3-d00f-f2bf-3eb2-49a163add143@infradead.org>
Date:   Sun, 14 Mar 2021 19:25:06 -0700
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Bhaskar Chowdhury <unixbhaskar@...il.com>, mdr@....com,
        jejb@...ux.ibm.com, martin.petersen@...cle.com,
        linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] scsi: Mundane spelling fixes in the file qla1280.c

On 3/14/21 7:16 PM, Bhaskar Chowdhury wrote:
> 
> s/quantites/quantities/
> s/Unfortunely/Unfortunately/
> 
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@...il.com>

Acked-by: Randy Dunlap <rdunlap@...radead.org>


> ---
>  drivers/scsi/qla1280.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c
> index 46de2541af25..95008811b2d2 100644
> --- a/drivers/scsi/qla1280.c
> +++ b/drivers/scsi/qla1280.c
> @@ -633,13 +633,13 @@ static int qla1280_read_nvram(struct scsi_qla_host *ha)
>  	 * to be read a word (two bytes) at a time.
>  	 *
>  	 * The net result of this would be that the word (and
> -	 * doubleword) quantites in the firmware would be correct, but
> +	 * doubleword) quantities in the firmware would be correct, but
>  	 * the bytes would be pairwise reversed.  Since most of the
> -	 * firmware quantites are, in fact, bytes, we do an extra
> +	 * firmware quantities are, in fact, bytes, we do an extra
>  	 * le16_to_cpu() in the firmware read routine.
>  	 *
>  	 * The upshot of all this is that the bytes in the firmware
> -	 * are in the correct places, but the 16 and 32 bit quantites
> +	 * are in the correct places, but the 16 and 32 bit quantities
>  	 * are still in little endian format.  We fix that up below by
>  	 * doing extra reverses on them */
>  	nv->isp_parameter = cpu_to_le16(nv->isp_parameter);
> @@ -687,7 +687,7 @@ qla1280_info(struct Scsi_Host *host)
>   * The mid-level driver tries to ensures that queuecommand never gets invoked
>   * concurrently with itself or the interrupt handler (although the
>   * interrupt handler may call this routine as part of request-completion
> - * handling).   Unfortunely, it sometimes calls the scheduler in interrupt
> + * handling).   Unfortunately, it sometimes calls the scheduler in interrupt
>   * context which is a big NO! NO!.
>   **************************************************************************/
>  static int
> --


-- 
~Randy