On Tue, Jan 2, 2018 at 3:42 PM, Sudeep Holla <sudeep.holla@xxxxxxx> wrote: > +#ifdef CONFIG_ARM_SCMI_PROTOCOL > +int scmi_driver_register(struct scmi_driver *driver, > + struct module *owner, const char *mod_name); > +void scmi_driver_unregister(struct scmi_driver *driver); > +#else > +static int scmi_driver_register(struct scmi_driver *driver, > + struct module *owner, const char *mod_name) > +{ > + return -EINVAL; > +} > +static void scmi_driver_unregister(struct scmi_driver *driver) {} > +#endif /* CONFIG_ARM_SCMI_PROTOCOL */ The dummy helpers must be 'static inline', not just 'static', otherwise you get a warning when this header is included by a file that doesn't call both of them. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html