From: sashiko-bot@kernel.org
To: "Pavitrakumar Managutte" <pavitrakumarm@vayavyalabs.com>
Cc: devicetree@vger.kernel.org, conor+dt@kernel.org, robh@kernel.org
Subject: Re: [PATCH v20 4/4] crypto: spacc - Add SPAcc Kconfig and Makefile
Date: Wed, 05 Aug 2026 16:59:55 +0000 [thread overview]
Message-ID: <20260805165956.5B2E31F000E9@smtp.kernel.org> (raw)
In-Reply-To: <20260805164815.249328-5-pavitrakumarm@vayavyalabs.com>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] Missing `select CRYPTO_SHA3` dependency for SPAcc ahash support.
--
--- Patch [4]: [PATCH v20 4/4] crypto: spacc - Add SPAcc Kconfig and Makefile ---
commit 4e5d1fffdbede78b50849c59ea51d64c31455d5c
Author: Pavitrakumar Managutte <pavitrakumarm@vayavyalabs.com>
crypto: spacc - Add SPAcc Kconfig and Makefile
This commit introduces the Kconfig and Makefile for the new dwc_spacc
Security Protocol Accelerator driver. It wires up the build system entries
and configuration options for various SPAcc features.
> diff --git a/drivers/crypto/dwc-spacc/Kconfig b/drivers/crypto/dwc-spacc/Kconfig
> new file mode 100644
> --- /dev/null
> +++ b/drivers/crypto/dwc-spacc/Kconfig
> @@ -0,0 +1,64 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +
> +config CRYPTO_DEV_SPACC
> + tristate "Support for dwc_spacc Security Protocol Accelerator"
> + depends on HAS_DMA
> + depends on HAS_IOMEM
> + select CRYPTO_ENGINE
> + select CRYPTO_HASH if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_SHA1 if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_MD5 if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_SHA256 if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_SHA512 if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_HMAC if CRYPTO_DEV_SPACC_HASH
[Severity: Medium]
Is a select for CRYPTO_SHA3 missing here?
The SPAcc driver implements hardware acceleration for SHA-3 algorithms and
requires the software SHA-3 algorithms as fallbacks. If CONFIG_CRYPTO_SHA3
is not enabled, the driver's probe sequence will fail to find the fallback
and silently skip registering the SHA-3 hardware acceleration algorithms.
> + select CRYPTO_CMAC if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_XCBC if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_AES if CRYPTO_DEV_SPACC_HASH
> + select CRYPTO_SM4_GENERIC if CRYPTO_DEV_SPACC_HASH
> + default n
[ ... ]
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260805164815.249328-1-pavitrakumarm@vayavyalabs.com?part=4
prev parent reply other threads:[~2026-08-05 16:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-05 16:48 [PATCH v20 0/4] crypto: spacc - Add SPAcc Crypto Driver Pavitrakumar Managutte
2026-08-05 16:48 ` [PATCH v20 1/4] dt-bindings: crypto: Document support for SPAcc Pavitrakumar Managutte
2026-08-05 16:48 ` [PATCH v20 2/4] crypto: spacc - Add SPAcc ahash support Pavitrakumar Managutte
2026-08-05 17:03 ` sashiko-bot
2026-08-05 16:48 ` [PATCH v20 3/4] crypto: spacc - add spacc autodetect support Pavitrakumar Managutte
2026-08-05 16:48 ` [PATCH v20 4/4] crypto: spacc - Add SPAcc Kconfig and Makefile Pavitrakumar Managutte
2026-08-05 16:59 ` sashiko-bot [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=20260805165956.5B2E31F000E9@smtp.kernel.org \
--to=sashiko-bot@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=pavitrakumarm@vayavyalabs.com \
--cc=robh@kernel.org \
--cc=sashiko-reviews@lists.linux.dev \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.