[PATCH] powerpc: Use str_plural() to fix Coccinelle warning

2024-03-31 Thread Thorsten Blum
Fixes the following Coccinelle/coccicheck warning reported by string_choices.cocci: opportunity for str_plural(tpc) Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/setup-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/setup-common.c

[PATCH] powerpc: Use str_enabled_disabled() helper function

2024-10-27 Thread Thorsten Blum
Remove hard-coded strings by using the str_enabled_disabled() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/secure_boot.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/secure_boot.c b/arch/powerpc/kernel/secure_boot.c index

Re: [PATCH] powerpc: Transliterate author name and remove FIXME

2024-11-10 Thread Thorsten Blum
On 11. Nov 2024, at 02:11, Michael Ellerman wrote: > Thorsten Blum writes: >> The name is Mimi Phuong-Thao Vo. > > Is that the correct spelling? According to [1] it's correct. Thanks, Thorsten [1] https://www.linkedin.com/in/mimi-phuong-thao-vo-5768033

[PATCH] powerpc: Transliterate author name and remove FIXME

2024-11-10 Thread Thorsten Blum
The name is Mimi Phuong-Thao Vo. Signed-off-by: Thorsten Blum --- arch/powerpc/boot/rs6000.h | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/arch/powerpc/boot/rs6000.h b/arch/powerpc/boot/rs6000.h index a9d879155ef9..16df8f3c43f1 100644 --- a/arch/powerpc/boot/rs6000.h

[PATCH] powerpc/xmon: Use str_yes_no() helper in dump_one_paca()

2024-11-29 Thread Thorsten Blum
Remove hard-coded strings by using the str_yes_no() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/xmon/xmon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c index e6cddbb2305f..6c84169b309d 100644

Re: [PATCH] powerpc: Transliterate author name and remove FIXME

2024-11-22 Thread Thorsten Blum
On 11. Nov 2024, at 02:11, Michael Ellerman wrote: > Thorsten Blum writes: >> The name is Mimi Phuong-Thao Vo. > > Is that the correct spelling? > > The github commit below suggests it's Mimi Phûông-Thåo Võ. > > And presumably the author preferred that spell

Re: [PATCH] powerpc: Transliterate author name and remove FIXME

2025-01-07 Thread Thorsten Blum
On 23. Nov 2024, at 11:19, Christophe Leroy wrote: > Isn't our file just a copy of the one from binutils ? Shouldn't we adjust it > based on commit https://github.com/bminor/binutils-gdb/commit/2ce18a16268a ? It looks like it's a copy and the name is spelled the same as in my patch: "Mimi Phuo

[PATCH] fadump: Use str_yes_no() helper in fadump_show_config()

2024-12-30 Thread Thorsten Blum
Remove hard-coded strings by using the str_yes_no() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/fadump.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c index 4b371c738213

[PATCH] powerpc: mpic: Use str_enabled_disabled() helper function

2025-02-04 Thread Thorsten Blum
Remove hard-coded strings by using the str_enabled_disabled() helper function. Use pr_debug() instead of printk(KERN_DEBUG) to silence a checkpatch warning. Signed-off-by: Thorsten Blum --- arch/powerpc/sysdev/mpic.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a

[PATCH] powerpc/mm/fault: Use str_write_read() helper function

2025-02-10 Thread Thorsten Blum
Remove hard-coded strings by using the str_write_read() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/mm/fault.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/mm/fault.c b/arch/powerpc/mm/fault.c index c156fe0d53c3..806c74e0d5ab 100644

Re: [RESEND PATCH] fadump: Use str_yes_no() helper in fadump_show_config()

2025-02-09 Thread Thorsten Blum
On 9. Feb 2025, at 10:16, Christophe Leroy wrote: > Le 09/02/2025 à 09:17, Thorsten Blum a écrit : >> Remove hard-coded strings by using the str_yes_no() helper function. >> Reviewed-by: Sourabh Jain >> Reviewed-by: Ritesh Harjani (IBM) >> Signed-off-by: Thorste

[PATCH] powerpc/ps3: Use str_write_read() in ps3_notification_read_write()

2025-02-10 Thread Thorsten Blum
Remove hard-coded strings by using the str_write_read() helper function. Signed-off-by: Thorsten Blum Suggested-by: Christophe Leroy --- arch/powerpc/platforms/ps3/device-init.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/ps3/device-init.c b

[PATCH] powerpc/iommu: Use str_disabled_enabled() helper

2025-02-10 Thread Thorsten Blum
Remove hard-coded strings by using the str_disabled_enabled() helper. Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/iommu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/iommu.c b/arch/powerpc/kernel/iommu.c index 0ebae6e4c19d..244eb4857e7f

Re: [PATCH] powerpc: Transliterate author name and remove FIXME

2025-02-11 Thread Thorsten Blum
On 7. Jan 2025, at 13:16, Thorsten Blum wrote: > On 23. Nov 2024, at 11:19, Christophe Leroy wrote: >> Isn't our file just a copy of the one from binutils ? Shouldn't we adjust it >> based on commit https://github.com/bminor/binutils-gdb/commit/2ce18a16268a ? > >

[PATCH] powerpc: Use str_on_off() helper in check_cache_coherency()

2024-12-10 Thread Thorsten Blum
Remove hard-coded strings by using the str_on_off() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/setup-common.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup-common.c index

[RESEND PATCH] powerpc: Use str_on_off() helper in check_cache_coherency()

2024-12-20 Thread Thorsten Blum
Remove hard-coded strings by using the str_on_off() helper function. Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/setup-common.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup-common.c index

[PATCH] powerpc/powermac: Use str_enabled_disabled() and str_on_off() helpers

2025-01-17 Thread Thorsten Blum
Remove hard-coded strings by using the str_enabled_disabled() and str_on_off() helper functions. Signed-off-by: Thorsten Blum --- arch/powerpc/platforms/powermac/setup.c | 4 ++-- arch/powerpc/platforms/powermac/time.c | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a

[RESEND PATCH] fadump: Use str_yes_no() helper in fadump_show_config()

2025-02-09 Thread Thorsten Blum
Remove hard-coded strings by using the str_yes_no() helper function. Reviewed-by: Sourabh Jain Reviewed-by: Ritesh Harjani (IBM) Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/fadump.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/powerpc/kernel/fadump.c

[PATCH] KVM: powerpc: Enable commented out BUILD_BUG_ON() assertion

2025-02-20 Thread Thorsten Blum
the comment. Signed-off-by: Thorsten Blum --- arch/powerpc/kvm/timing.h | 4 1 file changed, 4 deletions(-) diff --git a/arch/powerpc/kvm/timing.h b/arch/powerpc/kvm/timing.h index 45817ab82bb4..14b0e23f601f 100644 --- a/arch/powerpc/kvm/timing.h +++ b/arch/powerpc/kvm/timing.h @@ -38,11 +3

[PATCH] ASoC: fsl: fsl_qmc_audio: Remove unnecessary bool conversions

2025-02-23 Thread Thorsten Blum
Remove unnecessary bool conversions and simplify the code. Signed-off-by: Thorsten Blum --- sound/soc/fsl/fsl_qmc_audio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_qmc_audio.c b/sound/soc/fsl/fsl_qmc_audio.c index e257b8adafe0..b2979290c973 100644

Re: [RESEND PATCH] powerpc: mpic: Use str_enabled_disabled() helper function

2025-02-24 Thread Thorsten Blum
Hi Christophe, > On 24. Feb 2025, at 13:56, Christophe Leroy wrote: > Le 19/02/2025 à 12:20, Thorsten Blum a écrit : >> Remove hard-coded strings by using the str_enabled_disabled() helper >> function. >> Use pr_debug() instead of printk(KERN_DEBUG) to silence a checkpatch

[RESEND PATCH] powerpc/ps3: Use str_write_read() in ps3_notification_read_write()

2025-02-19 Thread Thorsten Blum
Remove hard-coded strings by using the str_write_read() helper function. Suggested-by: Christophe Leroy Signed-off-by: Thorsten Blum --- arch/powerpc/platforms/ps3/device-init.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/ps3/device-init.c b

[RESEND PATCH] powerpc: mpic: Use str_enabled_disabled() helper function

2025-02-19 Thread Thorsten Blum
Remove hard-coded strings by using the str_enabled_disabled() helper function. Use pr_debug() instead of printk(KERN_DEBUG) to silence a checkpatch warning. Reviewed-by: Ricardo B. Marlière Signed-off-by: Thorsten Blum --- arch/powerpc/sysdev/mpic.c | 7 --- 1 file changed, 4 insertions