[U-Boot] [PATCH v6 8/8] armv8: ls1046a: setup SEC ICIDs and fix up device tree

[Bharat Bhushan]

> -----Original Message-----
> From: Horia Geanta
> Sent: Tuesday, July 31, 2018 9:49 PM
> To: Laurentiu Tudor <laurentiu.tudor at nxp.com>; u-boot at lists.denx.de;
> Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; York Sun
> <york.sun at nxp.com>
> Cc: Bharat Bhushan <bharat.bhushan at nxp.com>
> Subject: Re: [PATCH v6 8/8] armv8: ls1046a: setup SEC ICIDs and fix up device
> tree
> 
> On 7/31/2018 5:53 PM, laurentiu.tudor at nxp.com wrote:
> > From: Laurentiu Tudor <laurentiu.tudor at nxp.com>
> >
> > Add support for SEC ICID configuration and apply it for ls1046a.
> > Also add code to make the necessary device tree fixups.
> >
> > Signed-off-by: Laurentiu Tudor <laurentiu.tudor at nxp.com>
> Reviewed-by: Horia Geantă <horia.geanta at nxp.com>

Reviewed-by: Bharat Bhushan <bharat.bhushan at nxp.com>

Thanks
-Bharat

> 
> [snip]
> > +#define SET_SEC_RTIC_ICID_ENTRY(rtic_num, streamid) \
> > +	SET_ICID_ENTRY(NULL, streamid, (((streamid) << 16) | (streamid)), \
> > +		offsetof(ccsr_sec_t, rticliodnr[rtic_num].ls) + \
> > +		CONFIG_SYS_FSL_SEC_ADDR, 0)
> > +
> RTIC nodes have a binding doc in Linux kernel:
> Documentation/devicetree/bindings/crypto/fsl-sec4.txt
> 
> Thus, in theory, "fsl,sec-v4.0-rtic-memory" could be used instead of NULL for
> the compat string.
> PPC code in arch/powerpc/include/asm/fsl_liodn.h does this.
> 
> In practice, this won't make any difference until a RTIC kernel driver is added.
> 
> Regards,
> Horia
>