Date:   Mon, 6 Dec 2021 09:38:21 +0800
From:   Shawn Guo <shawnguo@kernel.org>
To:     Martin Kepplinger <martin.kepplinger@puri.sm>
Cc:     devicetree@vger.kernel.org, festevam@gmail.com,
        kernel@pengutronix.de, Abel Vesa <abel.vesa@nxp.com>,
        kernel@puri.sm, krzk@kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com,
        linux-kernel@vger.kernel.org, robh@kernel.org
Subject: Re: [PATCH] arm64: dts: imx8mq: remove interconnect property from
 lcdif
Message-ID: <20211206013821.GP4216@dragon>
References: <20211124083703.730243-1-martin.kepplinger@puri.sm>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211124083703.730243-1-martin.kepplinger@puri.sm>
User-Agent: Mutt/1.9.4 (2018-02-28)
Precedence: bulk

+ 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 <martin.kepplinger@puri.sm>

Applied, thanks.

> ---
> 
> revision history
> ----------------
> this patch replaces the wrong 1:1 revert previously sent:
> https://lore.kernel.org/linux-arm-kernel/20211123114545.411787-1-martin.kepplinger@puri.sm/
> 
> 
>  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
>