Replaced old SSI function with new macros jira id - ODUHIGH-212
[o-du/l2.git] / src / mt / ss_task.c
index 73f9037..5255445 100644 (file)
@@ -191,14 +191,14 @@ PAIFS16 initFnct;           /* initialization function */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS350, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check initialization function */
    if (initFnct == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS351, ERRZERO, "Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -217,7 +217,7 @@ PAIFS16 initFnct;           /* initialization function */
    {
       SSLOGERROR(ERRCLS_DEBUG, ESS352, ERRZERO,
                      "Could not lock system task table");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    SS_ACQUIRE_ALL_SEMA(&osCp.tTskTblSem, ret);
@@ -229,7 +229,7 @@ PAIFS16 initFnct;           /* initialization function */
                  "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -246,11 +246,11 @@ PAIFS16 initFnct;           /* initialization function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS355, ERRZERO,
                       "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(ROUTRES);
+      return (ROUTRES);
    }
 
    /* check if entity and instance already registered */
@@ -266,11 +266,11 @@ PAIFS16 initFnct;           /* initialization function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS357, ERRZERO,
                          "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -312,11 +312,11 @@ PAIFS16 initFnct;           /* initialization function */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS358, ERRZERO,
                       "Could not give the Semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
    }
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SRegInit */
 
 
@@ -376,14 +376,14 @@ ActvTsk actvTsk;               /* activate task function */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS359, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check activation function */
    if (actvTsk == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS360, ERRZERO, "Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check task type */
@@ -391,14 +391,14 @@ ActvTsk actvTsk;               /* activate task function */
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS361, ERRZERO, "Invalid task type");
       /* ss021.103 - Addition of return fail for invalid task type */
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check task priority */
    if (prior > PRIOR3)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS362, ERRZERO, "Invalid task priority");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -408,7 +408,7 @@ ActvTsk actvTsk;               /* activate task function */
    {
       SSLOGERROR(ERRCLS_DEBUG, ESS363, ERRZERO,
                      "Could not lock system task table");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* lock TAPA task table */
@@ -420,13 +420,13 @@ ActvTsk actvTsk;               /* activate task function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS364, ERRZERO,
                          "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
  
       SSLOGERROR(ERRCLS_DEBUG, ESS365, ERRZERO,
                      "Could not lock TAPA task table");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
    /* ss021.103 - Modification for SRegInit not called yet */
    /* check if entity and instance already registered */
@@ -445,11 +445,11 @@ ActvTsk actvTsk;               /* activate task function */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS366, ERRZERO,
                          "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
          }
          SSLOGERROR(ERRCLS_INT_PAR, ESS367, ERRZERO, "Too many tasks");
-         RETVALUE(ROUTRES);
+         return (ROUTRES);
       }
 #endif
 
@@ -491,13 +491,13 @@ ActvTsk actvTsk;               /* activate task function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS368, ERRZERO,
                     "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS369, ERRZERO, 
                  "ActvTask already registered");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -513,7 +513,7 @@ ActvTsk actvTsk;               /* activate task function */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS370, ERRZERO,
                       "Could not give the Semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
    }
 
@@ -535,7 +535,7 @@ ActvTsk actvTsk;               /* activate task function */
 #endif
 
       SDeregTTsk(ent, inst);
-      RETVALUE(ret);
+      return (ret);
    }
 
 
@@ -555,7 +555,7 @@ ActvTsk actvTsk;               /* activate task function */
    }
 
 
-   RETVALUE(ret);
+   return (ret);
 
 } /* SRegActvTsk */
  
@@ -602,7 +602,7 @@ Inst inst;                     /* instance */
    if (ent >= SS_MAX_ENT  ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS373, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -613,7 +613,7 @@ Inst inst;                     /* instance */
    {
       SSLOGERROR(ERRCLS_DEBUG, ESS374, ERRZERO,
                      "Could not lock system task table");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* lock the TAPA task table */
@@ -626,7 +626,7 @@ Inst inst;                     /* instance */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -643,11 +643,11 @@ Inst inst;                     /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS377, ERRZERO,
                          "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -675,7 +675,7 @@ Inst inst;                     /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS378, ERRZERO,
                       "Could not give the Semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
    }
 
@@ -683,7 +683,7 @@ Inst inst;                     /* instance */
    /* deregister the TAPA task */
    if (SDeregTTsk(ent, inst) != ROK)
    {
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -704,7 +704,7 @@ Inst inst;                     /* instance */
                         "Could not lock system task table");
 #endif
 
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
 
       idx = (SsIdx) sTskId;
@@ -720,11 +720,11 @@ Inst inst;                     /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS380, ERRZERO,
                          "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
          }
 
-         RETVALUE(ROK);
+         return ROK;
       }
 
       /* check if the system task is dying already */
@@ -737,10 +737,10 @@ Inst inst;                     /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS381, ERRZERO,
                          "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
          }
-         RETVALUE(ROK);
+         return ROK;
       }
 
 
@@ -762,7 +762,7 @@ Inst inst;                     /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS382, ERRZERO,
                       "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
@@ -781,13 +781,13 @@ Inst inst;                     /* instance */
                         "Could not destroy system task");
 #endif
 
-            RETVALUE(RFAILED);
+            return RFAILED;
          }
       }
    }
 
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SDeregInitTskTmr */
 
 #endif /* SS_MULTIPLE_PROCS */
