[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87h68wnbe4.fsf@gmail.com>
Date: Mon, 28 Oct 2024 09:10:03 +0530
From: Ritesh Harjani (IBM) <ritesh.list@...il.com>
To: Thorsten Blum <thorsten.blum@...ux.dev>, Michael Ellerman <mpe@...erman.id.au>, Nicholas Piggin <npiggin@...il.com>, Christophe Leroy <christophe.leroy@...roup.eu>, Naveen N Rao <naveen@...nel.org>, Madhavan Srinivasan <maddy@...ux.ibm.com>, Rob Herring <robh@...nel.org>
Cc: Thorsten Blum <thorsten.blum@...ux.dev>, linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] powerpc: Use str_enabled_disabled() helper function
Thorsten Blum <thorsten.blum@...ux.dev> writes:
> Remove hard-coded strings by using the str_enabled_disabled() helper
> function.
>
Looks like now a days LKP also reports this - e.g. [1] "opportunity for str_enabled_disabled(enable)"
[1]: https://lore.kernel.org/all/202410071601.TFpXoqgW-lkp@intel.com/
I see more such oppotunities for later improvements in arch/powerpc/
root-> git grep "\"enabled\" : \"disabled\"" arch/powerpc/
arch/powerpc/kernel/secure_boot.c: pr_info("Secure boot mode %s\n", enabled ? "enabled" : "disabled");
arch/powerpc/kernel/secure_boot.c: pr_info("Trusted boot mode %s\n", enabled ? "enabled" : "disabled");
arch/powerpc/platforms/powermac/setup.c: "enabled" : "disabled");
arch/powerpc/sysdev/mpic.c: flags & HT_MSI_FLAGS_ENABLE ? "enabled" : "disabled", addr);
> Signed-off-by: Thorsten Blum <thorsten.blum@...ux.dev>
> ---
> arch/powerpc/kernel/secure_boot.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
For this patch it looks good to me. Please feel free to add -
Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@...il.com>
> diff --git a/arch/powerpc/kernel/secure_boot.c b/arch/powerpc/kernel/secure_boot.c
> index 9e0efb657f39..3a28795b4ed8 100644
> --- a/arch/powerpc/kernel/secure_boot.c
> +++ b/arch/powerpc/kernel/secure_boot.c
> @@ -5,6 +5,7 @@
> */
> #include <linux/types.h>
> #include <linux/of.h>
> +#include <linux/string_choices.h>
> #include <asm/secure_boot.h>
>
> static struct device_node *get_ppc_fw_sb_node(void)
> @@ -38,7 +39,7 @@ bool is_ppc_secureboot_enabled(void)
> of_node_put(node);
>
> out:
> - pr_info("Secure boot mode %s\n", enabled ? "enabled" : "disabled");
> + pr_info("Secure boot mode %s\n", str_enabled_disabled(enabled));
>
> return enabled;
> }
> @@ -62,7 +63,7 @@ bool is_ppc_trustedboot_enabled(void)
> of_node_put(node);
>
> out:
> - pr_info("Trusted boot mode %s\n", enabled ? "enabled" : "disabled");
> + pr_info("Trusted boot mode %s\n", str_enabled_disabled(enabled));
>
> return enabled;
> }
> --
> 2.47.0
Powered by blists - more mailing lists