[U-Boot] [RFC PATCH 3/3 V2] stdio/device: rework function naming convention

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sun May 10 16:17:02 CEST 2009


On 19:32 Sat 09 May     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> 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

>  
> > -#if defined(CONFIG_SPLASH_SCREEN) && !defined(CONFIG_SYS_DEVICE_NULLDEV)
> > -#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

Best Regards,
J.


More information about the U-Boot mailing list