Replaced old SSI function with new macros jira id - ODUHIGH-212
[o-du/l2.git] / src / 5gnrsch / rg_sch_tom.c
index afabf28..d80ed47 100755 (executable)
@@ -37,17 +37,7 @@ static int RLOG_FILE_ID=228;
 static int RLOG_MODULE_ID=4096;
 
 /* header include files -- defines (.h) */
-#include "envopt.h"        /* environment options */
-#include "envdep.h"        /* environment dependent */
-#include "envind.h"        /* environment independent */
-#include "gen.h"           /* general layer */
-#include "ssi.h"           /* system services */
-#include "cm5.h"           /* common timers defines */
-#include "cm_hash.h"       /* common hash list defines */
-#include "cm_llist.h"      /* common linked list defines */
-#include "cm_mblk.h"       /* memory management */
-#include "cm_tkns.h"       /* common tokens */
-#include "cm_lte.h"       /* common tokens */
+#include "common_def.h"
 #include "tfu.h"           /* RGU defines */
 #include "lrg.h"           /* layer management defines for LTE-MAC */
 #include "rgr.h"           /* layer management defines for LTE-MAC */
@@ -62,15 +52,6 @@ static int RLOG_MODULE_ID=4096;
 
 
 /* header/extern include files (.x) */
-#include "gen.x"           /* general layer typedefs */
-#include "ssi.x"           /* system services typedefs */
-#include "cm5.x"           /* common timers */
-#include "cm_hash.x"       /* common hash list */
-#include "cm_lib.x"        /* common library */
-#include "cm_llist.x"      /* common linked list */
-#include "cm_mblk.x"       /* memory management */
-#include "cm_tkns.x"       /* common tokens */
-#include "cm_lte.x"       /* common tokens */
 #include "tfu.x"           /* RGU types */
 #include "lrg.x"           /* layer management typedefs for MAC */
 #include "rgr.x"           /* layer management typedefs for MAC */
@@ -81,6 +62,8 @@ static int RLOG_MODULE_ID=4096;
 #ifdef EMTC_ENABLE
 #include "rg_sch_emtc_ext.x"
 #endif 
+U8 rgSCHCmnCalcPcqiBitSz(RgSchUeCb    *ueCb,U8 numTxAnt);
+PUBLIC S16 rgSCHDhm5gtfHqFdbkInd(RgSchUeCb *ue,RgSchCellCb *cell,CmLteTimingInfo timingInfo,TfuHqFdbk fdbk,RgSchErrInfo *err);
 /* local defines */
 #ifdef EMTC_ENABLE
 EXTERN  Bool rgSCHEmtcChkEmtcUe ARGS(
@@ -698,9 +681,9 @@ U32 rgSch5gtfCqi2Mcs[15] =
 /*HARQ Feedback interpretation in accordance with Femto Forum.
 Note: There is no value as '0' in Femto Forum Spec but in order to retain
 the existing usage in MAC (and its Acceptance), its being considered*/
-CONSTANT PRIVATE U8 rgSchTomHqFbkMap[8] = {0,1,0,0,4,4,4,4};
+//CONSTANT PRIVATE U8 rgSchTomHqFbkMap[8] = {0,1,0,0,4,4,4,4};
 /*added #defines instead of magic numbers*/
-CONSTANT PRIVATE U32 rgSCHTomBinCoe[RG_SCH_MAX_NUM_UE_SEL_SUBBANDS][RG_SCH_MAX_TOT_NUM_SUBBANDS]={
+/*CONSTANT PRIVATE U32 rgSCHTomBinCoe[RG_SCH_MAX_NUM_UE_SEL_SUBBANDS][RG_SCH_MAX_TOT_NUM_SUBBANDS]={
 {1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28},
 {0,1,3,6,10,15,21,28,36,45,55,66,78,91,105,120,136,153,171,190,210,231,253,276,300,325,351,378},
 {0,0,1,4,10,20,35,56,84,120,165,220,286,364,455,560,680,816,969,1140,1330,1540,1771,2024,2300,2600,2925,3276},
@@ -708,7 +691,7 @@ CONSTANT PRIVATE U32 rgSCHTomBinCoe[RG_SCH_MAX_NUM_UE_SEL_SUBBANDS][RG_SCH_MAX_T
 {0,0,0,0,1,6,21,56,126,252,462,792,1287,2002,3003,4368,6188,8568,11628,15504,20349,26334,33649,42504,53130,65780,80730,98280},
 {0,0,0,0,0,1,7,28,84,210,462,924,1716,3003,5005,8008,12376,18564,27132,38760,54264,74613,100947,134596,177100,230230,296010,376740}
 };
-
+*/
 
 /*ccpu00116923 - ADD - SRS present support*/
 /*Tables Derived from 3GPP TS 36.211 Section 5.5.3.3 */
@@ -806,7 +789,7 @@ RgSchUeCb       **ue;
       *ue = rgSCHCmnGetPoUe(cell, rapId, timingInfo);
       printf("     ==== inding UE from PDCCH Order Mapping\n");
    }
-   RETVALUE(ROK);
+   return ROK;
 }
 /**
  * @brief Handler for processing Random Access request indication 
@@ -860,7 +843,7 @@ TfuRaReqIndInfo *raReqInd;
       RLOG_ARG3(L_ERROR,DBG_CELLID,cell->cellId, 
          "rgSCHTomRaReqInd(): No cell found with raReq cellId = (%d) errorType (%d)"
          " errorCause(%d)",raReqInd->cellId, err.errType, err.errCause);
-      RETVALUE(RFAILED);
+      return RFAILED;
    } 
    
    for (raRntiCnt = 0; raRntiCnt < raReqInd->nmbOfRaRnti; raRntiCnt++)
@@ -906,7 +889,7 @@ TfuRaReqIndInfo *raReqInd;
          }
       }
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomRaReqInd */
 
 
@@ -954,7 +937,7 @@ TfuUlCqiIndInfo *ulCqiInd;
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,
          "rgSCHTomUlCqiInd() Unable to get the ulCqiInd cell with id(%d)", 
          ulCqiInd->cellId);
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    for (;node; node=node->next)
@@ -982,7 +965,7 @@ TfuUlCqiIndInfo *ulCqiInd;
       /* wideband cqi is directly reported now. and also isTxPort0 */
       rgSCHUtlUlCqiInd(cell, ue, ulCqiInfo);
    }
-   RETVALUE(ROK);
+       return ROK;
 }  /* rgSCHTomUlCqiInd */
 
 /**
@@ -1030,7 +1013,7 @@ TfuPucchDeltaPwrIndInfo *pucchDeltaPwr;
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,
          "rgSCHTomPucchDeltaPwrInd() Unable to get the pucchDeltaPwr cell with id(%d)", 
          pucchDeltaPwr->cellId);
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    node =  pucchDeltaPwr->pucchDeltaPwrLst.first;
@@ -1051,7 +1034,7 @@ TfuPucchDeltaPwrIndInfo *pucchDeltaPwr;
       }
       rgSCHUtlPucchDeltaPwrInd(cell, ue, ueElem->pucchDeltaPwr);
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomPucchDeltaPwrInd */
 
 /**
@@ -1108,8 +1091,8 @@ TfuHqIndInfo    *harqAckInd;
          " the cell for cellId (%d)", harqAckInd->cellId);
       err.errType = RGSCHERR_TOM_HARQACKIND;
       err.errCause = RGSCHERR_TOM_INV_CELL_ID;
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 #ifdef RG_5GTF   
    node =  harqAckInd->hqIndLst.first;
    for (;node; node=node->next)
@@ -1243,7 +1226,7 @@ TfuHqIndInfo    *harqAckInd;
       }
    }
 #endif 
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomHarqAckInd */
 
 
@@ -1295,8 +1278,8 @@ TfuSrIndInfo    *srInd;
          ":%d ", srInd->cellId);
       err.errType = RGSCHERR_TOM_SRIND;
       err.errCause = RGSCHERR_TOM_INV_CELL_ID;
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
 
    node =  srInd->srLst.first;
@@ -1327,7 +1310,7 @@ TfuSrIndInfo    *srInd;
          continue;
       }
    }
-   RETVALUE(ret);
+   return ROK;
 }  /* end of rgSCHTomSrInd */
 
 /**
@@ -1373,8 +1356,8 @@ TfuDoaIndInfo   *doaInd;
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for doaInd cellId"
          ":%d", doaInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
 
    node =  doaInd->doaRpt.first;
@@ -1390,7 +1373,7 @@ TfuDoaIndInfo   *doaInd;
       }
       rgSCHUtlDoaInd(cell, ue, doaInfo);
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomDoaInd */
 /**
  * @brief Handler for processing downlink CQI indication recieved from 
@@ -1435,8 +1418,8 @@ TfuDlCqiIndInfo *dlCqiInd;
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for cellId"
          ":%d", dlCqiInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
 
    node =  dlCqiInd->dlCqiRptsLst.first;
@@ -1453,7 +1436,7 @@ TfuDlCqiIndInfo *dlCqiInd;
       rgSCHUtlDlCqiInd(cell, ue, dlCqiInfo, dlCqiInd->timingInfo);
       rgSCHUtlHdlUlTransInd(cell, ue, dlCqiInd->timingInfo);
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomDlCqiInd */
 
 /**
@@ -1526,7 +1509,7 @@ PRIVATE S16 rgSCHTomUtlMovePcqiNxtOccasion(cell, ue, cqiCb)
    rgSCHUtlSCellHndlCqiCollsn(cqiCb);
 #endif
 
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlMovePcqiNxtOccasion */
 
 /**
@@ -1649,7 +1632,7 @@ PRIVATE S16 rgSCHTomUtlMovePriNxtOccasion(cell, ue, riCb)
 #ifdef LTE_ADV
    rgSCHUtlSCellHndlRiCollsn(riCb);
 #endif
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlMovePriNxtOccasion */
 
 /**
@@ -1705,7 +1688,7 @@ PRIVATE S16 rgSCHTomUtlMoveSrNxtOccasion(cell, ue)
    cmLListAdd2Tail(&cell->pCqiSrsSrLst[ue->srCb.nSrTrIdx].srLst, 
          &ue->srCb.srLstEnt);
 
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlMoveSrNxtOccasion */
 
 /**
@@ -1791,8 +1774,7 @@ PRIVATE S16 rgSCHTomUtlMoveSrsNxtOccasion(cell, ue)
    }   
    ue->srsCb.nSrsTrIdx = srsIdx;
    ue->srsCb.srsDist = srsDist; 
-
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlMoveSrsNxtOccasion */
 
 
@@ -1854,8 +1836,8 @@ PUBLIC S16 rgSCHTomRawCqiInd(cell, rawCqiInd)
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for cellId"
             ":%d", rawCqiInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
 
    node =  rawCqiInd->rawCqiRpt.first;
@@ -2068,7 +2050,7 @@ PUBLIC S16 rgSCHTomRawCqiInd(cell, rawCqiInd)
          rlsHqBufs->numUes = 0;
       }
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomRawCqiInd */
 
 /**
@@ -2115,8 +2097,8 @@ PUBLIC S16 rgSCHTomSrsInd(cell, srsInd)
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for cellId"
          ":%d", srsInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
    node =  srsInd->srsRpt.first;
    for (;node; node=node->next)
@@ -2132,7 +2114,7 @@ PUBLIC S16 rgSCHTomSrsInd(cell, srsInd)
       rgSCHUtlSrsInd(cell, ue, srsInfo, srsInd->timingInfo);
       rgSCHUtlHdlUlTransInd(cell, ue, srsInd->timingInfo);
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomSrsInd */
 
 /*
@@ -2192,7 +2174,7 @@ PRIVATE S16 rgSCHTomUtlGenIndices(label, posM, valN, valK, sbInfo)
       xmin = idx+1;
       label = label-binCoe;
    }
-   RETVALUE(ROK);
+   return ROK;
 } /* end of rgSCHTomUtlGenIndices*/
 #endif
 #endif 
@@ -2261,8 +2243,8 @@ TfuCrcIndInfo *crcInd;
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for cellId"
          ":%d", crcInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 #ifdef RG_ULSCHED_AT_CRC
 #ifndef LTE_ADV
    {
@@ -2438,7 +2420,7 @@ TfuCrcIndInfo *crcInd;
       {
          RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate TfuCntrlReqInfo "
                "for cell");
-         RETVALUE(ret);
+         return ret;
       }
       rgSCHCmnUlSch(cell);
 #ifdef LTE_L2_MEAS
@@ -2452,10 +2434,10 @@ TfuCrcIndInfo *crcInd;
          RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"rgSCHTomCrcInd() Unable to process"
                   " downlink subframe for cellId %d", crcInd->cellId);
          err.errType = RGSCHERR_TOM_TTIIND;
-         RETVALUE(RFAILED);
-      }
+         return RFAILED;
+               }
 #endif /* RG_ULSCHED_AT_CRC */
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomCrcInd */
 
 /**
@@ -2503,8 +2485,8 @@ TfuTimingAdvIndInfo *timingAdvInd;
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to get the cell for cellId"
          "=(%d)", timingAdvInd->cellId);
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
 
    node =  timingAdvInd->timingAdvLst.first;
@@ -2520,7 +2502,7 @@ TfuTimingAdvIndInfo *timingAdvInd;
       }
       rgSCHDhmUpdTa (cell, ue, timingAdvInfo->timingAdv);
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomTimingAdvInd */
 
 /**
@@ -2883,7 +2865,7 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate TfuRecpReqInfo "
          "for cell");
       err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-      RETVALUE(ret);
+      return ret;
    }    
    recpReqInfo->cellId = cell->cellId;
    cmLListInit(&recpReqInfo->ueRecpReqLst);
@@ -2900,7 +2882,7 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to fill Data recption "
          "requests for cell");
       RGSCH_FREE_MEM(recpReqInfo);
-      RETVALUE(ret);
+      return ret;
    } 
    /* Filling HARQ Reception requests */
    ret = rgSCHTomUtlFillHqFdbkRecpReq (recpReqInfo, cell, validIdx,err);
@@ -2909,14 +2891,14 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to fill Harq Feedback "
          "reception requests for cell");
       RGSCH_FREE_MEM(recpReqInfo);
-      RETVALUE(ret);
+      return ret;
    }
    /* sending the RecpReq to Phy */
    if (rgSCHUtlTfuRecpReq(inst, cell->tfuSap->sapCfg.suId, recpReqInfo) != ROK)
    {
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to send Cntrl info for cell");
    }
-   RETVALUE(ROK);
+   return ROK;
 } /* end of rgSCHTomUtlProcUlSf */ 
 
 #ifdef LTE_TDD
@@ -2964,7 +2946,7 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"rgSCHTomUtlPrcUlTddSpclSf() Unable to "
                "Allocate TfuRecpReqInfo for cell");
       err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-      RETVALUE(ret);
+      return ret;
    }    
    recpReqInfo->cellId = cell->cellId;
    cmLListInit(&recpReqInfo->ueRecpReqLst);
@@ -2991,7 +2973,7 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"rgSCHTomUtlPrcUlTddSpclSf() Unable to fill"
             " SRS recption requests for cell");;
       RGSCH_FREE_MEM(recpReqInfo);
-      RETVALUE(ret);
+      return ret;
    }
    /* sending the RecpReq to Phy */
    if (rgSCHUtlTfuRecpReq(inst, cell->tfuSap->sapCfg.suId, recpReqInfo) != ROK)
@@ -2999,7 +2981,7 @@ RgSchErrInfo       *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"rgSCHTomUtlPrcUlTddSpclSf() Unable to send "
                                "Cntrl info for cell");
    }
-   RETVALUE(ROK);
+   return ROK;
 } /* end of rgSCHTomUtlPrcUlTddSpclSf */ 
 #endif
 #endif
@@ -3082,7 +3064,7 @@ RgSchErrInfo         *err;
             RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to send PHICH info "
                      "for cellId (%d)\n", cell->cellId));
             RGSCH_FREE_MEM(cntrlInfo);
-            RETVALUE(ret);
+            return ret;
          }
          if ((ret = rgSCHTomUtlFillUlPdcch (cell, cntrlInfo, ulSf, err)) != 
                         ROK)
@@ -3090,7 +3072,7 @@ RgSchErrInfo         *err;
             RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to send PDCCH info "
                      "for cellId (%d)\n", cell->cellId));
             RGSCH_FREE_MEM(cntrlInfo);
-            RETVALUE(ret);
+            return ret;
          }
       }
 #ifdef EMTC_ENABLE
@@ -3112,7 +3094,7 @@ RgSchErrInfo         *err;
             RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to send PDCCH info "
                      "for cellId (%d)\n", cell->cellId));
             RGSCH_FREE_MEM(cntrlInfo);
-            RETVALUE(ret);
+            return ret;
          }
          rgBwAlloInfo[dlSf->sfNum] += dlSf->bwAssigned;
          rgBwAlcnt[dlSf->sfNum] ++;
@@ -3137,8 +3119,8 @@ if(0 == cntrlInfo->dlMpdcchLst.count)
          RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to send Cntrl info "
             "for cell");
       }
+   return ROK;
 
-   RETVALUE(ROK);
 } 
 
 
@@ -3195,7 +3177,7 @@ RgSchErrInfo       *err;
                                 &(cntrlInfo->memCp))) != ROK)
       {
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
       harqAck->txPower = 0;
@@ -3218,7 +3200,7 @@ RgSchErrInfo       *err;
       harqAck->lnk.node = (PTR)harqAck;
       node = node->next;
    } /* end of while */
-   RETVALUE(ret);
+   return ret;
 } /* end of */ 
 
 
@@ -3367,7 +3349,7 @@ RgSchErrInfo    *err;
                                 &(cntrlInfo->memCp))) != ROK)
       {
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
 #ifdef LTEMAC_SPS
       tfuPdcch->crnti       = pdcch->crnti;
@@ -3411,7 +3393,7 @@ RgSchErrInfo    *err;
          gDlNumUePerTti[numUePerTti-1]++;
       }
    }
-   RETVALUE(ret);
+   return ret;
 } /* end of rgSCHTomUtlFillDlPdcch*/ 
 
 #ifdef RGSCH_SPS_STATS
@@ -3504,7 +3486,7 @@ RgSchErrInfo    *err;
                   &(cntrlInfo->memCp))) != ROK)
       {
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
       tfuPdcch->rnti       = pdcch->rnti;
 #ifdef LTE_ADV
@@ -3536,7 +3518,7 @@ RgSchErrInfo    *err;
      //   abort();
    }
 #endif
-   RETVALUE(ret);
+   return ret;
 } /* end of rgSCHTomUtlFillUlPdcch*/ 
 
 /** @brief This function does the processing for Timing adjustment.
@@ -3603,8 +3585,7 @@ RgSchCellCb    *cell;
          ue->dl.taCb.numRemSf--;
       }
    } /* end of taUeLst */
-
-   RETVALUE(ROK);
+   return ROK;
 } /* end of rgSCHTomUtlProcTA */ 
 
 /** @brief This function handles filling of Hq reception request to
@@ -3717,7 +3698,7 @@ RgSchErrInfo            *err;
                RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
                         "TfuUeRecpReqInfo for cell");
                err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-               RETVALUE(ret);
+               return ret;
             }
             pucchRecpInfo->type = TFU_RECP_REQ_PUCCH;
             if ((hqCb->hqE->ue != NULLP) /*&& 
@@ -3816,7 +3797,7 @@ RgSchErrInfo            *err;
          break;
       }
    }
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillHqFdbkInfo */
 
 #ifdef RG_5GTF 
@@ -3867,17 +3848,15 @@ PRIVATE S16 rgSCHTomUtlFillHqFdbkFor5gtf (recpReqInfo, cell, validIdx, hqCb, dlS
    Inst                 inst = cell->instIdx;
 #endif
    S16                  ret;
-   CmLList              *hqPNode;
    RgSchUeCb            *ue;
    TfuUePucchRecpReq    *pucchReqInfo = NULLP;
 
-   hqPNode = dlSfHqInfo->hqPLst.first;
    ue = (RgSchUeCb*)dlSfHqInfo->dlSfUeLnk.node;
    
    if (ue == NULLP)
    {
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
    ret = rgSCHUtlGetEventMem((Ptr *)&pucchRecpInfo,
          sizeof(TfuUeRecpReqInfo), &(recpReqInfo->memCp));
    if (ret != ROK)
@@ -3885,7 +3864,7 @@ PRIVATE S16 rgSCHTomUtlFillHqFdbkFor5gtf (recpReqInfo, cell, validIdx, hqCb, dlS
       RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to Allocate "
                "TfuUeRecpReqInfo for cellId=%d \n", cell->cellId));
       err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-      RETVALUE(ret);
+      return ret;
    }
    pucchRecpInfo->type = TFU_RECP_REQ_PUCCH;
    pucchRecpInfo->rnti = ue->ueId; /* Even for Rel pdcch also setting CRNTI
@@ -3910,7 +3889,7 @@ PRIVATE S16 rgSCHTomUtlFillHqFdbkFor5gtf (recpReqInfo, cell, validIdx, hqCb, dlS
    
    cmLListAdd2Tail(&recpReqInfo->ueRecpReqLst, &(pucchRecpInfo->lnk));
    pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillHqFdbkForFrmt1B */
 #endif
 
