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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aG1cCK1U7cs+Jf1n@lizhi-Precision-Tower-5810>
Date: Tue, 8 Jul 2025 13:57:28 -0400
From: Frank Li <Frank.li@....com>
To: Frieder Schrempf <frieder@...s.de>
Cc: linux-arm-kernel@...ts.infradead.org,
	Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
	imx@...ts.linux.dev, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	linux-kernel@...r.kernel.org, Rob Herring <robh@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Shawn Guo <shawnguo@...nel.org>,
	Eberhard Stoll <eberhard.stoll@...tron.de>,
	Frieder Schrempf <frieder.schrempf@...tron.de>,
	Fabio Estevam <festevam@...il.com>,
	Pengutronix Kernel Team <kernel@...gutronix.de>
Subject: Re: [PATCH v2 1/2] ARM: dts: imx6ul-kontron-sl-common: Add SPI NOR
 partitions

On Tue, Jul 08, 2025 at 02:56:04PM +0200, Frieder Schrempf wrote:
> From: Eberhard Stoll <eberhard.stoll@...tron.de>
>
> Describe the partitions for the bootloader and the environment
> on the SPI NOR.
>
> Signed-off-by: Eberhard Stoll <eberhard.stoll@...tron.de>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@...tron.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@...00 {
> +				label = "env";
> +				reg = <0xf0000 0x8000>;
> +			};
> +
> +			partition@...00 {
> +				label = "env_redundant";
> +				reg = <0xf8000 0x8000>;
> +			};
> +		};
>  	};
>  };
>
> --
> 2.50.0
>