2015-06-03 Alexey Klimov <klimov.linux@xxxxxxxxx>:
> Hi Gustavo,
>
> On Wed, Jun 3, 2015 at 5:30 PM, Gustavo Padovan <gustavo@xxxxxxxxxxx> wrote:
> > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> >
> > Check error and call DRM_ERROR if clk_prepare_enable() fails.
> >
> > Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> > ---
> > drivers/gpu/drm/exynos/exynos7_drm_decon.c | 29 ++++++++++++++++++++++++----
> > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 14 ++++++++++++--
> > drivers/gpu/drm/exynos/exynos_mixer.c | 31 +++++++++++++++++++++++++-----
> > 3 files changed, 63 insertions(+), 11 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/exynos/exynos7_drm_decon.c b/drivers/gpu/drm/exynos/exynos7_drm_decon.c
> > index d659ba2..ffd7c3b 100644
> > --- a/drivers/gpu/drm/exynos/exynos7_drm_decon.c
> > +++ b/drivers/gpu/drm/exynos/exynos7_drm_decon.c
> > @@ -606,6 +606,7 @@ static void decon_init(struct decon_context *ctx)
> > static void decon_enable(struct exynos_drm_crtc *crtc)
> > {
> > struct decon_context *ctx = crtc->ctx;
> > + int ret;
> >
> > if (!ctx->suspended)
> > return;
> > @@ -614,10 +615,30 @@ static void decon_enable(struct exynos_drm_crtc *crtc)
> >
> > pm_runtime_get_sync(ctx->dev);
> >
> > - clk_prepare_enable(ctx->pclk);
> > - clk_prepare_enable(ctx->aclk);
> > - clk_prepare_enable(ctx->eclk);
> > - clk_prepare_enable(ctx->vclk);
> > + ret = clk_prepare_enable(ctx->pclk);
> > + if (ret < 0) {
> > + DRM_ERROR("Failed to prepare_enable the pclk [%d]\n", ret);
> > + return;
> > + goto pclk_err;
>
> This goto after return probably got here by mistake. Debug/rebase leftover?
Yes. Thanks for pointing it out. I'll send an updated patch fixing this.
>
> <..snip..>
>
>
> By the way, are you using some branch to prepare this patch?
> Could you please check if you're able to apple it?
> In current master branch that i see similar DRM_ERROR() messages are
> already in place in similar functions. For example, in master branch i
> see decon_poweron() instead of decon_enable() and fimd_poweron()
> instead of fimd_enable() in your patch. Is there any chance that it's
> slightly outdated?
This is based on tree of today, with the atomic modesetting patches
applied.
Gustavo
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
