Hi Arnaud, On 28 February 2013 12:07, Arnaud Patard <arnaud.patard@xxxxxxxxxxx> wrote: > Javier Martin <javier.martin@xxxxxxxxxxxxxxxxx> writes: > > Hi, > >> SAHARA2 HW module is included in the i.MX27 SoC from >> Freescale. It is capable of performing cipher algorithms >> such as AES, 3DES..., hashing and RNG too. > > The rmmod oops is gone with this version, thanks. > >> >> This driver provides support for AES-CBC and AES-ECB >> by now. >> >> Signed-off-by: Javier Martin <javier.martin@xxxxxxxxxxxxxxxxx> >> --- >> drivers/crypto/Kconfig | 10 + >> drivers/crypto/Makefile | 1 + >> drivers/crypto/sahara.c | 1070 +++++++++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 1081 insertions(+) >> create mode 100644 drivers/crypto/sahara.c >> >> diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig >> index 87ec4d0..7d73f5a 100644 >> --- a/drivers/crypto/Kconfig >> +++ b/drivers/crypto/Kconfig >> @@ -276,6 +276,16 @@ config CRYPTO_DEV_PICOXCELL >> >> Saying m here will build a module named pipcoxcell_crypto. >> >> +config CRYPTO_DEV_SAHARA >> + tristate "Support for SAHARA crypto accelerator" >> + depends on MACH_MX27 && EXPERIMENTAL && OF > > maybe switch to ARCH_MXC instead of MAC_MX27 ? > > [...] > >> +static struct platform_driver sahara_driver = { >> + .probe = sahara_probe, >> + .remove = __devexit_p(sahara_remove), > > Please remove the __devexit_p too. > > > For information, you'll find below what I've changed to test the driver on my > imx51. It's not perfect but it's enough for testing things. > > Since I have to send v3 anyway to comply with Sascha's request maybe it would be worth to include your changes to provide support for i.mx51 too. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander. Cantabria. Spain +34 942 25 32 60 www.vista-silicon.com -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html