@@ -877,13 +877,13 @@ ActvTsk actvTsk;                /* activation function */
    if ((proc == SS_INV_PROCID) || (ent >= SS_MAX_ENT) ||  (inst >= SS_MAX_INST))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS384, ERRZERO, "Invalid proc/entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #else /* SS_MULTIPLE_PROCS */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS385, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -891,7 +891,7 @@ ActvTsk actvTsk;                /* activation function */
    if (actvTsk == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS386, ERRZERO, "Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check task type */
@@ -899,14 +899,14 @@ ActvTsk actvTsk;                /* activation function */
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS387, ERRZERO, "Invalid task type");
       /* ss021.103 - Addition of return fail for invalid task type */
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check task priority */
    if (prior > PRIOR3)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS388, ERRZERO, "Invalid task priority");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -923,7 +923,7 @@ ActvTsk actvTsk;                /* activation function */
                      "Could not find proc table index");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -943,7 +943,7 @@ ActvTsk actvTsk;                /* activation function */
    {
       SSLOGERROR(ERRCLS_DEBUG, ESS390, ERRZERO,
                      "Could not lock system task table");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    SS_ACQUIRE_ALL_SEMA(&osCp.tTskTblSem, ret);
@@ -961,11 +961,11 @@ ActvTsk actvTsk;                /* activation function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS392, ERRZERO,
                       "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -982,11 +982,11 @@ ActvTsk actvTsk;                /* activation function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS394, ERRZERO,
                       "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(ROUTRES);
+      return (ROUTRES);
    }
 
    /* check if entity and instance already registered */
@@ -1007,11 +1007,11 @@ ActvTsk actvTsk;                /* activation function */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS396, ERRZERO,
                       "Could not give the Semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -1111,10 +1111,10 @@ ActvTsk actvTsk;                /* activation function */
       SSLOGERROR(ERRCLS_DEBUG, ESS397, ERRZERO,
                       "Could not give the Semaphore");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SRegTTsk */
  /* ss002.301 Additions */
 /*
@@ -1179,13 +1179,13 @@ ActvTsk cbTsk;
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESSXXX, ERRZERO, "SRegCbTsk() : Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    if (cbTsk == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESSXXX, ERRZERO, " SRegCbTsk() : Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -1201,7 +1201,7 @@ ActvTsk cbTsk;
                      " SRegCbTsk() : Could not find proc table index");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -1213,7 +1213,7 @@ ActvTsk cbTsk;
       SSLOGERROR(ERRCLS_DEBUG, ESSXXX, ERRZERO,
                      "SRegCbTsk() : Could not lock TAPA task table");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 #ifdef SS_MULTIPLE_PROCS
@@ -1227,7 +1227,7 @@ ActvTsk cbTsk;
    {
       SS_RELEASE_ALL_SEMA(&osCp.tTskTblSem);
       SSLOGERROR(ERRCLS_INT_PAR, ESSXXX, ERRZERO, "SRegCbTsk() : Unknown task");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
    tTsk = &osCp.tTskTbl[idx];
@@ -1235,7 +1235,7 @@ ActvTsk cbTsk;
 
    /* unlock the table */
    SS_RELEASE_ALL_SEMA(&osCp.tTskTblSem);
-   RETVALUE(ROK);
+   return ROK;
 }
 
 \f
@@ -1310,13 +1310,13 @@ Inst inst;                      /* instance */
    if ((proc == SS_INV_PROCID) || (ent >= SS_MAX_ENT) ||  (inst >= SS_MAX_INST))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS398, ERRZERO, "Invalid processor/entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #else /* SS_MULTIPLE_PROCS */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS399, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -1333,7 +1333,7 @@ Inst inst;                      /* instance */
       SSLOGERROR(ERRCLS_DEBUG, ESS400, ERRZERO,
                              "Could not find proc id index");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -1347,7 +1347,7 @@ Inst inst;                      /* instance */
       SSLOGERROR(ERRCLS_DEBUG, ESS401, ERRZERO,
                              "Could not lock timer table");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    for (idx = 0;  idx < SS_MAX_TMRS;  idx++)
@@ -1400,7 +1400,7 @@ Inst inst;                      /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
              SSLOGERROR(ERRCLS_DEBUG, ESS402, ERRZERO,
                       "Could not give the Semaphore");
-             RETVALUE(RFAILED);
+             return RFAILED;
 #endif
           }
 
@@ -1408,7 +1408,7 @@ Inst inst;                      /* instance */
             SSLOGERROR(ERRCLS_DEBUG, ESS403, ERRZERO,
                         "Could not deregister task's timer(s)");
 #endif
-            RETVALUE(RFAILED);
+            return RFAILED;
 
          }
       }
@@ -1421,7 +1421,7 @@ Inst inst;                      /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
              SSLOGERROR(ERRCLS_DEBUG, ESS404, ERRZERO,
                       "Could not give the Semaphore");
-             RETVALUE(RFAILED);
+             return RFAILED;
 #endif
           }
  /* ss002.301 Additions */
