[Skiboot] [PATCH 1/2] opal-prd: Add get_prd_flags to host interfaces
jk at ozlabs.org
Wed Sep 14 13:56:05 AEST 2016
> My intention was that the flags for OPAL and PHYP would be distinct.
> That would handle the specific case we're dealing with right now. I
> don't want to force a change into PHYP to set a bit to indicate they
> don't have a bug we found in OPAL/opal-prd. Adding features into the
> mix does make the idea of a comon set of flags more enticing though.
> I'll have to search around for the notes I sent out months ago related
> to a 'get_capabilities' function to see if there are more ideas we forgot.
> I still like the idea of multiple sets, so here's attempt #3.
> /* Common features */
> #define HBRT_CAPABILITIES_SET0_COMMON 0
> #define HBRT_CAPABILITIES_COMMON_AWESOME_FEATURE (1ul << 0)
> #define HBRT_CAPABILITIES_COMMON_NEW_INTERFACE_PARMS (2ul << 0)
> /* OPAL fixes */
> #define HBRT_CAPABILITIES_SET1_OPAL 1
> #define HBRT_CAPABILITIES_OPAL_HAS_XSCOM_RC (1ul << 0)
> /* PHYP fixes */
> #define HBRT_CAPABILITIES_SET2_PHYP 2
> #define HBRT_CAPABILITIES_PHYP_FIXED_BROKEN_THING (1ul << 0)
OK, that almost exactly lines up with the new revision that I'm working
>> Or should we make the sets based on functionality, rather than
> Not completely sure what you are suggesting here.
This was more grouping the sets along the lines of the functionality
that they affect, rather than the implementer of the interface (ie,
OPAL vs PHYP).
However - since HBRT knows whether it's running on OPAL vs. PHYP, it
does makes sense to have fixes grouped into OPAL / PHYP categories. As
long as we do use the COMMON set(s) when appropriate, and coordinate on
changes to that, we should be fine.
Are you okay with an unknown set returning 0?
More information about the Skiboot