On Thu, Nov 25, 2010 at 05:52:23PM +0200, Srikar wrote: > +static struct regulator_consumer_supply rx51_vdac_supply[] = { > + { > +#if defined(CONFIG_FB_OMAP2) || defined(CONFIG_FB_OMAP2_MODULE) The ifdefs here aren't really saving much... > + .supply = "vdda_dac", > + .dev = &rx51_display_device.dev, dev_name is preferred, though I'm not sure if we'll ever get round to phasing out plain dev. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html