@@ -1439,7 +1439,7 @@ Inst inst;                      /* instance */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -1459,7 +1459,7 @@ Inst inst;                      /* instance */
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS412, ERRZERO, "Unknown task");
  /* ss002.301 Additions */
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    tTsk = &osCp.tTskTbl[idx];
@@ -1492,7 +1492,7 @@ Inst inst;                      /* instance */
        SSLOGERROR(ERRCLS_DEBUG, ESS415, ERRZERO, "Could not get a message");
 #endif
 
-       RETVALUE(RFAILED);
+       return RFAILED;
        }
 
        /* Update message*/
@@ -1528,12 +1528,12 @@ Inst inst;                      /* instance */
 #if (ERRCLASS & ERRCLS_DEBUG)
        SSLOGERROR(ERRCLS_INT_PAR, ESS416, ERRZERO, "Could not write to demand queue");
 #endif
-       RETVALUE(RFAILED);
+       return RFAILED;
        }
    }
 #endif
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SDeregTTsk */
 
 \f
@@ -1578,7 +1578,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
    if (tskId == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS417, ERRZERO, "Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check system task priority */
@@ -1586,7 +1586,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS418, ERRZERO,
                      "Invalid system task priority");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -1601,7 +1601,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
                      "Could not lock system task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -1647,10 +1647,10 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS420, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
-      RETVALUE(ROK);
+      return ROK;
    }
 #endif /* SS_SINGLE_THREADED */
 
@@ -1665,7 +1665,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS421, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
@@ -1673,7 +1673,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
       SSLOGERROR(ERRCLS_ADD_RES, ESS422, ERRZERO, "Too many system tasks");
 #endif
 
-      RETVALUE(ROUTRES);
+      return (ROUTRES);
    }
 
 
@@ -1695,7 +1695,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS423, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
@@ -1704,7 +1704,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
                   "Could not initialize demand queue");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* initialize the system task entry lock */
@@ -1718,7 +1718,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS425, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
@@ -1727,7 +1727,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
                   "Could not initialize system task entry lock");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -1750,7 +1750,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS427, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
@@ -1759,7 +1759,7 @@ SSTskId *tskId;                 /* filled in with system task ID */
                   "Could not create system task");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* success, update the table */
@@ -1781,11 +1781,11 @@ SSTskId *tskId;                 /* filled in with system task ID */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS429, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
-   RETVALUE(ROK);
+   return ROK;
 
 } /* SCreateSTsk */
 
@@ -1846,7 +1846,7 @@ U16 port
    /* Initialize the lock, return on failure */
    if( SInitLock(&(osCp.logger.bufLock),SS_LOCK_MUTEX) != ROK)
        {
-      RETVALUE(RFAILED);
+      return RFAILED;
    } /* if */
 
    SLock(&osCp.logger.bufLock);
@@ -1857,7 +1857,7 @@ U16 port
    if( osCp.logger.started == TRUE)
    {
       (Void)SUnlock(&(osCp.logger.bufLock));
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
        /* set the mode of the logging */
@@ -1882,7 +1882,7 @@ U16 port
       {
          SDisplay(0,"Error Opening Log File \n");
          (Void)SUnlock(&(osCp.logger.bufLock));
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
       strcpy(osCp.logger.filePath,path);
        }
@@ -1895,7 +1895,7 @@ U16 port
                        {
                           SDisplay(0, "Error at WSAStartup!\n");
             (Void)SUnlock(&(osCp.logger.bufLock));
-                               RETVALUE(RFAILED);
+                               return RFAILED;
                        }
 #endif
       osCp.logger.socketdes = socket(AF_INET, SOCK_DGRAM, 0);
@@ -1906,7 +1906,7 @@ U16 port
                        WSACleanup();
 #endif
          (Void)SUnlock(&(osCp.logger.bufLock));
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
                /* bind to local address */
                localAddr.sin_family = AF_INET;
@@ -1922,7 +1922,7 @@ U16 port
                        close(osCp.logger.socketdes);
 #endif
          (Void)SUnlock(&(osCp.logger.bufLock));
-         RETVALUE(RFAILED);
+         return RFAILED;
                }
                /* remote address */
                remoteAddr.sin_family = AF_INET;
@@ -1937,7 +1937,7 @@ U16 port
                        close(osCp.logger.socketdes);
 #endif
          (Void)SUnlock(&(osCp.logger.bufLock));
-         RETVALUE(RFAILED);
+         return RFAILED;
                }
                remoteAddr.sin_addr.s_addr = inet_addr(IPA);
                osCp.logger.remoteAddr = remoteAddr;
@@ -1958,7 +1958,7 @@ U16 port
 
    (Void)SUnlock(&(osCp.logger.bufLock));
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SRegLogCfg */
 
 /*
@@ -2007,7 +2007,7 @@ S8 *logBuf
        {
       writeNum = sendto(osCp.logger.socketdes, buffer, cmStrlen((U8 *)buffer), 0, (struct sockaddr*)&osCp.logger.remoteAddr, sizeof(osCp.logger.remoteAddr));
       /*ss013.301 : Returned after sending to socket*/
-      RETVALUE(ROK);
+      return ROK;
        }
    osCp.logger.curNumFlush++;
    if(osCp.logger.curNumFlush == osCp.logger.maxNumFlush)
@@ -2030,7 +2030,7 @@ S8 *logBuf
    /*SPutSBuf(DFLT_REGION,DFLT_POOL,
             (Data *)buffer, (Size)(osCp.logger.maxBufSiz * sizeof(S8)));*/
  
