On Wed, 2014-03-26 at 10:03 +0100, Wolfgang Denk wrote: > Dear Ian Campbell, > > In message <1395822781.29683.12.ca...@dagon.hellion.org.uk> you wrote: > > On Mon, 2014-03-24 at 21:54 +0100, Marek Vasut wrote: > > > > > > +int sunxi_gpio_set_cfgpin(u32 pin, u32 val); > > > > +int sunxi_gpio_get_cfgpin(u32 pin); > > > > +int sunxi_gpio_set_drv(u32 pin, u32 val); > > > > +int sunxi_gpio_set_pull(u32 pin, u32 val); > > > > +int name_to_gpio(const char *name); > > > > +#define name_to_gpio name_to_gpio > > > > > > What is this ugly define doing here ? > > > > common/cmd_gpio.c uses the #ifndef name_to_gpio pattern to provide (or > > not) a default fallback implementation. I think this is a reasonably > > (but not very) common idiom for such cases where the non-default variant > > is not best expressed as a macro. > > Please add a comment to explain that.
Unless you object I think I'll do as Marek suggested name the function sunxi_name_to_gpio and make the #define to that, it seems more consistent that way. Ian. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
