diff mbox series

[stable,4.19,1/4] ARM: dts: am4372: Set memory bandwidth limit for DISPC

Message ID 20191015065937.23169-1-mathieu.poirier@linaro.org (mailing list archive)
State New, archived
Headers show
Series [stable,4.19,1/4] ARM: dts: am4372: Set memory bandwidth limit for DISPC | expand

Commit Message

Mathieu Poirier Oct. 15, 2019, 6:59 a.m. UTC
From: Peter Ujfalusi <peter.ujfalusi@ti.com>

commit f90ec6cdf674248dcad85bf9af6e064bf472b841 upstream

Set memory bandwidth limit to filter out resolutions above 720p@60Hz to
avoid underflow errors due to the bandwidth needs of higher resolutions.

am43xx can not provide enough bandwidth to DISPC to correctly handle
'high' resolutions.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Cc: stable <stable@vger.kernel.org> # 4.19
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
---
 arch/arm/boot/dts/am4372.dtsi | 2 ++
 1 file changed, 2 insertions(+)

Comments

Greg KH Oct. 16, 2019, 6:31 p.m. UTC | #1
On Tue, Oct 15, 2019 at 12:59:34AM -0600, Mathieu Poirier wrote:
> From: Peter Ujfalusi <peter.ujfalusi@ti.com>
> 
> commit f90ec6cdf674248dcad85bf9af6e064bf472b841 upstream
> 
> Set memory bandwidth limit to filter out resolutions above 720p@60Hz to
> avoid underflow errors due to the bandwidth needs of higher resolutions.
> 
> am43xx can not provide enough bandwidth to DISPC to correctly handle
> 'high' resolutions.
> 
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Signed-off-by: Tony Lindgren <tony@atomide.com>
> Cc: stable <stable@vger.kernel.org> # 4.19
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
>  arch/arm/boot/dts/am4372.dtsi | 2 ++
>  1 file changed, 2 insertions(+)

What about 5.3?  Is this ok there?

thanks,

greg k-h
Mathieu Poirier Oct. 17, 2019, 4:50 p.m. UTC | #2
On Wed, 16 Oct 2019 at 12:31, Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Tue, Oct 15, 2019 at 12:59:34AM -0600, Mathieu Poirier wrote:
> > From: Peter Ujfalusi <peter.ujfalusi@ti.com>
> >
> > commit f90ec6cdf674248dcad85bf9af6e064bf472b841 upstream
> >
> > Set memory bandwidth limit to filter out resolutions above 720p@60Hz to
> > avoid underflow errors due to the bandwidth needs of higher resolutions.
> >
> > am43xx can not provide enough bandwidth to DISPC to correctly handle
> > 'high' resolutions.
> >
> > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> > Signed-off-by: Tony Lindgren <tony@atomide.com>
> > Cc: stable <stable@vger.kernel.org> # 4.19
> > Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> > ---
> >  arch/arm/boot/dts/am4372.dtsi | 2 ++
> >  1 file changed, 2 insertions(+)
>
> What about 5.3?  Is this ok there?
>

Yes - all the patches in this series are also applicable (and apply) to 5.3.y

Thanks,
Mathieu

> thanks,
>
> greg k-h
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index d4b7c59eec68..cf1e4f747242 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -1142,6 +1142,8 @@ 
 				ti,hwmods = "dss_dispc";
 				clocks = <&disp_clk>;
 				clock-names = "fck";
+
+				max-memory-bandwidth = <230000000>;
 			};
 
 			rfbi: rfbi@4832a800 {