@@ -3997,8 +3976,8 @@ RgSchErrInfo            *err;
    
    if (ue == NULLP)
    {
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
    ret = rgSCHUtlGetEventMem((Ptr *)&pucchRecpInfo,
          sizeof(TfuUeRecpReqInfo), &(recpReqInfo->memCp));
    if (ret != ROK)
@@ -4006,7 +3985,7 @@ RgSchErrInfo            *err;
       RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to Allocate "
                "TfuUeRecpReqInfo for cellId=%d \n", cell->cellId));
       err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-      RETVALUE(ret);
+      return ret;
    }
    pucchRecpInfo->type = TFU_RECP_REQ_PUCCH;
    pucchRecpInfo->rnti = ue->ueId; /* Even for Rel pdcch also setting CRNTI
@@ -4243,7 +4222,7 @@ RgSchErrInfo            *err;
 #endif/*TFU_UPGRADE*/
    cmLListAdd2Tail(&recpReqInfo->ueRecpReqLst, &(pucchRecpInfo->lnk));
    pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillHqFdbkForFrmt1B */
 /** @brief This function handles filling of Hq reception request to
  *   Per Hq Proc.
@@ -4324,8 +4303,8 @@ RgSchErrInfo            *err;
    
    if (ue == NULLP)
    {
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
    ret = rgSCHUtlGetEventMem((Ptr *)&pucchRecpInfo,
          sizeof(TfuUeRecpReqInfo), &(recpReqInfo->memCp));
    if (ret != ROK)
@@ -4333,7 +4312,7 @@ RgSchErrInfo            *err;
       RGSCHDBGERRNEW(inst,(rgSchPBuf(inst),"Unable to Allocate "
                "TfuUeRecpReqInfo for cellId=%d \n", cell->cellId));
       err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-      RETVALUE(ret);
+      return ret;
    }
    pucchRecpInfo->type = TFU_RECP_REQ_PUCCH;
    pucchRecpInfo->rnti = ue->ueId; /* Even for Rel pdcch also setting CRNTI
@@ -4364,7 +4343,7 @@ RgSchErrInfo            *err;
 #endif/*TFU_UPGRADE*/
    cmLListAdd2Tail(&recpReqInfo->ueRecpReqLst, &(pucchRecpInfo->lnk));
    pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillHqFdbkForFrmt3 */
 
 #endif/*LTE_ADV*/
@@ -4479,7 +4458,7 @@ RgSchErrInfo            *err;
 
    /* Check with TDD Code */
    /* FOR ACK NACK REP */
-   RETVALUE(ROK);
+   return ROK;
 } /* end of rgSCHTomUtlFillHqFdbkRecpReq */ 
 #ifdef TFU_UPGRADE 
 /** @brief This function handles filling of SR reception request to
@@ -4546,7 +4525,7 @@ PRIVATE S16 rgSCHTomUtlFillSrRecpReq (recpReqInfo, cell, validIdx, err)
          RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
             "TfuUeRecpReqInfo for cell RNTI:%d",ue->ueId);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
      cmMemset((U8 *)&pucchRecpInfo->t.pucchRecpReq, 0, sizeof(TfuUePucchRecpReq));
@@ -4585,7 +4564,7 @@ PRIVATE S16 rgSCHTomUtlFillSrRecpReq (recpReqInfo, cell, validIdx, err)
       }
       pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
    }
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillSrRecpReq */
 #endif
 #endif
@@ -4633,8 +4612,8 @@ PRIVATE S16 rgSCHTomUtlWillUeRprtCqiRi ( ue, willueRprtCqiRi)
          {/*onDuration NOT running, do not expect cqi/pmi/ri*/
             *willueRprtCqiRi = FALSE;
          }
-         RETVALUE(ROK);
-      }
+         return ROK;
+               }
 #endif /*end of LTEMAC_R9*/
       /* ccpu00134258: Fix for CQI DRX issue*/
       if(ue->drxCb->onDurTmrLen > 2)
@@ -4645,8 +4624,7 @@ PRIVATE S16 rgSCHTomUtlWillUeRprtCqiRi ( ue, willueRprtCqiRi)
          }
       }
    }/*ue->isDrxEnabled*/
-   
-   RETVALUE(ROK);
+   return ROK; 
 } /*End of rgSCHTomUtlWillUeRprtCqiRi*/
 
 /** @brief This function handles filling of RI reception request to
@@ -4733,7 +4711,7 @@ PRIVATE S16 rgSCHTomUtlFillRiRecpReq (recpReqInfo, cell, validIdx, err)
             RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
                "TfuUeRecpReqInfo for cell RNTI:%d",ue->ueId);
             err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-            RETVALUE(ret);
+            return ret;
          }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
          cmMemset((U8 *)&pucchRecpInfo->t.pucchRecpReq, 0, sizeof(TfuUePucchRecpReq));
@@ -4768,7 +4746,7 @@ PRIVATE S16 rgSCHTomUtlFillRiRecpReq (recpReqInfo, cell, validIdx, err)
          riCb->riDist--; 
       }     
    }
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillRiRecpReq */
 #endif
 #ifdef RG_5GTF
