Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-08-19 Thread Marek Vasut
On 8/17/21 11:43 AM, Raphael Gallais-Pou wrote: On 7/2/21 8:07 PM, Marek Vasut wrote: On 7/2/21 11:23 AM, Raphael Gallais-Pou wrote: Hello Marek, Hi, Sorry for the late answer. No worries, take your time On 6/30/21 2:35 AM, Marek Vasut wrote: On 6/29/21 1:58 PM, Raphael GALLAIS-POU -

Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-08-17 Thread Raphael Gallais-Pou
On 7/2/21 8:07 PM, Marek Vasut wrote: > On 7/2/21 11:23 AM, Raphael Gallais-Pou wrote: >> Hello Marek, > > Hi, > >> Sorry for the late answer. > > No worries, take your time > >> On 6/30/21 2:35 AM, Marek Vasut wrote: >>> On 6/29/21 1:58 PM, Raphael GALLAIS-POU - foss wrote: >>> >>> [...] >>>

Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-07-06 Thread yannick Fertre
Hi Raphaël, thanks for the patch. Tested-by: Yannick Fertre On 6/29/21 1:58 PM, Raphael GALLAIS-POU - foss wrote: Bugzilla ticket: https://intbugzilla.st.com/show_bug.cgi?id=60620 Gerrit patch: https://gerrit.st.com/c/mpu/oe/st/linux-stm32/+/208093/ In the LTDC driver, pm_runtime_get_sync

Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-07-02 Thread Marek Vasut
On 7/2/21 11:23 AM, Raphael Gallais-Pou wrote: Hello Marek, Hi, Sorry for the late answer. No worries, take your time On 6/30/21 2:35 AM, Marek Vasut wrote: On 6/29/21 1:58 PM, Raphael GALLAIS-POU - foss wrote: [...] +++ b/drivers/gpu/drm/stm/ltdc.c @@ -425,10 +425,17 @@ static void

Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-07-02 Thread Raphael Gallais-Pou
Hello Marek, Sorry for the late answer. On 6/30/21 2:35 AM, Marek Vasut wrote: On 6/29/21 1:58 PM, Raphael GALLAIS-POU - foss wrote: [...] +++ b/drivers/gpu/drm/stm/ltdc.c @@ -425,10 +425,17 @@ static void ltdc_crtc_atomic_enable(struct drm_crtc *crtc,   {   struct ltdc_device *ldev

Re: [PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-06-29 Thread Marek Vasut
On 6/29/21 1:58 PM, Raphael GALLAIS-POU - foss wrote: [...] +++ b/drivers/gpu/drm/stm/ltdc.c @@ -425,10 +425,17 @@ static void ltdc_crtc_atomic_enable(struct drm_crtc *crtc, { struct ltdc_device *ldev = crtc_to_ltdc(crtc); struct drm_device *ddev = crtc->dev; + int ret;

[PATCH] drm/stm: ltdc: improve pm_runtime to stop clocks

2021-06-29 Thread Raphael GALLAIS-POU - foss
Bugzilla ticket: https://intbugzilla.st.com/show_bug.cgi?id=60620 Gerrit patch: https://gerrit.st.com/c/mpu/oe/st/linux-stm32/+/208093/ In the LTDC driver, pm_runtime_get_sync was wrongly used and caused the LTDC pixel clock to be systematically enabled in the clock summary. After one simple use