msm: audio: qdsp5: Fix for logical errors in qdsp5
Add missing break statement in AUDIO_FLUSH ioctl in audio_out driver.
Add missing braces for the conditional check for buffer size in
AUDIO_SET_STREAM_CONFIG ioctl for AMR NB, EVRC and QCELP drivers.
Correct the loop terminating logic in audpcm_in_flush() function.
Add break statement in AUDPP_MSG_FLUSH_ACK ioctl in WMA and WMAPro
drivers.
CRs-Fixed: 355773
Change-Id: Iba07194c9a7332ff45490533f2ca09ab6a8742aa
Signed-off-by: Phani Kumar Allada <pallad@codeaurora.org>
diff --git a/arch/arm/mach-msm/qdsp5/audio_amrnb_in.c b/arch/arm/mach-msm/qdsp5/audio_amrnb_in.c
index d4e4893..bec1d4c 100644
--- a/arch/arm/mach-msm/qdsp5/audio_amrnb_in.c
+++ b/arch/arm/mach-msm/qdsp5/audio_amrnb_in.c
@@ -821,13 +821,15 @@
}
/* Allow only single frame */
if (audio->mode == MSM_AUD_ENC_MODE_TUNNEL) {
- if (cfg.buffer_size != (FRAME_SIZE - 8))
+ if (cfg.buffer_size != (FRAME_SIZE - 8)) {
rc = -EINVAL;
break;
+ }
} else {
- if (cfg.buffer_size != (AMRNB_FRAME_SIZE + 14))
+ if (cfg.buffer_size != (AMRNB_FRAME_SIZE + 14)) {
rc = -EINVAL;
break;
+ }
}
audio->buffer_size = cfg.buffer_size;
break;