[Skiboot] [PATCH] core/opal: Fix address-cells in opal node in DT

[Vasant Hegde]

On 04/15/2019 07:41 PM, Jeremy Kerr wrote:
> Hi all,
> 
>>> This patch fix this FWTS warning:
>>>
>>> Warning (reg_format): "reg" property in /ibm,opal/flash at 0 has
>>> invalid length (8 bytes) (#address-cells == 0, #size-cells == 0)
> 
> This is a bit of a tricky one. The ibm,opal node doesn't reference a
> "bus" of any kind, so the #address-cells and #size-cells being zero is
> correct. What we should have done for the flash node is used some
> property (other than 'reg', which has a specific meaning) for the size
> of the flash device.
> 
> However, even if we patches that in OPAL and Linux now, renaming the
> property would break compatibility with existing OSes.
> 
>> But if we add "#size-cells" = 1 then its like telling you have `size`
>> without address.. which is also wrong.
> 
> That's not too much of an issue; the implication that this introduces is
> that any 'reg' node under /ibm,opal is a one-cell size value - which may
> not be too bad a thing.
> 
>> May be we can just add this patch to keep dtc happy.
>>
>> @jk, any thoughts?
> 
> It's not great, but it's probably not too awful either. Silencing the
> warning might be a reasonable justification here, and it could be argued
> that nodes under ibm,opal could reasonably describe sizes of things?

Agree. Lets add size property as "1" and silence warning.

@Maxim, Can you respin the patch? And fat comment explaining the rational behind 
this fix.


-Vasant