[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<TY2PPF5CB9A1BE68147624EDB4C0B6C041BF29BA@TY2PPF5CB9A1BE6.apcprd06.prod.outlook.com>
Date: Tue, 3 Feb 2026 05:31:26 +0000
From: Ryan Chen <ryan_chen@...eedtech.com>
To: Thomas Gleixner <tglx@...utronix.de>, kernel test robot <lkp@...el.com>
CC: "oe-kbuild-all@...ts.linux.dev" <oe-kbuild-all@...ts.linux.dev>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] irqchip/aspeed-scu-ic: Remove unused variable mask
Hello Thomas,
Thanks the patch. Appreciate.
Ryan
> Cc: oe-kbuild-all@...ts.linux.dev; linux-kernel@...r.kernel.org
> Subject: [PATCH] irqchip/aspeed-scu-ic: Remove unused variable mask
>
>
> The kernel test robot reports:
>
> drivers/irqchip/irq-aspeed-scu-ic.c:107:27: warning: variable 'mask' set but
> not used
> 107 | unsigned int sts, mask;
>
> Remove the leftover.
>
> Fixes: b2a0c13f8b4f ("irqchip/aspeed-scu-ic: Add support for AST2700 SCU
> interrupt controllers")
> Reported-by: kernel test robot <lkp@...el.com>
> Signed-off-by: Thomas Gleixner <tglx@...nel.org>
> Closes:
> https://lore.kernel.org/oe-kbuild-all/202602010957.9uuKqUkG-lkp@intel.com/
> ---
> Seems the aspeed people are too busy to care about their own code....
> ---
> drivers/irqchip/irq-aspeed-scu-ic.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> --- a/drivers/irqchip/irq-aspeed-scu-ic.c
> +++ b/drivers/irqchip/irq-aspeed-scu-ic.c
> @@ -104,11 +104,10 @@ static void aspeed_scu_ic_irq_handler_sp
> struct aspeed_scu_ic *scu_ic = irq_desc_get_handler_data(desc);
> struct irq_chip *chip = irq_desc_get_chip(desc);
> unsigned long bit, enabled, max, status;
> - unsigned int sts, mask;
> + unsigned int sts;
>
> chained_irq_enter(chip, desc);
>
> - mask = scu_ic->irq_enable;
> sts = readl(scu_ic->base + scu_ic->isr);
> enabled = sts & scu_ic->irq_enable;
> sts = readl(scu_ic->base + scu_ic->isr);
Powered by blists - more mailing lists