diff mbox

[U-Boot] zynq: Allow board specific files to over write CONFIG_EXTRA_ENV_SETTINGS.

Message ID 1461262472-732-1-git-send-email-philip@opensdr.com
State Accepted
Delegated to: Michal Simek
Headers show

Commit Message

Philip Balister April 21, 2016, 6:14 p.m. UTC
The zynq-common.h header file is usually included last in the board specific
header file. Check if the CONFIG_EXTRA_ENV_SETTINGS variable is set before
applying the default value. This makes it easier for a board to set a
board specific environment.

Cc: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Philip Balister <philip@opensdr.com>
---
 include/configs/zynq-common.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Michal Simek April 22, 2016, 4:41 a.m. UTC | #1
On 21.4.2016 20:14, Philip Balister wrote:
> The zynq-common.h header file is usually included last in the board specific
> header file. Check if the CONFIG_EXTRA_ENV_SETTINGS variable is set before
> applying the default value. This makes it easier for a board to set a
> board specific environment.
> 
> Cc: Michal Simek <michal.simek@xilinx.com>
> Signed-off-by: Philip Balister <philip@opensdr.com>
> ---
>  include/configs/zynq-common.h | 2 ++
>  1 file changed, 2 insertions(+)

Did you rebase it on the top of latest tree?

I have added this some time ago
"ARM: zynq: Enable option to overwrite default variables"
commit b7b3efe75a46c3d6bff43b7b314d1fff90ca65ec

Thanks,
Michal
diff mbox

Patch

diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h
index 9ff3df2..ea8b0b0 100644
--- a/include/configs/zynq-common.h
+++ b/include/configs/zynq-common.h
@@ -205,6 +205,7 @@ 
 #endif
 
 /* Default environment */
+#ifndef CONFIG_EXTRA_ENV_SETTINGS
 #define CONFIG_EXTRA_ENV_SETTINGS	\
 	"fit_image=fit.itb\0"		\
 	"load_addr=0x2000000\0"		\
@@ -227,6 +228,7 @@ 
 			"load usb 0 ${load_addr} ${fit_image} && " \
 			"bootm ${load_addr}; fi\0" \
 		DFU_ALT_INFO
+#endif
 
 #define CONFIG_BOOTCOMMAND		"run $modeboot"
 #define CONFIG_BOOTDELAY		3 /* -1 to Disable autoboot */