[PATCH 1/3] drm/exynos: Suspend/resume is unused if !PM
Andrzej Hajda
a.hajda at samsung.com
Fri Sep 25 01:47:05 PDT 2015
On 09/24/2015 12:14 PM, Thierry Reding wrote:
> From: Thierry Reding <treding at nvidia.com>
>
> 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 at nvidia.com>
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)
> {
>
More information about the dri-devel
mailing list