xilinx: common: Change macro handling in board_fdt_blob_setup()
authorMichal Simek <michal.simek@xilinx.com>
Mon, 4 Jan 2021 10:03:36 +0000 (11:03 +0100)
committerMichal Simek <michal.simek@xilinx.com>
Wed, 20 Jan 2021 07:34:53 +0000 (08:34 +0100)
Remove ifdef logic which is handled by preprocessor and move it link time
optimization to get full compile code coverage.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
board/xilinx/common/board.c

index cdc06a3..9f651db 100644 (file)
@@ -324,25 +324,29 @@ void *board_fdt_blob_setup(void)
 {
        void *fdt_blob;
 
-#if !defined(CONFIG_VERSAL_NO_DDR) && !defined(CONFIG_ZYNQMP_NO_DDR)
-       fdt_blob = (void *)CONFIG_XILINX_OF_BOARD_DTB_ADDR;
+       if (!IS_ENABLED(CONFIG_VERSAL_NO_DDR) &&
+           !IS_ENABLED(CONFIG_VERSAL_NO_DDR)) {
+               fdt_blob = (void *)CONFIG_XILINX_OF_BOARD_DTB_ADDR;
 
-       if (fdt_magic(fdt_blob) == FDT_MAGIC)
-               return fdt_blob;
+               if (fdt_magic(fdt_blob) == FDT_MAGIC)
+                       return fdt_blob;
 
-       debug("DTB is not passed via %p\n", fdt_blob);
-#endif
+               debug("DTB is not passed via %p\n", fdt_blob);
+       }
 
-#ifdef CONFIG_SPL_BUILD
-       /* FDT is at end of BSS unless it is in a different memory region */
-       if (IS_ENABLED(CONFIG_SPL_SEPARATE_BSS))
-               fdt_blob = (ulong *)&_image_binary_end;
-       else
-               fdt_blob = (ulong *)&__bss_end;
-#else
-       /* FDT is at end of image */
-       fdt_blob = (ulong *)&_end;
-#endif
+       if (IS_ENABLED(CONFIG_SPL_BUILD)) {
+               /*
+                * FDT is at end of BSS unless it is in a different memory
+                * region
+                */
+               if (IS_ENABLED(CONFIG_SPL_SEPARATE_BSS))
+                       fdt_blob = (ulong *)&_image_binary_end;
+               else
+                       fdt_blob = (ulong *)&__bss_end;
+       } else {
+               /* FDT is at end of image */
+               fdt_blob = (ulong *)&_end;
+       }
 
        if (fdt_magic(fdt_blob) == FDT_MAGIC)
                return fdt_blob;