Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 9 Oct 2022 19:42:41 +0700
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
Message-ID: <Y0LBwe4/K2hnchRC@debian.me>
References: <20221009091519.3152948-1-13667453960@163.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="FF6OHtQv1OVujNNT"
Content-Disposition: inline
In-Reply-To: <20221009091519.3152948-1-13667453960@163.com>
Precedence: bulk


--FF6OHtQv1OVujNNT
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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

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(-)
>=20
> 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)
> =20
> -/* 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)
=20
Oh, you refer to comment for ADF_RING_SIZE_BYTES_MIN() macro.
The patch does s/bufer/buffer/.

Thanks.

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

--FF6OHtQv1OVujNNT
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCY0LBuwAKCRD2uYlJVVFO
o3aFAP9y3fTmuQN/W+GzjlLzAAHPgHIU2nnTRWW2oB5BrrZWhAD+OIqszJ8iFlg7
GEHhezJNyBZOyYJCVvFXL9yOr3uWRwY=
=MU34
-----END PGP SIGNATURE-----

--FF6OHtQv1OVujNNT--