ASoC: soc-ops: cleanup cppcheck warning at snd_soc_put_xr_sx()
This patch cleanups below cppcheck warning. sound/soc/soc-ops.c:859:18: style: The scope of the variable 'regval' can be reduced. [variableScope] unsigned int i, regval, regmask; ^ sound/soc/soc-ops.c:859:26: style: The scope of the variable 'regmask' can be reduced. [variableScope] unsigned int i, regval, regmask; ^ sound/soc/soc-ops.c:860:6: style: The scope of the variable 'err' can be reduced. [variableScope] int err; ^ Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Link: https://lore.kernel.org/r/87zgtzunoz.wl-kuninori.morimoto.gx@renesas.com Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
b1ebecb90b
commit
b285b51018
|
@ -856,17 +856,16 @@ int snd_soc_put_xr_sx(struct snd_kcontrol *kcontrol,
|
|||
unsigned long mask = (1UL<<mc->nbits)-1;
|
||||
long max = mc->max;
|
||||
long val = ucontrol->value.integer.value[0];
|
||||
unsigned int i, regval, regmask;
|
||||
int err;
|
||||
unsigned int i;
|
||||
|
||||
if (invert)
|
||||
val = max - val;
|
||||
val &= mask;
|
||||
for (i = 0; i < regcount; i++) {
|
||||
regval = (val >> (regwshift*(regcount-i-1))) & regwmask;
|
||||
regmask = (mask >> (regwshift*(regcount-i-1))) & regwmask;
|
||||
err = snd_soc_component_update_bits(component, regbase+i,
|
||||
regmask, regval);
|
||||
unsigned int regval = (val >> (regwshift*(regcount-i-1))) & regwmask;
|
||||
unsigned int regmask = (mask >> (regwshift*(regcount-i-1))) & regwmask;
|
||||
int err = snd_soc_component_update_bits(component, regbase+i,
|
||||
regmask, regval);
|
||||
if (err < 0)
|
||||
return err;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue