From: Boris Brezillon <boris.brezillon@free-electrons.com> To: Boris Brezillon <boris.brezillon@free-electrons.com> Cc: Jason Cooper <jason@lakedaemon.net>, Arnaud Ebalard <arno@natisbad.org>, Gregory CLEMENT <gregory.clement@free-electrons.com>, Mark Rutland <mark.rutland@arm.com>, Thomas Petazzoni <info@free-electrons.com>, Herbert Xu <herbert@gondor.apana.org.au>, Pawel Moll <pawel.moll@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Eran Ben-Avi <benavi@marvell.com>, Nadav Haklai <nadavh@marvell.com>, devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, Andrew Lunn <andrew@lunn.ch>, linux-crypto@vger.kernel.org, Kumar Gala <galak@codeaurora.org>, Tawfik Bayouk <tawfik@marvell.com>, "David S. Miller" <davem@davemloft.net>, Lior Amsalem <alior@marvell.com>, Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> Subject: Re: [PATCH 0/2] crypto: add new driver for Marvell CESA Date: Fri, 17 Apr 2015 10:39:46 +0200 [thread overview] Message-ID: <20150417103946.2a46c243@bbrezillon> (raw) In-Reply-To: <20150417103356.6ce78145@bbrezillon> On Fri, 17 Apr 2015 10:33:56 +0200 Boris Brezillon <boris.brezillon@free-electrons.com> wrote: > Hi Jason, > > On Mon, 13 Apr 2015 20:11:46 +0000 > Jason Cooper <jason@lakedaemon.net> wrote: > > > > > > > > I'd appreciate if we'd look into it. I understand from on-list and > > > > off-list discussion that the rewrite was unavoidable. So I'm willing to > > > > concede that. Giving people time to migrate from old to new while still > > > > being able to update for other security fixes seems reasonable. > > > > > > Jason, what do you think of the approach above? > > > > I say keep it simple. We shouldn't use the DT changes to trigger one > > vice the other. We need to be able to build both, but only load one at > > a time. If that's anything other than simple to do, then we make it a > > Kconfig binary choice and move on. > > Actually I was planning to handle it with a Kconfig dependency rule > (NEW_DRIVER depends on !OLD_DRIVER and OLD_DRIVER depends > on !NEW_DRIVER). > I don't know how to make it a runtime check without adding new > compatible strings for the kirkwood, dove and orion platforms, and I'm > sure sure this is a good idea. ^ not > Do you have any ideas ? > > Regards, > > Boris > > -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: boris.brezillon@free-electrons.com (Boris Brezillon) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 0/2] crypto: add new driver for Marvell CESA Date: Fri, 17 Apr 2015 10:39:46 +0200 [thread overview] Message-ID: <20150417103946.2a46c243@bbrezillon> (raw) In-Reply-To: <20150417103356.6ce78145@bbrezillon> On Fri, 17 Apr 2015 10:33:56 +0200 Boris Brezillon <boris.brezillon@free-electrons.com> wrote: > Hi Jason, > > On Mon, 13 Apr 2015 20:11:46 +0000 > Jason Cooper <jason@lakedaemon.net> wrote: > > > > > > > > I'd appreciate if we'd look into it. I understand from on-list and > > > > off-list discussion that the rewrite was unavoidable. So I'm willing to > > > > concede that. Giving people time to migrate from old to new while still > > > > being able to update for other security fixes seems reasonable. > > > > > > Jason, what do you think of the approach above? > > > > I say keep it simple. We shouldn't use the DT changes to trigger one > > vice the other. We need to be able to build both, but only load one at > > a time. If that's anything other than simple to do, then we make it a > > Kconfig binary choice and move on. > > Actually I was planning to handle it with a Kconfig dependency rule > (NEW_DRIVER depends on !OLD_DRIVER and OLD_DRIVER depends > on !NEW_DRIVER). > I don't know how to make it a runtime check without adding new > compatible strings for the kirkwood, dove and orion platforms, and I'm > sure sure this is a good idea. ^ not > Do you have any ideas ? > > Regards, > > Boris > > -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
next prev parent reply other threads:[~2015-04-17 8:39 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 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 [this message] 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=20150417103946.2a46c243@bbrezillon \ --to=boris.brezillon@free-electrons.com \ --cc=alior@marvell.com \ --cc=andrew@lunn.ch \ --cc=arno@natisbad.org \ --cc=benavi@marvell.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.