riscv: cpu: Use CONFIG_IS_ENABLED(CPU) instead of plain ifdef
authorMichal Simek <michal.simek@amd.com>
Thu, 17 Jul 2025 06:26:15 +0000 (08:26 +0200)
committerLeo Yu-Chi Liang <ycliang@andestech.com>
Thu, 14 Aug 2025 06:32:00 +0000 (14:32 +0800)
ifdef CONFIG_CPU only works in U-Boot proper but macro is not working when
XPL phases are used. In this case CONFIG_SPL_CPU is also defined and can be
disabled which is causing compilation error.

Signed-off-by: Michal Simek <michal.simek@amd.com>
Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>
arch/riscv/cpu/cpu.c

index 15c4e14..d5123e4 100644 (file)
@@ -608,14 +608,14 @@ static inline bool supports_extension(char ext)
 
 static int riscv_cpu_probe(void)
 {
-#ifdef CONFIG_CPU
-       int ret;
+       if (CONFIG_IS_ENABLED(CPU)) {
+               int ret;
 
-       /* probe cpus so that RISC-V timer can be bound */
-       ret = cpu_probe_all();
-       if (ret)
-               return log_msg_ret("RISC-V cpus probe failed\n", ret);
-#endif
+               /* probe cpus so that RISC-V timer can be bound */
+               ret = cpu_probe_all();
+               if (ret)
+                       return log_msg_ret("RISC-V cpus probe failed\n", ret);
+       }
 
        return 0;
 }