linux-kernel - Re: [PATCH] crypto: qat - fix spelling typo in comment

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: <Y0LBwe4/K2hnchRC@debian.me>
Date:   Sun, 9 Oct 2022 19:42:41 +0700
From:   Bagas Sanjaya <bagasdotme@...il.com>
To:     Jiangshan Yi <13667453960@....com>
Cc:     herbert@...dor.apana.org.au, davem@...emloft.net,
        giovanni.cabiddu@...el.com, qat-linux@...el.com,
        linux-crypto@...r.kernel.org, linux-kernel@...r.kernel.org,
        Jiangshan Yi <yijiangshan@...inos.cn>,
        k2ci <kernel-bot@...inos.cn>
Subject: Re: [PATCH] crypto: qat - fix spelling typo in comment

On Sun, Oct 09, 2022 at 05:15:19PM +0800, Jiangshan Yi wrote:
> From: Jiangshan Yi <yijiangshan@...inos.cn>
> 
> Fix spelling typo in comment.
> 

What comment are you referring? I have to see the actual diff below.

Anyways, for similar typofixes patches, you need to describe what words
you are correcting and where you do the fix.

Also, if you send the patch from a different address than From: line
in the patch, you need to have Signed-off-by for the sender address.

> Reported-by: k2ci <kernel-bot@...inos.cn>
> Signed-off-by: Jiangshan Yi <yijiangshan@...inos.cn>
> ---
>  drivers/crypto/qat/qat_common/adf_transport_access_macros.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/crypto/qat/qat_common/adf_transport_access_macros.h b/drivers/crypto/qat/qat_common/adf_transport_access_macros.h
> index 3b6b0267bbec..d3667dbd9826 100644
> --- a/drivers/crypto/qat/qat_common/adf_transport_access_macros.h
> +++ b/drivers/crypto/qat/qat_common/adf_transport_access_macros.h
> @@ -37,7 +37,7 @@
>  #define ADF_SIZE_TO_RING_SIZE_IN_BYTES(SIZE) ((1 << (SIZE - 1)) << 7)
>  #define ADF_RING_SIZE_IN_BYTES_TO_SIZE(SIZE) ((1 << (SIZE - 1)) >> 7)
>  
> -/* Minimum ring bufer size for memory allocation */
> +/* Minimum ring buffer size for memory allocation */
>  #define ADF_RING_SIZE_BYTES_MIN(SIZE) \
>  	((SIZE < ADF_SIZE_TO_RING_SIZE_IN_BYTES(ADF_RING_SIZE_4K)) ? \
>  		ADF_SIZE_TO_RING_SIZE_IN_BYTES(ADF_RING_SIZE_4K) : SIZE)
 
Oh, you refer to comment for ADF_RING_SIZE_BYTES_MIN() macro.
The patch does s/bufer/buffer/.

Thanks.

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

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