@@ -4823,7 +4801,7 @@ PRIVATE S16 rgSCHTomUtlFillCqiRiRecpReq (recpReqInfo, cell, validIdx, err)
             RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
                "TfuUeRecpReqInfo for cell RNTI:%d ", ue->ueId);
             err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-            RETVALUE(ret);
+            return ret;
          }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
          cmMemset((U8 *)&pucchRecpInfo->t.pucchRecpReq, 0, sizeof(TfuUePucchRecpReq));
@@ -4840,8 +4818,7 @@ PRIVATE S16 rgSCHTomUtlFillCqiRiRecpReq (recpReqInfo, cell, validIdx, err)
          pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
       }
    }
-
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillCqiRiRecpReq */
 #endif
 #endif
@@ -4931,7 +4908,7 @@ PRIVATE S16 rgSCHTomUtlFillPcqiRecpReq (recpReqInfo, cell, validIdx, err)
          RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
             "TfuUeRecpReqInfo for cell RNTI:%d ", ue->ueId);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
       cmMemset((U8 *)&pucchRecpInfo->t.pucchRecpReq, 0, sizeof(TfuUePucchRecpReq));
@@ -4958,7 +4935,7 @@ PRIVATE S16 rgSCHTomUtlFillPcqiRecpReq (recpReqInfo, cell, validIdx, err)
       }
       pucchRecpInfo->lnk.node = (PTR)pucchRecpInfo;
    }
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillPcqiRecpReq */
 /** @brief This function handles filling of SRS reception request to
  * PHY.
@@ -5024,7 +5001,7 @@ PRIVATE S16 rgSCHTomUtlFillSrsRecpReq (recpReqInfo, cell, validIdx, err)
             RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
                "TfuUeRecpReqInfo for RNTI:%d ",ue->ueId);
             err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-            RETVALUE(ret);
+            return ret;
          }
 
 #ifdef TFU_ALLOC_EVENT_NO_INIT
@@ -5056,7 +5033,7 @@ PRIVATE S16 rgSCHTomUtlFillSrsRecpReq (recpReqInfo, cell, validIdx, err)
          ue->srsCb.srsDist--; 
       }
    }
-   RETVALUE(ROK);
+   return ROK;
 }/* end of rgSCHTomUtlFillSrsRecpReq */
 #endif 
 #ifndef TFU_UPGRADE
@@ -5122,7 +5099,7 @@ RgSchErrInfo         *err;
          RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
             "TfuUeRecpReqInfo for RNTI:%d ", alloc->ue->ueId);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
       if (!alloc->forMsg3)
       {
@@ -5167,7 +5144,7 @@ RgSchErrInfo         *err;
       
       alloc = rgSCHUtlNextRcptnReq (cell, alloc);
    } /* end of while */
-   RETVALUE(ROK); 
+   return ROK;
 } /* end of rgSCHTomUtlFillDatRecpReq */
 
 #else
@@ -5210,7 +5187,6 @@ PRIVATE S16 rgSCHTomUtlFillDatRecpReq (recpReqInfo, cell, validIdx, err)
    RgSchUlAlloc      *alloc;
    TfuUeRecpReqInfo  *datRecpInfo;
 
-   Bool              hqPres;   /*Set when HARQ Rec Req is present*/
    Bool              isAperiodic = FALSE; /*Set when Aperiodic CQI is expected */
    U8                numUePerTti = 0;
 
@@ -5236,15 +5212,13 @@ PRIVATE S16 rgSCHTomUtlFillDatRecpReq (recpReqInfo, cell, validIdx, err)
          RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Allocate "
             "TfuUeRecpReqInfo for RNTI:%d ", alloc->rnti);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
 #ifdef TFU_ALLOC_EVENT_NO_INIT
       datRecpInfo->t.puschRecpReq.initialNSrs.pres = FALSE;
       datRecpInfo->t.puschRecpReq.initialNumRbs.pres = FALSE;
 #endif
       datRecpInfo->type = TFU_RECP_REQ_PUSCH;
-      /* ccpu00131944 - Intializing hqPres in each iteration*/
-      hqPres = FALSE;
       /* Check if this if for MSG3 - no scope for feedback along with it. */
       if ((FALSE == alloc->forMsg3))
       {
@@ -5304,7 +5278,7 @@ PRIVATE S16 rgSCHTomUtlFillDatRecpReq (recpReqInfo, cell, validIdx, err)
       cell->ulNumUeSchedPerTti[numUePerTti-1]++;
       gUlNumUePerTti[numUePerTti - 1]++;
    }
-   RETVALUE(ROK); 
+   return ROK;
 } /* end of rgSCHTomUtlFillDatRecpReq */
 #endif
 /* rg009.201. Added changes of TFU_UPGRADE */
@@ -5340,8 +5314,8 @@ PUBLIC S16 rgSCHTomUtlFillRiBitWidthInfo(ueCb)
    if (ueCb->mimoInfo.txMode != RGR_UE_TM_3 &&
              ueCb->mimoInfo.txMode != RGR_UE_TM_4)
    {
-      RETVALUE(RFAILED);
-   }
+      return RFAILED;
+       }
 
    ueCb->rawCqiBitW[ueCb->cqiRiWritIdx].type = TFU_RECP_REQ_PUCCH;
    ueCb->rawCqiBitW[ueCb->cqiRiWritIdx].u.pucch.pucchRawCqiInfo.mode= 
