> -----Original Message----- > From: Colin King [mailto:colin.king@xxxxxxxxxxxxx] > Sent: Thursday, April 19, 2018 10:35 PM > To: Bard Liao; Oder Chiou; Liam Girdwood; Mark Brown; Jaroslav Kysela; > Takashi Iwai; alsa-devel@xxxxxxxxxxxxxxxx > Cc: kernel-janitors@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: [PATCH][next] ASoC: rt5668: fix incorrect 'and' operator > > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Currently logical and is being used instead of bitwise and. Fix this. > > Detected by CoverityScan, CID#1468008 ("Logical vs bitwise operator") > > Fixes: d59fb2856223 ("ASoC: rt5668: add rt5668B codec driver") > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Acked-By: Bard Liao <bardliao@xxxxxxxxxxx> > --- > sound/soc/codecs/rt5668.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/codecs/rt5668.c b/sound/soc/codecs/rt5668.c > index 52a343f96eb2..3c19d03f2446 100644 > --- a/sound/soc/codecs/rt5668.c > +++ b/sound/soc/codecs/rt5668.c > @@ -1194,7 +1194,7 @@ static int set_filter_clk(struct snd_soc_dapm_widget > *w, > int ref, val, reg, idx = -EINVAL; > static const int div[] = {1, 2, 3, 4, 6, 8, 12, 16, 24, 32, 48}; > > - val = snd_soc_component_read32(component, RT5668_GPIO_CTRL_1) > && > + val = snd_soc_component_read32(component, RT5668_GPIO_CTRL_1) & > RT5668_GP4_PIN_MASK; > if (w->shift == RT5668_PWR_ADC_S1F_BIT && > val == RT5668_GP4_PIN_ADCDAT2) > -- > 2.17.0 > > > ------Please consider the environment before printing this e-mail. ��.n��������+%������w��{.n����z�ޗ�����n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�
