[Skiboot] [PATCH] core/opal: Fix address-cells in opal node in DT
jk at ozlabs.org
Tue Apr 16 00:11:21 AEST 2019
> > 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?
More information about the Skiboot