* [PATCH] crypto: artpec6 - use memcpy_and_pad to simplify prepare_hash
@ 2026-03-09 21:11 Thorsten Blum
2026-03-21 8:45 ` Herbert Xu
0 siblings, 1 reply; 2+ messages in thread
From: Thorsten Blum @ 2026-03-09 21:11 UTC (permalink / raw)
To: Jesper Nilsson, Lars Persson, Herbert Xu, David S. Miller
Cc: Thorsten Blum, linux-arm-kernel, linux-crypto, linux-kernel
Use memcpy_and_pad() instead of memcpy() followed by memset() to
simplify artpec6_crypto_prepare_hash().
Also fix a duplicate word in a comment and remove a now-redundant one.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
drivers/crypto/axis/artpec6_crypto.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/crypto/axis/artpec6_crypto.c b/drivers/crypto/axis/artpec6_crypto.c
index b04d6379244a..a4793b76300c 100644
--- a/drivers/crypto/axis/artpec6_crypto.c
+++ b/drivers/crypto/axis/artpec6_crypto.c
@@ -1323,7 +1323,7 @@ static int artpec6_crypto_prepare_hash(struct ahash_request *areq)
artpec6_crypto_init_dma_operation(common);
- /* Upload HMAC key, must be first the first packet */
+ /* Upload HMAC key, it must be the first packet */
if (req_ctx->hash_flags & HASH_FLAG_HMAC) {
if (variant == ARTPEC6_CRYPTO) {
req_ctx->key_md = FIELD_PREP(A6_CRY_MD_OPER,
@@ -1333,11 +1333,8 @@ static int artpec6_crypto_prepare_hash(struct ahash_request *areq)
a7_regk_crypto_dlkey);
}
- /* Copy and pad up the key */
- memcpy(req_ctx->key_buffer, ctx->hmac_key,
- ctx->hmac_key_length);
- memset(req_ctx->key_buffer + ctx->hmac_key_length, 0,
- blocksize - ctx->hmac_key_length);
+ memcpy_and_pad(req_ctx->key_buffer, blocksize, ctx->hmac_key,
+ ctx->hmac_key_length, 0);
error = artpec6_crypto_setup_out_descr(common,
(void *)&req_ctx->key_md,
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] crypto: artpec6 - use memcpy_and_pad to simplify prepare_hash
2026-03-09 21:11 [PATCH] crypto: artpec6 - use memcpy_and_pad to simplify prepare_hash Thorsten Blum
@ 2026-03-21 8:45 ` Herbert Xu
0 siblings, 0 replies; 2+ messages in thread
From: Herbert Xu @ 2026-03-21 8:45 UTC (permalink / raw)
To: Thorsten Blum
Cc: Jesper Nilsson, Lars Persson, David S. Miller, linux-arm-kernel,
linux-crypto, linux-kernel
On Mon, Mar 09, 2026 at 10:11:21PM +0100, Thorsten Blum wrote:
> Use memcpy_and_pad() instead of memcpy() followed by memset() to
> simplify artpec6_crypto_prepare_hash().
>
> Also fix a duplicate word in a comment and remove a now-redundant one.
>
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
> drivers/crypto/axis/artpec6_crypto.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
Patch applied. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-21 8:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-09 21:11 [PATCH] crypto: artpec6 - use memcpy_and_pad to simplify prepare_hash Thorsten Blum
2026-03-21 8:45 ` Herbert Xu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox