+Ilko Iliev <[email protected]> again Hi Joe,
On Wed, Feb 27, 2013 at 12:40 AM, Joe Hershberger <[email protected]> wrote: > Hi Simon, > > On Mon, Feb 25, 2013 at 11:28 PM, Simon Glass <[email protected]> wrote: >> Hi Joe, >> >> On Sun, Feb 24, 2013 at 11:53 AM, Joe Hershberger >> <[email protected]> wrote: >>> Hi Simon, >>> >>> On Sun, Feb 24, 2013 at 11:26 AM, Simon Glass <[email protected]> wrote: >>>> This is not currently used, since autoboot is not enabled for this >>>> board, but the string is missing a parameter. Add it. >>>> >>> >>> Why not enable autoboot for this board so that this setting gets testing? >> >> Actually with autoconf this setting is tested, which is how I found >> the problem. The old code used #ifdef and so the problem was masked. >> >> Or do you think I should enable because that was probably the board >> vendor's indent? > > What I was thinking is that if the board sets the configuration for > this, then they surely want this feature to work. It should be > enabled by default for this board (unless the maintainer explicitly > declares no, in which case the configuration should be removed > completely). I don't disagree with you, but I think that is a matter for the board maintainer. I found a minor problem in the config file which I fixed. But I don't really think I want to enable CONFIG_AUTOBOOT_KEYED, select some values for CONFIG_AUTOBOOT_DELAY_STR2, etc. as well. For my purposes I just need the board to continue to boot. Somehow we dropped Ilko Iliev <[email protected]> off the Cc here, so have re-added. Regards, Simon _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
