On 3/5/2024 3:28 AM, Pavitrakumar M wrote: > Signed-off-by: shwetar <shwetar@xxxxxxxxxxxxxxx> > Signed-off-by: Pavitrakumar M <pavitrakumarm@xxxxxxxxxxxxxxx> > Acked-by: Ruud Derwig <Ruud.Derwig@xxxxxxxxxxxx> > --- > drivers/crypto/dwc-spacc/Kconfig | 95 +++++++++++++++++++++++++++++++ > drivers/crypto/dwc-spacc/Makefile | 16 ++++++ > 2 files changed, 111 insertions(+) > create mode 100644 drivers/crypto/dwc-spacc/Kconfig > create mode 100644 drivers/crypto/dwc-spacc/Makefile > > diff --git a/drivers/crypto/dwc-spacc/Kconfig b/drivers/crypto/dwc-spacc/Kconfig > new file mode 100644 > index 000000000000..6f40358f7932 > --- /dev/null > +++ b/drivers/crypto/dwc-spacc/Kconfig > @@ -0,0 +1,95 @@ > +# SPDX-License-Identifier: GPL-2.0-only > + > +config CRYPTO_DEV_SPACC > + tristate "Support for dw_spacc Security protocol accelerators" > + depends on HAS_DMA > + default y <snip> Why is the default y rather than n or m? I would prefer it to be a module, but I just want to understand why it was chosen to be default compiled in. Thanks, Easwar