lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 7 Jul 2020 15:37:35 +0100 From: Lee Jones <lee.jones@...aro.org> To: broonie@...nel.org, lgirdwood@...il.com, perex@...ex.cz, tiwai@...e.com Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, alsa-devel@...a-project.org, Lee Jones <lee.jones@...aro.org>, Peter Ujfalusi <peter.ujfalusi@...com>, Jarkko Nikula <jarkko.nikula@...mer.com>, Samuel Ortiz <samuel.ortiz@...ia.com>, linux-omap@...r.kernel.org Subject: [PATCH v2 21/28] ASoC: ti: omap-mcbsp-st: Remove set, but unused variable 'w' Looks like 'w' has remained unchecked since the driver's inception. Fixes the following W=1 kernel build warning(s): sound/soc/ti/omap-mcbsp-st.c: In function ‘omap_mcbsp_st_chgain’: sound/soc/ti/omap-mcbsp-st.c:145:6: warning: variable ‘w’ set but not used [-Wunused-but-set-variable] Peter suggested that the whole read can be removed, so that's been done too. Cc: Peter Ujfalusi <peter.ujfalusi@...com> Cc: Jarkko Nikula <jarkko.nikula@...mer.com> Cc: Samuel Ortiz <samuel.ortiz@...ia.com> Cc: linux-omap@...r.kernel.org Signed-off-by: Lee Jones <lee.jones@...aro.org> --- sound/soc/ti/omap-mcbsp-st.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/sound/soc/ti/omap-mcbsp-st.c b/sound/soc/ti/omap-mcbsp-st.c index 5a32b54bbf3bb..0bc7d26c660aa 100644 --- a/sound/soc/ti/omap-mcbsp-st.c +++ b/sound/soc/ti/omap-mcbsp-st.c @@ -142,11 +142,8 @@ static void omap_mcbsp_st_fir_write(struct omap_mcbsp *mcbsp, s16 *fir) static void omap_mcbsp_st_chgain(struct omap_mcbsp *mcbsp) { - u16 w; struct omap_mcbsp_st_data *st_data = mcbsp->st_data; - w = MCBSP_ST_READ(mcbsp, SSELCR); - MCBSP_ST_WRITE(mcbsp, SGAINCR, ST_CH0GAIN(st_data->ch0gain) | ST_CH1GAIN(st_data->ch1gain)); } -- 2.25.1
Powered by blists - more mailing lists