On 17/07/2020 15:54, Lee Jones wrote: > Looks like it hasn't ever been checked. > > Fixes the following W=1 kernel build warning(s): > > drivers/spi/spi-meson-spicc.c: In function ‘meson_spicc_reset_fifo’: > drivers/spi/spi-meson-spicc.c:365:6: warning: variable ‘data’ set but not > used [-Wunused-but-set-variable] > 365 | u32 data; > | ^~~~ > > Cc: Kevin Hilman <[email protected]> > Cc: Neil Armstrong <[email protected]> > Cc: [email protected] > Signed-off-by: Lee Jones <[email protected]> > --- > drivers/spi/spi-meson-spicc.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/spi/spi-meson-spicc.c b/drivers/spi/spi-meson-spicc.c > index 77f7d0e0e46ad..ecba6b4a5d85d 100644 > --- a/drivers/spi/spi-meson-spicc.c > +++ b/drivers/spi/spi-meson-spicc.c > @@ -362,8 +362,6 @@ static void meson_spicc_setup_xfer(struct > meson_spicc_device *spicc, > > static void meson_spicc_reset_fifo(struct meson_spicc_device *spicc) > { > - u32 data; > - > if (spicc->data->has_oen) > writel_bits_relaxed(SPICC_ENH_MAIN_CLK_AO, > SPICC_ENH_MAIN_CLK_AO, > @@ -373,7 +371,7 @@ static void meson_spicc_reset_fifo(struct > meson_spicc_device *spicc) > spicc->base + SPICC_TESTREG); > > while (meson_spicc_rxready(spicc)) > - data = readl_relaxed(spicc->base + SPICC_RXDATA); > + readl_relaxed(spicc->base + SPICC_RXDATA); > > if (spicc->data->has_oen) > writel_bits_relaxed(SPICC_ENH_MAIN_CLK_AO, 0, >
Acked-by: Neil Armstrong <[email protected]>
