* [PATCH v2] ARM: dts: imx6ull: fix ubi mount failed on MYS-6ULX-IOT board
@ 2021-03-17 15:45 dillon.minfei
2021-03-17 15:45 ` [PATCH v2] ARM: dts: imx6ull: fix ubi filesystem mount failed dillon.minfei
0 siblings, 1 reply; 3+ messages in thread
From: dillon.minfei @ 2021-03-17 15:45 UTC (permalink / raw)
To: robh+dt, shawnguo, s.hauer, parthiban, kernel, festevam,
linux-imx, devicetree, linux-arm-kernel, linux-kernel
Cc: dillon min
From: dillon min <dillon.minfei@gmail.com>
This patch intend to fix ubi filesystem mount failed on MYS-6ULX-IOT board.
Hi Shawn,
Sorry for the incorrect commit message, could you help to update the part regard
to it with git commit --amend?
thanks.
Changes in v2:
just correct commit message, no code changes.
dillon min (1):
ARM: dts: imx6ull: fix ubi filesystem mount failed
arch/arm/boot/dts/imx6ull-myir-mys-6ulx-eval.dts | 1 +
1 file changed, 1 insertion(+)
--
1.9.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2] ARM: dts: imx6ull: fix ubi filesystem mount failed
2021-03-17 15:45 [PATCH v2] ARM: dts: imx6ull: fix ubi mount failed on MYS-6ULX-IOT board dillon.minfei
@ 2021-03-17 15:45 ` dillon.minfei
2021-03-18 4:07 ` Shawn Guo
0 siblings, 1 reply; 3+ messages in thread
From: dillon.minfei @ 2021-03-17 15:45 UTC (permalink / raw)
To: robh+dt, shawnguo, s.hauer, parthiban, kernel, festevam,
linux-imx, devicetree, linux-arm-kernel, linux-kernel
Cc: dillon min
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(+)
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";
};
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] ARM: dts: imx6ull: fix ubi filesystem mount failed
2021-03-17 15:45 ` [PATCH v2] ARM: dts: imx6ull: fix ubi filesystem mount failed dillon.minfei
@ 2021-03-18 4:07 ` Shawn Guo
0 siblings, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2021-03-18 4:07 UTC (permalink / raw)
To: dillon.minfei
Cc: robh+dt, s.hauer, parthiban, kernel, festevam, linux-imx,
devicetree, linux-arm-kernel, linux-kernel
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-03-18 4:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-17 15:45 [PATCH v2] ARM: dts: imx6ull: fix ubi mount failed on MYS-6ULX-IOT board dillon.minfei
2021-03-17 15:45 ` [PATCH v2] ARM: dts: imx6ull: fix ubi filesystem mount failed dillon.minfei
2021-03-18 4:07 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).