Re: [PATCH 23/28] ARM: dts: DRA7: Add DT node for DES IP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/06/16 14:38, Tony Lindgren wrote:
* Tero Kristo <t-kristo@xxxxxx> [160601 02:09]:
From: Joel Fernandes <joelf@xxxxxx>

DRA7xx SoCs have a DES3DES IP. Add DT data for the same.

Are these dts changes safe to apply separately or do they
cause issues like extra warnings during boot?

DTS changes are fine to merge as is separately, some crypto functionality might not work properly though (well, the support is somewhat broken on am43xx/dra7 anyways.) I just gave the kernel a boot test with am43xx/dra7 platforms with the DT data only applied, and that part worked fine.

I noticed that if you merge the hwmod changes before you have the DTS data though, you will get some extra boot warnings of the following kind on am43xx/dra7:

<snip>
[    0.315623] omap_hwmod: aes1: no dt node
[    0.315631] ------------[ cut here ]------------
[ 0.315656] WARNING: CPU: 0 PID: 1 at arch/arm/mach-omap2/omap_hwmod.c:2497 _init+0x1d0/0x41c
[    0.315663] omap_hwmod: aes1: doesn't have mpu register target base
<snip>

Do you want to pick-up the DTS changes from this revision of series as is or shall I repost those also? I think the series would require a re-ordering of posting the DTS changes before the hwmod data.

-Tero
--
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



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux