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

[Bharat Bhushan]

> -----Original Message-----
> From: Horia Geanta
> Sent: Monday, July 30, 2018 8:03 PM
> To: Laurentiu Tudor <laurentiu.tudor at nxp.com>; Bharat Bhushan
> <bharat.bhushan at nxp.com>; u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>; York Sun <york.sun at nxp.com>
> Cc: Alison Wang <alison.wang at nxp.com>; Ruchika Gupta
> <ruchika.gupta at nxp.com>
> Subject: Re: [PATCH v5 8/8] armv8: ls1046a: setup SEC ICIDs and fix up device
> tree
> 
> On 7/30/2018 4:58 PM, Laurentiu Tudor wrote:
> >> The only way to implement a *mechanism* is to provide different ICIDs
> >> for all the blocks.
> >> Any other solution would be imposing a *policy*, thus restricting
> >> user's possibilities. Admittedly there are use cases less "popular"
> >> than others, but if possible it would be best not to decide for the user and
> provide full flexibility.
> >>
> >> Is there a resource (ICID values) shortage?
> >>
> > No. I can change the patch to have distinct ICIDs and resubmit. Please
> > let me know if you agree on this.


Yes, please assign separate ICIDs for each.

Thanks
-Bharat

> >
> Yes, this is my suggestion.
> IIUC it's also similar to the way LIODNs are allocated on PPC.
> True, LS102x is the exception - it assigns ICID 0x10 to all SEC blocks.

> 
> Regards,
> Horia