projects
/
linux-2.6-microblaze.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
2dc15ff
)
ALSA: hda: cs35l41: Fix missing error code in cs35l41_smart_amp()
author
Harshit Mogalapalli
<harshit.m.mogalapalli@oracle.com>
Mon, 30 Oct 2023 07:08:36 +0000
(
00:08
-0700)
committer
Takashi Iwai
<tiwai@suse.de>
Tue, 31 Oct 2023 17:30:16 +0000
(18:30 +0100)
When firmware status is invalid, assign -EINVAL to ret as ret is '0' at
that point and returning success is incorrect when firmware status is
invalid.
Fixes:
a51d8ba03a4f
("ALSA: hda: cs35l41: Check CSPL state after loading firmware")
Signed-off-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Link:
https://lore.kernel.org/r/20231030070836.3234385-1-harshit.m.mogalapalli@oracle.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/cs35l41_hda.c
patch
|
blob
|
history
diff --git
a/sound/pci/hda/cs35l41_hda.c
b/sound/pci/hda/cs35l41_hda.c
index
98a5123
..
b2db809
100644
(file)
--- a/
sound/pci/hda/cs35l41_hda.c
+++ b/
sound/pci/hda/cs35l41_hda.c
@@
-1042,6
+1042,7
@@
static int cs35l41_smart_amp(struct cs35l41_hda *cs35l41)
default:
dev_err(cs35l41->dev, "Firmware status is invalid: %u\n",
fw_status);
+ ret = -EINVAL;
goto clean_dsp;
}