-   RETVALUE(ROK);
+   return ROK;
 } /* SFlushBufToLog */
 
 /*
@@ -2062,7 +2062,7 @@ PUBLIC S16 SCleanUp()
    if(osCp.logger.started == FALSE)
    {
       (Void)SUnlock(&(osCp.logger.bufLock));
-      RETVALUE(ROK);
+      return ROK;
    }
    if(osCp.logger.curBufSiz)
    {
@@ -2089,7 +2089,7 @@ PUBLIC S16 SCleanUp()
    signal(SIGINT, App_SigExitCallBack);
 #endif 
 
-   RETVALUE(ROK);
+   return ROK;
 
 } /* end of SCleanUp */
 
@@ -2122,7 +2122,7 @@ PUBLIC S16 SDeregLogCfg()
  
    SCleanUp();   
 
-   RETVALUE(ROK);
+   return ROK;
 
 } /* SDeregLogCfg */
 
@@ -2221,7 +2221,7 @@ SCpuInfo *cpuInfo;  /* Information about the cores/threads per core */
         cpuInfo->numCores < 1 || cpuInfo->threadsPerCore < 1 )
    {
        SSLOGERROR(ERRCLS_INT_PAR, ESS430, ERRZERO, "Invalid number of cores\n");
-       RETVALUE(RFAILED);
+       return RFAILED;
    }
 
    /* lock mCInfo */
@@ -2243,7 +2243,7 @@ SCpuInfo *cpuInfo;  /* Information about the cores/threads per core */
    /* unlock mCInfo */
    SUnlock(&osCp.mCILock);
 
-   RETVALUE(ROK);
+   return ROK;
 
 } /* SRegCpuInfo */
 
@@ -2287,10 +2287,10 @@ U32 *coreId;                    /* the core/processor id to which the affinity i
     if (ret != ROK)
     {
        SSLOGERROR(ERRCLS_INT_PAR, ESS431, ERRZERO, "Failed to get affinity\n");
-       RETVALUE(RFAILED);
+       return RFAILED;
     } /* end if */
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SGetAffinity */
 
 
@@ -2335,7 +2335,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
        mode > SS_AFFINITY_MODE_EXCL)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS432, ERRZERO, "Invalid mode for setting core affinity\n");
-      RETVALUE(RFAILED);
+      return RFAILED;
    } /* end if */
 
    /* check the value of core id */
@@ -2343,7 +2343,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
        (coreId > osCp.mCInfo.cpuInfo.numCores || coreId < 0))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS433, ERRZERO, "Invalid core id\n");
-      RETVALUE(RFAILED);
+      return RFAILED;
    } /* end if */
 
    /* set affinity according to the mode supplied */
@@ -2374,7 +2374,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                              {
                                  SSLOGERROR(ERRCLS_INT_PAR, ESS434, ERRZERO, \
                                             "Failed to set core affinity\n");
-                                 RETVALUE(RFAILED);
+                                 return RFAILED;
                              } /* end if */
   
                              /* lock mCInfo */
@@ -2414,7 +2414,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
               {
                   SSLOGERROR(ERRCLS_INT_PAR, ESS436, ERRZERO,\
                              "Can not set core affinity, core is exclusively used for other task\n");
-                  RETVALUE(RFAILED);
+                  return RFAILED;
               } /* end if */
               /* call the implementation specific affinity function with the core id supplied by caller */
               ret = ssdSetAffinity(tskId, coreId);
@@ -2422,7 +2422,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
               if (ret != ROK)
               {
                   SSLOGERROR(ERRCLS_INT_PAR, ESS437, ERRZERO, "Failed to set core affinity\n");
-                  RETVALUE(RFAILED);
+                  return RFAILED;
               } /* end if */
 
               /* lock mCInfo */
@@ -2467,7 +2467,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                               if (ret != ROK)
                               {
                                   SSLOGERROR(ERRCLS_INT_PAR, ESS439, ERRZERO, "Failed to set core affinity\n");
-                                  RETVALUE(RFAILED);
+                                  return RFAILED;
                               } /* end if */
     
                               /* lock mCInfo */
@@ -2498,7 +2498,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                          {
                              SSLOGERROR(ERRCLS_INT_PAR, ESS441, ERRZERO, 
                                         "Can not set core affinity, core is exclusively used for other task\n");
-                             RETVALUE(RFAILED);
+                             return RFAILED;
                          } /* end else */
                          tskNotFound = FALSE;
                            break;
@@ -2510,7 +2510,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                if (coreIndex == SS_MAX_CORES)
                 {
                     SSLOGERROR(ERRCLS_INT_PAR, ESS442, ERRZERO, "Invalid core id\n");
-                    RETVALUE(RFAILED);
+                    return RFAILED;
                 } /* if */
              break;
           } /* end case SS_AFFINITY_MODE_ASSOC */
@@ -2527,7 +2527,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                  if (ret != ROK)
                  {
                     SSLOGERROR(ERRCLS_INT_PAR, ESS443, ERRZERO, "Failed to set core affinity\n");
-                    RETVALUE(RFAILED);
+                    return RFAILED;
                  } /* end if */
 
                  /* lock mCInfo */
@@ -2571,7 +2571,7 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                                 if (ret != ROK)
                                  {
                                     SSLOGERROR(ERRCLS_INT_PAR, ESS444, ERRZERO, "Failed to set core affinity\n");
-                                    RETVALUE(RFAILED);
+                                    return RFAILED;
                                  } /* end if */
      
                                  /* lock mCInfo */
@@ -2604,25 +2604,25 @@ SSTskId *tskAssociatedTskId;    /* filled in with system task ID of the associat
                   {
                      SSLOGERROR(ERRCLS_INT_PAR, ESS445, ERRZERO, 
                                 "Failed to set core affinity, no threads on cores available\n");
-                     RETVALUE(RFAILED);
+                     return RFAILED;
                   } /* end if no thrs on cores available */
 
               } /* else if */
               else
               {
                  SSLOGERROR(ERRCLS_INT_PAR, ESS446, ERRZERO, "Can not set exclusive affinity for the core\n");
-                 RETVALUE(RFAILED);
+                 return RFAILED;
               } /* end else */
             break;
          } /* end case SS_AFFINITY_MODE_EXCL */
          default:
          {
             SSLOGERROR(ERRCLS_INT_PAR, ESS447, ERRZERO, "Invalid mode for setting core affinity\n");
-            RETVALUE(RFAILED);
+            return RFAILED;
          } /* end default case */
     } /* end of switch */
 
-    RETVALUE(ROK);
+    return ROK;
                
 
 } /* SSetAffinity */
@@ -2678,7 +2678,7 @@ SSTskId tskId;                  /* system task to be destroyed */
    if (tskId >= SS_MAX_STSKS)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS448, (ErrVal) tskId, "Invalid task ID");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -2693,7 +2693,7 @@ SSTskId tskId;                  /* system task to be destroyed */
                      "Could not lock system task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -2715,13 +2715,13 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS450, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS451, (ErrVal) idx,
                         "Invalid system task ID");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
    else if (sTsk->termPend != FALSE)
    {
@@ -2732,13 +2732,13 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS452, ERRZERO,
                        "Could not give the Semaphore");
-           RETVALUE(RFAILED);
+           return RFAILED;
 #endif
       }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS453, (ErrVal) idx,
                         "Invalid system task ID");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -2756,7 +2756,7 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
              SSLOGERROR(ERRCLS_DEBUG, ESS454, ERRZERO,
                        "Could not give the Semaphore");
-             RETVALUE(RFAILED);
+             return RFAILED;
 #endif
          }
 
@@ -2765,7 +2765,7 @@ SSTskId tskId;                  /* system task to be destroyed */
                         "Could not lock system task entry");
 #endif
 
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
    }
 
@@ -2782,7 +2782,7 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
              SSLOGERROR(ERRCLS_DEBUG, ESS456, ERRZERO,
                        "Could not give the Semaphore");
-             RETVALUE(RFAILED);
+             return RFAILED;
 #endif
          }
       }
@@ -2793,7 +2793,7 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
              SSLOGERROR(ERRCLS_DEBUG, ESS457, ERRZERO,
                        "Could not give the Semaphore");
-             RETVALUE(RFAILED);
+             return RFAILED;
 #endif
          }
 
@@ -2802,7 +2802,7 @@ SSTskId tskId;                  /* system task to be destroyed */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -2840,7 +2840,7 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS459, ERRZERO,
                        "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
    }
@@ -2852,7 +2852,7 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS460, ERRZERO,
                      "Could not give the Semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
     }
 
@@ -2875,7 +2875,7 @@ SSTskId tskId;                  /* system task to be destroyed */
                   "Could not destroy system task");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
    
    /* multi-core support */
@@ -2932,12 +2932,12 @@ SSTskId tskId;                  /* system task to be destroyed */
 #if (ERRCLASS & ERRCLS_DEBUG)
        SSLOGERROR(ERRCLS_DEBUG, ESS462, ERRZERO,
                          "Could not unlock the Semaphore");
-       RETVALUE(RFAILED);
+       return RFAILED;
 #endif
    }
 #endif
    
-   RETVALUE(ROK);
+   return ROK;
 } /* SDestroySTsk */
 
 
@@ -3024,14 +3024,14 @@ SSTskId sTskId;                 /* system task to use */
    if ((proc == SS_INV_PROCID) || (ent >= SS_MAX_ENT) ||  (inst >= SS_MAX_INST))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS463, ERRZERO, "Invalid processor/entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #else /* SS_MULTIPLE_PROCS */
    /* check entity and instance range */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS464, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -3040,7 +3040,7 @@ SSTskId sTskId;                 /* system task to use */
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS465, (ErrVal) sTskId,
                   "Invalid system task ID");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3055,7 +3055,7 @@ SSTskId sTskId;                 /* system task to use */
       SSLOGERROR(ERRCLS_DEBUG, ESS466, ERRZERO,
                              "Could not find proc id index");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -3069,7 +3069,7 @@ SSTskId sTskId;                 /* system task to use */
                      "Could not lock system task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3087,13 +3087,13 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS468, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS469, (ErrVal) sTskId,
                      "Unknown system task ID");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3111,7 +3111,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS470, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
 
@@ -3120,7 +3120,7 @@ SSTskId sTskId;                 /* system task to use */
                         "Could not lock system task entry");
 #endif
 
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
    }
 
@@ -3137,7 +3137,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS472, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
       }
@@ -3148,11 +3148,11 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS473, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3169,7 +3169,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS474, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
       }
@@ -3181,7 +3181,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS475, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
@@ -3190,7 +3190,7 @@ SSTskId sTskId;                 /* system task to use */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3216,7 +3216,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS477, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
       }
@@ -3227,12 +3227,12 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS478, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS479, ERRZERO, "Unknown task");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3250,7 +3250,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS480, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
       }
@@ -3261,11 +3261,11 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS481, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3282,7 +3282,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS482, ERRZERO,
                      "Could not give the Semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
         }
       }
@@ -3293,11 +3293,11 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS483, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
-      RETVALUE(ROUTRES);
+      return (ROUTRES);
    }
 
 
@@ -3337,7 +3337,7 @@ SSTskId sTskId;                 /* system task to use */
                      "Could not find proc table index");
 #endif
 
-              RETVALUE(RFAILED);
+              return RFAILED;
         }
        (Void)(tTsk->initTsk)(tTsk->proc, tTsk->ent, tTsk->inst, sTsk->region, PWR_UP, &xxCb);
        /* 
@@ -3372,7 +3372,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS484, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
    }
@@ -3383,7 +3383,7 @@ SSTskId sTskId;                 /* system task to use */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS485, ERRZERO,
                      "Could not give the Semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
    }
 
@@ -3399,11 +3399,11 @@ SSTskId sTskId;                 /* system task to use */
 #else /* SS_MULTIPLE_PROCS */
       SDetachTTsk(ent, inst);
 #endif /* SS_MULTIPLE_PROCS */
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SAttachTTsk */
 
 \f
@@ -3478,14 +3478,14 @@ Inst inst;                      /* instance ID of the task */
    if ((proc == SS_INV_PROCID) || (ent >= SS_MAX_ENT) ||  (inst >= SS_MAX_INST))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS486, ERRZERO, "Invalid processor/entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #else /* SS_MULTIPLE_PROCS */
    /* check entity and instance ranges */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS487, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 #endif
@@ -3501,7 +3501,7 @@ Inst inst;                      /* instance ID of the task */
       SSLOGERROR(ERRCLS_DEBUG, ESS488, ERRZERO,
                              "Could not find proc id index");
 #endif
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 
@@ -3519,7 +3519,7 @@ Inst inst;                      /* instance ID of the task */
                      "Could not lock system task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3534,7 +3534,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS490, ERRZERO,
                      "Could not give the Semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
     }
 
@@ -3543,7 +3543,7 @@ Inst inst;                      /* instance ID of the task */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3567,12 +3567,12 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS492, ERRZERO,
                      "Could not give the Semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
      }
 
       SSLOGERROR(ERRCLS_INT_PAR, ESS493, ERRZERO, "Unknown task");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3596,10 +3596,10 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS494, ERRZERO,
                      "Could not give the Semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
      }
-      RETVALUE(ROK);
+      return ROK;
    }
 
 
@@ -3629,7 +3629,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS495, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
        }
 
@@ -3638,7 +3638,7 @@ Inst inst;                      /* instance ID of the task */
                         "Could not lock system task entry");
 #endif
 
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
    }
 
@@ -3654,7 +3654,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS497, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
       if ( SUnlock(&osCp.sTskTblLock) != ROK)
@@ -3662,7 +3662,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS498, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
@@ -3671,7 +3671,7 @@ Inst inst;                      /* instance ID of the task */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -3713,7 +3713,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS500, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
    }
@@ -3726,7 +3726,7 @@ Inst inst;                      /* instance ID of the task */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS501, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
@@ -3736,11 +3736,11 @@ Inst inst;                      /* instance ID of the task */
     */
    if (ret != ROK)
    {
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SDetachTTsk */
 
 \f
@@ -3819,7 +3819,7 @@ Buffer *mBuf;                   /* message to post */
    if (mBuf == NULLP)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS502, ERRZERO, "Invalid message buffer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* check the pst structure */
@@ -3827,7 +3827,7 @@ Buffer *mBuf;                   /* message to post */
    {
       SPutMsg(mBuf);
       SSLOGERROR(ERRCLS_INT_PAR, ESS503, ERRZERO, "Null pointer");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
    /* ss021.103 - Addition to check for valid route */
    /* ss023.103 - Modification to fix bug in route validation */
@@ -3836,7 +3836,7 @@ Buffer *mBuf;                   /* message to post */
    {
       SPutMsg(mBuf);
       SSLOGERROR(ERRCLS_INT_PAR, ESS504, ERRZERO, "No route defined");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3852,7 +3852,7 @@ Buffer *mBuf;                   /* message to post */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3874,7 +3874,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS506, ERRZERO,
                      "Could not release the semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
       }
 #endif
@@ -3884,7 +3884,7 @@ Buffer *mBuf;                   /* message to post */
       SSLOGERROR(ERRCLS_INT_PAR, ESS507, ERRZERO,
                   "Invalid source/destination entity/instance");
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif
 
@@ -3917,7 +3917,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS508, ERRZERO,
                      "Could not release the semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
           }
 #endif
@@ -3928,7 +3928,7 @@ Buffer *mBuf;                   /* message to post */
                      "Could not lock router task entry");
 #endif
 
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
 
       /* call the router activation function */
@@ -3942,7 +3942,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
           SSLOGERROR(ERRCLS_DEBUG, ESS510, ERRZERO,
                      "Could not give the Semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
       }
 
@@ -3955,11 +3955,11 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
             SSLOGERROR(ERRCLS_DEBUG, ESS511, ERRZERO,
                      "Could not release the semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
           }
 #endif
-         RETVALUE((r == RFAILED) ? RFAILED : ROK);
+         return ((r == RFAILED) ? RFAILED : ROK);
       }
    }
 #endif  /* SS_RTR_SUPPORT */
@@ -3986,7 +3986,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS512, ERRZERO,
                      "Could not release the semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
       }
 #endif
@@ -3997,7 +3997,7 @@ Buffer *mBuf;                   /* message to post */
       SSLOGERROR(ERRCLS_DEBUG, ESS513, ERRZERO, "Unknown task");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 #else /* SS_MULTIPLE_PROCS */
@@ -4022,7 +4022,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
       SSLOGERROR(ERRCLS_DEBUG, ESS514, ERRZERO,
                      "Could not release the semaphore");
-      RETVALUE(RFAILED);
+      return RFAILED;
 #endif
       }
 #endif
@@ -4032,7 +4032,7 @@ Buffer *mBuf;                   /* message to post */
       SSLOGERROR(ERRCLS_DEBUG, ESS515, ERRZERO, "Unknown task");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 #endif /* SS_MULTIPLE_PROCS */
@@ -4086,7 +4086,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
                   SSLOGERROR(ERRCLS_DEBUG, ESS516, ERRZERO,
                      "Could not release the semaphore");
-                  RETVALUE(RFAILED);
+                  return RFAILED;
 #endif
                }
 #endif
@@ -4097,7 +4097,7 @@ Buffer *mBuf;                   /* message to post */
                            "Could not lock driver task entry");
 #endif
 
-               RETVALUE(RFAILED);
+               return RFAILED;
             }
 #ifdef L2_L3_SPLIT
 }
@@ -4120,7 +4120,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
               SSLOGERROR(ERRCLS_DEBUG, ESS519, ERRZERO,
                      "Could not give the Semaphore");
-              RETVALUE(RFAILED);
+              return RFAILED;
 #endif
             }
 #ifdef L2_L3_SPLIT
@@ -4134,11 +4134,11 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
                SSLOGERROR(ERRCLS_DEBUG, ESS520, ERRZERO,
                      "Could not release the semaphore");
-               RETVALUE(RFAILED);
+               return RFAILED;
 #endif
             }
 #endif
-            RETVALUE(ROK);
+            return ROK;
          }
       }
 
@@ -4149,7 +4149,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS521, ERRZERO,
                      "Could not release the semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
       }
 #endif
@@ -4160,7 +4160,7 @@ Buffer *mBuf;                   /* message to post */
                   "Could not find a driver task to handle this proc ID");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif  /* SS_DRVR_SUPPORT */
  /* ss002.301 Modifications */
@@ -4181,7 +4181,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
                SSLOGERROR(ERRCLS_DEBUG, ESS512, ERRZERO,
                      "Could not release the semaphore");
-               RETVALUE(RFAILED);
+               return RFAILED;
 #endif
        }
 #endif
@@ -4189,7 +4189,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
        SSLOGERROR(ERRCLS_DEBUG, ESS513, ERRZERO, "call back function failed\n");
 #endif
-                       RETVALUE(ROK);
+                       return ROK;
                }
 #ifdef SS_MULTIPLE_PROCS
                dstIdx = osCp.tTskIds[dstProcIdIdx][tempPst.dstEnt][tempPst.dstInst];
@@ -4213,7 +4213,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS527, ERRZERO,
            "Could not release the semaphore");
-         RETVALUE(RFAILED);
+         return RFAILED;
 #endif
       }
 
@@ -4222,7 +4222,7 @@ Buffer *mBuf;                   /* message to post */
       if( ret != ROK )
       {
          /* No need to free the buffer, its already done in called fun */
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
 
       /* Allocate for the mBuf */
@@ -4231,7 +4231,7 @@ Buffer *mBuf;                   /* message to post */
       if( workPtr == NULLP )
       {
          SPutFpaMsg(wqBuf);
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
 
       /* Convert the pointers to physical address */
@@ -4239,7 +4239,7 @@ Buffer *mBuf;                   /* message to post */
       if( ret != ROK )
       {
          SPutFpaMsg(wqBuf);
-         RETVALUE(RFAILED);
+         return RFAILED;
       }
 
       /* Assign the values for work ptr */
@@ -4253,7 +4253,7 @@ Buffer *mBuf;                   /* message to post */
       cvmx_pow_work_submit(workPtr, workPtr->tag, workPtr->tag_type, \
             workPtr->qos, workPtr->grp);
 
-      RETVALUE(ROK);
+      return ROK;
    }
 #endif /* SS_SEUM_CAVIUM */
 
@@ -4273,7 +4273,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS523, ERRZERO,
                      "Could not release the semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
       }
 #endif
@@ -4284,7 +4284,7 @@ Buffer *mBuf;                   /* message to post */
                  "Destination TAPA task is not attached");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 #ifdef SS_LOCKLESS_MEMORY
@@ -4354,7 +4354,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
            SSLOGERROR(ERRCLS_DEBUG, ESS525, ERRZERO,
                      "Could not release the semaphore");
-            RETVALUE(RFAILED);
+            return RFAILED;
 #endif
       }
 #endif
@@ -4365,7 +4365,7 @@ Buffer *mBuf;                   /* message to post */
                      "Could not write to demand queue");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -4377,7 +4377,7 @@ Buffer *mBuf;                   /* message to post */
 #if (ERRCLASS & ERRCLS_DEBUG)
         SSLOGERROR(ERRCLS_DEBUG, ESS527, ERRZERO,
                   "Could not release the semaphore");
-        RETVALUE(RFAILED);
+        return RFAILED;
 #endif
    }
 #endif
@@ -4388,7 +4388,7 @@ Buffer *mBuf;                   /* message to post */
     */
    ssdPstTsk(tempPst, mBuf, tTsk);
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SPstTsk */
 
 /* ss001.301: additions */
@@ -4430,7 +4430,7 @@ Ent *ent;                        /* entity */
       ent[tskCnt] = osCp.tTskTbl[tskCnt].ent;
    }
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SGetTapaTskEntIds */
 
 /*
@@ -4475,7 +4475,7 @@ Ent ent;                        /* entity */
                      "Could not lock TAPA task table");
 #endif
 
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    for(tskCnt = 0; tskCnt < osCp.numTTsks; tskCnt++)
@@ -4492,11 +4492,11 @@ Ent ent;                        /* entity */
 #if (ERRCLASS & ERRCLS_DEBUG)
          SSLOGERROR(ERRCLS_DEBUG, ESS639, ERRZERO,
                    "Could not release the semaphore");
-          RETVALUE(RFAILED);
+          return RFAILED;
 #endif
     }
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SGetTapaTskEntIds */
 
 /*
@@ -4548,7 +4548,7 @@ Bool  *hstReg;                        /* entity */
       *hstReg = TRUE;
    }
 
-   RETVALUE(ROK);
+   return ROK;
 } /* SGetTapaTskEntIds */
 
 #endif /* SS_HISTOGRAM_SUPPORT */
@@ -4621,7 +4621,7 @@ void *data;
 #endif
    SPrint(prntBuf);
 
-   RETVALUE(ROK);
+   return ROK;
 }
 
 #ifdef ANSI
@@ -4660,7 +4660,7 @@ void;
    /* Implicit watchdog stop during dereg */
    SStopHrtBt();
 
-   RETVALUE(ROK);
+   return ROK;
 }
 
 #ifdef ANSI
@@ -4678,7 +4678,7 @@ U8 timeInterval;
    ssdStartWatchDgTmr(NULLP, SS_TMR_HRTBT, timeInterval);
    ssdSndHrtBtMsg(TRUE, SS_WD_HB_REQ);
 
-   RETVALUE(ROK);
+   return ROK;
 }
 
 #ifdef ANSI
@@ -4695,7 +4695,7 @@ void
 
    osCp.wdCp.globWd.watchdogStop = 1;
 
-   RETVALUE(ROK);
+   return ROK;
 }
 
 #ifdef ANSI
@@ -4723,7 +4723,7 @@ Buffer *mBuf;               /* message buffer */
 );
    SPrint(prntBuf);
 #endif /* DEBUGP */
-       RETVALUE(ROK);
+       return ROK;
 }
 
 #ifdef ANSI
@@ -4808,7 +4808,7 @@ Buffer *mBuf;               /* message buffer */
 #endif /* DEBUGP */
       }
    }
-       RETVALUE(ROK);
+       return ROK;
 }
 
 #endif /* SS_WATCHDOG */
@@ -4894,14 +4894,14 @@ U64 *totTime;
    if ((procId == SS_INV_PROCID) || (ent >= SS_MAX_ENT) ||  (inst >= SS_MAX_INST))
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS463, ERRZERO, "Invalid processor/entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #else /* SS_MULTIPLE_PROCS */
    /* check entity and instance range */
    if (ent >= SS_MAX_ENT ||  inst >= SS_MAX_INST)
    {
       SSLOGERROR(ERRCLS_INT_PAR, ESS464, ERRZERO, "Invalid entity/instance");
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 #endif /* SS_MULTIPLE_PROCS */
 #endif
@@ -4910,7 +4910,7 @@ U64 *totTime;
    ret = SLock(&osCp.sTskTblLock);
    if (ret != ROK)
    {
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
 
@@ -4919,7 +4919,7 @@ U64 *totTime;
 
     if (procIdIdx == SS_INV_PROCID_IDX)
     {
-      RETVALUE(RFAILED);
+      return RFAILED;
     }
 
     idx = osCp.tTskIds[procIdIdx][ent][inst];
@@ -4931,7 +4931,7 @@ U64 *totTime;
     if(tTsk == NULLP )
     {
              SUnlock(&osCp.sTskTblLock);
-             RETVALUE(RFAILED);
+             return RFAILED;
     }
     *curEvent      = osCp.tTskTbl[idx].curEvent;
     *curEvtTime    = osCp.tTskTbl[idx].curEvtTime;
@@ -4939,7 +4939,7 @@ U64 *totTime;
     *sTskId        = osCp.tTskTbl[idx].sTsk->tskId;
 
    SUnlock(&osCp.sTskTblLock);
-   RETVALUE(ROK);
+   return ROK;
 }
 #endif /* SS_THREAD_PROFILE */