diff mbox series

[v1] hwrng: Kconfig - Do not enable by default CN10K driver

Message ID 20240625195746.48905-1-francesco@dolcini.it
State Superseded
Headers show
Series [v1] hwrng: Kconfig - Do not enable by default CN10K driver | expand

Commit Message

Francesco Dolcini June 25, 2024, 7:57 p.m. UTC
From: Francesco Dolcini <francesco.dolcini@toradex.com>

Do not enable by default the CN10K HW random generator driver.

CN10K Random Number Generator is available only on some specific
Marvell SoCs, however the driver is in practice enabled by default on
all arm64 configs.

Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
as an alternative I could propose

default HW_RANDOM if ARCH_THUNDER=y

---
 drivers/char/hw_random/Kconfig | 1 -
 1 file changed, 1 deletion(-)

Comments

Herbert Xu July 6, 2024, 12:10 a.m. UTC | #1
On Tue, Jun 25, 2024 at 09:57:46PM +0200, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> Do not enable by default the CN10K HW random generator driver.
> 
> CN10K Random Number Generator is available only on some specific
> Marvell SoCs, however the driver is in practice enabled by default on
> all arm64 configs.
> 
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
> ---
> as an alternative I could propose
> 
> default HW_RANDOM if ARCH_THUNDER=y

Adding marvell Cc.

Cheers,
Herbert Xu July 12, 2024, 11:55 p.m. UTC | #2
On Tue, Jun 25, 2024 at 09:57:46PM +0200, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> Do not enable by default the CN10K HW random generator driver.
> 
> CN10K Random Number Generator is available only on some specific
> Marvell SoCs, however the driver is in practice enabled by default on
> all arm64 configs.
> 
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
> ---
> as an alternative I could propose
> 
> default HW_RANDOM if ARCH_THUNDER=y
> 
> ---
>  drivers/char/hw_random/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

Patch applied.  Thanks.
Francesco Dolcini July 22, 2024, 6:55 a.m. UTC | #3
Hello Herbert, Bharat

On Thu, Jul 18, 2024 at 10:10:31AM +0000, Bharat Bhushan wrote:
> > On Tue, Jun 25, 2024 at 09:57:46PM +0200, Francesco Dolcini wrote:
> >> From: Francesco Dolcini <mailto:francesco.dolcini@toradex.com>
> >> 
> >> Do not enable by default the CN10K HW random generator driver.
> >> 
> >> CN10K Random Number Generator is available only on some specific
> >> Marvell SoCs, however the driver is in practice enabled by default on
> >> all arm64 configs.
> >> 
> >> Signed-off-by: Francesco Dolcini <mailto:francesco.dolcini@toradex.com>
> >> ---
> >> as an alternative I could propose
> >> 
> >> default HW_RANDOM if ARCH_THUNDER=y
> 
> Yes, make default if ARCH_THUNDER is true

Herbert, this patch was merged, do you want a follow-up patch as Bharat wrote
or you are good with the current status?

Francesco
Herbert Xu July 24, 2024, 3:13 a.m. UTC | #4
On Mon, Jul 22, 2024 at 08:55:40AM +0200, Francesco Dolcini wrote:
>
> > Yes, make default if ARCH_THUNDER is true
> 
> Herbert, this patch was merged, do you want a follow-up patch as Bharat wrote
> or you are good with the current status?

Please send me a follow-up patch.  Thanks!
diff mbox series

Patch

diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig
index 442c40efb200..01e2e1ef82cf 100644
--- a/drivers/char/hw_random/Kconfig
+++ b/drivers/char/hw_random/Kconfig
@@ -555,7 +555,6 @@  config HW_RANDOM_ARM_SMCCC_TRNG
 config HW_RANDOM_CN10K
        tristate "Marvell CN10K Random Number Generator support"
        depends on HW_RANDOM && PCI && (ARM64 || (64BIT && COMPILE_TEST))
-       default HW_RANDOM
        help
 	 This driver provides support for the True Random Number
 	 generator available in Marvell CN10K SoCs.