lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