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

Jeremy Kerr jk at ozlabs.org
Tue Apr 16 00:11:21 AEST 2019


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?

Cheers,


Jeremy



More information about the Skiboot mailing list