From: Pan Bian Date: Mon, 28 Nov 2016 23:33:07 +0000 (+0800) Subject: ALSA: echoaudio: Fix improper return value in function load_asic X-Git-Tag: rel_imx_4.19.35_1.1.0~13556^2~11 X-Git-Url: https://git.somdevices.com/?a=commitdiff_plain;h=e59d8bb574f6d8097e7e14981440dc37425fddc6;p=linux.git ALSA: echoaudio: Fix improper return value in function load_asic When the second call to load_asic_generic() fails in function load_asic(), "false" is returned. The real value of "false" is 0, which indicates success in the context. As a result, the execution status and the return value may be inconsistent. This patch fixes the bug. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188761 Signed-off-by: Pan Bian Signed-off-by: Takashi Iwai --- diff --git a/sound/pci/echoaudio/layla24_dsp.c b/sound/pci/echoaudio/layla24_dsp.c index df28e5117359..c02bc1dcc170 100644 --- a/sound/pci/echoaudio/layla24_dsp.c +++ b/sound/pci/echoaudio/layla24_dsp.c @@ -135,7 +135,7 @@ static int load_asic(struct echoaudio *chip) err = load_asic_generic(chip, DSP_FNC_LOAD_LAYLA24_EXTERNAL_ASIC, FW_LAYLA24_2S_ASIC); if (err < 0) - return false; + return err; /* Now give the external ASIC a little time to set up */ mdelay(10);