On 05.04.2018 11:50, Enric Balletbo i Serra wrote:
> From: Tomasz Figa <tfiga@xxxxxxxxxxxx>
>
> It is not used anymore after last changes and it was not even correct to
> begin with as it assumed a 1:1 relation between a CRTC and encoder,
> while in fact a CRTC can be attached to multiple encoders.
>
> Signed-off-by: Tomasz Figa <tfiga@xxxxxxxxxxxx>
> Signed-off-by: Thierry Escande <thierry.escande@xxxxxxxxxxxxx>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
Reviewed-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
--
Regards
Andrzej
> ---
>
> drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 35 -----------------------------
> drivers/gpu/drm/rockchip/rockchip_drm_psr.h | 1 -
> 2 files changed, 36 deletions(-)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_psr.c b/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
> index 1bf5cba9a64d..b1988ac758d5 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
> @@ -40,23 +40,6 @@ struct psr_drv {
> int (*set)(struct drm_encoder *encoder, bool enable);
> };
>
> -static struct psr_drv *find_psr_by_crtc(struct drm_crtc *crtc)
> -{
> - struct rockchip_drm_private *drm_drv = crtc->dev->dev_private;
> - struct psr_drv *psr;
> -
> - mutex_lock(&drm_drv->psr_list_lock);
> - list_for_each_entry(psr, &drm_drv->psr_list, list) {
> - if (psr->encoder->crtc == crtc)
> - goto out;
> - }
> - psr = ERR_PTR(-ENODEV);
> -
> -out:
> - mutex_unlock(&drm_drv->psr_list_lock);
> - return psr;
> -}
> -
> static struct psr_drv *find_psr_by_encoder(struct drm_encoder *encoder)
> {
> struct rockchip_drm_private *drm_drv = encoder->dev->dev_private;
> @@ -173,24 +156,6 @@ static void rockchip_drm_do_flush(struct psr_drv *psr)
> mutex_unlock(&psr->lock);
> }
>
> -/**
> - * rockchip_drm_psr_flush - flush a single pipe
> - * @crtc: CRTC of the pipe to flush
> - *
> - * Returns:
> - * 0 on success, -errno on fail
> - */
> -int rockchip_drm_psr_flush(struct drm_crtc *crtc)
> -{
> - struct psr_drv *psr = find_psr_by_crtc(crtc);
> - if (IS_ERR(psr))
> - return PTR_ERR(psr);
> -
> - rockchip_drm_do_flush(psr);
> - return 0;
> -}
> -EXPORT_SYMBOL(rockchip_drm_psr_flush);
> -
> /**
> * rockchip_drm_psr_flush_all - force to flush all registered PSR encoders
> * @dev: drm device
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_psr.h b/drivers/gpu/drm/rockchip/rockchip_drm_psr.h
> index 40e026c14168..860c62494496 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_psr.h
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_psr.h
> @@ -16,7 +16,6 @@
> #define __ROCKCHIP_DRM_PSR___
>
> void rockchip_drm_psr_flush_all(struct drm_device *dev);
> -int rockchip_drm_psr_flush(struct drm_crtc *crtc);
>
> int rockchip_drm_psr_inhibit_put(struct drm_encoder *encoder);
> int rockchip_drm_psr_inhibit_get(struct drm_encoder *encoder);
--
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
