Hi Simon,
On 2016-05-15 02:49, Simon Glass wrote:
Allow this to be unset, such that gd->mon_len is invalid. This seems to
be
what the sh architecture does.
Signed-off-by: Simon Glass <s...@chromium.org>
---
common/board_f.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/board_f.c b/common/board_f.c
index 109025a..d405b5b 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -274,7 +274,7 @@ static int setup_mon_len(void)
gd->mon_len = CONFIG_SYS_MONITOR_LEN;
#elif defined(CONFIG_NDS32)
gd->mon_len = (ulong)(&__bss_end) - (ulong)(&_start);
-#else
+#elif defined(CONFIG_SYS_MONITOR_BASE)
/* TODO: use (ulong)&__bss_end - (ulong)&__text_start; ? */
gd->mon_len = (ulong)&__bss_end - CONFIG_SYS_MONITOR_BASE;
before this produced an compile error for undefiend
CONFIG_SYS_MONITOR_BASE, now it will be silently ignored. Why not leave
the else case and use the TODO statement instead?
At best we should equalize the linker files and remove the whole
ifdiffery here.
Andreas
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot