2022-10-09 09:19:08

by Yi Jiangshan

[permalink] [raw]

Subject: [PATCH] crypto: qat - fix spelling typo in comment

---

From: Jiangshan Yi <[email protected]>

Fix spelling typo in comment.

Reported-by: k2ci <[email protected]>
Signed-off-by: Jiangshan Yi <[email protected]>
---
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)
--
2.25.1


No virus found
Checked by Hillstone Network AntiVirus

2022-10-09 12:48:04

by Bagas Sanjaya

[permalink] [raw]

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 <[email protected]>
>
> 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 <[email protected]>
> Signed-off-by: Jiangshan Yi <[email protected]>
> ---
> 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

---

Attachments:

(No filename) (1.63 kB)
signature.asc (235.00 B)
Download all attachments