[lkml]   [2019]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] ARM: dts: imx7ulp: add crypto support
On 6/12/2019 4:06 PM, Shawn Guo wrote:
> On Wed, Jun 12, 2019 at 11:45:18AM +0000, Horia Geanta wrote:
>> On 6/12/2019 1:40 PM, Shawn Guo wrote:
>>> On Thu, Jun 06, 2019 at 11:02:55AM +0300, Horia Geantă wrote:
>>>> From: Iuliana Prodan <>
>>>> Add crypto node in device tree for CAAM support.
>>>> Noteworthy is that on 7ulp the interrupt line is shared
>>>> between the two job rings.
>>>> Signed-off-by: Iuliana Prodan <>
>>>> Signed-off-by: Franck LENORMAND <>
>>>> Signed-off-by: Horia Geantă <>
>>>> ---
>>>> I've just realized that this patch should be merged through the crypto tree,
>>>> else bisectability could be affected due to cryptodev-2.6
>>>> commit 385cfc84a5a8 ("crypto: caam - disable some clock checks for iMX7ULP")
>>>> ( )
>>>> which should come first.
>>> I'm not sure I follow it. This is a new device added to imx7ulp DT.
>>> It's never worked before on imx7ulp. How would it affect git bisect?
>> Driver corresponding to this device (drivers/crypto/caam) has to be updated
>> before adding the node in DT.
>> Is there any guarantee wrt. merge order of the crypto and DT trees?
> Do not merge DT changes until driver part hits mainline.
That would mean driver changes would be merged in v5.3 and DT node in v5.4.

Would going through the crypto tree with this patch be such a big issue?
I don't think it's the first time (relatively small) DT patches
are merged via other trees.


 \ /
  Last update: 2019-06-12 15:16    [W:0.071 / U:46.168 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site