From: George Moussalem Date: Tue, 6 May 2025 06:18:09 +0000 (+0400) Subject: qualcommax: ipq50xx: remove ECC user config from board files X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F18723%2Fhead;p=thirdparty%2Fopenwrt.git qualcommax: ipq50xx: remove ECC user config from board files With the spi-qpic-snand driver now supporting reading ECC requirements from the NAND chip itself, there's no need to set those in the DTS anymore avoiding issues with devices of the same revision using different NAND types with varying ECC requirements. Signed-off-by: George Moussalem Link: https://github.com/openwrt/openwrt/pull/18723 Signed-off-by: Robert Marko --- diff --git a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-gl-b3000.dts b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-gl-b3000.dts index 684b6d95c53..9abf20fec58 100644 --- a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-gl-b3000.dts +++ b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-gl-b3000.dts @@ -226,8 +226,6 @@ #size-cells = <1>; nand-ecc-engine = <&qpic_nand>; - nand-ecc-strength = <4>; - nand-ecc-step-size = <512>; nand-bus-width = <8>; partitions { diff --git a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mr5500.dts b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mr5500.dts index 8dbd8a5d88e..99bd9a71a24 100644 --- a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mr5500.dts +++ b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mr5500.dts @@ -28,12 +28,6 @@ }; }; -&qpic_nand { - nand@0 { - nand-ecc-strength = <4>; - }; -}; - /* * =============================================================== * _______________________ _______________________ diff --git a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mx-base.dtsi b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mx-base.dtsi index a015fcbd34b..0cc70cceb7c 100644 --- a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mx-base.dtsi +++ b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-mx-base.dtsi @@ -130,9 +130,6 @@ #size-cells = <1>; nand-ecc-engine = <&qpic_nand>; - - nand-ecc-strength = <8>; - nand-ecc-step-size = <512>; nand-bus-width = <8>; partitions { diff --git a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-wrc-x3000gs2.dts b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-wrc-x3000gs2.dts index fc3e86b8702..69f124cb9d7 100644 --- a/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-wrc-x3000gs2.dts +++ b/target/linux/qualcommax/files/arch/arm64/boot/dts/qcom/ipq5018-wrc-x3000gs2.dts @@ -164,10 +164,6 @@ #size-cells = <1>; nand-ecc-engine = <&qpic_nand>; - - /* strength=8 breaks NAND I/O, use 4 instead */ - nand-ecc-strength = <4>; - nand-ecc-step-size = <512>; nand-bus-width = <8>; partitions {