[v2,1/5] arm64: dts: imx8mm: Add common -u-boot.dtsi

Message ID 20210315174543.104002-2-jagan@amarulasolutions.com
State New
Headers show
Series
  • board: imx8mm: Add Engicam i.Core MX8M Mini SoM
Related show

Commit Message

Jagan Teki March 15, 2021, 5:45 p.m. UTC
/soc@0
aips1
aips2
aips3
clk
iomuxc
osc_24m

are common node enablements across imx8mm platform for
dm-spi, dm-pre-reloc stages.

Move them into common dtsi, imx8mm-u-boot.dtsi

Cc: Tim Harvey <tharvey@gateworks.com>
Cc: Adam Ford <aford173@gmail.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Teresa Remmet <t.remmet@phytec.de>
Cc: Igor Opaniuk <igor.opaniuk@toradex.com>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
Changes for v2:
- add venice changes

 arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi    | 37 +-----------------
 arch/arm/dts/imx8mm-evk-u-boot.dtsi           | 37 +-----------------
 arch/arm/dts/imx8mm-u-boot.dtsi               | 39 +++++++++++++++++++
 arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi | 35 +----------------
 arch/arm/dts/imx8mm-venice-u-boot.dtsi        | 37 +-----------------
 arch/arm/dts/imx8mm-verdin-u-boot.dtsi        | 38 ++----------------
 arch/arm/dts/phycore-imx8mm-u-boot.dtsi       | 37 +-----------------
 7 files changed, 51 insertions(+), 209 deletions(-)
 create mode 100644 arch/arm/dts/imx8mm-u-boot.dtsi

Comments

Peng Fan April 8, 2021, 2:12 a.m. UTC | #1
On 2021/3/16 1:45, Jagan Teki wrote:
> /soc@0
> aips1
> aips2
> aips3
> clk
> iomuxc
> osc_24m
> 
> are common node enablements across imx8mm platform for
> dm-spi, dm-pre-reloc stages.
> 
> Move them into common dtsi, imx8mm-u-boot.dtsi

This may conflict with 
https://patchwork.ozlabs.org/project/uboot/patch/20210406035904.26598-1-peng.fan@oss.nxp.com/

Thanks,
Peng.

