On Wednesday 22 January 2014, Hans de Goede wrote: > --- a/include/linux/ahci_platform.h > +++ b/include/linux/ahci_platform.h > @@ -50,4 +50,11 @@ int ahci_platform_init_host(struct platform_device *pdev, > unsigned int force_port_map, > unsigned int mask_port_map); > > +#ifdef CONFIG_PM_SLEEP > +int ahci_platform_suspend_host(struct device *dev); > +int ahci_platform_resume_host(struct device *dev); > +int ahci_platform_suspend(struct device *dev); > +int ahci_platform_resume(struct device *dev); > +#endif > + Not sure if the #ifdef does any good here. Normally, we don't hide declarations so we can do stuff like if (IS_ENABLED(CONFIG_PM_SLEEP)) ret = ahci_platform_suspend_host(dev); and expect the code to compile and link just fine. 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