Looks as though the result of snd_soc_update_bits() has never been checked.

Fixes the following W=1 kernel build warning(s):

 sound/soc/codecs/ak4458.c: In function ‘ak4458_set_dai_mute’:
 sound/soc/codecs/ak4458.c:408:16: warning: variable ‘ret’ set but not used 
[-Wunused-but-set-variable]

Cc: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Cc: Junichi Wakasugi <wakasugi...@om.asahi-kasei.co.jp>
Cc: Mihai Serban <mihai.ser...@nxp.com>
Signed-off-by: Lee Jones <lee.jo...@linaro.org>
---
 sound/soc/codecs/ak4458.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/codecs/ak4458.c b/sound/soc/codecs/ak4458.c
index f180cb5dfe4f1..39ae089dcd1d8 100644
--- a/sound/soc/codecs/ak4458.c
+++ b/sound/soc/codecs/ak4458.c
@@ -405,7 +405,7 @@ static int ak4458_set_dai_mute(struct snd_soc_dai *dai, int 
mute)
 {
        struct snd_soc_component *component = dai->component;
        struct ak4458_priv *ak4458 = snd_soc_component_get_drvdata(component);
-       int nfs, ndt, ret, reg;
+       int nfs, ndt, reg;
        int ats;
 
        nfs = ak4458->fs;
@@ -416,14 +416,14 @@ static int ak4458_set_dai_mute(struct snd_soc_dai *dai, 
int mute)
        ndt = att_speed[ats] / (nfs / 1000);
 
        if (mute) {
-               ret = snd_soc_component_update_bits(component, 
AK4458_01_CONTROL2,  0x01, 1);
+               snd_soc_component_update_bits(component, AK4458_01_CONTROL2,  
0x01, 1);
                mdelay(ndt);
                if (ak4458->mute_gpiod)
                        gpiod_set_value_cansleep(ak4458->mute_gpiod, 1);
        } else {
                if (ak4458->mute_gpiod)
                        gpiod_set_value_cansleep(ak4458->mute_gpiod, 0);
-               ret = snd_soc_component_update_bits(component, 
AK4458_01_CONTROL2, 0x01, 0);
+               snd_soc_component_update_bits(component, AK4458_01_CONTROL2, 
0x01, 0);
                mdelay(ndt);
        }
 
-- 
2.25.1

Reply via email to