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.openwrt.org/?a=commitdiff_plain;h=01b72ce61e8f7d050785981ab83d059e6cba4c27;p=openwrt%2Fstaging%2Flinusw.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 684b6d95c5..9abf20fec5 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 8dbd8a5d88..99bd9a71a2 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 a015fcbd34..0cc70cceb7 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 fc3e86b870..69f124cb9d 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 {