* [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() @ 2025-01-14 20:25 Thorsten Blum 2025-01-14 20:29 ` John Paul Adrian Glaubitz 0 siblings, 1 reply; 4+ messages in thread From: Thorsten Blum @ 2025-01-14 20:25 UTC (permalink / raw) To: David S. Miller, Andreas Larsson, Randy Dunlap, Bjorn Helgaas, Sam Ravnborg, Dawei Li Cc: Thorsten Blum, sparclinux, linux-kernel Remove the unnecessary if check and return the result directly. Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> --- arch/sparc/kernel/irq_64.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/sparc/kernel/irq_64.c b/arch/sparc/kernel/irq_64.c index aff0d24f8c6f..81fb49b089a8 100644 --- a/arch/sparc/kernel/irq_64.c +++ b/arch/sparc/kernel/irq_64.c @@ -146,9 +146,7 @@ static int hv_irq_version; */ static bool sun4v_cookie_only_virqs(void) { - if (hv_irq_version >= 3) - return true; - return false; + return hv_irq_version >= 3; } static void __init irq_init_hv(void) -- 2.47.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() 2025-01-14 20:25 [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() Thorsten Blum @ 2025-01-14 20:29 ` John Paul Adrian Glaubitz 2025-01-15 16:11 ` Thorsten Blum 0 siblings, 1 reply; 4+ messages in thread From: John Paul Adrian Glaubitz @ 2025-01-14 20:29 UTC (permalink / raw) To: Thorsten Blum, David S. Miller, Andreas Larsson, Randy Dunlap, Bjorn Helgaas, Sam Ravnborg, Dawei Li Cc: sparclinux, linux-kernel Hi Thorsten, On Tue, 2025-01-14 at 21:25 +0100, Thorsten Blum wrote: > Remove the unnecessary if check and return the result directly. > > Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> > --- > arch/sparc/kernel/irq_64.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/sparc/kernel/irq_64.c b/arch/sparc/kernel/irq_64.c > index aff0d24f8c6f..81fb49b089a8 100644 > --- a/arch/sparc/kernel/irq_64.c > +++ b/arch/sparc/kernel/irq_64.c > @@ -146,9 +146,7 @@ static int hv_irq_version; > */ > static bool sun4v_cookie_only_virqs(void) > { > - if (hv_irq_version >= 3) > - return true; > - return false; > + return hv_irq_version >= 3; > } > > static void __init irq_init_hv(void) I'm not sure whether this change is really necessary nor will it improve the readability of the code. I think the compiler will optimize the code the same way, for both variants. Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer `. `' Physicist `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() 2025-01-14 20:29 ` John Paul Adrian Glaubitz @ 2025-01-15 16:11 ` Thorsten Blum 2025-01-17 15:21 ` Andreas Larsson 0 siblings, 1 reply; 4+ messages in thread From: Thorsten Blum @ 2025-01-15 16:11 UTC (permalink / raw) To: John Paul Adrian Glaubitz Cc: David S. Miller, Andreas Larsson, Randy Dunlap, Bjorn Helgaas, Sam Ravnborg, Dawei Li, sparclinux, linux-kernel Hi Adrian, On 14. Jan 2025, at 21:29, John Paul Adrian Glaubitz wrote: > On Tue, 2025-01-14 at 21:25 +0100, Thorsten Blum wrote: >> Remove the unnecessary if check and return the result directly. >> >> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> >> --- >> arch/sparc/kernel/irq_64.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/arch/sparc/kernel/irq_64.c b/arch/sparc/kernel/irq_64.c >> index aff0d24f8c6f..81fb49b089a8 100644 >> --- a/arch/sparc/kernel/irq_64.c >> +++ b/arch/sparc/kernel/irq_64.c >> @@ -146,9 +146,7 @@ static int hv_irq_version; >> */ >> static bool sun4v_cookie_only_virqs(void) >> { >> - if (hv_irq_version >= 3) >> - return true; >> - return false; >> + return hv_irq_version >= 3; >> } >> >> static void __init irq_init_hv(void) > > I'm not sure whether this change is really necessary nor will it improve > the readability of the code. I think the compiler will optimize the code > the same way, for both variants. It's not strictly necessary, but it's shorter, less verbose, and imo improves the code's readability. Personally, I'd even consider the current version a code smell, but feel free to ignore the patch. Thanks, Thorsten ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() 2025-01-15 16:11 ` Thorsten Blum @ 2025-01-17 15:21 ` Andreas Larsson 0 siblings, 0 replies; 4+ messages in thread From: Andreas Larsson @ 2025-01-17 15:21 UTC (permalink / raw) To: Thorsten Blum, John Paul Adrian Glaubitz Cc: David S. Miller, Randy Dunlap, Bjorn Helgaas, Sam Ravnborg, Dawei Li, sparclinux, linux-kernel On 2025-01-15 17:11, Thorsten Blum wrote: > Hi Adrian, > > On 14. Jan 2025, at 21:29, John Paul Adrian Glaubitz wrote: >> On Tue, 2025-01-14 at 21:25 +0100, Thorsten Blum wrote: >>> Remove the unnecessary if check and return the result directly. >>> >>> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> >>> --- >>> arch/sparc/kernel/irq_64.c | 4 +--- >>> 1 file changed, 1 insertion(+), 3 deletions(-) >>> >>> diff --git a/arch/sparc/kernel/irq_64.c b/arch/sparc/kernel/irq_64.c >>> index aff0d24f8c6f..81fb49b089a8 100644 >>> --- a/arch/sparc/kernel/irq_64.c >>> +++ b/arch/sparc/kernel/irq_64.c >>> @@ -146,9 +146,7 @@ static int hv_irq_version; >>> */ >>> static bool sun4v_cookie_only_virqs(void) >>> { >>> - if (hv_irq_version >= 3) >>> - return true; >>> - return false; >>> + return hv_irq_version >= 3; >>> } >>> >>> static void __init irq_init_hv(void) >> >> I'm not sure whether this change is really necessary nor will it improve >> the readability of the code. I think the compiler will optimize the code >> the same way, for both variants. > > It's not strictly necessary, but it's shorter, less verbose, and imo > improves the code's readability. Personally, I'd even consider the > current version a code smell, but feel free to ignore the patch. I could see it both ways, but personally agree about it being slightly more readable. As I'll pick up the other sun4v_cookie_only_virqs() related patch I might as well take this one as well. Reviewed-by: Andreas Larsson <andreas@gaisler.com> Picking this up to my for-next. Thanks, Andreas ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-01-17 15:21 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-01-14 20:25 [PATCH] sparc/irq: Remove unneeded if check in sun4v_cookie_only_virqs() Thorsten Blum 2025-01-14 20:29 ` John Paul Adrian Glaubitz 2025-01-15 16:11 ` Thorsten Blum 2025-01-17 15:21 ` Andreas Larsson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.