[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20230514030525.GK727834@dragon>
Date: Sun, 14 May 2023 11:05:25 +0800
From: Shawn Guo <shawnguo@...nel.org>
To: Andrejs Cainikovs <andrejs.cainikovs@...il.com>
Cc: linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
devicetree@...r.kernel.org,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
NXP Linux Team <linux-imx@....com>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Marcel Ziswiler <marcel.ziswiler@...adex.com>,
Emanuele Ghidoli <emanuele.ghidoli@...adex.com>,
Francesco Dolcini <francesco.dolcini@...adex.com>,
Andrejs Cainikovs <andrejs.cainikovs@...adex.com>
Subject: Re: [PATCH v1 4/4] arm64: dts: colibri-imx8x: delete adc1 and dsp
On Mon, Apr 17, 2023 at 07:38:30PM +0200, Andrejs Cainikovs wrote:
> i.MX8, i.MX8X, i.MX8XP and i.MX8XL SOC device trees are all based on
> imx8-ss-*.dtsi files. For i.MX8X and i.MX8XP these device trees
> should be updated with some peripherals removed or updated, similar
> to i.MX8XL (imx8dxl-ss-*.dtsi files). However, it looks like only
> i.MX8 and i.MX8XL are up to date, but for i.MX8X and i.MX8XP some
> of the peripherals got inherited from imx8-ss-*.dtsi files, but in
> reality they are not present on SOC.
> As a result, during resource partition ownership check U-Boot receives
> messages from SCU firmware about these resources not owned by boot
> partition. In reality, these resources are not owned by anyone, as
> they simply does not exist, but are defined in Linux device tree.
> This change removes those peripherals, which are listed during
> U-Boot resource partition ownership check as warnings:
>
> ## Flattened Device Tree blob at 9d400000
> Booting using the fdt blob at 0x9d400000
> Loading Device Tree to 00000000fd652000, end 00000000fd67efff ... OK
> Disable clock-controller@...80000 rsrc 512 not owned
> Disable clock-controller@...90000 rsrc 102 not owned
>
> Starting kernel ...
>
> Fixes: ba5a5615d54f ("arm64: dts: freescale: add initial support for colibri imx8x")
>
> Signed-off-by: Andrejs Cainikovs <andrejs.cainikovs@...adex.com>
Applying of the patch will result in a different patch author and SoB email.
Shawn
> ---
> arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> index b0d6f632622c..49d105eb4769 100644
> --- a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> @@ -769,3 +769,10 @@ pinctrl_wifi: wifigrp {
> fsl,pins = <IMX8QXP_SCU_BOOT_MODE3_SCU_DSC_RTC_CLOCK_OUTPUT_32K 0x20>;
> };
> };
> +
> +/* Delete peripherals which are not present on SOC, but are defined in imx8-ss-*.dtsi */
> +
> +/delete-node/ &adc1;
> +/delete-node/ &adc1_lpcg;
> +/delete-node/ &dsp;
> +/delete-node/ &dsp_lpcg;
> --
> 2.34.1
>
Powered by blists - more mailing lists