diff options
Diffstat (limited to 'src/usr/pnor')
-rw-r--r-- | src/usr/pnor/ast_mboxdd.C | 5 | ||||
-rw-r--r-- | src/usr/pnor/sfc_ast2400.C | 7 | ||||
-rw-r--r-- | src/usr/pnor/sfc_ast2500.C | 5 |
3 files changed, 16 insertions, 1 deletions
diff --git a/src/usr/pnor/ast_mboxdd.C b/src/usr/pnor/ast_mboxdd.C index b7504e934..5b6e58300 100644 --- a/src/usr/pnor/ast_mboxdd.C +++ b/src/usr/pnor/ast_mboxdd.C @@ -365,6 +365,11 @@ errlHndl_t astMbox::initializeMbox(void) do { + // The BMC may have disabled SIO access, but there's not much point in + // testing whether it's available as there's no alternative action if + // it is not. Instead, just try the SIO accesses and bail out with the + // errl if they fail. + //First disable SIO Mailbox engine to configure it // 0x30 - Enable/Disable Reg l_data = SIO::DISABLE_DEVICE; diff --git a/src/usr/pnor/sfc_ast2400.C b/src/usr/pnor/sfc_ast2400.C index e8e612efa..aa0e2bb07 100644 --- a/src/usr/pnor/sfc_ast2400.C +++ b/src/usr/pnor/sfc_ast2400.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2014,2016 */ +/* Contributors Listed Below - COPYRIGHT 2014,2018 */ /* [+] Google Inc. */ /* [+] International Business Machines Corp. */ /* */ @@ -95,6 +95,11 @@ errlHndl_t SfcAST2400::hwInit( ) uint32_t l_lpc_addr; do { + // The BMC may have disabled SIO access, but there's not much point in + // testing whether it's available as there's no alternative action if + // it is not. Instead, just try the SIO accesses and bail out with the + // errl if they fail. + /* Enable device 0x0D*/ uint8_t l_data = SIO::ENABLE_DEVICE; size_t l_len = sizeof(l_data); diff --git a/src/usr/pnor/sfc_ast2500.C b/src/usr/pnor/sfc_ast2500.C index 39465f9b2..9f4d88ae1 100644 --- a/src/usr/pnor/sfc_ast2500.C +++ b/src/usr/pnor/sfc_ast2500.C @@ -94,6 +94,11 @@ errlHndl_t SfcAST2500::hwInit( ) uint32_t l_lpc_addr; do { + // The BMC may have disabled SIO access, but there's not much point in + // testing whether it's available as there's no alternative action if + // it is not. Instead, just try the SIO accesses and bail out with the + // errl if they fail. + /* Enable device 0x0D*/ uint8_t l_data = SIO::ENABLE_DEVICE; size_t l_len = sizeof(l_data); |