From: Stephan Mueller <smueller@chronox.de> To: Boris Brezillon <boris.brezillon@free-electrons.com> Cc: Herbert Xu <herbert@gondor.apana.org.au>, "David S. Miller" <davem@davemloft.net>, linux-crypto@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, devicetree@vger.kernel.org, Tawfik Bayouk <tawfik@marvell.com>, Lior Amsalem <alior@marvell.com>, Nadav Haklai <nadavh@marvell.com>, Eran Ben-Avi <benavi@marvell.com>, Thomas Petazzoni <info@free-electrons.com>, Gregory CLEMENT <gregory.clement@free-electrons.com>, Jason Cooper <jason@lakedaemon.net>, Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>, Andrew Lunn <andrew@lunn.ch>, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Arnaud Ebalard <arno@natisbad.org> Subject: Re: [PATCH 0/2] crypto: add new driver for Marvell CESA Date: Thu, 09 Apr 2015 17:52:42 +0200 [thread overview] Message-ID: <13353890.ylqfTNBYXC@tauon> (raw) In-Reply-To: <1428591523-1780-1-git-send-email-boris.brezillon@free-electrons.com> Am Donnerstag, 9. April 2015, 16:58:41 schrieb Boris Brezillon: Hi Boris, >Hello, > >This is an attempt to replace the mv_cesa driver by a new one to address >some limitations of the existing driver. >From a performance and CPU load point of view the most important >limitation is the lack of DMA support, thus preventing us from chaining >crypto operations. > >I know we usually try to adapt existing drivers instead of replacing them >by new ones, but after trying to refactor the mv_cesa driver I realized it >would take longer than writing an new one from scratch. > >Here are the main features brought by this new driver: >- support for armada SoCs (up to 38x) while keeping support for older ones > (Orion and Kirkwood) >- DMA mode to offload the CPU in case of intensive crypto usage >- new algorithms: SHA256, DES and 3DES > >I'd like to thank Arnaud, who has carefully reviewed several iterations of >this driver, helped me improved my implementation, provided support for >several crypto algorithms, provided support for armada-370 and tested >the driver on different platforms, hence the SoB and dual MODULE_AUTHOR >in the driver code. Your patch 1/2 did not make it to the crypto list. To big? It is on the lkml list though. > >Best Regards, > >Boris > >Boris Brezillon (2): > crypto: add new driver for Marvell CESA > crypto: marvell/CESA: update DT bindings documentation > > .../devicetree/bindings/crypto/mv_cesa.txt | 50 +- > drivers/crypto/Kconfig | 2 + > drivers/crypto/Makefile | 2 +- > drivers/crypto/marvell/Makefile | 1 + > drivers/crypto/marvell/cesa.c | 539 ++++++++ > drivers/crypto/marvell/cesa.h | 802 ++++++++++++ > drivers/crypto/marvell/cipher.c | 761 +++++++++++ > drivers/crypto/marvell/hash.c | 1349 >++++++++++++++++++++ drivers/crypto/marvell/tdma.c | >223 ++++ > drivers/crypto/mv_cesa.c | 1193 ----------------- > drivers/crypto/mv_cesa.h | 150 --- > 11 files changed, 3716 insertions(+), 1356 deletions(-) > create mode 100644 drivers/crypto/marvell/Makefile > create mode 100644 drivers/crypto/marvell/cesa.c > create mode 100644 drivers/crypto/marvell/cesa.h > create mode 100644 drivers/crypto/marvell/cipher.c > create mode 100644 drivers/crypto/marvell/hash.c > create mode 100644 drivers/crypto/marvell/tdma.c > delete mode 100644 drivers/crypto/mv_cesa.c > delete mode 100644 drivers/crypto/mv_cesa.h Ciao Stephan
WARNING: multiple messages have this Message-ID (diff)
From: smueller@chronox.de (Stephan Mueller) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 0/2] crypto: add new driver for Marvell CESA Date: Thu, 09 Apr 2015 17:52:42 +0200 [thread overview] Message-ID: <13353890.ylqfTNBYXC@tauon> (raw) In-Reply-To: <1428591523-1780-1-git-send-email-boris.brezillon@free-electrons.com> Am Donnerstag, 9. April 2015, 16:58:41 schrieb Boris Brezillon: Hi Boris, >Hello, > >This is an attempt to replace the mv_cesa driver by a new one to address >some limitations of the existing driver. >From a performance and CPU load point of view the most important >limitation is the lack of DMA support, thus preventing us from chaining >crypto operations. > >I know we usually try to adapt existing drivers instead of replacing them >by new ones, but after trying to refactor the mv_cesa driver I realized it >would take longer than writing an new one from scratch. > >Here are the main features brought by this new driver: >- support for armada SoCs (up to 38x) while keeping support for older ones > (Orion and Kirkwood) >- DMA mode to offload the CPU in case of intensive crypto usage >- new algorithms: SHA256, DES and 3DES > >I'd like to thank Arnaud, who has carefully reviewed several iterations of >this driver, helped me improved my implementation, provided support for >several crypto algorithms, provided support for armada-370 and tested >the driver on different platforms, hence the SoB and dual MODULE_AUTHOR >in the driver code. Your patch 1/2 did not make it to the crypto list. To big? It is on the lkml list though. > >Best Regards, > >Boris > >Boris Brezillon (2): > crypto: add new driver for Marvell CESA > crypto: marvell/CESA: update DT bindings documentation > > .../devicetree/bindings/crypto/mv_cesa.txt | 50 +- > drivers/crypto/Kconfig | 2 + > drivers/crypto/Makefile | 2 +- > drivers/crypto/marvell/Makefile | 1 + > drivers/crypto/marvell/cesa.c | 539 ++++++++ > drivers/crypto/marvell/cesa.h | 802 ++++++++++++ > drivers/crypto/marvell/cipher.c | 761 +++++++++++ > drivers/crypto/marvell/hash.c | 1349 >++++++++++++++++++++ drivers/crypto/marvell/tdma.c | >223 ++++ > drivers/crypto/mv_cesa.c | 1193 ----------------- > drivers/crypto/mv_cesa.h | 150 --- > 11 files changed, 3716 insertions(+), 1356 deletions(-) > create mode 100644 drivers/crypto/marvell/Makefile > create mode 100644 drivers/crypto/marvell/cesa.c > create mode 100644 drivers/crypto/marvell/cesa.h > create mode 100644 drivers/crypto/marvell/cipher.c > create mode 100644 drivers/crypto/marvell/hash.c > create mode 100644 drivers/crypto/marvell/tdma.c > delete mode 100644 drivers/crypto/mv_cesa.c > delete mode 100644 drivers/crypto/mv_cesa.h Ciao Stephan
next prev parent reply other threads:[~2015-04-09 15:52 UTC|newest] Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-04-09 14:58 [PATCH 0/2] crypto: add new driver for Marvell CESA Boris Brezillon 2015-04-09 14:58 ` Boris Brezillon 2015-04-09 14:58 ` [PATCH 1/2] " Boris Brezillon [not found] ` <1428591523-1780-2-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> 2015-04-10 10:38 ` Paul Bolle 2015-04-10 10:38 ` Paul Bolle 2015-04-10 10:38 ` Paul Bolle 2015-04-10 11:17 ` Boris Brezillon 2015-04-10 11:17 ` Boris Brezillon 2015-04-09 14:58 ` [PATCH 2/2] crypto: marvell/CESA: update DT bindings documentation Boris Brezillon 2015-04-09 14:58 ` Boris Brezillon 2015-04-09 15:18 ` [PATCH 0/2] crypto: add new driver for Marvell CESA Andrew Lunn 2015-04-09 15:18 ` Andrew Lunn [not found] ` <20150409172826.18916274@bbrezillon> 2015-04-09 15:37 ` Andrew Lunn 2015-04-09 15:37 ` Andrew Lunn 2015-04-09 15:37 ` Andrew Lunn 2015-04-09 15:37 ` Andrew Lunn 2015-04-09 15:34 ` Sebastian Hesselbarth 2015-04-09 15:34 ` Sebastian Hesselbarth 2015-04-09 15:57 ` Boris Brezillon 2015-04-09 15:57 ` Boris Brezillon 2015-04-09 23:21 ` Arnaud Ebalard 2015-04-09 23:21 ` Arnaud Ebalard 2015-04-09 23:21 ` Arnaud Ebalard 2015-04-09 15:52 ` Stephan Mueller [this message] 2015-04-09 15:52 ` Stephan Mueller 2015-04-10 13:50 ` Jason Cooper 2015-04-10 13:50 ` Jason Cooper 2015-04-10 15:11 ` Boris Brezillon 2015-04-10 15:11 ` Boris Brezillon 2015-04-10 22:30 ` Jason Cooper 2015-04-10 22:30 ` Jason Cooper 2015-04-13 9:39 ` Gregory CLEMENT 2015-04-13 9:39 ` Gregory CLEMENT 2015-04-13 12:47 ` Jason Cooper 2015-04-13 12:47 ` Jason Cooper 2015-04-13 16:06 ` Arnaud Ebalard 2015-04-13 16:06 ` Arnaud Ebalard 2015-04-13 20:11 ` Jason Cooper 2015-04-13 20:11 ` Jason Cooper 2015-04-17 8:33 ` Boris Brezillon 2015-04-17 8:33 ` Boris Brezillon 2015-04-17 8:39 ` Boris Brezillon 2015-04-17 8:39 ` Boris Brezillon 2015-04-17 10:59 ` Jason Cooper 2015-04-17 10:59 ` Jason Cooper 2015-04-17 13:01 ` Gregory CLEMENT 2015-04-17 13:01 ` Gregory CLEMENT 2015-04-17 14:19 ` Boris Brezillon 2015-04-17 14:19 ` Boris Brezillon 2015-04-17 14:32 ` Maxime Ripard 2015-04-17 14:32 ` Maxime Ripard 2015-04-17 14:40 ` Gregory CLEMENT 2015-04-17 14:40 ` Gregory CLEMENT 2015-04-17 14:50 ` Maxime Ripard 2015-04-17 14:50 ` Maxime Ripard 2015-04-17 15:01 ` Gregory CLEMENT 2015-04-17 15:01 ` Gregory CLEMENT 2015-04-17 15:01 ` Gregory CLEMENT 2015-04-17 15:49 ` Maxime Ripard 2015-04-17 15:49 ` Maxime Ripard 2015-04-17 16:04 ` Gregory CLEMENT 2015-04-17 16:04 ` Gregory CLEMENT 2015-04-17 16:04 ` Gregory CLEMENT 2015-04-28 19:52 ` Boris Brezillon 2015-04-28 19:52 ` Boris Brezillon 2015-04-29 9:49 ` Herbert Xu 2015-04-29 9:49 ` Herbert Xu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=13353890.ylqfTNBYXC@tauon \ --to=smueller@chronox.de \ --cc=alior@marvell.com \ --cc=andrew@lunn.ch \ --cc=arno@natisbad.org \ --cc=benavi@marvell.com \ --cc=boris.brezillon@free-electrons.com \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=galak@codeaurora.org \ --cc=gregory.clement@free-electrons.com \ --cc=herbert@gondor.apana.org.au \ --cc=ijc+devicetree@hellion.org.uk \ --cc=info@free-electrons.com \ --cc=jason@lakedaemon.net \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-crypto@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=nadavh@marvell.com \ --cc=pawel.moll@arm.com \ --cc=robh+dt@kernel.org \ --cc=sebastian.hesselbarth@gmail.com \ --cc=tawfik@marvell.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.