From: Bagas Sanjaya <bagasdotme@gmail.com>
To: Jiangshan Yi <13667453960@163.com>
Cc: herbert@gondor.apana.org.au, davem@davemloft.net,
giovanni.cabiddu@intel.com, qat-linux@intel.com,
linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org,
Jiangshan Yi <yijiangshan@kylinos.cn>,
k2ci <kernel-bot@kylinos.cn>
Subject: Re: [PATCH] crypto: qat - fix spelling typo in comment
Date: Sun, 9 Oct 2022 19:42:41 +0700 [thread overview]
Message-ID: <Y0LBwe4/K2hnchRC@debian.me> (raw)
In-Reply-To: <20221009091519.3152948-1-13667453960@163.com>
[-- Attachment #1: Type: text/plain, Size: 1674 bytes --]
On Sun, Oct 09, 2022 at 05:15:19PM +0800, Jiangshan Yi wrote:
> From: Jiangshan Yi <yijiangshan@kylinos.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@kylinos.cn>
> Signed-off-by: Jiangshan Yi <yijiangshan@kylinos.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
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
prev parent reply other threads:[~2022-10-09 12:42 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-09 9:15 [PATCH] crypto: qat - fix spelling typo in comment Jiangshan Yi
2022-10-09 12:42 ` Bagas Sanjaya [this message]
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=Y0LBwe4/K2hnchRC@debian.me \
--to=bagasdotme@gmail.com \
--cc=13667453960@163.com \
--cc=davem@davemloft.net \
--cc=giovanni.cabiddu@intel.com \
--cc=herbert@gondor.apana.org.au \
--cc=kernel-bot@kylinos.cn \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=qat-linux@intel.com \
--cc=yijiangshan@kylinos.cn \
/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