On Tue, Dec 17, 2019 at 3:11 AM Horia Geanta <horia.geanta@xxxxxxx> wrote: > > On 12/13/2019 5:39 PM, Adam Ford wrote: > > Both the i.MX8MQ and i.MX8M Mini support the CAAM driver, but it > So do the Layerscape ARMv8-based SoCs: > LS1012A, LS1028A, LS1043A, LS1046A, LS1088A, LS2088A, LX2160A > > > is currently not enabled by default. > > > > This patch enables this driver by default. > > > > Signed-off-by: Adam Ford <aford173@xxxxxxxxx> > > --- > > V2: New to series > > > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > > index 6a83ba2aea3e..0212975b908b 100644 > > --- a/arch/arm64/configs/defconfig > > +++ b/arch/arm64/configs/defconfig > > @@ -845,6 +845,7 @@ CONFIG_SECURITY=y > > CONFIG_CRYPTO_ECHAINIV=y > > CONFIG_CRYPTO_ANSI_CPRNG=y > > CONFIG_CRYPTO_DEV_SUN8I_CE=m > > +CONFIG_CRYPTO_DEV_FSL_CAAM=y > This should probably be "m" instead. Out of curiosity, what is the rule for when things are 'm' vs 'y'? In the Code Aurora repo, it is set to 'y' and the mainline kernel for the i.MX6/7, the imx_v6_v7_defconfig is also set to 'y' which is why I used 'y' here. I can do a V3 to address the other items you noted, but I want to understand the rules about the defconfig so I don't make the same mistake again. thanks, adam > > Horia
