[Skiboot] [PATCH 1/3] errorlog : Fixes duplicate use of SRC component ID's

Stewart Smith stewart at linux.vnet.ibm.com
Thu Jul 14 18:05:40 AEST 2016


Mukesh Ojha <mukesh02 at linux.vnet.ibm.com> writes:
> Same SRC component id was used for two components. This patch fixes it and allocate
> them unused component id's.
>
> Signed-off-by: Mukesh Ojha <mukesh02 at linux.vnet.ibm.com>

So, I had a look at the other defines around here, put them in order and
noticed another duplicate:

#define OPAL_CU                         0x1000
#define OPAL_VP                         0x1000
and
#define OPAL_SE                         0x2000
#define OPAL_CN                         0x2000

So, what I've done is re-ordered the table into numerical order.

Then, finding that OPAL_VP was unused, I just changed it to a free
number 0x1a00, same with OPAL_CN (now 0x1b00).

I don't particularly like the two letter codes there though.. they're
not obvious as to what they are. A patch to change them to be something
like OPAL_SRC_COMPONENT_XSCOM would be welcome.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list