* [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth
@ 2019-07-25 17:12 Kevin Brodsky
2019-07-26 9:50 ` Sudeep Holla
2019-08-05 10:31 ` Linus Walleij
0 siblings, 2 replies; 3+ messages in thread
From: Kevin Brodsky @ 2019-07-25 17:12 UTC (permalink / raw)
To: linux-arm-kernel
Cc: devicetree, Pawel Moll, Ruben Ayrapetyan, Linus Walleij,
Kevin Brodsky, Liviu Dudau, dri-devel, Sudeep Holla,
Robin Murphy, Brian Starkey
It is unclear why max-memory-bandwidth should be set for CLCD on the
fast model. Removing that property allows allocating and using 32bpp
buffers, which may be desirable on certain platforms such as
Android.
Reported-by: Ruben Ayrapetyan <ruben.ayrapetyan@arm.com>
Signed-off-by: Kevin Brodsky <kevin.brodsky@arm.com>
---
Changes in v2:
- Remove the attribute completely instead of increasing its value. It is
optional and there is no clear reason why it should be set at all.
arch/arm64/boot/dts/arm/fvp-base-revc.dts | 8 --------
arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi | 2 --
2 files changed, 10 deletions(-)
diff --git a/arch/arm64/boot/dts/arm/fvp-base-revc.dts b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
index 687707020ec1..62ab0d54ff71 100644
--- a/arch/arm64/boot/dts/arm/fvp-base-revc.dts
+++ b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
@@ -265,13 +265,5 @@
<0 0 42 &gic 0 0 GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>,
<0 0 43 &gic 0 0 GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>,
<0 0 44 &gic 0 0 GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>;
-
- motherboard {
- iofpga@3,00000000 {
- clcd@1f0000 {
- max-memory-bandwidth = <130000000>; /* 16bpp @ 63.5MHz */
- };
- };
- };
};
};
diff --git a/arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi b/arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi
index 454cf6c44c49..03a7bf079c8f 100644
--- a/arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi
+++ b/arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi
@@ -188,8 +188,6 @@
interrupts = <14>;
clocks = <&v2m_oscclk1>, <&v2m_clk24mhz>;
clock-names = "clcdclk", "apb_pclk";
- /* 800x600 16bpp @36MHz works fine */
- max-memory-bandwidth = <54000000>;
memory-region = <&vram>;
port {
--
2.22.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth
2019-07-25 17:12 [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth Kevin Brodsky
@ 2019-07-26 9:50 ` Sudeep Holla
2019-08-05 10:31 ` Linus Walleij
1 sibling, 0 replies; 3+ messages in thread
From: Sudeep Holla @ 2019-07-26 9:50 UTC (permalink / raw)
To: Kevin Brodsky
Cc: devicetree, Pawel Moll, Ruben Ayrapetyan, Linus Walleij,
Liviu Dudau, dri-devel, Sudeep Holla, Robin Murphy,
Brian Starkey, linux-arm-kernel
On Thu, Jul 25, 2019 at 06:12:14PM +0100, Kevin Brodsky wrote:
> It is unclear why max-memory-bandwidth should be set for CLCD on the
> fast model. Removing that property allows allocating and using 32bpp
> buffers, which may be desirable on certain platforms such as
> Android.
>
> Reported-by: Ruben Ayrapetyan <ruben.ayrapetyan@arm.com>
> Signed-off-by: Kevin Brodsky <kevin.brodsky@arm.com>
> ---
>
> Changes in v2:
> - Remove the attribute completely instead of increasing its value. It is
> optional and there is no clear reason why it should be set at all.
>
> arch/arm64/boot/dts/arm/fvp-base-revc.dts | 8 --------
> arch/arm64/boot/dts/arm/rtsm_ve-motherboard.dtsi | 2 --
I was about to ask about the above file in v1, and then saw v2.
I will take this for v5.4. Thanks
--
Regards,
Sudeep
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth
2019-07-25 17:12 [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth Kevin Brodsky
2019-07-26 9:50 ` Sudeep Holla
@ 2019-08-05 10:31 ` Linus Walleij
1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2019-08-05 10:31 UTC (permalink / raw)
To: Kevin Brodsky
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Pawel Moll, Ruben Ayrapetyan, Liviu Dudau,
open list:DRM PANEL DRIVERS, Sudeep Holla, Robin Murphy,
Brian Starkey, Linux ARM
On Thu, Jul 25, 2019 at 7:12 PM Kevin Brodsky <kevin.brodsky@arm.com> wrote:
> It is unclear why max-memory-bandwidth should be set for CLCD on the
> fast model. Removing that property allows allocating and using 32bpp
> buffers, which may be desirable on certain platforms such as
> Android.
>
> Reported-by: Ruben Ayrapetyan <ruben.ayrapetyan@arm.com>
> Signed-off-by: Kevin Brodsky <kevin.brodsky@arm.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-08-05 10:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-25 17:12 [PATCH v2] arm64: dts: fast models: Remove clcd's max-memory-bandwidth Kevin Brodsky
2019-07-26 9:50 ` Sudeep Holla
2019-08-05 10:31 ` Linus Walleij
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).