@@ -5373,7 +5347,7 @@ PUBLIC S16 rgSCHTomUtlFillRiBitWidthInfo(ueCb)
    }
    
    RG_SCH_INCR_CQIRI_INDEX(ueCb->cqiRiWritIdx);
-   RETVALUE(ROK);
+   return ROK;
 }
 
 /***********************************************************
@@ -5523,7 +5497,7 @@ PUBLIC U8 rgSCHTomUtlFetchPcqiBitSz(ueCb, numTxAnt, ri)
    }
    
    RG_SCH_INCR_CQIRI_INDEX(ueCb->cqiRiWritIdx);
-   RETVALUE(pcqiSz);
+   return pcqiSz;
 }
 
 
@@ -5575,7 +5549,7 @@ PUBLIC S16 rgSCHTomUtlPcqiSbCalcBpIdx(crntTimInfo, ueCb, cqiCb)
       cqiCb->prioLvl = RG_SCH_CQI_PRIO_LVL_0;
 #endif
    }
-   RETVALUE(ROK);
+   return ROK;
 }
 
 
@@ -5674,7 +5648,7 @@ PUBLIC S16 rgSCHTomUtlMoveNxtOccasion(cell, ue, validIdx)
    {
       rgSCHTomUtlMoveSrNxtOccasion(cell, ue); 
    }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlMoveNxtOccasion */
 
 
@@ -5969,7 +5943,7 @@ PUBLIC S16 rgSCHTomUtlFillDatAperRecpReq(cell, cqiReq, alloc, datRecpInfo, timeI
    datRecpInfo->rnti = alloc->rnti;
    rgSCHUtlAllocRcptInfo (cell, alloc, timeInfo,
             &datRecpInfo->t.puschRecpReq.ulSchInfo);
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillDatAperRecpReq */
 
 
@@ -6065,7 +6039,7 @@ hqPres, validIdx)
    datRecpInfo->rnti = alloc->rnti;
    rgSCHUtlAllocRcptInfo (cell, alloc, timeInfo,
          &datRecpInfo->t.puschRecpReq.ulSchInfo);
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillDatPriRecpReq */
 
 
@@ -6135,7 +6109,7 @@ PUBLIC S16 rgSCHTomUtlFillDatPCqiRecpReq(cell, alloc, datRecpInfo,
    {
       RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"Unable to Fill "
          "CqiPmi size RNTI:%d",alloc->rnti);
-      RETVALUE(RFAILED);
+      return RFAILED;
    }
 
    /* Fill only the first RI index since Periodic can come 
@@ -6182,7 +6156,7 @@ PUBLIC S16 rgSCHTomUtlFillDatPCqiRecpReq(cell, alloc, datRecpInfo,
    datRecpInfo->rnti = alloc->rnti;
    rgSCHUtlAllocRcptInfo (cell, alloc, timeInfo,
             &datRecpInfo->t.puschRecpReq.ulSchInfo);
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillDatPCqiRecpReq */
 
 /**
@@ -6244,7 +6218,7 @@ PUBLIC S16 rgSCHTomUtlFillDatSrsRecpReq(cell, alloc, datRecpInfo, timeInfo,
    datRecpInfo->rnti = alloc->rnti;
    rgSCHUtlAllocRcptInfo (cell, alloc, timeInfo, 
             &datRecpInfo->t.puschRecpReq.ulSchInfo);
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillDatSrsRecpReq */
 
 /**
@@ -6300,7 +6274,7 @@ PUBLIC S16 rgSCHTomFillOnlySrsRecpReq(cell, alloc, datRecpInfo)
     * Refer Section 5.2.2.6 of TS 36.212 V8.5.0*/
    datRecpInfo->t.puschRecpReq.ulSchInfo.nSrs = 1;
 
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomFillOnlySrsRecpReq */
 
 /**
@@ -6539,7 +6513,7 @@ Bool              isDatPresOnSecCell;
             {
                RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"RNTI:%d "
                   "Unable to Fill CqiPmi size", ue->ueId);
-               RETVALUE(RFAILED);
+               return RFAILED;
             }
             if(pucchRecpInfo->t.pucchRecpReq.uciInfo == TFU_PUCCH_HARQ_SR)
             {
@@ -6623,7 +6597,7 @@ Bool              isDatPresOnSecCell;
       }
    }            
    UNUSED(dropCqi);
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillCqiSrSrsWithHq */
 
 /**
@@ -6750,8 +6724,8 @@ U16               validIdx;
             {
                RLOG_ARG1(L_ERROR,DBG_CELLID,cell->cellId,"RNTI:%d"
                   " Unable to Fill CqiPmi size", ue->ueId);
-               RETVALUE(RFAILED);
-            }
+               return RFAILED;
+                               }
 
             pucchRecpInfo->t.pucchRecpReq.uciInfo = TFU_PUCCH_SR_CQI;
          }
@@ -6812,7 +6786,7 @@ U16               validIdx;
 
          }
       }
-   RETVALUE(ROK);
+   return ROK;
 }  /* rgSCHTomUtlFillCqiSrsWithSr */
 
 #endif 
@@ -6948,7 +6922,7 @@ noFdbks, memCp, elemIdx, nxtDlsf)
             RLOG_ARG1(L_ERROR,DBG_CELLID,cellCb->cellId,"Unable to"
                "Allocate TfuUeRecpReqInfo for RNTI:%d ", ueCb->ueId);
             err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-            RETVALUE(ret);
