Hi Simon,

On 04.04.2016 15:18, Simon Glass wrote:
On Apr 4, 2016 03:16, "Stefan Roese" <s...@denx.de <mailto:s...@denx.de>> wrote:
 >
 > Hi Simon, Hi Tom!
 >
 >
 > On 02.04.2016 03:55, Tom Rini wrote:
 >>
 >> On Sat, Mar 19, 2016 at 02:18:38AM -0600, Simon Glass wrote:
 >>
 >>> Command parsing and processing code is not needed when the command
line is
 >>> disabled. Remove this code in that case.
 >>>
 >>> Signed-off-by: Simon Glass <s...@chromium.org <mailto:s...@chromium.org>>
 >>> Reviewed-by: Tom Rini <tr...@konsulko.com <mailto:tr...@konsulko.com>>
 >>
 >>
 >> Applied to u-boot/master, thanks!
 >
 >
 > I just noticed that this patch breaks U-Boot on some boards. All that
 > don't have CONFIG_SYS_HUSH_PARSER or CONFIG_CMDLINE enabled (e.g.
 > some MVEBU board, like the "db-mv784mp-gp".
 >
 > How should this be solved? Should we enable CONFIG_CMDLINE per
 > default (via config_defaults.h ?)? Or should we enable CONFIG_CMDLINE
 > in all board config headers that are currently missing it?

All boards should have CONFIG_CMDLINE enables by default.

That's currently not the case. At least some MVEBU (Armada XP / 38x)
boards don't have it enabled. Here a short (most likely incomplete)
list:

clearfog
db-88f6820-gp
db-mv784mp-gp_defconfig
ds414
maxbcm

And probably some (most?) Orion & Kirkwood based board as well. I
assume that most boards that include "mv-common.h" have it not
enabled (I didn't check all of them).

Some board, like theadorable have HUSH support enabled but
CMDLINE not. These boards are not broken.

Is the problem
with the build or at run-time?

Run-time.

Thanks,
Stefan

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to