[U-Boot] [RFC PATCH 3/3 V2] stdio/device: rework function naming convention
Wolfgang Denk
wd at denx.de
Sun May 10 21:37:38 CEST 2009
Dear Jean-Christophe PLAGNIOL-VILLARD,
In message <20090510141702.GA21079 at game.jcrosoft.org> you wrote:
>
> > In message <1241863009-26036-3-git-send-email-plagnioj at jcrosoft.com> you wrote:
> > > actually the console API use the following naming convention
> >
> > Please look up the difference in meaning between "actually" and "cur-
> > rently"; I think you usually mean "currently" when you say "actually".
> >
> >
> > This patch should preceed the previous one, where you already use the
> > new, not yet introduced "stdio_" naming scheme.
> >
> > > ======Extract======
> > > typedef struct device_t;
> > >
> > > int device_register (device_t * dev);
> > > int devices_init (void);
> > > int device_deregister(char *devname);
> > > struct list_head* device_get_list(void);
> > > device_t* device_get_by_name(char* name);
> > > device_t* device_clone(device_t *dev);
> > > =======
> > >
> > > which is not console dependent and is confusing
> >
> > I think this explanation is confusing, too, as I don't know whjat you
> > mean by "console dependent".
> device is a generic name
Can you please try and explain what you find confusing, and what you
mean by "not console dependent"? I cannot even guess what you mean
here.
> > > -#define CONFIG_SYS_DEVICE_NULLDEV 1
> > > +#if defined(CONFIG_SPLASH_SCREEN) && !defined(CONFIG_SYS_STDIO_NULLDEV)
> > > +#define CONFIG_SYS_STDIO_NULLDEV 1
> > > #endif
> >
> > I'm not happy about the CONFIG_SYS_STDIO_NULLDEV; the "NULL device" is
> > actually a more general concept and not limited to the stdio handling.
> > Please do not put a "STDIO" in this name.
> here it's to active the stdio null device
No. If you are talking about "a more general concept" which is "not
limited to the stdio handling", then we should do it right. Then
there is no such thing as a "stdio null device". There is only a null
device, and we assign it to one or more stdio channels here. Please
do not name this "stdio null device" or ...STDIO_NULLDEV.
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
PROGRAM - n. A magic spell cast over a computer allowing it to turn
one's input into error messages.
v. tr. - To engage in a pastime similar to banging one's head against
a wall, but with fewer opportunities for reward.
More information about the U-Boot
mailing list