linux-kernel - Re: [PATCH] microblaze: mb: Use str_yes_no() helper in show

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: <CAHTX3dKYrSqLk+1nODm+yMVFfEeW_WvfboiZ_A13DN4_iCeuag@mail.gmail.com>
Date: Fri, 15 Nov 2024 10:28:20 +0100
From: Michal Simek <monstr@...str.eu>
To: Thorsten Blum <thorsten.blum@...ux.dev>
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH] microblaze: mb: Use str_yes_no() helper in show_cpuinfo()

čt 14. 11. 2024 v 23:49 odesílatel Thorsten Blum
<thorsten.blum@...ux.dev> napsal:
>
> Remove hard-coded strings by using the str_yes_no() helper function.
>
> Signed-off-by: Thorsten Blum <thorsten.blum@...ux.dev>
> ---
>  arch/microblaze/kernel/cpu/mb.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/microblaze/kernel/cpu/mb.c b/arch/microblaze/kernel/cpu/mb.c
> index 9581d194d9e4..37cb2898216b 100644
> --- a/arch/microblaze/kernel/cpu/mb.c
> +++ b/arch/microblaze/kernel/cpu/mb.c
> @@ -66,10 +66,10 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>                    " MSR:\t\t%s\n"
>                    " PCMP:\t\t%s\n"
>                    " DIV:\t\t%s\n",
> -                  (cpuinfo.use_instr & PVR0_USE_BARREL_MASK) ? "yes" : "no",
> -                  (cpuinfo.use_instr & PVR2_USE_MSR_INSTR) ? "yes" : "no",
> -                  (cpuinfo.use_instr & PVR2_USE_PCMP_INSTR) ? "yes" : "no",
> -                  (cpuinfo.use_instr & PVR0_USE_DIV_MASK) ? "yes" : "no");
> +                  str_yes_no(cpuinfo.use_instr & PVR0_USE_BARREL_MASK),
> +                  str_yes_no(cpuinfo.use_instr & PVR2_USE_MSR_INSTR),
> +                  str_yes_no(cpuinfo.use_instr & PVR2_USE_PCMP_INSTR),
> +                  str_yes_no(cpuinfo.use_instr & PVR0_USE_DIV_MASK));
>
>         seq_printf(m, " MMU:\t\t%x\n", cpuinfo.mmu);
>
> @@ -120,7 +120,7 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>
>         seq_printf(m,
>                    "HW-Debug:\t%s\n",
> -                  cpuinfo.hw_debug ? "yes" : "no");
> +                  str_yes_no(cpuinfo.hw_debug));
>
>         seq_printf(m,
>                    "PVR-USR1:\t%02x\n"
> --
> 2.47.0
>

Applied.

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs