On Wed, Dec 12, 2012 at 12:58 PM, Jean-Christophe PLAGNIOL-VILLARD <[email protected]> wrote:
>> > +#ifdef CONFIG_PINCTRL >> > + struct dev_pin_info pins; >> > +#endif > can do do not put ifdef in the struct as if will force us to put ifdef in the > c code too instead of just use a inline function Oh, I'm just planning to use this from the pinctrl core code itself and the code in drivers/base/pinctrl.c (which is only compiled when CONFIG_PINCTRL is set) so there is no risk for that right now. I would agree if we exposed to drivers, like the dev_pm_ops... but that's not the plan right now. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
