summaryrefslogtreecommitdiff
path: root/packages
diff options
context:
space:
mode:
authorSam Nelson <sam.nelson@ti.com>2016-02-17 14:04:48 -0500
committerSam Nelson <sam.nelson@ti.com>2016-02-18 13:47:16 -0500
commit6ea722baaed7b227aafdc4cc15d3712baec13d6d (patch)
tree2f065aa242b749f2c5e1c4555e9dc424ec6836c1 /packages
parentd004c4e1ea6957e6a2a1e3592046fd988644a83a (diff)
downloadipc-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')
-rw-r--r--packages/ti/ipc/family/tci6614/Interrupt.c8
-rw-r--r--packages/ti/sdo/ipc/family/arctic/InterruptArp32.c6
-rw-r--r--packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c8
-rw-r--r--packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c8
-rw-r--r--packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c8
-rw-r--r--packages/ti/sdo/ipc/family/vayu/InterruptArp32.c8
-rw-r--r--packages/ti/sdo/ipc/family/vayu/InterruptDsp.c8
-rw-r--r--packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c8
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))