2021-11-24 08:37:52

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH] arm64: dts: imx8mq: remove interconnect property from lcdif

The mxsfb driver handling imx8mq lcdif doesn't yet request the
interconnect bandwidth that's needed at runtime when the description is
present in the DT node.

So remove that description and bring it back when it's supported.

Fixes: ad1abc8a03fd ("arm64: dts: imx8mq: Add interconnect for lcdif")
Signed-off-by: Martin Kepplinger <[email protected]>
---

revision history
----------------
this patch replaces the wrong 1:1 revert previously sent:
https://lore.kernel.org/linux-arm-kernel/[email protected]/


arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 972766b67a15..71bf497f99c2 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -524,8 +524,6 @@ lcdif: lcd-controller@30320000 {
<&clk IMX8MQ_VIDEO_PLL1>,
<&clk IMX8MQ_VIDEO_PLL1_OUT>;
assigned-clock-rates = <0>, <0>, <0>, <594000000>;
- interconnects = <&noc IMX8MQ_ICM_LCDIF &noc IMX8MQ_ICS_DRAM>;
- interconnect-names = "dram";
status = "disabled";

port@0 {
--
2.30.2



2021-12-06 01:38:35

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mq: remove interconnect property from lcdif

+ Abel

On Wed, Nov 24, 2021 at 09:37:03AM +0100, Martin Kepplinger wrote:
> The mxsfb driver handling imx8mq lcdif doesn't yet request the
> interconnect bandwidth that's needed at runtime when the description is
> present in the DT node.
>
> So remove that description and bring it back when it's supported.
>
> Fixes: ad1abc8a03fd ("arm64: dts: imx8mq: Add interconnect for lcdif")
> Signed-off-by: Martin Kepplinger <[email protected]>

Applied, thanks.

> ---
>
> revision history
> ----------------
> this patch replaces the wrong 1:1 revert previously sent:
> https://lore.kernel.org/linux-arm-kernel/[email protected]/
>
>
> arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> index 972766b67a15..71bf497f99c2 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -524,8 +524,6 @@ lcdif: lcd-controller@30320000 {
> <&clk IMX8MQ_VIDEO_PLL1>,
> <&clk IMX8MQ_VIDEO_PLL1_OUT>;
> assigned-clock-rates = <0>, <0>, <0>, <594000000>;
> - interconnects = <&noc IMX8MQ_ICM_LCDIF &noc IMX8MQ_ICS_DRAM>;
> - interconnect-names = "dram";
> status = "disabled";
>
> port@0 {
> --
> 2.30.2
>