On Thu, Mar 14, 2013 at 02:01:20PM -0300, Fabio Estevam wrote: > Hi Wolfgang, > > On Thu, Mar 14, 2013 at 9:31 AM, Wolfgang Denk <w...@denx.de> wrote: > > Dear Fabio Estevam, > > > > In message <1363228354-29534-1-git-send-email-feste...@gmail.com> you wrote: > >> From: Fabio Estevam <fabio.este...@freescale.com> > >> > >> Add initial support for Wandboard. > > > > Subject and commit message are redundant, resulting in text like this: > > > > Add initial support for Wandboard dual lite and solo. > > > > Add initial support for Wandboard. > > > > Please remove the second line. > > Thanks for the review. I will address your comments and send v5 after > the nitrogen patches are applied into u-boot.imx. > > >> +#define CONFIG_BOOTDELAY 1 > > > > Is there any reason for not chosing the more standard 5 second delay? > > Ok, so let's go with 3 seconds then ;-)
There's a pretty even distribution of 1 3 and 5 second delays (with a few 10s, 2s and 6s). If they want 1, let them have 1, it's not hard to break into, you have U-Boot starting + 1sec. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
