[U-Boot] [PATCH 1/1 v2] console: USB: KBD: Fix incorrect autoboot timeout

Jim Lin jilin at nvidia.com
Thu Jan 24 11:38:05 CET 2013


On Thu, 2013-01-24 at 03:33 +0800, Wolfgang Denk wrote:
> Dear Jim Lin,
> 
> In message <1358937511-32664-1-git-send-email-jilin at nvidia.com> you wrote:
> > Autoboot timeout defined by CONFIG_BOOTDELAY will not be accurate if
> > CONFIG_USB_KEYBOARD and CONFIG_SYS_USB_EVENT_POLL are defined in
> > configuration file and when tstc() function for checking key pressed
> > takes longer time than 10 ms (e.g., 50 ms) to finish.
> 
> Thanks.  One minor nitpick...
> 
> > +		/* delay 1000 ms */
> > +		ts = get_timer(0);
> > +		do {
> >  			if (tstc()) {	/* we got a key press	*/
> >  				abort  = 1;	/* don't auto boot	*/
> >  				bootdelay = 0;	/* no more delay	*/
> > @@ -263,8 +263,7 @@ int abortboot(int bootdelay)
> >  # endif
> >  				break;
> >  			}
> > -			udelay(10000);
> > -		}
> > +		} while (!abort && get_timer(ts) < 1000);
> 
> I recommend to keep a short udelay() [say, an udelay(1000)] in the
> loop, as this will make sure that watchdog still gets triggered on
> systems that need this.
> 
I will keep
   udelay(10000);
for safety in next patch.

Thanks.

-- nvpublic



More information about the U-Boot mailing list