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
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
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
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
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
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
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
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
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
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
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
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
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
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 ?
>
>
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
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
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
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
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
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
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
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
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
23 matches
Mail list logo