* [PATCH] arm: smp: remove unused variable
@ 2020-12-28 12:01 Wolfram Sang
2021-01-26 10:04 ` Geert Uytterhoeven
0 siblings, 1 reply; 6+ messages in thread
From: Wolfram Sang @ 2020-12-28 12:01 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marc Zyngier, Russell King, linux-kernel, linux-renesas-soc,
Wolfram Sang, Thomas Gleixner
Not used anymore after refactoring:
arch/arm/kernel/smp.c: In function ‘show_ipi_list’:
arch/arm/kernel/smp.c:543:16: warning: variable ‘irq’ set but not used [-Wunused-but-set-variable]
543 | unsigned int irq;
Fixes: 88c637748e31 ("ARM: smp: Use irq_desc_kstat_cpu() in show_ipi_list()")
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Marc Zyngier <maz@kernel.org>
---
arch/arm/kernel/smp.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c
index 5c48eb4fd0e5..74679240a9d8 100644
--- a/arch/arm/kernel/smp.c
+++ b/arch/arm/kernel/smp.c
@@ -540,12 +540,9 @@ void show_ipi_list(struct seq_file *p, int prec)
unsigned int cpu, i;
for (i = 0; i < NR_IPI; i++) {
- unsigned int irq;
-
if (!ipi_desc[i])
continue;
- irq = irq_desc_get_irq(ipi_desc[i]);
seq_printf(p, "%*s%u: ", prec - 1, "IPI", i);
for_each_online_cpu(cpu)
--
2.29.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] arm: smp: remove unused variable
2020-12-28 12:01 [PATCH] arm: smp: remove unused variable Wolfram Sang
@ 2021-01-26 10:04 ` Geert Uytterhoeven
2021-01-26 10:41 ` Russell King - ARM Linux admin
0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2021-01-26 10:04 UTC (permalink / raw)
To: Wolfram Sang
Cc: Marc Zyngier, Russell King, Linux Kernel Mailing List,
Linux-Renesas, Thomas Gleixner, Linux ARM
Hi Wolfram,
On Mon, Dec 28, 2020 at 1:03 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Not used anymore after refactoring:
>
> arch/arm/kernel/smp.c: In function ‘show_ipi_list’:
> arch/arm/kernel/smp.c:543:16: warning: variable ‘irq’ set but not used [-Wunused-but-set-variable]
> 543 | unsigned int irq;
>
> Fixes: 88c637748e31 ("ARM: smp: Use irq_desc_kstat_cpu() in show_ipi_list()")
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
Known issue since Dec 15, and still not fixed...
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm: smp: remove unused variable
2021-01-26 10:04 ` Geert Uytterhoeven
@ 2021-01-26 10:41 ` Russell King - ARM Linux admin
2021-01-26 13:02 ` Wolfram Sang
0 siblings, 1 reply; 6+ messages in thread
From: Russell King - ARM Linux admin @ 2021-01-26 10:41 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Marc Zyngier, Linux Kernel Mailing List, Linux-Renesas,
Wolfram Sang, Thomas Gleixner, Linux ARM
On Tue, Jan 26, 2021 at 11:04:47AM +0100, Geert Uytterhoeven wrote:
> Hi Wolfram,
>
> On Mon, Dec 28, 2020 at 1:03 PM Wolfram Sang
> <wsa+renesas@sang-engineering.com> wrote:
> > Not used anymore after refactoring:
> >
> > arch/arm/kernel/smp.c: In function ‘show_ipi_list’:
> > arch/arm/kernel/smp.c:543:16: warning: variable ‘irq’ set but not used [-Wunused-but-set-variable]
> > 543 | unsigned int irq;
> >
> > Fixes: 88c637748e31 ("ARM: smp: Use irq_desc_kstat_cpu() in show_ipi_list()")
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Marc Zyngier <maz@kernel.org>
>
> Known issue since Dec 15, and still not fixed...
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Those who cause breakage really should be the ones to look at patches
that fix their breakage.
The way patches get applied is if they end up in my patch system... if
they don't make it there, they don't get applied.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm: smp: remove unused variable
2021-01-26 10:41 ` Russell King - ARM Linux admin
@ 2021-01-26 13:02 ` Wolfram Sang
2021-01-26 13:40 ` Russell King - ARM Linux admin
0 siblings, 1 reply; 6+ messages in thread
From: Wolfram Sang @ 2021-01-26 13:02 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Marc Zyngier, Linux Kernel Mailing List, Linux-Renesas,
Geert Uytterhoeven, Thomas Gleixner, Linux ARM
[-- Attachment #1.1: Type: text/plain, Size: 450 bytes --]
Hi Russell,
> Those who cause breakage really should be the ones to look at patches
> that fix their breakage.
Does it mean you want an explicit ack from Thomas or that it should go
via his tree?
> The way patches get applied is if they end up in my patch system... if
> they don't make it there, they don't get applied.
The patch itself is here:
https://www.armlinux.org.uk/developer/patches/viewpatch.php?id=9047/1
All the best,
Wolfram
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm: smp: remove unused variable
2021-01-26 13:02 ` Wolfram Sang
@ 2021-01-26 13:40 ` Russell King - ARM Linux admin
2021-01-26 13:52 ` Wolfram Sang
0 siblings, 1 reply; 6+ messages in thread
From: Russell King - ARM Linux admin @ 2021-01-26 13:40 UTC (permalink / raw)
To: Wolfram Sang
Cc: Marc Zyngier, Linux Kernel Mailing List, Linux-Renesas,
Geert Uytterhoeven, Thomas Gleixner, Linux ARM
On Tue, Jan 26, 2021 at 02:02:40PM +0100, Wolfram Sang wrote:
> Hi Russell,
>
> > Those who cause breakage really should be the ones to look at patches
> > that fix their breakage.
>
> Does it mean you want an explicit ack from Thomas or that it should go
> via his tree?
What I'm saying is... don't expect me to always review patches that
are for fixing code that other people have contributed - I wish those
who introduce regressions would stick around and attend to breakage
that they cause, instead of hoping that someone else will do that
for them. It's a reasonable ask.
> > The way patches get applied is if they end up in my patch system... if
> > they don't make it there, they don't get applied.
>
> The patch itself is here:
>
> https://www.armlinux.org.uk/developer/patches/viewpatch.php?id=9047/1
Thanks - I'll get around to it in due course. Since I'm no longer
supported as 32-bit ARM maintainer, I only apply patches once or maybe
twice per kernel release. Quite how long this will be sustainable for,
I've no idea (it depends whether I get any actual paying work, and
how much.)
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm: smp: remove unused variable
2021-01-26 13:40 ` Russell King - ARM Linux admin
@ 2021-01-26 13:52 ` Wolfram Sang
0 siblings, 0 replies; 6+ messages in thread
From: Wolfram Sang @ 2021-01-26 13:52 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Marc Zyngier, Linux Kernel Mailing List, Linux-Renesas,
Wolfram Sang, Geert Uytterhoeven, Thomas Gleixner, Linux ARM
> What I'm saying is... don't expect me to always review patches that
> are for fixing code that other people have contributed - I wish those
> who introduce regressions would stick around and attend to breakage
> that they cause, instead of hoping that someone else will do that
> for them. It's a reasonable ask.
Yes, I am much with you here.
> Thanks - I'll get around to it in due course. Since I'm no longer
> supported as 32-bit ARM maintainer, I only apply patches once or maybe
> twice per kernel release. Quite how long this will be sustainable for,
> I've no idea (it depends whether I get any actual paying work, and
> how much.)
I didn't know that, thanks for the heads up. I understand that problem
and hope there will be income for the work you do.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-01-26 13:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-28 12:01 [PATCH] arm: smp: remove unused variable Wolfram Sang
2021-01-26 10:04 ` Geert Uytterhoeven
2021-01-26 10:41 ` Russell King - ARM Linux admin
2021-01-26 13:02 ` Wolfram Sang
2021-01-26 13:40 ` Russell King - ARM Linux admin
2021-01-26 13:52 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).