@@ -194,8 +194,7 @@ DSP_STATUS CFG_GetExecFile(struct CFG_DEVNODE *hDevNode, u32 ulBufSize,
ulBufSize, pstrExecFile);
if (!hDevNode)
status = CFG_E_INVALIDHDEVNODE;
-
- if (!pstrExecFile)
+ else if (!pstrExecFile)
status = CFG_E_INVALIDPOINTER;
if (DSP_SUCCEEDED(status)) {
@@ -277,9 +276,13 @@ DSP_STATUS CFG_GetObject(OUT u32 *pdwValue, u32 dwType)
switch (dwType) {
case (REG_DRV_OBJECT):
status = REG_GetValue(DRVOBJECT, (u8 *)pdwValue, &dwBufSize);
+ if (DSP_FAILED(status))
+ status = CFG_E_RESOURCENOTAVAIL;
break;
case (REG_MGR_OBJECT):
status = REG_GetValue(MGROBJECT, (u8 *)pdwValue, &dwBufSize);
+ if (DSP_FAILED(status))
+ status = CFG_E_RESOURCENOTAVAIL;
break;
default:
break;
@@ -289,7 +292,6 @@ DSP_STATUS CFG_GetObject(OUT u32 *pdwValue, u32 dwType)
"CFG_GetObject SUCCESS DrvObject: "
"0x%x\n ", *pdwValue);
} else {
- status = CFG_E_RESOURCENOTAVAIL;
*pdwValue = 0;
GT_0trace(CFG_debugMask, GT_6CLASS, "CFG_GetObject Failed \n");
}
@@ -173,9 +173,7 @@ DSP_STATUS CLK_Enable(IN enum SERVICES_ClkId clk_id)
pClk = SERVICES_Clks[clk_id].clk_handle;
if (pClk) {
- if (clk_enable(pClk) == 0x0) {
- /* Success ? */
- } else {
+ if (clk_enable(pClk)) {
pr_err("CLK_Enable: failed to Enable CLK %s, "
"CLK dev id = %s\n",
SERVICES_Clks[clk_id].clk_name,
@@ -209,8 +207,7 @@ DSP_STATUS CLK_Set_32KHz(IN enum SERVICES_ClkId clk_id)
DSP_STATUS status = DSP_SOK;
struct clk *pClk;
struct clk *pClkParent;
- enum SERVICES_ClkId sys_32k_id = SERVICESCLK_sys_32k_ck;
- pClkParent = SERVICES_Clks[sys_32k_id].clk_handle;
+ pClkParent = SERVICES_Clks[SERVICESCLK_sys_32k_ck].clk_handle;
DBC_Require(clk_id < SERVICESCLK_NOT_DEFINED);
GT_2trace(CLK_debugMask, GT_6CLASS, "CLK_Set_32KHz: CLK %s, "
@@ -52,17 +52,14 @@ static unsigned int crefs; /* module counter */
DSP_STATUS REG_DeleteValue(IN CONST char *pstrValue)
{
DSP_STATUS status;
- DBC_Require(strlen(pstrValue) < REG_MAXREGPATHLENGTH);
+ DBC_Require(strlen(pstrValue) < REG_MAXREGPATHLENGTH);
GT_0trace(REG_debugMask, GT_ENTER, "REG_DeleteValue: entered\n");
SYNC_EnterCS(reglock);
- if (regsupDeleteValue(pstrValue) == DSP_SOK)
- status = DSP_SOK;
- else
- status = DSP_EFAIL;
-
+ status = regsupDeleteValue(pstrValue);
SYNC_LeaveCS(reglock);
+
return status;
}
@@ -169,18 +166,17 @@ DSP_STATUS REG_SetValue(IN CONST char *pstrValue, IN u8 *pbData,
DBC_Require(pstrValue && pbData);
DBC_Require(dwDataSize > 0);
- DBC_Require(strlen(pstrValue) < REG_MAXREGPATHLENGTH);
+ DBC_Require(strlen(pstrValue) < REG_MAXREGPATHLENGTH);
SYNC_EnterCS(reglock);
- /* We need to use regsup calls... */
- /* ...for now we don't need the key handle or */
- /* the subkey, all we need is the value to lookup. */
- if (regsupSetValue((char *)pstrValue, pbData, dwDataSize) == DSP_SOK)
- status = DSP_SOK;
- else
- status = DSP_EFAIL;
-
+ /*
+ * We need to use regsup calls
+ * for now we don't need the key handle or
+ * the subkey, all we need is the value to lookup.
+ */
+ status = regsupSetValue((char *)pstrValue, pbData, dwDataSize);
SYNC_LeaveCS(reglock);
+
return status;
}
@@ -187,7 +187,6 @@ DSP_STATUS SYNC_ResetEvent(struct SYNC_OBJECT *hEvent)
if (MEM_IsValidHandle(hEvent, SIGNATURE)) {
pEvent->state = so_reset;
- status = DSP_SOK;
} else {
status = DSP_EHANDLE;
GT_1trace(SYNC_debugMask, GT_6CLASS,