Fix coding conventions in a few headers
authorChris Ring <cring@ti.com>
Mon, 24 Jun 2013 22:14:03 +0000 (15:14 -0700)
committerChris Ring <cring@ti.com>
Mon, 24 Jun 2013 22:14:03 +0000 (15:14 -0700)
Correct some IPC header issues recently identified by static
analysis.

packages/ti/ipc/family/omap54xx/InterruptIpu.h
packages/ti/ipc/family/vayu/InterruptProxy.h
packages/ti/ipc/mm/MmType.h
packages/ti/srvmgr/omaprpc/OmapRpc.h

index cfd56de8c3790bdff53c7ceb2ef7414b2b3a1e08..2327ead185aab085421745776ac412df5cd3de75 100644 (file)
  */
 /*
  *  ======== InterruptIpu.h ========
- *  OMAP4430/Ducati Interrupt Manger
+ *  OMAP54xx IPU Interrupt Manger
  */
 
+#ifndef ti_ipc_family_omap54xx_InterruptIpu__include
+#define ti_ipc_family_omap54xx_InterruptIpu__include
+
+#if defined(__cplusplus)
+extern "C" {
+#endif
+
 #include <ti/sysbios/hal/Hwi.h>
 
 #define INVALIDPAYLOAD       (0xFFFFFFFF)
@@ -66,3 +73,10 @@ Void InterruptIpu_intSend(UInt16 remoteProcId,  UArg arg);
  *  Clear interrupt
  */
 UInt InterruptIpu_intClear();
+
+
+#if defined(__cplusplus)
+}
+#endif
+
+#endif /* ti_ipc_family_omap54xx_InterruptIpu__include */
index 21dff80431836dd7538beb965a9fd201f25c7451..e32bdbd185fd0b5fe782dd189d2d57c48532d527 100644 (file)
@@ -34,8 +34,8 @@
  *  Proxy Interrupt Manager
  */
 
-#ifndef ti_ipc_rpmsg_InterruptProxy__include
-#define ti_ipc_rpmsg_InterruptProxy__include
+#ifndef ti_ipc_family_vayu_InterruptProxy__include
+#define ti_ipc_family_vayu_InterruptProxy__include
 
 #if defined(__cplusplus)
 extern "C" {
@@ -88,4 +88,4 @@ extern "C" {
 }
 #endif /* defined (__cplusplus) */
 
-#endif /* ti_ipc_rpmsg_InterruptProxy__include */
+#endif /* ti_ipc_family_vayu_InterruptProxy__include */
index b60e866c797f17eda898f6d87f69ac581104cb82..5b9d583801e60921a60e87ca21c604fd73b7706b 100644 (file)
@@ -63,7 +63,7 @@ extern "C" {
  *              The given array must be the actual array buffer,
  *              not a pointer to it.
  */
-#define MmType_NumElem(x) (sizeof(x)/sizeof(x[0]))
+#define MmType_NumElem(x) (sizeof(x)/sizeof((x)[0]))
 
 /*!
  *  @brief      Parameter Direction
@@ -97,7 +97,7 @@ typedef enum {
     MmType_Param_MAX
 } MmType_ParamType;
 
-#define MmType_PtrType(type)    (type | MmType_Param_PTR)
+#define MmType_PtrType(type)    ((type) | MmType_Param_PTR)
 
 /*!
  *  @brief      Parameter Signature
index b5b241d56a1d078ed26a0a200c126c7c9e393ca6..9469fe3a7aef04db51e12a4fae8acf781f62b152 100644 (file)
@@ -61,16 +61,16 @@ extern "C" {
 #define OmapRpc_E_FAIL                 -1
 
 #define OmapRpc_NUM_PARAMETERS(size) \
-    (size/sizeof(struct OmapRpc_Parameter))
+    ((size)/sizeof(struct OmapRpc_Parameter))
 
 #define OmapRpc_PAYLOAD(ptr, type) \
     ((struct type *)&(ptr)[sizeof(struct OmapRpc_MsgHeader)])
 
-#define OmapRpc_PARAM(param, type) (param.size == sizeof(type) ? (type)param.data : 0)
+#define OmapRpc_PARAM(param, type) ((param).size == sizeof(type) ? (type)(param).data : 0)
 
 #define OmapRpc_Stringerize(func)   #func
 
-#define OmapRpc_dimof(x)    (sizeof(x)/sizeof(x[0]))
+#define OmapRpc_dimof(x)    (sizeof(x)/sizeof((x)[0]))
 
 #define OmapRpc_DESC_EXEC_SYNC  (0x0100)
 #define OmapRpc_DESC_EXEC_ASYNC (0x0200)
@@ -81,8 +81,8 @@ extern "C" {
 #define OmapRpc_JOBID_DISCRETE  (0)
 #define OmapRpc_POOLID_DEFAULT  (0x8000)
 
-#define OmapRpc_SET_FXN_IDX(idx)    (idx | 0x80000000)
-#define OmapRpc_FXN_MASK(idx)       (idx & 0x7FFFFFFF)
+#define OmapRpc_SET_FXN_IDX(idx)    ((idx) | 0x80000000)
+#define OmapRpc_FXN_MASK(idx)       ((idx) & 0x7FFFFFFF)
 
 #define OMAPRPC_MAX_CHANNEL_NAMELEN (64)
 #define OMAPRPC_MAX_FUNC_NAMELEN    (64)
@@ -192,7 +192,7 @@ typedef enum OmapRpc_Param_Type {
     OmapRpc_Param_MAX
 } OmapRpc_Param_Type;
 
-#define OmapRpc_PtrType(type)   (type | OmapRpc_Param_PTR)
+#define OmapRpc_PtrType(type)   ((type) | OmapRpc_Param_PTR)
 
 typedef struct OmapRpc_ParamSignature {
     UInt32 direction;   /**< @see OmapRpc_Direction */