Android: Fix compilation warnings
authorAngela Stegmaier <angelabaker@ti.com>
Wed, 1 Nov 2017 16:52:54 +0000 (11:52 -0500)
committerAngela Stegmaier <angelabaker@ti.com>
Mon, 20 Nov 2017 17:50:32 +0000 (11:50 -0600)
Fix compilation warnings seen with Android build.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
17 files changed:
hlos_common/src/utils/MultiProc.c
linux/include/GateHWSpinlock.h
linux/src/api/Ipc.c
linux/src/api/MessageQ.c
linux/src/api/gates/GateHWSpinlock.c
linux/src/api/gates/GateMP.c
linux/src/api/gates/GateMutex.c
linux/src/daemon/GateHWSpinlock.c
linux/src/daemon/GateMP_daemon.c
linux/src/daemon/MessageQ_daemon.c
linux/src/daemon/NameServer_daemon.c
linux/src/heaps/HeapStd.c
linux/src/tests/NameServerApp.c
linux/src/tests/ping_rpmsg.c
linux/src/transport/TransportRpmsg.c
linux/src/utils/LAD_client.c
packages/ti/ipc/mm/MmRpc.c

index d7f505f0b6f172b19ace6c794cbe7963ba2a628e..83638ae7911335d18d7f49fad5fef7d3dbd295ac 100644 (file)
@@ -119,8 +119,7 @@ Int MultiProc_setLocalId(UInt16 id)
      *     To call setLocalId, the id must have been set to invalid.
      *  2. Make sure the call is made before module startup
      */
