MLK-22304-1 drm/imx: lcdif-dsim: fix build warnings if CONFIG_PM_SLEEP off
authorFancy Fang <chen.fang@nxp.com>
Tue, 23 Jul 2019 08:02:18 +0000 (16:02 +0800)
committerFancy Fang <chen.fang@nxp.com>
Wed, 24 Jul 2019 01:56:38 +0000 (09:56 +0800)
When CONFIG_PM_SLEEP is disabled, the suspend and resume hooks
are implemented as dummy functions, but GCC will report below
build warnings:

drivers/gpu/imx/lcdif/lcdif-common.c:731:12: warning: ‘imx_lcdif_suspend’ defined but not used [-Wunused-function]
 static int imx_lcdif_suspend(struct device *dev)
            ^
drivers/gpu/imx/lcdif/lcdif-common.c:735:12: warning: ‘imx_lcdif_resume’ defined but not used [-Wunused-function]
 static int imx_lcdif_resume(struct device *dev)
            ^
drivers/gpu/drm/imx/sec_mipi_dsim-imx.c:324:12: warning: ‘imx_sec_dsim_suspend’ defined but not used [-Wunused-function]
 static int imx_sec_dsim_suspend(struct device *dev)
            ^
drivers/gpu/drm/imx/sec_mipi_dsim-imx.c:329:12: warning: ‘imx_sec_dsim_resume’ defined but not used [-Wunused-function]
 static int imx_sec_dsim_resume(struct device *dev)
            ^

So remove these dummy functions to avoid these build warnings.

Signed-off-by: Fancy Fang <chen.fang@nxp.com>
drivers/gpu/drm/imx/sec_mipi_dsim-imx.c
drivers/gpu/imx/lcdif/lcdif-common.c

index bb833f8..a3fe4aa 100644 (file)
@@ -320,16 +320,6 @@ static int imx_sec_dsim_resume(struct device *dev)
 {
        return imx_sec_dsim_runtime_resume(dev);
 }
-#else
-static int imx_sec_dsim_suspend(struct device *dev)
-{
-       return 0;
-}
-
-static int imx_sec_dsim_resume(struct device *dev)
-{
-       return 0;
-}
 #endif
 
 #ifdef CONFIG_PM
index c190419..9425605 100644 (file)
@@ -727,15 +727,6 @@ static int imx_lcdif_resume(struct device *dev)
 {
        return imx_lcdif_runtime_resume(dev);
 }
-#else
-static int imx_lcdif_suspend(struct device *dev)
-{
-       return 0;
-}
-static int imx_lcdif_resume(struct device *dev)
-{
-       return 0;
-}
 #endif
 
 #ifdef CONFIG_PM