We are not updating the BMC status in the switch()( *default case* ), which may 
lead to installing the IPMI protocol for the failure case too. Could you please 
initialize the BMC status in the default case also?

switch ( SelfTestResult->Result ) {

case IPMI_APP_SELFTEST_NO_ERROR:

case IPMI_APP_SELFTEST_NOT_IMPLEMENTED:

.......
.....
...

case IPMI_APP_SELFTEST_ERROR:

.......
.....
...

case IPMI_APP_SELFTEST_FATAL_HW_ERROR:

.......
.....
...

*default:*

IpmiInstance->BmcStatus = BMC_HARDFAIL;

//

// Call routine to check device specific failures.

//

GetDeviceSpecificTestResults (IpmiInstance);

}

Thanks,
Arun


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#100945): https://edk2.groups.io/g/devel/message/100945
Mute This Topic: https://groups.io/mt/97485243/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to