diff mbox series

ARM: dts: imx6ull: fix ubi filesystem mount failed

Message ID 1615270520-16951-2-git-send-email-dillon.minfei@gmail.com
State Superseded
Headers show
Series ARM: dts: imx6ull: fix ubi filesystem mount failed | expand

Commit Message

Dillon Min March 9, 2021, 6:15 a.m. UTC
From: dillon min <dillon.minfei@gmail.com>

since Micron MT29F2G08ABAEAWP's ecc error management:

|Description           			| Requirement
|Minimum required ECC  			| 4-bit ECC per 528 bytes
|Minimum ECC with internal ECC enabled  | 4-bit ECC per 516 bytes (user data) +
                                         8bytes (parity data)

to avoid unnecessary overheads related to bigger ecc calculations.
need choose to use fsl,use-minimum-ecc, else will run into ecc error.

[    9.449265] ubi0: scanning is finished
[    9.463968] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading
22528 bytes from PEB 513:4096, read only 22528 bytes, retry
[    9.486940] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading
22528 bytes from PEB 513:4096, read only 22528 bytes, retry
[    9.509906] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading
22528 bytes from PEB 513:4096, read only 22528 bytes, retry
[    9.532845] ubi0 error: ubi_io_read: error -74 (ECC error) while reading
22528 bytes from PEB 513:4096, read 22528 bytes

Fixes: f9ecf10cb88c ("ARM: dts: imx6ull: add MYiR MYS-6ULX SBC")
Signed-off-by: dillon min <dillon.minfei@gmail.com>
---
 arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Shawn Guo March 17, 2021, 1 p.m. UTC | #1
On Tue, Mar 09, 2021 at 02:15:20PM +0800, dillon.minfei@gmail.com wrote:
> From: dillon min <dillon.minfei@gmail.com>

> 

> since Micron MT29F2G08ABAEAWP's ecc error management:

> 

> |Description           			| Requirement

> |Minimum required ECC  			| 4-bit ECC per 528 bytes

> |Minimum ECC with internal ECC enabled  | 4-bit ECC per 516 bytes (user data) +

>                                          8bytes (parity data)

> 

> to avoid unnecessary overheads related to bigger ecc calculations.

> need choose to use fsl,use-minimum-ecc, else will run into ecc error.

> 

> [    9.449265] ubi0: scanning is finished

> [    9.463968] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading

> 22528 bytes from PEB 513:4096, read only 22528 bytes, retry

> [    9.486940] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading

> 22528 bytes from PEB 513:4096, read only 22528 bytes, retry

> [    9.509906] ubi0 warning: ubi_io_read: error -74 (ECC error) while reading

> 22528 bytes from PEB 513:4096, read only 22528 bytes, retry

> [    9.532845] ubi0 error: ubi_io_read: error -74 (ECC error) while reading

> 22528 bytes from PEB 513:4096, read 22528 bytes

> 

> Fixes: f9ecf10cb88c ("ARM: dts: imx6ull: add MYiR MYS-6ULX SBC")

> Signed-off-by: dillon min <dillon.minfei@gmail.com>


Applied, thanks.
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts b/arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts
index ecbb2cc5b9ab..79cc45728cd2 100644
--- a/arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts
+++ b/arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts
@@ -14,5 +14,6 @@ 
 };
 
 &gpmi {
+	fsl,use-minimum-ecc;
 	status = "okay";
 };