[PATCH 2/3] drm/exynos: fimc: Clock control is unused if !PM
Andrzej Hajda
a.hajda at samsung.com
Fri Sep 25 01:41:32 PDT 2015
On 09/24/2015 12:14 PM, Thierry Reding wrote:
> From: Thierry Reding <treding at nvidia.com>
>
> Protect the fimc_clk_ctrl() function with an #ifdef CONFIG_PM guard to
> avoid "defined but not used" warnings.
>
> Signed-off-by: Thierry Reding <treding at nvidia.com>
Reviewed-by: Andrzej Hajda <a.hajda at samsung.com>
Regards
Andrzej
> ---
> drivers/gpu/drm/exynos/exynos_drm_fimc.c | 36 ++++++++++++++++----------------
> 1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c b/drivers/gpu/drm/exynos/exynos_drm_fimc.c
> index 2a652359af64..dd3a5e6d58c8 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_fimc.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimc.c
> @@ -1206,23 +1206,6 @@ static struct exynos_drm_ipp_ops fimc_dst_ops = {
> .set_addr = fimc_dst_set_addr,
> };
>
> -static int fimc_clk_ctrl(struct fimc_context *ctx, bool enable)
> -{
> - DRM_DEBUG_KMS("enable[%d]\n", enable);
> -
> - if (enable) {
> - clk_prepare_enable(ctx->clocks[FIMC_CLK_GATE]);
> - clk_prepare_enable(ctx->clocks[FIMC_CLK_WB_A]);
> - ctx->suspended = false;
> - } else {
> - clk_disable_unprepare(ctx->clocks[FIMC_CLK_GATE]);
> - clk_disable_unprepare(ctx->clocks[FIMC_CLK_WB_A]);
> - ctx->suspended = true;
> - }
> -
> - return 0;
> -}
> -
> static irqreturn_t fimc_irq_handler(int irq, void *dev_id)
> {
> struct fimc_context *ctx = dev_id;
> @@ -1780,6 +1763,24 @@ static int fimc_remove(struct platform_device *pdev)
> return 0;
> }
>
> +#ifdef CONFIG_PM
> +static int fimc_clk_ctrl(struct fimc_context *ctx, bool enable)
> +{
> + DRM_DEBUG_KMS("enable[%d]\n", enable);
> +
> + if (enable) {
> + clk_prepare_enable(ctx->clocks[FIMC_CLK_GATE]);
> + clk_prepare_enable(ctx->clocks[FIMC_CLK_WB_A]);
> + ctx->suspended = false;
> + } else {
> + clk_disable_unprepare(ctx->clocks[FIMC_CLK_GATE]);
> + clk_disable_unprepare(ctx->clocks[FIMC_CLK_WB_A]);
> + ctx->suspended = true;
> + }
> +
> + return 0;
> +}
> +
> #ifdef CONFIG_PM_SLEEP
> static int fimc_suspend(struct device *dev)
> {
> @@ -1806,7 +1807,6 @@ static int fimc_resume(struct device *dev)
> }
> #endif
>
> -#ifdef CONFIG_PM
> static int fimc_runtime_suspend(struct device *dev)
> {
> struct fimc_context *ctx = get_fimc_context(dev);
>
More information about the dri-devel
mailing list