From: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx>
Pointless dead code path - there is nothing to fail before the
DSP_SUCCEEDED check.
Signed-off-by: Phil Carmody <ext-phil.2.carmody@xxxxxxxxx>
Signed-off-by: Hiroshi DOYU <Hiroshi.DOYU@xxxxxxxxx>
---
drivers/dsp/bridge/pmgr/wcd.c | 11 ++++-------
1 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers/dsp/bridge/pmgr/wcd.c b/drivers/dsp/bridge/pmgr/wcd.c
index ee15b02..9c3fe6f 100644
--- a/drivers/dsp/bridge/pmgr/wcd.c
+++ b/drivers/dsp/bridge/pmgr/wcd.c
@@ -709,14 +709,11 @@ u32 PROCWRAP_Ctrl(union Trapped_Args *args)
goto func_end;
}
cbDataSize += sizeof(u32);
- if (DSP_SUCCEEDED(status)) {
- pArgs = MEM_Alloc(cbDataSize, MEM_NONPAGED);
- if (pArgs == NULL) {
- status = DSP_EMEMORY;
- goto func_end;
- }
- } else
+ pArgs = MEM_Alloc(cbDataSize, MEM_NONPAGED);
+ if (pArgs == NULL) {
+ status = DSP_EMEMORY;
goto func_end;
+ }
cp_fm_usr(pArgs, args->ARGS_PROC_CTRL.pArgs, status,
cbDataSize);
--
1.6.0.4
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
