[PATCH] arm: dts: xilinx: parallella: Enable QSPI and NOR flash

[PATCH] arm: dts: xilinx: parallella: Enable QSPI and NOR flash

[Aleksander Jan Bajkowski]

The Parallella board has a SPI-NOR flash connected to the QSPI controller.
Enable the QSPI controller and add the corresponding SPI-NOR flash node
to allow the system to access it.

Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl>
---
 arch/arm/boot/dts/xilinx/zynq-parallella.dts | 33 ++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/arch/arm/boot/dts/xilinx/zynq-parallella.dts b/arch/arm/boot/dts/xilinx/zynq-parallella.dts
index 366af4fcf8d9..1903aa1cd83b 100644
--- a/arch/arm/boot/dts/xilinx/zynq-parallella.dts
+++ b/arch/arm/boot/dts/xilinx/zynq-parallella.dts
@@ -78,6 +78,39 @@ ldo2 {
 	};
 };
 
+&qspi {
+	bootph-all;
+	status = "okay";
+	num-cs = <1>;
+
+	flash@0 {
+		compatible = "jedec,spi-nor";
+		reg = <0>;
+		spi-tx-bus-width = <4>;
+		spi-rx-bus-width = <4>;
+		spi-max-frequency = <50000000>;
+		m25p,fast-read;
+
+		partitions {
+			compatible = "fixed-partitions";
+			#address-cells = <1>;
+			#size-cells = <1>;
+
+			partition@0 {
+				label = "uboot";
+				reg = <0x0 0x4e0000>;
+				read-only;
+			};
+
+			partition@4e0000 {
+				label = "uboot-env";
+				reg = <0x4e0000 0x10000>;
+				read-only;
+			};
+		};
+	};
+};
+
 &sdhci1 {
 	status = "okay";
 };
-- 
2.53.0

Re: [PATCH] arm: dts: xilinx: parallella: Enable QSPI and NOR flash

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] dts: uboot-env partition is marked read-only preventing fw_setenv updates
--

commit be87c0d3af6ecbab826ce9e10b9f662f4e9662e5
Author: Aleksander Jan Bajkowski <olek2@wp.pl>

arm: dts: xilinx: parallella: Enable QSPI and NOR flash

This commit enables the QSPI controller on the Parallella board and adds the
corresponding SPI-NOR flash node along with its fixed partitions.

> diff --git a/arch/arm/boot/dts/xilinx/zynq-parallella.dts b/arch/arm/boot/dts/xilinx/zynq-parallella.dts
> --- a/arch/arm/boot/dts/xilinx/zynq-parallella.dts
> +++ b/arch/arm/boot/dts/xilinx/zynq-parallella.dts
[ ... ]
> +		partitions {
> +			compatible = "fixed-partitions";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +
> +			partition@0 {
> +				label = "uboot";
> +				reg = <0x0 0x4e0000>;
> +				read-only;
> +			};
> +
> +			partition@4e0000 {
> +				label = "uboot-env";
> +				reg = <0x4e0000 0x10000>;
> +				read-only;

[Severity: Medium]
Will this prevent standard userspace tools like fw_setenv from updating the
U-Boot environment?

The kernel MTD subsystem parses the read-only property and clears the
MTD_WRITEABLE flag for this partition. If a root user attempts to open or
write to the corresponding /dev/mtdX device, the operation will fail with
-EROFS or -EPERM.

While some board configurations intentionally restrict write access to the
bootloader environment as a policy choice, is it intended to structurally
prevent dynamic updates from Linux on the Parallella board?

> +			};
> +		};
> +	};
> +};

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260917225317.319745-1-olek2@wp.pl?part=1