Hi Easwar, The default will be a module, my bad. I will rectify this in the v1 patch, which I am already working on. - PK On Wed, Mar 6, 2024 at 11:12 AM Easwar Hariharan <eahariha@xxxxxxxxxxxxxxxxxxx> wrote: > > 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 >