On 09/24/2015 12:14 PM, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > Protect the suspend and resume callbacks with an #ifdef CONFIG_PM_SLEEP > guard to avoid "defined but not used" warnings. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> The best solution would be to merge legacy exynos_drm_(suspend|resume) PM functions with exynos_drm_sys_(suspend|resume). There is no reason to keep both. Could you do it, if not I can prepare quick patch. Regards Andrzej > --- > drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c b/drivers/gpu/drm/exynos/exynos_drm_drv.c > index f0a5839bd226..e07a0fe16432 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_drv.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.c > @@ -304,6 +304,7 @@ int exynos_atomic_commit(struct drm_device *dev, struct drm_atomic_state *state, > return 0; > } > > +#ifdef CONFIG_PM_SLEEP > static int exynos_drm_suspend(struct drm_device *dev, pm_message_t state) > { > struct drm_connector *connector; > @@ -340,6 +341,7 @@ static int exynos_drm_resume(struct drm_device *dev) > > return 0; > } > +#endif > > static int exynos_drm_open(struct drm_device *dev, struct drm_file *file) > { > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html