[U-Boot] [PATCH 2/2] api: export LCD and video to external apps

Wolfgang Denk
2011-10-07T09:05:51Z

Dear Che-liang Chiou,

In message <CANJuy2KuhG+4kueEupj0PBeFKJdhAJJnt_8-vULXYUXh9oCWKQ@mail.gmail.com> you wrote:
>
> >> + ub_display_clear();
> >
> > What happens here if no display is available?
>
> It is equivalent to a no-op. Or do you think we should print a warning
> message if no display is available?

Please either add an if() and don;t run the function at all, or add a
comment that explains that this callis harmless if no display is
present.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"He was so narrow minded he could see through a keyhole with both
eyes ..."