On 16.08.21 15:19, Marek Behún wrote:
We do not need to guard code in board_init() and board_late_init()
functions with the CONFIG_SPL_BUILD macro, since these functions are not
called in SPL.

Signed-off-by: Marek Behún <[email protected]>

Reviewed-by: Stefan Roese <[email protected]>

Thanks,
Stefan

---
  board/CZ.NIC/turris_omnia/turris_omnia.c | 8 --------
  1 file changed, 8 deletions(-)

diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c 
b/board/CZ.NIC/turris_omnia/turris_omnia.c
index a7e5f56eed..a84a409f43 100644
--- a/board/CZ.NIC/turris_omnia/turris_omnia.c
+++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
@@ -129,7 +129,6 @@ static int omnia_mcu_read(u8 cmd, void *buf, int len)
        return dm_i2c_read(chip, cmd, buf, len);
  }
-#ifndef CONFIG_SPL_BUILD
  static int omnia_mcu_write(u8 cmd, const void *buf, int len)
  {
        struct udevice *chip;
@@ -158,7 +157,6 @@ static bool disable_mcu_watchdog(void)
return true;
  }
-#endif
static bool omnia_detect_sata(void)
  {
@@ -325,7 +323,6 @@ struct mv_ddr_topology_map *mv_ddr_topology_map_get(void)
                return &board_topology_map_1g;
  }
-#ifndef CONFIG_SPL_BUILD
  static int set_regdomain(void)
  {
        struct omnia_eeprom oep;
@@ -394,7 +391,6 @@ static void handle_reset_button(void)
                }
        }
  }
-#endif
int board_early_init_f(void)
  {
@@ -428,19 +424,15 @@ int board_init(void)
        /* address of boot parameters */
        gd->bd->bi_boot_params = mvebu_sdram_bar(0) + 0x100;
-#ifndef CONFIG_SPL_BUILD
        disable_mcu_watchdog();
-#endif
return 0;
  }
int board_late_init(void)
  {
-#ifndef CONFIG_SPL_BUILD
        set_regdomain();
        handle_reset_button();
-#endif
        pci_init();
return 0;



Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: [email protected]

Reply via email to