xilinx: common: Check return value from variable setup
authorMichal Simek <michal.simek@xilinx.com>
Wed, 12 Aug 2020 10:16:49 +0000 (12:16 +0200)
committerMichal Simek <michal.simek@xilinx.com>
Thu, 20 Aug 2020 07:58:16 +0000 (09:58 +0200)
env_set..() can failed that's why check return status and report it back to
make sure that user is aware that's something went wrong.

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

index 901591ba2a854bbc9bde6908d67601020da60b9c..581c88ad49a48cd43a7a804c43ed63541dfa1491 100644 (file)
@@ -77,10 +77,15 @@ void *board_fdt_blob_setup(void)
 
 int board_late_init_xilinx(void)
 {
-       env_set_hex("script_offset_f", CONFIG_BOOT_SCRIPT_OFFSET);
+       u32 ret = 0;
 
-       env_set_addr("bootm_low", (void *)gd->ram_base);
-       env_set_addr("bootm_size", (void *)gd->ram_size);
+       ret |= env_set_hex("script_offset_f", CONFIG_BOOT_SCRIPT_OFFSET);
+
+       ret |= env_set_addr("bootm_low", (void *)gd->ram_base);
+       ret |= env_set_addr("bootm_size", (void *)gd->ram_size);
+
+       if (ret)
+               printf("%s: Saving run time variables FAILED\n", __func__);
 
        return 0;
 }