[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20260120-alt-128-v4-1-0e5c491a532c@nvidia.com>
Date: Tue, 20 Jan 2026 12:34:14 -0800
From: Marc Olberding <molberding@...dia.com>
To: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>,
Andrew Jeffery <andrew@...econstruct.com.au>
Cc: devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-aspeed@...ts.ozlabs.org, linux-kernel@...r.kernel.org,
Marc Olberding <molberding@...dia.com>
Subject: [PATCH v4] ARM: dts: aspeed: add an alt 128M flash layout
Add a 128M layout for the BMC flash chip we didn't boot from. Including
this allows the user to write to each partition on the alternate spi
chip. This dtsi follows the existing standard of using the same layout
as non alt version and prepending `alt` to each partition's name.
This patch also adds this alt spi flash layout to
aspeed-bmc-nvidia-msx4-bmc.dts
Tested by running on target and catting sysfs nodes
for devdir in /sys/class/mtd/mtd*; do
[[ -d $devdir && -r $devdir/name ]] || continue
name=$(<"$devdir/name")
[[ $name == alt* ]] || continue
size=$(<"$devdir/size")
offset=0
[[ -r $devdir/offset ]] && offset=$(<"$devdir/offset")
dev=$(basename "$devdir")
printf "%s name=%s offset=0x%X size=0x%X\n" "$dev" "$name" \
"$offset" "$size"
done
mtd10 name=alt-rofs offset=0xA00000 size=0x5600000
mtd11 name=alt-rwfs offset=0x6000000 size=0x2000000
mtd7 name=alt-u-boot offset=0x0 size=0xE0000
mtd8 name=alt-u-boot-env offset=0xE0000 size=0x20000
mtd9 name=alt-kernel offset=0x100000 size=0x900000
Signed-off-by: Marc Olberding <molberding@...dia.com>
---
Add a layout for an alternate 128M flash layout. This is useful for the
aspeed ast2500[2] and ast2600[1] SoCs, where it has the ability to boot
from one of two spi chips, nominally the primary and backup SPIs. Adding
a layout allows userspace to flash the alternate spi by partition and
switch over to it, allowing lower downtime for firmware flash, or for
updating the primary flash from the golden backup flash. This is already
an established scheme for the 32M and 64M layouts, where the layout is
the same but each partition has `alt` prepended to its name. This allows
userspace to just prepend alt to the images in the update process to
specify which flash chip to update. This series also adds support for
the msx4 to use this layout.
References:
[1] https://www.aspeedtech.com/server_ast2600/
[2] https://www.aspeedtech.com/server_ast2500/
---
Changes in v4:
- Cut out unnecessary testing details
- Squashed patchset down to one patch.
- Link to v3: https://lore.kernel.org/r/20260120-alt-128-v3-0-a81bdafc00a7@nvidia.com
Changes in v3:
- Reworded commit adding alt flash layout and fixed formatting issues
- Expanded testing section
- Readded the Changes in v2 section, somehow they got lost
- Link to v2: https://lore.kernel.org/r/20251218-alt-128-v2-0-10499a0b7eb9@nvidia.com
Changes in v2:
- Retargetted series for the bmc tree
- Added a usage of the dtsi in the MSX4 device tree
- Link to v1: https://lore.kernel.org/all/20251211-alt-128-layout-v1-1-100f57d2bc4a@nvidia.com/
---
.../boot/dts/aspeed/aspeed-bmc-nvidia-msx4-bmc.dts | 1 +
.../dts/aspeed/openbmc-flash-layout-128-alt.dtsi | 32 ++++++++++++++++++++++
2 files changed, 33 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-nvidia-msx4-bmc.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-nvidia-msx4-bmc.dts
index 44f95a3986cb..a12d4164de4a 100644
--- a/arch/arm/boot/dts/aspeed/aspeed-bmc-nvidia-msx4-bmc.dts
+++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-nvidia-msx4-bmc.dts
@@ -67,6 +67,7 @@ flash@1 {
spi-tx-bus-width = <1>;
spi-rx-bus-width = <1>;
status = "okay";
+ #include "openbmc-flash-layout-128-alt.dtsi"
};
};
diff --git a/arch/arm/boot/dts/aspeed/openbmc-flash-layout-128-alt.dtsi b/arch/arm/boot/dts/aspeed/openbmc-flash-layout-128-alt.dtsi
new file mode 100644
index 000000000000..08ce2eab472c
--- /dev/null
+++ b/arch/arm/boot/dts/aspeed/openbmc-flash-layout-128-alt.dtsi
@@ -0,0 +1,32 @@
+// SPDX-License-Identifier: GPL-2.0+
+
+partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ u-boot@0 {
+ reg = <0x0 0xe0000>; // 896KB
+ label = "alt-u-boot";
+ };
+
+ u-boot-env@...00 {
+ reg = <0xe0000 0x20000>; // 128KB
+ label = "alt-u-boot-env";
+ };
+
+ kernel@...000 {
+ reg = <0x100000 0x900000>; // 9MB
+ label = "alt-kernel";
+ };
+
+ rofs@...000 {
+ reg = <0xa00000 0x5600000>; // 86MB
+ label = "alt-rofs";
+ };
+
+ rwfs@...0000 {
+ reg = <0x6000000 0x2000000>; // 32MB
+ label = "alt-rwfs";
+ };
+};
---
base-commit: 459a5aa171c0f13fcd78faa9594dc4aa5a95770b
change-id: 20251218-alt-128-19270d894872
Best regards,
--
Marc Olberding <molberding@...dia.com>
Powered by blists - more mailing lists