> 
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: Adam Ford <aford173@gmail.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Teresa Remmet <t.remmet@phytec.de>
> Cc: Igor Opaniuk <igor.opaniuk@toradex.com>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---
> Changes for v2:
> - add venice changes
> 
>   arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi    | 37 +-----------------
>   arch/arm/dts/imx8mm-evk-u-boot.dtsi           | 37 +-----------------
>   arch/arm/dts/imx8mm-u-boot.dtsi               | 39 +++++++++++++++++++
>   arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi | 35 +----------------
>   arch/arm/dts/imx8mm-venice-u-boot.dtsi        | 37 +-----------------
>   arch/arm/dts/imx8mm-verdin-u-boot.dtsi        | 38 ++----------------
>   arch/arm/dts/phycore-imx8mm-u-boot.dtsi       | 37 +-----------------
>   7 files changed, 51 insertions(+), 209 deletions(-)
>   create mode 100644 arch/arm/dts/imx8mm-u-boot.dtsi
> 
> diff --git a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
> index 6d80a529ae..73ddfdade6 100644
> --- a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
> @@ -3,6 +3,8 @@
>    * Copyright 2020 Compass Electronics Group, LLC
>    */
>   
> +#include "imx8mm-u-boot.dtsi"
> +
>   / {
>   	wdt-reboot {
>   		compatible = "wdt-reboot";
> @@ -11,32 +13,6 @@
>   	};
>   };
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
>   &reg_usdhc2_vmmc {
>   	u-boot,off-on-delay-us = <20000>;
>   };
> @@ -65,15 +41,6 @@
>   	u-boot,dm-spl;
>   };
>   
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
>   &pca6416_0 {
>   	compatible = "ti,tca6416";
>   };
> diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> index e843a5648e..79dffe7ccd 100644
> --- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
> @@ -3,6 +3,8 @@
>    * Copyright 2019 NXP
>    */
>   
> +#include "imx8mm-u-boot.dtsi"
> +
>   / {
>   	wdt-reboot {
>   		compatible = "wdt-reboot";
> @@ -11,41 +13,6 @@
>   	};
>   };
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> -
>   &reg_usdhc2_vmmc {
>   	u-boot,off-on-delay-us = <20000>;
>   };
> diff --git a/arch/arm/dts/imx8mm-u-boot.dtsi b/arch/arm/dts/imx8mm-u-boot.dtsi
> new file mode 100644
> index 0000000000..f833d9df59
> --- /dev/null
> +++ b/arch/arm/dts/imx8mm-u-boot.dtsi
> @@ -0,0 +1,39 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright (C) 2020 Jagan Teki <jagan@amarulasolutions.com>
> + */
> +
> +&{/soc@0} {
> +	u-boot,dm-pre-reloc;
> +	u-boot,dm-spl;
> +};
> +
> +&aips1 {
> +	u-boot,dm-spl;
> +	u-boot,dm-pre-reloc;
> +};
> +
> +&aips2 {
> +	u-boot,dm-spl;
> +};
> +
> +&aips3 {
> +	u-boot,dm-spl;
> +};
> +
> +&clk {
> +	u-boot,dm-spl;
> +	u-boot,dm-pre-reloc;
> +	/delete-property/ assigned-clocks;
> +	/delete-property/ assigned-clock-parents;
> +	/delete-property/ assigned-clock-rates;
> +};
> +
> +&iomuxc {
> +	u-boot,dm-spl;
> +};
> +
> +&osc_24m {
> +	u-boot,dm-spl;
> +	u-boot,dm-pre-reloc;
> +};
> diff --git a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
> index a4487e20c3..1a15d6a2ad 100644
> --- a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
> @@ -3,40 +3,7 @@
>    * Copyright 2021 Gateworks Corporation
>    */
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> +#include "imx8mm-u-boot.dtsi"
>   
>   &gpio1 {
>   	u-boot,dm-spl;
> diff --git a/arch/arm/dts/imx8mm-venice-u-boot.dtsi b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
> index 5b06d872c6..42b2903f04 100644
> --- a/arch/arm/dts/imx8mm-venice-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
> @@ -3,6 +3,8 @@
>    * Copyright 2021 Gateworks Corporation
>    */
>   
> +#include "imx8mm-u-boot.dtsi"
> +
>   / {
>   	wdt-reboot {
>   		compatible = "wdt-reboot";
> @@ -11,41 +13,6 @@
>   	};
>   };
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> -
>   &gpio1 {
>   	u-boot,dm-spl;
>   };
> diff --git a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> index 249b0f8f66..67c31c49b6 100644
> --- a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> @@ -2,6 +2,9 @@
>   /*
>    * Copyright 2020 Toradex
>    */
> +
> +#include "imx8mm-u-boot.dtsi"
> +
>   / {
>   	wdt-reboot {
>   		compatible = "wdt-reboot";
> @@ -10,27 +13,6 @@
>   	};
>   };
>   
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
>   &gpio1 {
>   	u-boot,dm-spl;
>   };
> @@ -55,15 +37,6 @@
>   	u-boot,dm-spl;
>   };
>   
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
>   &pinctrl_i2c1 {
>   	u-boot,dm-spl;
>   };
> @@ -84,11 +57,6 @@
>   	u-boot,dm-spl;
>   };
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
>   &{/soc@0/bus@30800000/i2c@30a20000/pmic} {
>   	u-boot,dm-spl;
>   };
> diff --git a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
> index fc0fa22d1b..91515b8df9 100644
> --- a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
> +++ b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
> @@ -4,6 +4,8 @@
>    * Author: Teresa Remmet <t.remmet@phytec.de>
>    */
>   
> +#include "imx8mm-u-boot.dtsi"
> +
>   / {
>   	wdt-reboot {
>   		compatible = "wdt-reboot";
> @@ -12,41 +14,6 @@
>   	};
>   };
>   
> -&{/soc@0} {
> -	u-boot,dm-pre-reloc;
> -	u-boot,dm-spl;
> -};
> -
> -&clk {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -	/delete-property/ assigned-clocks;
> -	/delete-property/ assigned-clock-parents;
> -	/delete-property/ assigned-clock-rates;
> -};
> -
> -&osc_24m {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips1 {
> -	u-boot,dm-spl;
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&aips2 {
> -	u-boot,dm-spl;
> -};
> -
> -&aips3 {
> -	u-boot,dm-spl;
> -};
> -
> -&iomuxc {
> -	u-boot,dm-spl;
> -};
> -
>   &pinctrl_uart3 {
>   	u-boot,dm-spl;
>   };
>
Stefano Babic April 8, 2021, 9:05 a.m. UTC | #2
On 08.04.21 04:12, Peng Fan (OSS) wrote:
> 
> On 2021/3/16 1:45, Jagan Teki wrote:
>> /soc@0
>> aips1
>> aips2
>> aips3
>> clk
>> iomuxc
>> osc_24m
>>
>> are common node enablements across imx8mm platform for
>> dm-spi, dm-pre-reloc stages.
>>
>> Move them into common dtsi, imx8mm-u-boot.dtsi
> 
> This may conflict with 
> https://patchwork.ozlabs.org/project/uboot/patch/20210406035904.26598-1-peng.fan@oss.nxp.com/ 

Right - Jagan, I have merged Peng's PR, I am merging the other 
conversions to DM. Your choice if you prefer to wait until I push on the 
server (maybe the best way to do), then you can rebase on top of it.

Regards,
Stefano

> 
> 
> Thanks,
> Peng.
> 
>>
>> Cc: Tim Harvey <tharvey@gateworks.com>
>> Cc: Adam Ford <aford173@gmail.com>
>> Cc: Peng Fan <peng.fan@nxp.com>
>> Cc: Teresa Remmet <t.remmet@phytec.de>
>> Cc: Igor Opaniuk <igor.opaniuk@toradex.com>
>> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
>> ---
>> Changes for v2:
>> - add venice changes
>>
>>   arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi    | 37 +-----------------
>>   arch/arm/dts/imx8mm-evk-u-boot.dtsi           | 37 +-----------------
>>   arch/arm/dts/imx8mm-u-boot.dtsi               | 39 +++++++++++++++++++
>>   arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi | 35 +----------------
>>   arch/arm/dts/imx8mm-venice-u-boot.dtsi        | 37 +-----------------
>>   arch/arm/dts/imx8mm-verdin-u-boot.dtsi        | 38 ++----------------
>>   arch/arm/dts/phycore-imx8mm-u-boot.dtsi       | 37 +-----------------
>>   7 files changed, 51 insertions(+), 209 deletions(-)
>>   create mode 100644 arch/arm/dts/imx8mm-u-boot.dtsi
>>
>> diff --git a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
>> index 6d80a529ae..73ddfdade6 100644
>> --- a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
>> @@ -3,6 +3,8 @@
>>    * Copyright 2020 Compass Electronics Group, LLC
>>    */
>> +#include "imx8mm-u-boot.dtsi"
>> +
>>   / {
>>       wdt-reboot {
>>           compatible = "wdt-reboot";
>> @@ -11,32 +13,6 @@
>>       };
>>   };
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>>   &reg_usdhc2_vmmc {
>>       u-boot,off-on-delay-us = <20000>;
>>   };
>> @@ -65,15 +41,6 @@
>>       u-boot,dm-spl;
>>   };
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>>   &pca6416_0 {
>>       compatible = "ti,tca6416";
>>   };
>> diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> index e843a5648e..79dffe7ccd 100644
>> --- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
>> @@ -3,6 +3,8 @@
>>    * Copyright 2019 NXP
>>    */
>> +#include "imx8mm-u-boot.dtsi"
>> +
>>   / {
>>       wdt-reboot {
>>           compatible = "wdt-reboot";
>> @@ -11,41 +13,6 @@
>>       };
>>   };
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> -
>>   &reg_usdhc2_vmmc {
>>       u-boot,off-on-delay-us = <20000>;
>>   };
>> diff --git a/arch/arm/dts/imx8mm-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-u-boot.dtsi
>> new file mode 100644
>> index 0000000000..f833d9df59
>> --- /dev/null
>> +++ b/arch/arm/dts/imx8mm-u-boot.dtsi
>> @@ -0,0 +1,39 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +/*
>> + * Copyright (C) 2020 Jagan Teki <jagan@amarulasolutions.com>
>> + */
>> +
>> +&{/soc@0} {
>> +    u-boot,dm-pre-reloc;
>> +    u-boot,dm-spl;
>> +};
>> +
>> +&aips1 {
>> +    u-boot,dm-spl;
>> +    u-boot,dm-pre-reloc;
>> +};
>> +
>> +&aips2 {
>> +    u-boot,dm-spl;
>> +};
>> +
>> +&aips3 {
>> +    u-boot,dm-spl;
>> +};
>> +
>> +&clk {
>> +    u-boot,dm-spl;
>> +    u-boot,dm-pre-reloc;
>> +    /delete-property/ assigned-clocks;
>> +    /delete-property/ assigned-clock-parents;
>> +    /delete-property/ assigned-clock-rates;
>> +};
>> +
>> +&iomuxc {
>> +    u-boot,dm-spl;
>> +};
>> +
>> +&osc_24m {
>> +    u-boot,dm-spl;
>> +    u-boot,dm-pre-reloc;
>> +};
>> diff --git a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
>> index a4487e20c3..1a15d6a2ad 100644
>> --- a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
>> @@ -3,40 +3,7 @@
>>    * Copyright 2021 Gateworks Corporation
>>    */
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> +#include "imx8mm-u-boot.dtsi"
>>   &gpio1 {
>>       u-boot,dm-spl;
>> diff --git a/arch/arm/dts/imx8mm-venice-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
>> index 5b06d872c6..42b2903f04 100644
>> --- a/arch/arm/dts/imx8mm-venice-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
>> @@ -3,6 +3,8 @@
>>    * Copyright 2021 Gateworks Corporation
>>    */
>> +#include "imx8mm-u-boot.dtsi"
>> +
>>   / {
>>       wdt-reboot {
>>           compatible = "wdt-reboot";
>> @@ -11,41 +13,6 @@
>>       };
>>   };
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> -
>>   &gpio1 {
>>       u-boot,dm-spl;
>>   };
>> diff --git a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi 
>> b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
>> index 249b0f8f66..67c31c49b6 100644
>> --- a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
>> @@ -2,6 +2,9 @@
>>   /*
>>    * Copyright 2020 Toradex
>>    */
>> +
>> +#include "imx8mm-u-boot.dtsi"
>> +
>>   / {
>>       wdt-reboot {
>>           compatible = "wdt-reboot";
>> @@ -10,27 +13,6 @@
>>       };
>>   };
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>>   &gpio1 {
>>       u-boot,dm-spl;
>>   };
>> @@ -55,15 +37,6 @@
>>       u-boot,dm-spl;
>>   };
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>>   &pinctrl_i2c1 {
>>       u-boot,dm-spl;
>>   };
>> @@ -84,11 +57,6 @@
>>       u-boot,dm-spl;
>>   };
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>>   &{/soc@0/bus@30800000/i2c@30a20000/pmic} {
>>       u-boot,dm-spl;
>>   };
>> diff --git a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi 
>> b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
>> index fc0fa22d1b..91515b8df9 100644
>> --- a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
>> +++ b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
>> @@ -4,6 +4,8 @@
>>    * Author: Teresa Remmet <t.remmet@phytec.de>
>>    */
>> +#include "imx8mm-u-boot.dtsi"
>> +
>>   / {
>>       wdt-reboot {
>>           compatible = "wdt-reboot";
>> @@ -12,41 +14,6 @@
>>       };
>>   };
>> -&{/soc@0} {
>> -    u-boot,dm-pre-reloc;
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&clk {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -    /delete-property/ assigned-clocks;
>> -    /delete-property/ assigned-clock-parents;
>> -    /delete-property/ assigned-clock-rates;
>> -};
>> -
>> -&osc_24m {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips1 {
>> -    u-boot,dm-spl;
>> -    u-boot,dm-pre-reloc;
>> -};
>> -
>> -&aips2 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&aips3 {
>> -    u-boot,dm-spl;
>> -};
>> -
>> -&iomuxc {
>> -    u-boot,dm-spl;
>> -};
>> -
>>   &pinctrl_uart3 {
>>       u-boot,dm-spl;
>>   };
>>
Jagan Teki April 8, 2021, 9:09 a.m. UTC | #3
On Thu, Apr 8, 2021 at 2:36 PM Stefano Babic <sbabic@denx.de> wrote:
>
> On 08.04.21 04:12, Peng Fan (OSS) wrote:
> >
> > On 2021/3/16 1:45, Jagan Teki wrote:
> >> /soc@0
> >> aips1
> >> aips2
> >> aips3
> >> clk
> >> iomuxc
> >> osc_24m
> >>
> >> are common node enablements across imx8mm platform for
> >> dm-spi, dm-pre-reloc stages.
> >>
> >> Move them into common dtsi, imx8mm-u-boot.dtsi
> >
> > This may conflict with
> > https://patchwork.ozlabs.org/project/uboot/patch/20210406035904.26598-1-peng.fan@oss.nxp.com/
>
> Right - Jagan, I have merged Peng's PR, I am merging the other
> conversions to DM. Your choice if you prefer to wait until I push on the
> server (maybe the best way to do), then you can rebase on top of it.

Yes, would work. I will rebase on top of your changes and send v3.

Jagan.

Patch

diff --git a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
index 6d80a529ae..73ddfdade6 100644
--- a/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-beacon-kit-u-boot.dtsi
@@ -3,6 +3,8 @@ 
  * Copyright 2020 Compass Electronics Group, LLC
  */
 
+#include "imx8mm-u-boot.dtsi"
+
 / {
 	wdt-reboot {
 		compatible = "wdt-reboot";
@@ -11,32 +13,6 @@ 
 	};
 };
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
 &reg_usdhc2_vmmc {
 	u-boot,off-on-delay-us = <20000>;
 };
@@ -65,15 +41,6 @@ 
 	u-boot,dm-spl;
 };
 
-&iomuxc {
-	u-boot,dm-spl;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
 &pca6416_0 {
 	compatible = "ti,tca6416";
 };
diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
index e843a5648e..79dffe7ccd 100644
--- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
@@ -3,6 +3,8 @@ 
  * Copyright 2019 NXP
  */
 
+#include "imx8mm-u-boot.dtsi"
+
 / {
 	wdt-reboot {
 		compatible = "wdt-reboot";
@@ -11,41 +13,6 @@ 
 	};
 };
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&iomuxc {
-	u-boot,dm-spl;
-};
-
 &reg_usdhc2_vmmc {
 	u-boot,off-on-delay-us = <20000>;
 };
diff --git a/arch/arm/dts/imx8mm-u-boot.dtsi b/arch/arm/dts/imx8mm-u-boot.dtsi
new file mode 100644
index 0000000000..f833d9df59
--- /dev/null
+++ b/arch/arm/dts/imx8mm-u-boot.dtsi
@@ -0,0 +1,39 @@ 
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2020 Jagan Teki <jagan@amarulasolutions.com>
+ */
+
+&{/soc@0} {
+	u-boot,dm-pre-reloc;
+	u-boot,dm-spl;
+};
+
+&aips1 {
+	u-boot,dm-spl;
+	u-boot,dm-pre-reloc;
+};
+
+&aips2 {
+	u-boot,dm-spl;
+};
+
+&aips3 {
+	u-boot,dm-spl;
+};
+
+&clk {
+	u-boot,dm-spl;
+	u-boot,dm-pre-reloc;
+	/delete-property/ assigned-clocks;
+	/delete-property/ assigned-clock-parents;
+	/delete-property/ assigned-clock-rates;
+};
+
+&iomuxc {
+	u-boot,dm-spl;
+};
+
+&osc_24m {
+	u-boot,dm-spl;
+	u-boot,dm-pre-reloc;
+};
diff --git a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
index a4487e20c3..1a15d6a2ad 100644
--- a/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-venice-gw700x-u-boot.dtsi
@@ -3,40 +3,7 @@ 
  * Copyright 2021 Gateworks Corporation
  */
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&iomuxc {
-	u-boot,dm-spl;
-};
+#include "imx8mm-u-boot.dtsi"
 
 &gpio1 {
 	u-boot,dm-spl;
diff --git a/arch/arm/dts/imx8mm-venice-u-boot.dtsi b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
index 5b06d872c6..42b2903f04 100644
--- a/arch/arm/dts/imx8mm-venice-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-venice-u-boot.dtsi
@@ -3,6 +3,8 @@ 
  * Copyright 2021 Gateworks Corporation
  */
 
+#include "imx8mm-u-boot.dtsi"
+
 / {
 	wdt-reboot {
 		compatible = "wdt-reboot";
@@ -11,41 +13,6 @@ 
 	};
 };
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&iomuxc {
-	u-boot,dm-spl;
-};
-
 &gpio1 {
 	u-boot,dm-spl;
 };
diff --git a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
index 249b0f8f66..67c31c49b6 100644
--- a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
@@ -2,6 +2,9 @@ 
 /*
  * Copyright 2020 Toradex
  */
+
+#include "imx8mm-u-boot.dtsi"
+
 / {
 	wdt-reboot {
 		compatible = "wdt-reboot";
@@ -10,27 +13,6 @@ 
 	};
 };
 
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
 &gpio1 {
 	u-boot,dm-spl;
 };
@@ -55,15 +37,6 @@ 
 	u-boot,dm-spl;
 };
 
-&iomuxc {
-	u-boot,dm-spl;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
 &pinctrl_i2c1 {
 	u-boot,dm-spl;
 };
@@ -84,11 +57,6 @@ 
 	u-boot,dm-spl;
 };
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
 &{/soc@0/bus@30800000/i2c@30a20000/pmic} {
 	u-boot,dm-spl;
 };
diff --git a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
index fc0fa22d1b..91515b8df9 100644
--- a/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
+++ b/arch/arm/dts/phycore-imx8mm-u-boot.dtsi
@@ -4,6 +4,8 @@ 
  * Author: Teresa Remmet <t.remmet@phytec.de>
  */
 
+#include "imx8mm-u-boot.dtsi"
+
 / {
 	wdt-reboot {
 		compatible = "wdt-reboot";
@@ -12,41 +14,6 @@ 
 	};
 };
 
-&{/soc@0} {
-	u-boot,dm-pre-reloc;
-	u-boot,dm-spl;
-};
-
-&clk {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-	/delete-property/ assigned-clocks;
-	/delete-property/ assigned-clock-parents;
-	/delete-property/ assigned-clock-rates;
-};
-
-&osc_24m {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips1 {
-	u-boot,dm-spl;
-	u-boot,dm-pre-reloc;
-};
-
-&aips2 {
-	u-boot,dm-spl;
-};
-
-&aips3 {
-	u-boot,dm-spl;
-};
-
-&iomuxc {
-	u-boot,dm-spl;
-};
-
 &pinctrl_uart3 {
 	u-boot,dm-spl;
 };