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

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

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

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

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()

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

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

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()

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

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

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()

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()

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

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

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()

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()

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

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()

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

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

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

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

Remove hard-coded strings by using the str_write_read() helper function. Suggested-by: Christophe Leroy Reviewed-by: Geert Uytterhoeven 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

[PATCH] bus: fsl-mc: Use strscpy() instead of strscpy_pad()

) when the argument is omitted. This makes the explicit size arguments unnecessary. No functional changes intended. Signed-off-by: Thorsten Blum --- drivers/bus/fsl-mc/dprc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/bus/fsl-mc/dprc.c b/drivers/bus/fsl-mc/dp

Re: [RESEND PATCH] bus: fsl-mc: Use strscpy() instead of strscpy_pad()

Hi Christophe, On 29. Apr 2025, at 13:58, Christophe Leroy wrote: > Le 29/04/2025 à 13:47, Ioana Ciornei a écrit: >> On Tue, Apr 29, 2025 at 12:41:48PM +0200, Thorsten Blum wrote: >>> Both destination buffers are already zero-initialized, making strscpy() >>> su

[RESEND PATCH] bus: fsl-mc: Use strscpy() instead of strscpy_pad()

) when the argument is omitted. This makes the explicit size arguments unnecessary. No functional changes intended. Signed-off-by: Thorsten Blum --- drivers/bus/fsl-mc/dprc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/bus/fsl-mc/dprc.c b/drivers/bus/fsl-mc/dp

[PATCH] powerpc: Replace strcpy() with strscpy() in proc_ppc64_init()

.kernel.org Signed-off-by: Thorsten Blum --- arch/powerpc/kernel/proc_powerpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/proc_powerpc.c b/arch/powerpc/kernel/proc_powerpc.c index 3816a2bf2b84..d083b4517065 100644 --- a/arch/powerpc/kernel/proc_pow

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

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

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

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

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

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

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

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

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/rtas: Replace one-element array with flexible array member

/KSPP/linux/issues/79 Link: https://github.com/KSPP/linux/issues/83 Signed-off-by: Thorsten Blum --- arch/powerpc/include/asm/rtas-types.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/include/asm/rtas-types.h b/arch/powerpc/include/asm/rtas-types.h index

[PATCH] powerpc: Fix indentation and replace typedef with struct name

Indent several struct members using tabs instead of spaces. Replace the typedef alias AOUTHDR with an explicit struct name to silence a checkpatch warning. Signed-off-by: Thorsten Blum --- arch/powerpc/boot/hack-coff.c | 2 +- arch/powerpc/boot/rs6000.h| 84