[Skiboot] [PATCH v2] Add BMC platform to enable correct OEM IPMI commands

Stewart Smith stewart at linux.vnet.ibm.com
Tue Nov 8 17:31:05 AEDT 2016


Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
>> @@ -462,8 +468,19 @@ static void sel_pnor(uint8_t access)
>>   		if (granted)
>>   			occ_pnor_set_owner(PNOR_OWNER_EXTERNAL);
>>
>> +		if (bmc_platform->ipmi_oem_pnor_access_status == 0) {
>
> Here we endup setting flag and then erroring out. Better to add this check 
> before entering switch case itself. That way host can continue to
> access PNOR?

Oh, good catch! Patch incoming.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list