+            return ret;
          }
          pucchRecpInfo->rnti    =  ueCb->ueId;
 #ifndef TFU_UPGRADE
@@ -6998,8 +6972,7 @@ noFdbks, memCp, elemIdx, nxtDlsf)
       /* Go to the next node */
       node = node->next;
    }
-
-   RETVALUE(ROK);
+   return ROK;
 }
 
 /** @brief This function handles filling of HARQ feedback recption request
@@ -7133,8 +7106,8 @@ RgSchDlHqProcCb         *prvHqCb;
                if((anInfo == NULLP) || 
                      (anInfo->latestMIdx != dlSf->dlFdbkInfo.m))
                {
-                  RETVALUE(ROK);
-               }
+                                          return ROK; 
+                                       }
             }
             else
             {
@@ -7156,7 +7129,7 @@ RgSchDlHqProcCb         *prvHqCb;
          }
          else
          {
-            RETVALUE(ROK);
+                          return ROK;
          }
 
          /* Do not proceed if PUSCH
@@ -7195,7 +7168,7 @@ RgSchDlHqProcCb         *prvHqCb;
                   n1PucchTkn, &alloc, hqSz);
             if (ret != ROK)
             {
-               RETVALUE(ret);
+               return ret;
             }
             /* TODO:: In case of F1BCS and CSI in same subframe
              * UE shall drop the CSI if there was at least one 
@@ -7275,7 +7248,7 @@ RgSchDlHqProcCb         *prvHqCb;
          prvHqCb = hqCb;
       }
    }
-   RETVALUE(ROK);
+   return ROK;
 }
 
 #ifdef LTE_ADV
@@ -7794,7 +7767,7 @@ PRIVATE S16 rgSCHTomUtlFillSfHqFdbkForFrmt1BCS(hqCb,hqRecpReq,noFdbks,dlSf,pdcch
       default:
          break;
    }
-   RETVALUE(ROK);
+   return ROK;
 }
 #endif
 
@@ -7881,7 +7854,7 @@ U8                      hqSz;
          RLOG_ARG1(L_ERROR,DBG_CELLID,cellCb->cellId,"Unable to "
             "Allocate TfuUeRecpReqInfo for cell RNTI:%d",rnti);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
       alloc = TRUE;
 #ifdef TFU_ALLOC_EVENT_NO_INIT
@@ -7896,7 +7869,7 @@ U8                      hqSz;
          RLOG_ARG1(L_ERROR,DBG_CELLID,cellCb->cellId,"Unable to "
             "Allocate TfuUeRecpReqInfo for cell RNTI:%d",rnti);
          err->errCause = RGSCHERR_TOM_MEM_EXHAUST;
-         RETVALUE(ret);
+         return ret;
       }
       cmMemset((U8 *)pucchInfo->pucchRecpInfo->t.pucchRecpReq.hqInfo.hqRes,0xff,sizeof(U16)*TFU_MAX_HQ_RES);
 #ifdef TFU_ALLOC_EVENT_NO_INIT
@@ -7961,8 +7934,7 @@ U8                      hqSz;
                     because of kworks*/
                    RGSCHDBGERRNEW(cellCb->instIdx,(rgSchPBuf(cellCb->instIdx),
                             "ANInfo should not be NULL for cellId=%d \n", cellCb->cellId));
-                   RETVALUE(RFAILED);
-
+                   return RFAILED;
                 }
 #endif
              }
@@ -7971,7 +7943,7 @@ U8                      hqSz;
        {/* This needs to be revisited while
            adding support for PUCCH format 3 */
           RGSCHDBGERRNEW(cellCb->instIdx,(rgSchPBuf(cellCb->instIdx),"Invalid Pucch format configured.."));
-          RETVALUE(RFAILED);
+          return RFAILED;
        }
     }
     else
@@ -8123,7 +8095,7 @@ U8                      hqSz;
 
    *pucchInfoRef = pucchInfo;
    *allocRef     = alloc;
-   RETVALUE(ROK);
+   return ROK;
 }
 #endif
 
@@ -8188,7 +8160,7 @@ RgSchErrInfo         *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to send PHICH info for "
             "cell");
       RGSCH_FREE_MEM(cntrlInfo);
-      RETVALUE(ret);
+      return ret;
    }
 
    /* Fill UL Pdcch */
@@ -8197,7 +8169,7 @@ RgSchErrInfo         *err;
       RLOG_ARG0(L_ERROR,DBG_CELLID,cell->cellId,"Unable to send PDCCH info for "
             "cell");
       RGSCH_FREE_MEM(cntrlInfo);
-      RETVALUE(ret);
+      return ret;
    }
    
 #ifdef EMTC_ENABLE
@@ -8224,8 +8196,7 @@ RgSchErrInfo         *err;
    {
       RGSCH_FREE_MEM(cntrlInfo);
    }
-
-   RETVALUE(ROK);
+   return ROK;
 } /* end of */ 
 #endif /* #ifdef RG_ULSCHED_AT_CRC*/
 
@@ -8295,7 +8266,7 @@ RgSchCellCb          *cell;
  *     Function: rgSCHDynTDDMrkCrntSfIdx 
  *     Purpose:  update the dyn tdd sunframe index
  *  @param[in]  Inst   schInst
- *  @RETVALUE   None
+ *  @return    None
  */
 #ifdef UNUSED_FUNC
 #ifdef ANSI