-    if ((_MultiProc_cfg.id == MultiProc_INVALIDID) /* &&
-        (Startup_rtsDone() == FALSE) */  )  {
+    if (_MultiProc_cfg.id == MultiProc_INVALIDID)  {
         /* It is ok to set the id */
         _MultiProc_cfg.id = id;
         return (MultiProc_S_SUCCESS);
index b809227ebebc95c8d7eca1744b8632f3cbfacfc6..09b3a752aab0bfba94c0fa725586322751ce557a 100644 (file)
@@ -218,8 +218,8 @@ Int GateHWSpinlock_stop(Void);
 
 /* Function to create an instance of GateHWSpinlock */
 GateHWSpinlock_Handle
-GateHWSpinlock_create (      GateHWSpinlock_LocalProtect localProtect,
-                       const GateHWSpinlock_Params *     params);
+GateHWSpinlock_create (      GateMP_LocalProtect     localProtect,
+                       const GateHWSpinlock_Params * params);
 
 /* Function to delete an instance of GateHWSpinlock */
 Int
index d963da71f38ea60e0469bfa71c28ce9e9c7d2bd0..4f87d3a924c1cbd25c43caec82f3e9ffc6835d2d 100644 (file)
@@ -410,6 +410,8 @@ exit:
 
 static void cleanup(int arg)
 {
+    (Void)arg;
+
     printf("Ipc: Caught SIGINT, calling Ipc_stop...\n");
     Ipc_stop();
     exit(0);
index f114810eafc71bb3b53a7db2f0404cae084b0f7b..242cf2cf32629f16be0dcfd17744e03d3f138f5e 100644 (file)
@@ -1015,7 +1015,7 @@ Int MessageQ_get(MessageQ_Handle handle, MessageQ_Msg *msg, UInt timeout)
     }
 #endif
 
-    if (timeout == MessageQ_FOREVER) {
+    if (timeout == (UInt)MessageQ_FOREVER) {
         sem_wait(&obj->synchronizer);
     }
     else {
@@ -1067,7 +1067,10 @@ Int MessageQ_get(MessageQ_Handle handle, MessageQ_Msg *msg, UInt timeout)
 Int MessageQ_count(MessageQ_Handle handle)
 {
     Int               count = -1;
-#if 0
+#if 1
+    (Void)handle;
+
+#else
     MessageQ_Object * obj   = (MessageQ_Object *) handle;
     socklen_t         optlen;
 
@@ -1279,6 +1282,7 @@ Void MessageQ_setMsgTrace(MessageQ_Msg msg, Bool traceFlag)
 SizeT MessageQ_sharedMemReq(Ptr sharedAddr)
 {
     SizeT memReq = 0u;
+    (Void)sharedAddr;
 
     /* Do nothing, as this is a copy transport. */
 
index 5e4538c9a5ca799cfc9b3264be4c6d7ff20e306a..c818e36310e9c3f85a100b45818e3048c54aef9f 100644 (file)
@@ -276,11 +276,12 @@ Void GateHWSpinlock_Params_init(GateHWSpinlock_Params *params)
  * Create a GateHWSpinlock instance
  */
 /* TODO: change the function to accept a local gate. Do this on all platforms */
-GateHWSpinlock_Handle GateHWSpinlock_create(GateHWSpinlock_LocalProtect
+GateHWSpinlock_Handle GateHWSpinlock_create(GateMP_LocalProtect
     localProtect, const GateHWSpinlock_Params * params)
 {
     GateHWSpinlock_Object * obj = (GateHWSpinlock_Object *)calloc(1,
         sizeof (GateHWSpinlock_Object));
+    (Void)localProtect;
 
     if (!obj) {
         PRINTVERBOSE0("GateHWSpinlock_create: memory allocation failure")
index 4aaadbd5109e9cec171b4a84bd8ac24d666b9694..47c1377278e8c716bfbb6470f9efa4bb4ffb0240 100644 (file)
@@ -416,7 +416,7 @@ Int GateMP_open(String name, GateMP_Handle *handle)
                     another processor")
                 status = GateMP_E_FAIL;
             }
-            else if (nsValue[0] != getpid()) {
+            else if (nsValue[0] != (UInt32)getpid()) {
                 /* error: trying to open another process's private gate */
                 *handle = NULL;
                 PRINTVERBOSE0("GateMP_open: cannot open private gate from \
@@ -808,7 +808,7 @@ static Int GateMP_Instance_init(GateMP_Object *obj,
             else {
                 /* created instance */
                 obj->resourceId = GateMP_getFreeResource(obj->remoteProtect);
-                if (obj->resourceId == -1) {
+                if (obj->resourceId == (UInt)-1) {
                     return (GateMP_E_RESOURCE);
                 }
             }
@@ -872,6 +872,7 @@ static Int GateMP_Instance_init(GateMP_Object *obj,
 static Void GateMP_Instance_finalize(GateMP_Object *obj, Int status)
 {
     GateMP_Object ** remoteGates = NULL;
+    (Void)status;
 
     /* remove from NameServer */
     if (obj->nsKey != 0) {
index bb5f21ccc9364288f0d5c7b30e8b05965cf060d8..4b2c80453a3386a58b8709aa7ba12415e5e3269c 100644 (file)
@@ -84,6 +84,8 @@ GateMutex_Handle GateMutex_create(const GateMutex_Params * params,
     Error_Block *eb)
 {
     pthread_mutexattr_t attr;
+    (Void)params;
+    (Void)eb;
 
     GateMutex_Object * obj = (GateMutex_Object *)calloc(1,
         sizeof(GateMutex_Object));
@@ -134,6 +136,7 @@ Void GateMutex_leave (GateMutex_Handle gmHandle, IArg key)
 {
     GateMutex_Object * obj = (GateMutex_Object *)gmHandle;
     int ret;
+    (Void)key;
 
     ret = pthread_mutex_unlock(&(obj->mutex));
     assert(ret == 0);
index adae4fca0451098566e9a48d0a8e6d99fede11b0..47822217d91e9438e4134d834fc428c6511c7360 100644 (file)
@@ -213,11 +213,12 @@ Void GateHWSpinlock_Params_init(GateHWSpinlock_Params *params)
  * Create a GateHWSpinlock instance
  */
 /* TODO: change the function to accept a local gate. Do this on all platforms */
-GateHWSpinlock_Handle GateHWSpinlock_create(GateHWSpinlock_LocalProtect
+GateHWSpinlock_Handle GateHWSpinlock_create(GateMP_LocalProtect
     localProtect, const GateHWSpinlock_Params * params)
 {
     GateHWSpinlock_Object * obj = (GateHWSpinlock_Object *)calloc(1,
         sizeof (GateHWSpinlock_Object));
+    (Void)localProtect;
 
     if (!obj) {
         LOG0("GateHWSpinlock_create: memory allocation failure")
index 56ab5802f7974909298917ebe5402cbc9482c3ce..6ad761156e8026c847a0b20e5a56ea1da336b0af 100644 (file)
@@ -674,7 +674,7 @@ Int GateMP_releaseResource(UInt id, GateMP_RemoteProtect type)
             break;
     }
 
-    if ((inUse != NULL) && (id < num)) {
+    if ((inUse != NULL) && (id < (UInt)num)) {
         assert(GateMP_module->defaultGate != NULL);
         key = GateMP_enter(GateMP_module->defaultGate);
         inUse[id] = UNUSED;
index 4ad26a176e322cef16dc36d2ba1eebdc21d28c3f..84479e6b16464953374c3bcba60bc9c740b34f2a 100644 (file)
@@ -185,6 +185,7 @@ Int MessageQ_setup(const MessageQ_Config * cfg)
 {
     Int                    status = MessageQ_S_SUCCESS;
     NameServer_Params      params;
+    (Void)cfg;
 
     pthread_mutex_lock(&(MessageQ_module->gate));
 
index f7982c799642ac78a0b51ccaa5c7ca9686ef719e..caf549c104b37090adfa57e2ec89cfe8774e4af6 100644 (file)
@@ -415,7 +415,7 @@ static void *listener_cb(void *arg)
     int i;
     UInt16 procId;
     struct  sockaddr_rpmsg  fromAddr;
-    unsigned int len;
+    socklen_t len;
     NameServerRemote_Msg msg;
     int nbytes;
     UInt16 numProcs = MultiProc_getNumProcsInCluster();
@@ -424,6 +424,7 @@ static void *listener_cb(void *arg)
     uint64_t event;
     Bool run = TRUE;
     Bool reconnect = FALSE;
+    (Void)arg;
 
     LOG0("listener_cb: Entered Listener thread.\n")
 
@@ -1048,7 +1049,7 @@ Int NameServer_getRemote(NameServer_Handle handle,
     struct timeval tv;
     uint64_t buf = 1;
     int err;
-    int i;
+    UInt i;
     static int seqNum = 0;
     Bool done = FALSE;
     UInt16 clusterId;
@@ -1129,7 +1130,7 @@ Int NameServer_getRemote(NameServer_Handle handle,
             /* Process response: */
             replyMsg = &NameServer_module->nsMsg;
 
-            if (replyMsg->seqNum != seqNum - 1) {
+            if ((int)(replyMsg->seqNum) != seqNum - 1) {
                 /* Ignore responses without current sequence # */
                 continue;
             }
index f42606215b7aa3ac912cef1c4a704bcbe7ce878f..74ddf2e4db1681392657bc931ee90b679f5200c9 100644 (file)
@@ -103,6 +103,7 @@ HeapStd_Handle HeapStd_downCast(IHeap_Handle base)
 void *HeapStd_alloc(void *handle, size_t size)
 {
     void *block;
+    (void)handle;
 
     block = malloc(size);
     return (block);
@@ -113,5 +114,7 @@ void *HeapStd_alloc(void *handle, size_t size)
  */
 void HeapStd_free(void *handle, void *block)
 {
+    (void)handle;
+
     free(block);
 }
index 7a12633d0cddedf33f85dc5eab2779aa608caffc..571ee790fe0f7691359a44921e772261ce2228b5 100644 (file)
@@ -267,7 +267,7 @@ Int testNS(NameServer_Handle nsHandle, String name)
         status = NameServer_getUInt32(nsHandle, key, &val, NULL);
         printf("NameServer_getUInt32(%s) returned %d, val=0x%x (was 0x00c0ffee)\n", key, status, val);
 
-        if (val != (0x0badc0de + i)) {
+        if (val != (UInt32)(0x0badc0de + i)) {
             printf("get val (0x%x) != add val (0x%x)!\n", val, 0x0badc0de + i);
         }
     }
@@ -451,6 +451,9 @@ NameServerApp_shutdown()
 int
 main (int argc, char ** argv)
 {
+    (Void)argc;
+    (Void)argv;
+
     NameServerApp_startup();
     NameServerApp_execute();
     NameServerApp_shutdown();
index 80cdd2db69357bb7355339310630ef9bf484ea57..36edcb5bbc23a5875c6a8575a615f2ff3d5078da 100644 (file)
@@ -78,7 +78,7 @@ int main (int argc, char ** argv)
     char buf[512];
     struct timespec   start,end;
     long              elapsed=0,delta;
-    int i;
+    unsigned int i;
     fd_set rfds;
 
     /* Parse Args: */
index 2a6abe32a3ec2f588b32d8f265248f7ab1f747aa..98482da0011132e28c55b69114112ce4d3d9a1b6 100644 (file)
@@ -459,6 +459,7 @@ Bool TransportRpmsg_put(Void *handle, Ptr pmsg)
     int     sock;
     int     err;
     UInt16  clusterId;
+    (Void)handle;
 
     /*
      * Retrieve the socket for the AF_SYSLINK protocol associated with this
@@ -496,6 +497,9 @@ exit:
  */
 Bool TransportRpmsg_control(Void *handle, UInt cmd, UArg cmdArg)
 {
+    (Void)handle;
+    (Void)cmd;
+    (Void)cmdArg;
     return FALSE;
 }
 
@@ -511,14 +515,14 @@ void *rpmsgThreadFxn(void *arg)
     fd_set   rfds;
     int      maxFd;
     int      nfds;
-    MessageQ_Msg     retMsg;
+    MessageQ_Msg     retMsg = NULL;
     MessageQ_QueueId queueId;
     MessageQ_Handle handle;
     Bool run = TRUE;
     int i;
     int j;
     int fd;
-
+    (Void)arg;
 
     while (run) {
         maxFd = TransportRpmsg_module->maxFd;
@@ -653,7 +657,7 @@ static Int transportGet(int sock, MessageQ_Msg *retMsg)
     Int           status    = MessageQ_S_SUCCESS;
     MessageQ_Msg  msg;
     struct sockaddr_rpmsg fromAddr;  /* [Socket address of sender] */
-    unsigned int  len;
+    socklen_t     len;
     int           byteCount;
 
     /*
@@ -729,12 +733,12 @@ Void bindFdToQueueIndex(TransportRpmsg_Object *obj, Int fd, UInt16 queuePort)
     UInt oldSize;
     UInt newCount;
     UInt queueIndex;
-    int i;
+    UInt i;
 
     /* subtract port offset from queue index */
     queueIndex = queuePort - MessageQ_PORTOFFSET;
 
-    if (queueIndex >= obj->numQueues) {
+    if (queueIndex >= (UInt)obj->numQueues) {
         newCount = queueIndex + TransportRpmsg_GROWSIZE;
         PRINTVERBOSE1("TransportRpmsg_bind: growing numQueues to %d\n",
                 newCount);
index 9c858c4b047d89c974f8fe32ec181e6a15775cd6..7a3987a6d8bc380ba9e989007ca0fb6c59d0b672 100644 (file)
@@ -98,7 +98,7 @@ LAD_ClientHandle LAD_findHandle(Void)
     pid = getpid();
 
     for (i = 0; i < LAD_MAXNUMCLIENTS; i++) {
-        if (clientInfo[i].PID == pid &&
+        if (clientInfo[i].PID == (UInt)pid &&
             clientInfo[i].connectedToLAD == TRUE) {
             break;
         }
index 15d0391fb8e1ed5356c72ef2ab54153c3f51e36c..2df7420cc4acfe7e1dc48d86193ae90075a40fa5 100644 (file)
@@ -115,6 +115,7 @@ int MmRpc_create(const char *service, const MmRpc_Params *params,
     int             status = MmRpc_S_SUCCESS;
     MmRpc_Object *  obj = NULL;
     char            cbuf[RPPC_MAX_INST_NAMELEN+16];
+    (void)params;
 
     if (service == NULL || handlePtr == NULL) {
         status = MmRpc_E_INVALIDPARAM;
@@ -208,7 +209,7 @@ int MmRpc_call(MmRpc_Handle handle, MmRpc_FxnCtx *ctx, int32_t *ret)
     MmRpc_Param *param;
     void *msg = NULL;
     int len;
-    int i;
+    unsigned int i;
 
     if (handle == NULL || ctx == NULL || ret == NULL) {
         status = MmRpc_E_INVALIDPARAM;