Re: [PATCH v2 1/2] ARM: dts: imx6ul-kontron-sl-common: Add SPI NOR partitions

[Frank Li <Frank.li@nxp.com>]

On Tue, Jul 08, 2025 at 02:56:04PM +0200, Frieder Schrempf wrote:
> From: Eberhard Stoll <eberhard.stoll@kontron.de>
>
> Describe the partitions for the bootloader and the environment
> on the SPI NOR.
>
> Signed-off-by: Eberhard Stoll <eberhard.stoll@kontron.de>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
> Changes for v2:
> * Fix name order in SoB
> ---
>  .../dts/nxp/imx/imx6ul-kontron-sl-common.dtsi | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6ul-kontron-sl-common.dtsi b/arch/arm/boot/dts/nxp/imx/imx6ul-kontron-sl-common.dtsi
> index dcf88f6103466..ab149c40a9cc1 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6ul-kontron-sl-common.dtsi
> +++ b/arch/arm/boot/dts/nxp/imx/imx6ul-kontron-sl-common.dtsi
> @@ -28,6 +28,27 @@ flash@0 {
>  		compatible = "mxicy,mx25v8035f", "jedec,spi-nor";
>  		spi-max-frequency = <50000000>;
>  		reg = <0>;
> +
> +		partitions {
> +			compatible = "fixed-partitions";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +
> +			partition@0 {
> +				label = "u-boot";
> +				reg = <0x0 0xf0000>;

I think reg should first property.
first is 'compatible', the second is 'reg'.

Frank

> +			};
> +
> +			partition@f0000 {
> +				label = "env";
> +				reg = <0xf0000 0x8000>;
> +			};
> +
> +			partition@f8000 {
> +				label = "env_redundant";
> +				reg = <0xf8000 0x8000>;
> +			};
> +		};
>  	};
>  };
>
> --
> 2.50.0
>