[U-Boot] [PATCH 1/1] make MODEM SUPPORT generic instead of duplicate it
Wolfgang Denk
wd at denx.de
Fri Apr 24 21:54:30 CEST 2009
Dear Jean-Christophe PLAGNIOL-VILLARD,
In message <20090405085601.GA31840 at game.jcrosoft.org> you wrote:
>
> > > diff --git a/common/modem.c b/common/modem.c
> > > new file mode 100644
> > > index 0000000..27e164a
> > > --- /dev/null
> > > +++ b/common/modem.c
> > ...
> > > +/* called from main loop (common/main.c) */
> > > +/* 'inline' - We have to do it fast */
> > > +static inline void mdm_readline(char *buf, int bufsiz)
> >
> > How is this supposed to work? You cannot call a static function from
> > another source file.
> this function is only use in this file
>
> actualy the code is lib_arm/board.c and lib_ppc/board.c is identical
>
> I just move it to modem.c
>
> the only function use outiside of this file is mdm_init
Then please fix the comment - it says: "called from main loop
(common/main.c)"
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
Winners never talk about glorious victories. That's because they're
the ones who see what the battlefield looks like afterwards. It's
only the losers who have glorious victories.
- Terry Pratchett, _Small Gods_
More information about the U-Boot
mailing list