On 03/02/2016 05:25 PM, Simon Glass wrote:
Hi Stephen,

On 29 February 2016 at 16:39, Stephen Warren <swar...@wwwdotorg.org> wrote:
On 02/25/2016 09:00 PM, Simon Glass wrote:

Don't try to run commands when not supported.


diff --git a/arch/sandbox/cpu/start.c b/arch/sandbox/cpu/start.c


+int board_run_command(const char *cmdline)
+{
+       printf("## Commands are disabled. Please enable
CONFIG_CMDLINE.\n");
+
+       return 1;
+}
+


Isn't this part of patch 5/9?

No - this is the sandbox support. That patch merely declares the
function in a header. Any board can implement it, and each would do so
in a separate patch.

OK, but this patch implements the function before there's a prototype for it in the header. That doesn't seem like the correct order.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to