ipc: Remove warnings found with MISRA checkers
authorSam Nelson <sam.nelson@ti.com>
Wed, 17 Feb 2016 19:04:48 +0000 (14:04 -0500)
committerSam Nelson <sam.nelson@ti.com>
Thu, 18 Feb 2016 18:47:16 +0000 (13:47 -0500)
- 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>
packages/ti/ipc/family/tci6614/Interrupt.c
packages/ti/sdo/ipc/family/arctic/InterruptArp32.c
packages/ti/sdo/ipc/family/tda3xx/InterruptArp32.c
packages/ti/sdo/ipc/family/tda3xx/InterruptDsp.c
packages/ti/sdo/ipc/family/tda3xx/NotifyDriverMbx.c
packages/ti/sdo/ipc/family/vayu/InterruptArp32.c
packages/ti/sdo/ipc/family/vayu/InterruptDsp.c
packages/ti/sdo/ipc/family/vayu/NotifyDriverMbx.c

index 600bddb19fd8b43a8de460d33e05f1b2c222676f..6bab6d94ba56c3ce874ec4140e5a5a05c56dc9d7 100644 (file)
@@ -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
index 1fa3cd62dcc1d438e578c2f00dc037df17a6d353..bc7ec29560457215d900fa3882d3c38f7579585e 100644 (file)
 #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
 
index f9c665c7c37e9de160c102e3c5cd9d4ac0c98e3e..57cf1bf321f422a63d7152912840161b1968ab22 100644 (file)
 #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))
index e4f0eea6d64096f23d806c65f2517739cfaebc02..1d7e32f1dd48cb27eb370d7fc4558fb24c027b7e 100644 (file)
 #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))
 
index 3172b689047927ca00d251e298defff4e80bce19..5e6d7e904ff1cf05d9820c487803195ad6324061 100644 (file)
 #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))
 
index 8b0fdbe2c9e1c6afe3cb8a5f0ebc69875ab2e8cd..6b199ca90cae04f84d72514e1806960d5d883f4c 100644 (file)
 #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))
index bd5a1bd4a00d5c553fae42e92b168295b83b005f..ce2b4d9c7460730acd68da9fbe0c1817fdf3fe69 100644 (file)
 #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))
 
index bdedc24a32cee162edcfafb8b6b6702053e9c7e6..37bafa93a38f1ea73c74318ca1077d8bb476749a 100644 (file)
 #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))