[PATCH] riscv: cpu_ops_sbi: smp_processor_id() returns int, not unsigned int

Ben Dooks posted 1 patch 1 month ago
arch/riscv/kernel/cpu_ops_sbi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] riscv: cpu_ops_sbi: smp_processor_id() returns int, not unsigned int
Posted by Ben Dooks 1 month ago
The print in sbi_cpu_stop() assumes smp_processor_id() returns an
unsigned int, when it is actually an int. Fix the format string to
avoid mismatch type warnings in rht pr_crit().

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 arch/riscv/kernel/cpu_ops_sbi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/riscv/kernel/cpu_ops_sbi.c b/arch/riscv/kernel/cpu_ops_sbi.c
index 87d655944803..00aff669f5f2 100644
--- a/arch/riscv/kernel/cpu_ops_sbi.c
+++ b/arch/riscv/kernel/cpu_ops_sbi.c
@@ -85,7 +85,7 @@ static void sbi_cpu_stop(void)
 	int ret;
 
 	ret = sbi_hsm_hart_stop();
-	pr_crit("Unable to stop the cpu %u (%d)\n", smp_processor_id(), ret);
+	pr_crit("Unable to stop the cpu %d (%d)\n", smp_processor_id(), ret);
 }
 
 static int sbi_cpu_is_stopped(unsigned int cpuid)
-- 
2.37.2.352.g3c44437643
Re: [PATCH] riscv: cpu_ops_sbi: smp_processor_id() returns int, not unsigned int
Posted by Paul Walmsley 1 month ago
On Fri, 2 Jan 2026, Ben Dooks wrote:

> The print in sbi_cpu_stop() assumes smp_processor_id() returns an
> unsigned int, when it is actually an int. Fix the format string to
> avoid mismatch type warnings in rht pr_crit().
> 
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>

Thanks, queued for v6.19-rc.


- Paul