arm: riscv: Remove additional ifdef from code guarded by CONFIG_IS_ENABLED
authorMichal Simek <michal.simek@amd.com>
Thu, 7 Jul 2022 08:47:16 +0000 (10:47 +0200)
committerMichal Simek <michal.simek@amd.com>
Tue, 26 Jul 2022 06:23:54 +0000 (08:23 +0200)
CONFIG_OF_LIBFDT is used twice for guarding the same code. It is enough to
do it once that's why remove additional ifdefs from arm and risc-v code.

Fixes: 0c303f9a6628 ("image: Drop IMAGE_ENABLE_OF_LIBFDT")
Signed-off-by: Michal Simek <michal.simek@amd.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Link: https://lore.kernel.org/r/f8e3ff9124195cbd957874de9a65ef79760ef5e7.1657183634.git.michal.simek@amd.com
arch/arm/lib/bootm.c
arch/riscv/lib/bootm.c

index a59a5e6..fc22bef 100644 (file)
@@ -200,12 +200,10 @@ static void boot_prep_linux(bootm_headers_t *images)
        char *commandline = env_get("bootargs");
 
        if (CONFIG_IS_ENABLED(OF_LIBFDT) && images->ft_len) {
-#ifdef CONFIG_OF_LIBFDT
                debug("using: FDT\n");
                if (image_setup_linux(images)) {
                        panic("FDT creation failed!");
                }
-#endif
        } else if (BOOTM_ENABLE_TAGS) {
                debug("using: ATAGS\n");
                setup_start_tag(gd->bd);
index 2e1e286..c1ac339 100644 (file)
@@ -65,13 +65,11 @@ static void announce_and_cleanup(int fake)
 static void boot_prep_linux(bootm_headers_t *images)
 {
        if (CONFIG_IS_ENABLED(OF_LIBFDT) && images->ft_len) {
-#ifdef CONFIG_OF_LIBFDT
                debug("using: FDT\n");
                if (image_setup_linux(images)) {
                        printf("FDT creation failed! hanging...");
                        hang();
                }
-#endif
        } else {
                printf("Device tree not found or missing FDT support\n");
                hang();