Commit 79b094c9 authored by Lee Jones's avatar Lee Jones Committed by Mark Brown

ASoC: ux500: ux500_msp_i2s: Remove unused variables 'reg_val_DR' and 'reg_val_TSTDR'

Looks like these have been unchecked since the driver's inception in 2012.

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

sound/soc/ux500/ux500_msp_i2s.c: In function ‘flush_fifo_rx’:
sound/soc/ux500/ux500_msp_i2s.c:398:6: warning: variable ‘reg_val_DR’
set but not used [-Wunused-but-set-variable]
sound/soc/ux500/ux500_msp_i2s.c: In function ‘flush_fifo_tx’:
sound/soc/ux500/ux500_msp_i2s.c:415:6: warning: variable
‘reg_val_TSTDR’ set but not used [-Wunused-but-set-variable]
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: zhong jiang <zhongjiang@huawei.com>
Cc: Ola Lilja <ola.o.lilja@stericsson.com>
Cc: Roger Nilsson <roger.xr.nilsson@stericsson.com>
Cc: Sandeep Kaushik <sandeep.kaushik@st.com>
Link: https://lore.kernel.org/r/20200709162328.259586-7-pierre-louis.bossart@linux.intel.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 701f4727
...@@ -395,7 +395,7 @@ static int enable_msp(struct ux500_msp *msp, struct ux500_msp_config *config) ...@@ -395,7 +395,7 @@ static int enable_msp(struct ux500_msp *msp, struct ux500_msp_config *config)
static void flush_fifo_rx(struct ux500_msp *msp) static void flush_fifo_rx(struct ux500_msp *msp)
{ {
u32 reg_val_DR, reg_val_GCR, reg_val_FLR; u32 reg_val_GCR, reg_val_FLR;
u32 limit = 32; u32 limit = 32;
reg_val_GCR = readl(msp->registers + MSP_GCR); reg_val_GCR = readl(msp->registers + MSP_GCR);
...@@ -403,7 +403,7 @@ static void flush_fifo_rx(struct ux500_msp *msp) ...@@ -403,7 +403,7 @@ static void flush_fifo_rx(struct ux500_msp *msp)
reg_val_FLR = readl(msp->registers + MSP_FLR); reg_val_FLR = readl(msp->registers + MSP_FLR);
while (!(reg_val_FLR & RX_FIFO_EMPTY) && limit--) { while (!(reg_val_FLR & RX_FIFO_EMPTY) && limit--) {
reg_val_DR = readl(msp->registers + MSP_DR); readl(msp->registers + MSP_DR);
reg_val_FLR = readl(msp->registers + MSP_FLR); reg_val_FLR = readl(msp->registers + MSP_FLR);
} }
...@@ -412,7 +412,7 @@ static void flush_fifo_rx(struct ux500_msp *msp) ...@@ -412,7 +412,7 @@ static void flush_fifo_rx(struct ux500_msp *msp)
static void flush_fifo_tx(struct ux500_msp *msp) static void flush_fifo_tx(struct ux500_msp *msp)
{ {
u32 reg_val_TSTDR, reg_val_GCR, reg_val_FLR; u32 reg_val_GCR, reg_val_FLR;
u32 limit = 32; u32 limit = 32;
reg_val_GCR = readl(msp->registers + MSP_GCR); reg_val_GCR = readl(msp->registers + MSP_GCR);
...@@ -421,7 +421,7 @@ static void flush_fifo_tx(struct ux500_msp *msp) ...@@ -421,7 +421,7 @@ static void flush_fifo_tx(struct ux500_msp *msp)
reg_val_FLR = readl(msp->registers + MSP_FLR); reg_val_FLR = readl(msp->registers + MSP_FLR);
while (!(reg_val_FLR & TX_FIFO_EMPTY) && limit--) { while (!(reg_val_FLR & TX_FIFO_EMPTY) && limit--) {
reg_val_TSTDR = readl(msp->registers + MSP_TSTDR); readl(msp->registers + MSP_TSTDR);
reg_val_FLR = readl(msp->registers + MSP_FLR); reg_val_FLR = readl(msp->registers + MSP_FLR);
} }
writel(0x0, msp->registers + MSP_ITCR); writel(0x0, msp->registers + MSP_ITCR);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment