Re: [PATCH] drm/exynos: add check for the device power status

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Wed, Jan 23, 2013 at 8:18 PM, Sean Paul <seanpaul@xxxxxxxxxxxx> wrote:
On Wed, Jan 23, 2013 at 5:10 AM, Shirish S <s.shirish@xxxxxxxxxxx> wrote:
> The hdmi and mixer win_commit calls currently are
> not checking the status of IP before updating the
> respective registers, this patch adds this check.
>
> Signed-off-by: Shirish S <s.shirish@xxxxxxxxxxx>
> ---
>  drivers/gpu/drm/exynos/exynos_hdmi.c  | 3 +++
>  drivers/gpu/drm/exynos/exynos_mixer.c | 3 +++
>  2 files changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c
> index 2c46b6c..ae79688 100644
> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c
> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
> @@ -2170,6 +2170,9 @@ static void hdmi_commit(void *ctx)
>
>         DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__);
>
> +       if (!hdata->powered)

The rest of the driver protects reads & writes to powered with hdmi_mutex

> +               return;
> +
>         hdmi_conf_apply(hdata);
>  }
>
> diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c
> index 21db895..4f5433f 100644
> --- a/drivers/gpu/drm/exynos/exynos_mixer.c
> +++ b/drivers/gpu/drm/exynos/exynos_mixer.c
> @@ -775,6 +775,9 @@ static void mixer_win_commit(void *ctx, int win)
>
>         DRM_DEBUG_KMS("[%d] %s, win: %d\n", __LINE__, __func__, win);
>
> +       if (!mixer_ctx->powered)

This is also protected everywhere else in the driver.

Thanks Sean, have uploaded patch set 2. 
> +               return;
> +
>         if (win > 1 && mixer_ctx->vp_enabled)
>                 vp_video_buffer(mixer_ctx, win);
>         else
> --
> 1.8.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Regards,
Shirish S
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux