| Message ID | 1554283453-2879-1-git-send-email-yannick.fertre@st.com |
|---|---|
| State | New |
| Headers | show |
| Series | drm/stm: ltdc: reset controller to avoid partial refresh | expand |
On 4/3/19 11:24 AM, Yannick Fertré wrote: > Display controller reset must be done as soon as possible after enable > the clock to avoid partial refresh on screen. > > Signed-off-by: Yannick Fertré <yannick.fertre@st.com> > --- > drivers/gpu/drm/stm/ltdc.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > index 32fd6a3..7bbe61c 100644 > --- a/drivers/gpu/drm/stm/ltdc.c > +++ b/drivers/gpu/drm/stm/ltdc.c > @@ -1134,6 +1134,12 @@ int ltdc_load(struct drm_device *ddev) > return -ENODEV; > } > > + if (!IS_ERR(rstc)) { > + reset_control_assert(rstc); > + usleep_range(10, 20); > + reset_control_deassert(rstc); > + } > + > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > ldev->regs = devm_ioremap_resource(dev, res); > if (IS_ERR(ldev->regs)) { > @@ -1156,12 +1162,6 @@ int ltdc_load(struct drm_device *ddev) > } > } > > - if (!IS_ERR(rstc)) { > - reset_control_assert(rstc); > - usleep_range(10, 20); > - reset_control_deassert(rstc); > - } > - Dear Yannick, Thank you for your patch, Acked-by: Philippe Cornu <philippe.cornu@st.com> Philippe :) > /* Disable interrupts */ > reg_clear(ldev->regs, LTDC_IER, > IER_LIE | IER_RRIE | IER_FUIE | IER_TERRIE); >
Le ven. 5 avr. 2019 à 18:31, Philippe CORNU <philippe.cornu@st.com> a écrit : > > > > On 4/3/19 11:24 AM, Yannick Fertré wrote: > > Display controller reset must be done as soon as possible after enable > > the clock to avoid partial refresh on screen. > > > > Signed-off-by: Yannick Fertré <yannick.fertre@st.com> > > --- > > drivers/gpu/drm/stm/ltdc.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > > index 32fd6a3..7bbe61c 100644 > > --- a/drivers/gpu/drm/stm/ltdc.c > > +++ b/drivers/gpu/drm/stm/ltdc.c > > @@ -1134,6 +1134,12 @@ int ltdc_load(struct drm_device *ddev) > > return -ENODEV; > > } > > > > + if (!IS_ERR(rstc)) { > > + reset_control_assert(rstc); > > + usleep_range(10, 20); > > + reset_control_deassert(rstc); > > + } > > + > > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > > ldev->regs = devm_ioremap_resource(dev, res); > > if (IS_ERR(ldev->regs)) { > > @@ -1156,12 +1162,6 @@ int ltdc_load(struct drm_device *ddev) > > } > > } > > > > - if (!IS_ERR(rstc)) { > > - reset_control_assert(rstc); > > - usleep_range(10, 20); > > - reset_control_deassert(rstc); > > - } > > - > > Dear Yannick, > Thank you for your patch, > > Acked-by: Philippe Cornu <philippe.cornu@st.com> Applied on drm-misc-next after doing a rebase. Thanks, Benjamin > > Philippe :) > > > /* Disable interrupts */ > > reg_clear(ldev->regs, LTDC_IER, > > IER_LIE | IER_RRIE | IER_FUIE | IER_TERRIE); > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index 32fd6a3..7bbe61c 100644 --- a/drivers/gpu/drm/stm/ltdc.c +++ b/drivers/gpu/drm/stm/ltdc.c @@ -1134,6 +1134,12 @@ int ltdc_load(struct drm_device *ddev) return -ENODEV; } + if (!IS_ERR(rstc)) { + reset_control_assert(rstc); + usleep_range(10, 20); + reset_control_deassert(rstc); + } + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); ldev->regs = devm_ioremap_resource(dev, res); if (IS_ERR(ldev->regs)) { @@ -1156,12 +1162,6 @@ int ltdc_load(struct drm_device *ddev) } } - if (!IS_ERR(rstc)) { - reset_control_assert(rstc); - usleep_range(10, 20); - reset_control_deassert(rstc); - } - /* Disable interrupts */ reg_clear(ldev->regs, LTDC_IER, IER_LIE | IER_RRIE | IER_FUIE | IER_TERRIE);
Display controller reset must be done as soon as possible after enable the clock to avoid partial refresh on screen. Signed-off-by: Yannick Fertré <yannick.fertre@st.com> --- drivers/gpu/drm/stm/ltdc.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)