diff options
author | Sam Nelson <sam.nelson@ti.com> | 2016-02-17 14:04:48 -0500 |
---|---|---|
committer | Sam Nelson <sam.nelson@ti.com> | 2016-02-18 13:47:16 -0500 |
commit | 6ea722baaed7b227aafdc4cc15d3712baec13d6d (patch) | |
tree | 2f065aa242b749f2c5e1c4555e9dc424ec6836c1 /packages | |
parent | d004c4e1ea6957e6a2a1e3592046fd988644a83a (diff) | |
download | ipc-6ea722baaed7b227aafdc4cc15d3712baec13d6d.tar.gz |
ipc: Remove warnings found with MISRA checkers
- Removed following warning "In the definition of a function-like
macro each instance of a parameter shall be enclosed in parentheses
unless it is used as the operand of # or ##
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
Diffstat (limited to 'packages')
8 files changed, 31 insertions, 31 deletions
diff --git a/packages/ti/ipc/family/tci6614/Interrupt.c b/packages/ti/ipc/family/tci6614/Interrupt.c index 600bddb..6bab6d9 100644 --- a/packages/ti/ipc/family/tci6614/Interrupt.c +++ b/packages/ti/ipc/family/tci6614/Interrupt.c @@ -56,17 +56,17 @@ Void Interrupt_isr(UArg arg); /* Shift to source bit id for CORES 0-3 */ #define MAP_TO_BITPOS(intId) \ - (intId == Interrupt_SRCS_BITPOS_CORE0 ? (intId + DNUM) : intId) + ((intId) == (Interrupt_SRCS_BITPOS_CORE0) ? ((intId) + (DNUM)) : (intId)) /* * Map remoteProcId to CORE ID [0-3] * NOTE: This assumes that HOST is at MultiProcId == 0, and CORE0 at 1i */ -#define MAP_RPROCID_TO_COREID(rProcId) (rProcId-1) +#define MAP_RPROCID_TO_COREID(rProcId) ((rProcId)-1) #define MAP_RPROCID_TO_SRCC(rProcId, intId) \ - (intId == Interrupt_SRCS_BITPOS_CORE0 ? \ - (intId + (rProcId-1)) : intId) + ((intId) == (Interrupt_SRCS_BITPOS_CORE0) ? \ + ((intId) + ((rProcId)-1)) : (intId)) /* ************************************************************************* * Module functions diff --git a/packages/ti/sdo/ipc/family/arctic/InterruptArp32.c b/packages/ti/sdo/ipc/family/arctic/InterruptArp32.c index 1fa3cd6..bc7ec29 100644 --- a/packages/ti/sdo/ipc/family/arctic/InterruptArp32.c +++ b/packages/ti/sdo/ipc/family/arctic/InterruptArp32.c @@ -63,11 +63,11 @@ #define MBX_INTR_TO_ARP32 0 #define MBX_INTR_TO_DSP 1 -#define MAILBOX_IRQSTATUS_CLR(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x104 + (INTR_NUM * 0x10)) +#define MAILBOX_IRQSTATUS_CLR(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x104 + ((INTR_NUM) * 0x10)) -#define MAILBOX_IRQENABLE_SET(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x108 + (INTR_NUM * 0x10)) +#define MAILBOX_IRQENABLE_SET(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x108 + ((INTR_NUM) * 0x10)) -#define MAILBOX_IRQENABLE_CLR(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x10C + (INTR_NUM * 0x10)) +#define MAILBOX_IRQENABLE_CLR(INTR_NUM) (InterruptArp32_mailboxBaseAddr + 0x10C + ((INTR_NUM) * 0x10)) #define ARP32INT 29 diff --git a/packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c b/packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c index f9c665c..57cf1bf 100644 --- a/packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c +++ b/packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c @@ -51,13 +51,13 @@ #define REG16(A) (*(volatile UInt16 *) (A)) #define REG32(A) (*(volatile UInt32 *) (A)) -#define PROCID(IDX) (InterruptArp32_procIdTable[IDX]) +#define PROCID(IDX) (InterruptArp32_procIdTable[(IDX)]) #define MBX_TABLE_IDX(SRC, DST) ((PROCID(SRC) * InterruptArp32_NUM_CORES) + \ PROCID(DST)) -#define SUBMBX_IDX(IDX) (InterruptArp32_mailboxTable[IDX] & 0xFF) -#define MBX_USER_IDX(IDX) ((InterruptArp32_mailboxTable[IDX] >> 8) \ +#define SUBMBX_IDX(IDX) (InterruptArp32_mailboxTable[(IDX)] & 0xFF) +#define MBX_USER_IDX(IDX) ((InterruptArp32_mailboxTable[(IDX)] >> 8) \ & 0xFF) -#define MBX_BASEADDR_IDX(IDX) ((InterruptArp32_mailboxTable[IDX] >> 16) \ +#define MBX_BASEADDR_IDX(IDX) ((InterruptArp32_mailboxTable[(IDX)] >> 16) \ & 0xFFFF) #define MAILBOX_REG_VAL(M) (0x1 << (2 * M)) diff --git a/packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c b/packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c index e4f0eea..1d7e32f 100644 --- a/packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c +++ b/packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c @@ -52,12 +52,12 @@ #define REG16(A) (*(volatile UInt16 *) (A)) #define REG32(A) (*(volatile UInt32 *) (A)) -#define PROCID(IDX) (InterruptDsp_procIdTable[IDX]) +#define PROCID(IDX) (InterruptDsp_procIdTable[(IDX)]) #define MBX_TABLE_IDX(SRC, DST) ((PROCID(SRC) * InterruptDsp_NUM_CORES) + \ PROCID(DST)) -#define SUBMBX_IDX(IDX) (InterruptDsp_mailboxTable[IDX] & 0xFF) -#define MBX_USER_IDX(IDX) ((InterruptDsp_mailboxTable[IDX] >> 8) & 0xFF) -#define MBX_BASEADDR_IDX(IDX) ((InterruptDsp_mailboxTable[IDX] >> 16) & 0xFFFF) +#define SUBMBX_IDX(IDX) (InterruptDsp_mailboxTable[(IDX)] & 0xFF) +#define MBX_USER_IDX(IDX) ((InterruptDsp_mailboxTable[(IDX)] >> 8) & 0xFF) +#define MBX_BASEADDR_IDX(IDX) ((InterruptDsp_mailboxTable[(IDX)] >> 16) & 0xFFFF) #define MAILBOX_REG_VAL(M) (0x1 << (2 * M)) diff --git a/packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c b/packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c index 3172b68..5e6d7e9 100644 --- a/packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c +++ b/packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c @@ -69,10 +69,10 @@ #define REG32(A) (*(volatile UInt32 *)(A)) #define MAILBOX_FIFOLENGTH 4 -#define PROCID(idx) (NotifyDriverMbx_procIdTable[idx]) +#define PROCID(idx) (NotifyDriverMbx_procIdTable[(idx)]) #define MBX_BASEADDR_IDX(idx) \ - ((NotifyDriverMbx_mailboxTable[idx] >> 16) & 0xFFFF) + ((NotifyDriverMbx_mailboxTable[(idx)] >> 16) & 0xFFFF) #define MAILBOX_ADDR(idx) \ (NotifyDriverMbx_mailboxBaseAddr[MBX_BASEADDR_IDX(idx)]) @@ -80,9 +80,9 @@ #define MBX_TABLE_IDX(src, dst) \ ((PROCID(src) * NotifyDriverMbx_NUM_CORES) + PROCID(dst)) -#define SUBMBX_IDX(idx) (NotifyDriverMbx_mailboxTable[idx] & 0xFF) +#define SUBMBX_IDX(idx) (NotifyDriverMbx_mailboxTable[(idx)] & 0xFF) -#define MBX_USER_IDX(idx) ((NotifyDriverMbx_mailboxTable[idx] >> 8) & 0xFF) +#define MBX_USER_IDX(idx) ((NotifyDriverMbx_mailboxTable[(idx)] >> 8) & 0xFF) #define MAILBOX_REG_VAL(m) (0x1 << (2 * m)) diff --git a/packages/ti/sdo/ipc/family/vayu/InterruptArp32.c b/packages/ti/sdo/ipc/family/vayu/InterruptArp32.c index 8b0fdbe..6b199ca 100644 --- a/packages/ti/sdo/ipc/family/vayu/InterruptArp32.c +++ b/packages/ti/sdo/ipc/family/vayu/InterruptArp32.c @@ -51,13 +51,13 @@ #define REG16(A) (*(volatile UInt16 *) (A)) #define REG32(A) (*(volatile UInt32 *) (A)) -#define PROCID(IDX) (InterruptArp32_procIdTable[IDX]) +#define PROCID(IDX) (InterruptArp32_procIdTable[(IDX)]) #define MBX_TABLE_IDX(SRC, DST) ((PROCID(SRC) * InterruptArp32_NUM_CORES) + \ PROCID(DST)) -#define SUBMBX_IDX(IDX) (InterruptArp32_mailboxTable[IDX] & 0xFF) -#define MBX_USER_IDX(IDX) ((InterruptArp32_mailboxTable[IDX] >> 8) \ +#define SUBMBX_IDX(IDX) (InterruptArp32_mailboxTable[(IDX)] & 0xFF) +#define MBX_USER_IDX(IDX) ((InterruptArp32_mailboxTable[(IDX)] >> 8) \ & 0xFF) -#define MBX_BASEADDR_IDX(IDX) ((InterruptArp32_mailboxTable[IDX] >> 16) \ +#define MBX_BASEADDR_IDX(IDX) ((InterruptArp32_mailboxTable[(IDX)] >> 16) \ & 0xFFFF) #define MAILBOX_REG_VAL(M) (0x1 << (2 * M)) diff --git a/packages/ti/sdo/ipc/family/vayu/InterruptDsp.c b/packages/ti/sdo/ipc/family/vayu/InterruptDsp.c index bd5a1bd..ce2b4d9 100644 --- a/packages/ti/sdo/ipc/family/vayu/InterruptDsp.c +++ b/packages/ti/sdo/ipc/family/vayu/InterruptDsp.c @@ -52,12 +52,12 @@ #define REG16(A) (*(volatile UInt16 *) (A)) #define REG32(A) (*(volatile UInt32 *) (A)) -#define PROCID(IDX) (InterruptDsp_procIdTable[IDX]) +#define PROCID(IDX) (InterruptDsp_procIdTable[(IDX)]) #define MBX_TABLE_IDX(SRC, DST) ((PROCID(SRC) * InterruptDsp_NUM_CORES) + \ PROCID(DST)) -#define SUBMBX_IDX(IDX) (InterruptDsp_mailboxTable[IDX] & 0xFF) -#define MBX_USER_IDX(IDX) ((InterruptDsp_mailboxTable[IDX] >> 8) & 0xFF) -#define MBX_BASEADDR_IDX(IDX) ((InterruptDsp_mailboxTable[IDX] >> 16) & 0xFFFF) +#define SUBMBX_IDX(IDX) (InterruptDsp_mailboxTable[(IDX)] & 0xFF) +#define MBX_USER_IDX(IDX) ((InterruptDsp_mailboxTable[(IDX)] >> 8) & 0xFF) +#define MBX_BASEADDR_IDX(IDX) ((InterruptDsp_mailboxTable[(IDX)] >> 16) & 0xFFFF) #define MAILBOX_REG_VAL(M) (0x1 << (2 * M)) diff --git a/packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c b/packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c index bdedc24..37bafa9 100644 --- a/packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c +++ b/packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c @@ -73,10 +73,10 @@ #define REG32(A) (*(volatile UInt32 *)(A)) #define MAILBOX_FIFOLENGTH 4 -#define PROCID(idx) (NotifyDriverMbx_procIdTable[idx]) +#define PROCID(idx) (NotifyDriverMbx_procIdTable[(idx)]) #define MBX_BASEADDR_IDX(idx) \ - ((NotifyDriverMbx_mailboxTable[idx] >> 16) & 0xFFFF) + ((NotifyDriverMbx_mailboxTable[(idx)] >> 16) & 0xFFFF) #define MAILBOX_ADDR(idx) \ (NotifyDriverMbx_mailboxBaseAddr[MBX_BASEADDR_IDX(idx)]) @@ -84,9 +84,9 @@ #define MBX_TABLE_IDX(src, dst) \ ((PROCID(src) * NotifyDriverMbx_NUM_CORES) + PROCID(dst)) -#define SUBMBX_IDX(idx) (NotifyDriverMbx_mailboxTable[idx] & 0xFF) +#define SUBMBX_IDX(idx) (NotifyDriverMbx_mailboxTable[(idx)] & 0xFF) -#define MBX_USER_IDX(idx) ((NotifyDriverMbx_mailboxTable[idx] >> 8) & 0xFF) +#define MBX_USER_IDX(idx) ((NotifyDriverMbx_mailboxTable[(idx)] >> 8) & 0xFF) #define MAILBOX_REG_VAL(m) (0x1 << (2 * m)) |