diff mbox series

[v2] ARM: dts: imx6ull: fix ubi filesystem mount failed

Message ID 1615995909-6240-2-git-send-email-dillon.minfei@gmail.com
State Accepted
Commit e4817a1b6b77db538bc0141c3b138f2df803ce87
Headers show
Series [v2] ARM: dts: imx6ull: fix ubi filesystem mount failed | expand

Commit Message

Dillon Min March 17, 2021, 3:45 p.m. UTC 
From: dillon min <dillon.minfei@gmail.com>

For NAND Ecc layout, there is a dependency from old kernel's nand driver
setting and current. if old kernel use 4 bit ecc , we should use 4 bit
in new kernel either. else will run into following error at filesystem
mounting.

So, enable fsl,use-minimum-ecc from device tree, to fix this mismatch

[    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>
Reviewed-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
---
v2:
correct commit message according to Fabio Estevam's suggestion.

 arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Shawn Guo March 18, 2021, 4:07 a.m. UTC | #1 
On Wed, Mar 17, 2021 at 11:45:09PM +0800, dillon.minfei@gmail.com wrote:
> From: dillon min <dillon.minfei@gmail.com>

> 

> For NAND Ecc layout, there is a dependency from old kernel's nand driver

> setting and current. if old kernel use 4 bit ecc , we should use 4 bit

> in new kernel either. else will run into following error at filesystem

> mounting.

> 

> So, enable fsl,use-minimum-ecc from device tree, to fix this mismatch

> 

> [    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>

> Reviewed-by: Fabio Estevam <festevam@gmail.com>

> Signed-off-by: Shawn Guo <shawnguo@kernel.org>


Replaced with this version.

Shawn
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";
 };