[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