lkml.org 
[lkml]   [2019]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] hpet: Drop unused variable 'm' in hpet_interrupt()
On Thu, Jul 11, 2019 at 09:32:38PM +0800, Kefeng Wang wrote:
> ../drivers/char/hpet.c:159:17: warning: variable ‘m’ set but not used
> [-Wunused-but-set-variable]
> unsigned long m, t, mc, base, k;
> ^
> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
> ---
> drivers/char/hpet.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c
> index 9ac6671bb514..039398cb14aa 100644
> --- a/drivers/char/hpet.c
> +++ b/drivers/char/hpet.c
> @@ -156,12 +156,11 @@ static irqreturn_t hpet_interrupt(int irq, void *data)
> * This has the effect of treating non-periodic like periodic.
> */
> if ((devp->hd_flags & (HPET_IE | HPET_PERIODIC)) == HPET_IE) {
> - unsigned long m, t, mc, base, k;
> + unsigned long t, mc, base, k;
> struct hpet __iomem *hpet = devp->hd_hpet;
> struct hpets *hpetp = devp->hd_hpets;
>
> t = devp->hd_ireqfreq;
> - m = read_counter(&devp->hd_timer->hpet_compare);

Are you sure this is ok to remove? You are reading from the hardware
here, and often times that is required in order to achive something
else.

So this function does not "do nothing", be careful and you will have to
test this before I can take it.

thanks,

greg k-h

\
 
 \ /
  Last update: 2019-07-25 14:34    [W:0.047 / U:0.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site