[U-Boot] [PATCH] POST: Remove duplicated post_hotkey_pressed() functions
Stefan Roese
sr at denx.de
Wed Nov 25 11:05:27 CET 2009
Hi Wolfgang,
On Tuesday 24 November 2009 23:35:02 Wolfgang Denk wrote:
> Hm... but the implementations were actually different. Some boards
>
> used ctrlc() so they would catch a (prebably previously pressed) ^C:
> > --- a/board/amcc/katmai/katmai.c
> > +++ b/board/amcc/katmai/katmai.c
>
> ..
>
> > -int post_hotkeys_pressed(void)
> > -{
> > - return (ctrlc());
> > -}
> > -#endif
>
> ..while others unconditionally return 09, without checking for ^C:
> > --- a/board/amcc/kilauea/kilauea.c
> > +++ b/board/amcc/kilauea/kilauea.c
>
> ..
>
> > -int post_hotkeys_pressed(void)
> > -{
> > - return 0; /* No hotkeys supported */
> > -}
>
> Are you sure the suggested change does not change the behavious on any
> of these boards?
Yes. The boards with this ctrlc() checking don't have POST support enabled at
all (e.g. Katmai). So the behaviour is not changed.
Cheers,
Stefan
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de
More information about the U-Boot
mailing list