Drop some #ifdefs that are not needed or can be converted to compile-time
checks.

Signed-off-by: Simon Glass <s...@chromium.org>
Reviewed-by: Wolfgang Wallner <wolfgang.wall...@br-automation.com>
---

(no changes since v1)

 arch/x86/cpu/cpu.c      | 58 ++++++++++++++++++++---------------------
 arch/x86/cpu/i386/cpu.c |  2 --
 2 files changed, 28 insertions(+), 32 deletions(-)

diff --git a/arch/x86/cpu/cpu.c b/arch/x86/cpu/cpu.c
index 23a4d633d2..d0720fb7fb 100644
--- a/arch/x86/cpu/cpu.c
+++ b/arch/x86/cpu/cpu.c
@@ -66,10 +66,8 @@ static const char *const x86_vendor_name[] = {
 
 int __weak x86_cleanup_before_linux(void)
 {
-#ifdef CONFIG_BOOTSTAGE_STASH
        bootstage_stash((void *)CONFIG_BOOTSTAGE_STASH_ADDR,
                        CONFIG_BOOTSTAGE_STASH_SIZE);
-#endif
 
        return 0;
 }
@@ -200,18 +198,19 @@ int last_stage_init(void)
 
        write_tables();
 
-#ifdef CONFIG_GENERATE_ACPI_TABLE
-       fadt = acpi_find_fadt();
+       if (IS_ENABLED(CONFIG_GENERATE_ACPI_TABLE)) {
+               fadt = acpi_find_fadt();
 
-       /* Don't touch ACPI hardware on HW reduced platforms */
-       if (fadt && !(fadt->flags & ACPI_FADT_HW_REDUCED_ACPI)) {
-               /*
-                * Other than waiting for OSPM to request us to switch to ACPI
-                * mode, do it by ourselves, since SMI will not be triggered.
-                */
-               enter_acpi_mode(fadt->pm1a_cnt_blk);
+               /* Don't touch ACPI hardware on HW reduced platforms */
+               if (fadt && !(fadt->flags & ACPI_FADT_HW_REDUCED_ACPI)) {
+                       /*
+                        * Other than waiting for OSPM to request us to switch
+                        * to ACPI * mode, do it by ourselves, since SMI will
+                        * not be triggered.
+                        */
+                       enter_acpi_mode(fadt->pm1a_cnt_blk);
+               }
        }
-#endif
 
        return 0;
 }
@@ -219,19 +218,20 @@ int last_stage_init(void)
 
 static int x86_init_cpus(void)
 {
-#ifdef CONFIG_SMP
-       debug("Init additional CPUs\n");
-       x86_mp_init();
-#else
-       struct udevice *dev;
+       if (IS_ENABLED(CONFIG_SMP)) {
+               debug("Init additional CPUs\n");
+               x86_mp_init();
+       } else {
+               struct udevice *dev;
 
-       /*
-        * This causes the cpu-x86 driver to be probed.
-        * We don't check return value here as we want to allow boards
-        * which have not been converted to use cpu uclass driver to boot.
-        */
-       uclass_first_device(UCLASS_CPU, &dev);
-#endif
+               /*
+                * This causes the cpu-x86 driver to be probed.
+                * We don't check return value here as we want to allow boards
+                * which have not been converted to use cpu uclass driver to
+                * boot.
+                */
+               uclass_first_device(UCLASS_CPU, &dev);
+       }
 
        return 0;
 }
@@ -269,13 +269,11 @@ int cpu_init_r(void)
 #ifndef CONFIG_EFI_STUB
 int reserve_arch(void)
 {
-#ifdef CONFIG_ENABLE_MRC_CACHE
-       mrccache_reserve();
-#endif
+       if (IS_ENABLED(CONFIG_ENABLE_MRC_CACHE))
+               mrccache_reserve();
 
-#ifdef CONFIG_SEABIOS
-       high_table_reserve();
-#endif
+       if (IS_ENABLED(CONFIG_SEABIOS))
+               high_table_reserve();
 
        if (IS_ENABLED(CONFIG_HAVE_ACPI_RESUME)) {
                acpi_s3_reserve();
diff --git a/arch/x86/cpu/i386/cpu.c b/arch/x86/cpu/i386/cpu.c
index 9809ac5111..fca3f79b69 100644
--- a/arch/x86/cpu/i386/cpu.c
+++ b/arch/x86/cpu/i386/cpu.c
@@ -626,7 +626,6 @@ int cpu_jump_to_64bit_uboot(ulong target)
        return -EFAULT;
 }
 
-#ifdef CONFIG_SMP
 int x86_mp_init(void)
 {
        int ret;
@@ -639,4 +638,3 @@ int x86_mp_init(void)
 
        return 0;
 }
-#endif
-- 
2.27.0.290.gba653c62da-goog

Reply via email to