[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <257e006b-daab-4eed-9e1e-189d1b7a3167@foss.st.com>
Date: Fri, 19 Sep 2025 09:07:27 +0200
From: Patrice CHOTARD <patrice.chotard@...s.st.com>
To: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Maxime Coquelin
<mcoquelin.stm32@...il.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>
CC: <devicetree@...r.kernel.org>, <linux-stm32@...md-mailman.stormreply.com>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<stable@...r.kernel.org>
Subject: Re: [PATCH v3] arm64: dts: st: Add memory-region-names property for
stm32mp257f-ev1
On 8/11/25 15:28, Patrice Chotard wrote:
> In order to set the AMCR register, which configures the
> memory-region split between ospi1 and ospi2, we need to
> identify the ospi instance.
>
> By using memory-region-names, it allows to identify the
> ospi instance this memory-region belongs to.
>
> Fixes: cad2492de91c ("arm64: dts: st: Add SPI NOR flash support on stm32mp257f-ev1 board")
> Cc: stable@...r.kernel.org
> Signed-off-by: Patrice Chotard <patrice.chotard@...s.st.com>
> ---
> Changes in v3:
> - Set again "Cc: <stable@...r.kernel.org>"
> - Link to v2: https://lore.kernel.org/r/20250811-upstream_fix_dts_omm-v2-1-00ff55076bd5@foss.st.com
>
> Changes in v2:
> - Update commit message.
> - Use correct memory-region-names value.
> - Remove "Cc: <stable@...r.kernel.org>" tag as the fixed patch is not part of a LTS.
> - Link to v1: https://lore.kernel.org/r/20250806-upstream_fix_dts_omm-v1-1-e68c15ed422d@foss.st.com
> ---
> arch/arm64/boot/dts/st/stm32mp257f-ev1.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/st/stm32mp257f-ev1.dts b/arch/arm64/boot/dts/st/stm32mp257f-ev1.dts
> index 2f561ad4066544445e93db78557bc4be1c27095a..7bd8433c1b4344bb5d58193a5e6314f9ae89e0a4 100644
> --- a/arch/arm64/boot/dts/st/stm32mp257f-ev1.dts
> +++ b/arch/arm64/boot/dts/st/stm32mp257f-ev1.dts
> @@ -197,6 +197,7 @@ &i2c8 {
>
> &ommanager {
> memory-region = <&mm_ospi1>;
> + memory-region-names = "ospi1";
> pinctrl-0 = <&ospi_port1_clk_pins_a
> &ospi_port1_io03_pins_a
> &ospi_port1_cs0_pins_a>;
>
> ---
> base-commit: 038d61fd642278bab63ee8ef722c50d10ab01e8f
> change-id: 20250806-upstream_fix_dts_omm-c006b69042f1
>
> Best regards,
Hi Krzysztof
Do you have any additional remarks on this patch ?
I would like this patch to be part of next kernel v6.18.
Thanks
Patrice
Powered by blists - more mailing lists