On Wed, Jan 23, 2013 at 8:18 PM, Sean Paul <seanpaul at chromium.org> wrote:
> On Wed, Jan 23, 2013 at 5:10 AM, Shirish S <s.shirish at samsung.com> 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 at samsung.com>
> > ---
> > 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 at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
Regards,
Shirish S
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130124/e540e529/attachment.html>
