[Skiboot] [PATCH v2 4/5] external/opal-prd: Support manufacturing command HTMGT and attribute override
stewart at linux.vnet.ibm.com
Wed Aug 26 09:54:56 AEST 2015
Patrick Williams <patrick at stwcx.xyz> writes:
> The Hostboot version of this file says the following:
> // Reserve some space for future growth.
> // do NOT ever change this number, even if you add functions.
> // The value of 32 was somewhat arbitrarily chosen.
> // If either side modifies the interface.h file we're suppose to be able to
> // tolerate the other side not supporting the function yet. The function
> // pointer can be NULL. So if we require a new interface from OPAL,
> // like "read_iic", we need to be able to tolerate that function pointer
> // being NULL and do something sane (and erroring out is not consider sane).
> // The purpose of this is to give us the ability to update Hostboot and
> // OPAL independently. It is pretty rare that we both have function ready
> // at the same time. The "reserve" is there so that the structures are
> // allocated with sufficient space and populated with NULL function pointers.
> // 32 is big enough that we should not likely add that many functions from
> // either direction in between any two levels of support.
> Is there a reason why this file isn't just copied verbatim from
I can't see one myself, and it probably makes sense for it to be
I'm happy for a patch to make them identical to come after this series
More information about the Skiboot