Re: [PATCH -next] mmc: remove set but not used variables 'len, clock_out'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 7 Dec 2018 at 07:26, YueHaibing <yuehaibing@xxxxxxxxxx> wrote:
>
> Fixes gcc '-Wunused-but-set-variable' warning:
>
> drivers/mmc/host/alcor.c: In function 'alcor_data_set_dma':
> drivers/mmc/host/alcor.c:124:12: warning:
>  variable 'len' set but not used [-Wunused-but-set-variable]
>
> drivers/mmc/host/alcor.c:648:15: warning:
>  variable 'clock_out' set but not used [-Wunused-but-set-variable]
>
> It never used since introduction in commit 2a9a4742e825 ("mmc: add new Alcor
> Micro Cardreader SD/MMC driver")
>
> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>

Applied for next, thanks!

Kind regards
Uffe

> ---
>  drivers/mmc/host/alcor.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/alcor.c b/drivers/mmc/host/alcor.c
> index f729a64..c712b7d 100644
> --- a/drivers/mmc/host/alcor.c
> +++ b/drivers/mmc/host/alcor.c
> @@ -121,7 +121,7 @@ static void alcor_reset(struct alcor_sdmmc_host *host, u8 val)
>  static void alcor_data_set_dma(struct alcor_sdmmc_host *host)
>  {
>         struct alcor_pci_priv *priv = host->alcor_pci;
> -       u32 addr, len;
> +       u32 addr;
>
>         if (!host->sg_count)
>                 return;
> @@ -138,7 +138,6 @@ static void alcor_data_set_dma(struct alcor_sdmmc_host *host)
>
>
>         addr = (u32)sg_dma_address(host->sg);
> -       len = sg_dma_len(host->sg);
>
>         alcor_write32(priv, addr, AU6601_REG_SDMA_ADDR);
>         host->sg = sg_next(host->sg);
> @@ -645,7 +644,6 @@ static irqreturn_t alcor_irq(int irq, void *d)
>  static void alcor_set_clock(struct alcor_sdmmc_host *host, unsigned int clock)
>  {
>         struct alcor_pci_priv *priv = host->alcor_pci;
> -       unsigned int clock_out = 0;
>         int i, diff = 0x7fffffff, tmp_clock = 0;
>         u16 clk_src = 0;
>         u8 clk_div = 0;
> @@ -670,7 +668,6 @@ static void alcor_set_clock(struct alcor_sdmmc_host *host, unsigned int clock)
>                         diff = tmp_diff;
>                         clk_src = cfg->clk_src_reg;
>                         clk_div = tmp_div;
> -                       clock_out = tmp_clock;
>                 }
>         }
>
>
>
>
>
>



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux