diff --git a/src/occ_405/firdata/ast_mboxdd.h b/src/occ_405/firdata/ast_mboxdd.h index d48e50eb..a9a8f5ec 100644 --- a/src/occ_405/firdata/ast_mboxdd.h +++ b/src/occ_405/firdata/ast_mboxdd.h @@ -26,8 +26,6 @@ #ifndef __AST_MBOXDD_H #define __AST_MBOXDD_H -#include - /** @file ast_mboxdd.H * @brief Provides the interfaces Aspeed MBOX hardware */ @@ -120,7 +118,11 @@ errorHndl_t readRegSIO(uint8_t i_regAddr, */ inline uint8_t get8( mboxMessage_t *i_msg, uint8_t i_index) { - assert( i_index < BMC_MBOX_ARGS_REGS); + if ( i_index >= BMC_MBOX_ARGS_REGS ) + { + return 0; + } + return i_msg->iv_args[i_index]; } @@ -131,9 +133,13 @@ inline uint8_t get8( mboxMessage_t *i_msg, uint8_t i_index) * @param[in] i_index: Index into args section to be written to * @param[in] i_value: data to be written */ -inline void put8( mboxMessage_t *i_msg, uint8_t i_index, uint8_t i_value ) +inline void put8( mboxMessage_t *i_msg, uint8_t i_index, uint8_t i_value ) { - assert( i_index < BMC_MBOX_ARGS_REGS); + if ( i_index >= BMC_MBOX_ARGS_REGS ) + { + return; + } + i_msg->iv_args[i_index] = i_value; } @@ -146,7 +152,11 @@ inline void put8( mboxMessage_t *i_msg, uint8_t i_index, uint8_t i_value ) */ inline uint16_t get16( mboxMessage_t *i_msg, uint8_t i_index ) { - assert( i_index < (BMC_MBOX_ARGS_REGS - 1)); + if ( i_index >= (BMC_MBOX_ARGS_REGS-1) ) + { + return 0; + } + return i_msg->iv_args[i_index] | (i_msg->iv_args[i_index + 1] << 8); } @@ -159,7 +169,11 @@ inline uint16_t get16( mboxMessage_t *i_msg, uint8_t i_index ) */ inline void put16( mboxMessage_t *i_msg, uint8_t i_index, uint16_t i_value ) { - assert( i_index < (BMC_MBOX_ARGS_REGS - 1)); + if ( i_index >= (BMC_MBOX_ARGS_REGS-1) ) + { + return; + } + i_msg->iv_args[i_index] = i_value & 0xff; i_msg->iv_args[i_index + 1] = i_value >> 8; } @@ -174,7 +188,11 @@ inline void put16( mboxMessage_t *i_msg, uint8_t i_index, uint16_t i_value ) */ inline uint32_t get32( mboxMessage_t *i_msg, uint8_t i_index ) { - assert( i_index < (BMC_MBOX_ARGS_REGS - 3)); + if ( i_index >= (BMC_MBOX_ARGS_REGS-3) ) + { + return 0; + } + return i_msg->iv_args[i_index] | (i_msg->iv_args[i_index + 1] << 8) | (i_msg->iv_args[i_index + 2] << 16) | @@ -189,11 +207,15 @@ inline uint32_t get32( mboxMessage_t *i_msg, uint8_t i_index ) */ inline void put32( mboxMessage_t *i_msg, uint8_t i_index, uint32_t i_value ) { - assert( i_index < (BMC_MBOX_ARGS_REGS - 3)); - i_msg->iv_args[i_index] = i_value & 0xff; - i_msg->iv_args[i_index + 1] = (i_value >> 8) & 0xff; - i_msg->iv_args[i_index + 2] = (i_value >> 16) & 0xff; - i_msg->iv_args[i_index + 3 ] = i_value >> 24; + if ( i_index >= (BMC_MBOX_ARGS_REGS-3) ) + { + return; + } + + i_msg->iv_args[i_index] = i_value & 0xff; + i_msg->iv_args[i_index + 1] = (i_value >> 8) & 0xff; + i_msg->iv_args[i_index + 2] = (i_value >> 16) & 0xff; + i_msg->iv_args[i_index + 3 ] = i_value >> 24; } typedef struct { diff --git a/src/occ_405/firdata/pnor_mboxdd.c b/src/occ_405/firdata/pnor_mboxdd.c index a5624ff9..5253ccf1 100644 --- a/src/occ_405/firdata/pnor_mboxdd.c +++ b/src/occ_405/firdata/pnor_mboxdd.c @@ -36,7 +36,6 @@ #include #include #include -#include extern int TRACE_PNOR_MBOX; int TRACE_PNOR_MBOX = 0; @@ -106,7 +105,11 @@ errorHndl_t readFlash(pnorMbox_t* i_pnorMbox, do { // Ensure we are operating on a 4-byte boundary - assert( i_size % 4 == 0); + if (i_size % 4 != 0) + { + TRAC_ERR("readFlash: not on 4-byte boundary"); + return FAIL; + } TRAC_INFO("readFlash(i_addr=0x%.8X)> ", i_addr); @@ -161,7 +164,11 @@ errorHndl_t writeFlash(pnorMbox_t* i_pnorMbox, do { // Ensure we are operating on a 4-byte boundary - assert( i_size % 4 == 0 ); + if (i_size % 4 != 0) + { + TRAC_ERR("writeFlash: not on 4-byte boundary"); + return FAIL; + } TRAC_INFO(ENTER_MRK"writeFlash(i_address=0x%llx)> ", i_addr); diff --git a/src/occ_405/firdata/pnor_util.c b/src/occ_405/firdata/pnor_util.c index 17a63500..c7841cbd 100644 --- a/src/occ_405/firdata/pnor_util.c +++ b/src/occ_405/firdata/pnor_util.c @@ -27,7 +27,6 @@ #include #include -#include #include #include diff --git a/src/occ_405/occLinkInputFile b/src/occ_405/occLinkInputFile index dd4e75ef..34d93cea 100644 --- a/src/occ_405/occLinkInputFile +++ b/src/occ_405/occLinkInputFile @@ -98,7 +98,6 @@ INPUT ( amec_amester.o timer.o trac_interface.o wof.o - ecc.o firData.o fir_data_collect.o fsi.o diff --git a/src/occ_405/topfiles.mk b/src/occ_405/topfiles.mk index 445f7c3c..ecc773d3 100644 --- a/src/occ_405/topfiles.mk +++ b/src/occ_405/topfiles.mk @@ -65,7 +65,6 @@ TOP-C-SOURCES = amec/amec_analytics.c \ dimm/dimm_control.c \ errl/errl.c \ firdata/ast_mboxdd.c \ - firdata/ecc.c \ firdata/firData.c \ firdata/fir_data_collect.c \ firdata/fsi.c \