2022-01-12 13:59:04

by Geert Uytterhoeven

Subject: [PATCH] crypto: marvell - Fix platform dependency for CRYPTO_DEV_OCTEONTX2_CPT

Arnd pointed out that OcteonTX2 is Thunder, not ThunderX2
(ThunderX2 is CN99XX, formerly named Broadcom Vulcan).

Fixes: 42e6f351dcb05fd1 ("crypto: marvell - CRYPTO_DEV_OCTEONTX2_CPT should depend on ARCH_THUNDER2")
Reported-by: Arnd Bergmann <[email protected]>
Signed-off-by: Geert Uytterhoeven <[email protected]>
---
drivers/crypto/marvell/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/marvell/Kconfig b/drivers/crypto/marvell/Kconfig
index 9125199f1702bbeb..3221a9d39dec604c 100644
--- a/drivers/crypto/marvell/Kconfig
+++ b/drivers/crypto/marvell/Kconfig
@@ -38,7 +38,7 @@ config CRYPTO_DEV_OCTEONTX_CPT

config CRYPTO_DEV_OCTEONTX2_CPT
tristate "Marvell OcteonTX2 CPT driver"
- depends on ARCH_THUNDER2 || COMPILE_TEST
+ depends on ARCH_THUNDER || COMPILE_TEST
depends on PCI_MSI && 64BIT
depends on CRYPTO_LIB_AES
depends on NET_VENDOR_MARVELL
--
2.25.1

2022-01-22 00:34:55

by Shijith Thotton

Subject: Re: [PATCH] crypto: marvell - Fix platform dependency for CRYPTO_DEV_OCTEONTX2_CPT

OcteonTX2 doesn't fall under ARCH_THUNDER and ARCH_THUNDER2. A better approach
would be to define and use a new SOC type as Arnd mentioned at
https://patchwork.kernel.org/project/linux-crypto/patch/266065918e47e8965bb6a0ab486da070278788e4.1641996057.git.geert+renesas@glider.be/
Please wait for him to post that patch or use arm64 as dependency.

Thanks,
Shijith