* imx27: Unsupported ECC algorithm @ 2020-09-16 13:49 Fabio Estevam 2020-09-17 16:13 ` Miquel Raynal 2020-09-21 9:40 ` Sascha Hauer 0 siblings, 2 replies; 19+ messages in thread From: Fabio Estevam @ 2020-09-16 13:49 UTC (permalink / raw) To: Miquel Raynal; +Cc: linux-mtd, Sascha Hauer Hi Miquel, On a imx27-phytec-phycard-s-rdk board running linux-next, the following warning is seen: 10:15:26.232069 ------------[ cut here ]------------ 10:15:26.232880 WARNING: CPU: 0 PID: 1 at drivers/mtd/nand/raw/nand_base.c:5242 nand_scan_with_ids+0x9bc/0x1008 10:15:26.241132 Unsupported ECC algorithm! I believe it has been caused by 74a89d7f9ed7 ("mtd: rawnand: Use the ECC framework user input parsing bits") Its dts uses nand-ecc-mode = "hw" : &nfc { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_nfc>; nand-bus-width = <8>; nand-ecc-mode = "hw"; nand-on-flash-bbt; status = "okay"; }; I don't have access to this hardware, but I saw the report by the Pengutronix lab in kernelci: https://storage.kernelci.org/next/master/next-20200916/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html Any ideas? Thanks, Fabio Estevam ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-16 13:49 imx27: Unsupported ECC algorithm Fabio Estevam @ 2020-09-17 16:13 ` Miquel Raynal 2020-09-17 16:45 ` Miquel Raynal 2020-09-21 9:40 ` Sascha Hauer 1 sibling, 1 reply; 19+ messages in thread From: Miquel Raynal @ 2020-09-17 16:13 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-mtd, Sascha Hauer Hi Fabio, Fabio Estevam <festevam@gmail.com> wrote on Wed, 16 Sep 2020 10:49:03 -0300: > Hi Miquel, > > On a imx27-phytec-phycard-s-rdk board running linux-next, the > following warning is seen: > > 10:15:26.232069 ------------[ cut here ]------------ > 10:15:26.232880 WARNING: CPU: 0 PID: 1 at > drivers/mtd/nand/raw/nand_base.c:5242 nand_scan_with_ids+0x9bc/0x1008 > 10:15:26.241132 Unsupported ECC algorithm! > > I believe it has been caused by 74a89d7f9ed7 ("mtd: rawnand: Use the > ECC framework user input parsing bits") > > Its dts uses nand-ecc-mode = "hw" : > > &nfc { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_nfc>; > nand-bus-width = <8>; > nand-ecc-mode = "hw"; > nand-on-flash-bbt; > status = "okay"; > }; > > I don't have access to this hardware, but I saw the report by the > Pengutronix lab in kernelci: > https://storage.kernelci.org/next/master/next-20200916/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > Any ideas? Thanks for the report, I don't know what is happening yet but I'll investigate. As I'll be off next week, it might take a bit longer than expected to see this solved if I don't come up with something by tomorrow... Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-17 16:13 ` Miquel Raynal @ 2020-09-17 16:45 ` Miquel Raynal 2020-09-17 17:12 ` Fabio Estevam 0 siblings, 1 reply; 19+ messages in thread From: Miquel Raynal @ 2020-09-17 16:45 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-mtd, Sascha Hauer Hi Fabio, Miquel Raynal <miquel.raynal@bootlin.com> wrote on Thu, 17 Sep 2020 18:13:18 +0200: > Hi Fabio, > > Fabio Estevam <festevam@gmail.com> wrote on Wed, 16 Sep 2020 10:49:03 > -0300: > > > Hi Miquel, > > > > On a imx27-phytec-phycard-s-rdk board running linux-next, the > > following warning is seen: > > > > 10:15:26.232069 ------------[ cut here ]------------ > > 10:15:26.232880 WARNING: CPU: 0 PID: 1 at > > drivers/mtd/nand/raw/nand_base.c:5242 nand_scan_with_ids+0x9bc/0x1008 > > 10:15:26.241132 Unsupported ECC algorithm! > > > > I believe it has been caused by 74a89d7f9ed7 ("mtd: rawnand: Use the > > ECC framework user input parsing bits") > > > > Its dts uses nand-ecc-mode = "hw" : > > > > &nfc { > > pinctrl-names = "default"; > > pinctrl-0 = <&pinctrl_nfc>; > > nand-bus-width = <8>; > > nand-ecc-mode = "hw"; > > nand-on-flash-bbt; > > status = "okay"; > > }; > > > > I don't have access to this hardware, but I saw the report by the > > Pengutronix lab in kernelci: > > https://storage.kernelci.org/next/master/next-20200916/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > Any ideas? > > Thanks for the report, I don't know what is happening yet but I'll > investigate. > > As I'll be off next week, it might take a bit longer than expected > to see this solved if I don't come up with something by tomorrow... Perhaps Tudor's series is related to your issue, would you mind testing it? https://patchwork.ozlabs.org/project/linux-mtd/list/?series=202344 Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-17 16:45 ` Miquel Raynal @ 2020-09-17 17:12 ` Fabio Estevam 0 siblings, 0 replies; 19+ messages in thread From: Fabio Estevam @ 2020-09-17 17:12 UTC (permalink / raw) To: Miquel Raynal; +Cc: linux-mtd, Sascha Hauer Hi Miquel, On Thu, Sep 17, 2020 at 1:45 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > Perhaps Tudor's series is related to your issue, would you mind testing > it? > > https://patchwork.ozlabs.org/project/linux-mtd/list/?series=202344 I don't have access to the imx27 hardware, just noticed the problem in kernelci, but I can't test it myself. Thanks ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-16 13:49 imx27: Unsupported ECC algorithm Fabio Estevam 2020-09-17 16:13 ` Miquel Raynal @ 2020-09-21 9:40 ` Sascha Hauer 2020-09-28 15:20 ` Miquel Raynal 1 sibling, 1 reply; 19+ messages in thread From: Sascha Hauer @ 2020-09-21 9:40 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-mtd, Sascha Hauer, Miquel Raynal Hi Fabio, On Wed, Sep 16, 2020 at 10:49:03AM -0300, Fabio Estevam wrote: > Hi Miquel, > > On a imx27-phytec-phycard-s-rdk board running linux-next, the > following warning is seen: > > 10:15:26.232069 ------------[ cut here ]------------ > 10:15:26.232880 WARNING: CPU: 0 PID: 1 at > drivers/mtd/nand/raw/nand_base.c:5242 nand_scan_with_ids+0x9bc/0x1008 > 10:15:26.241132 Unsupported ECC algorithm! Thanks for spotting this. > > I believe it has been caused by 74a89d7f9ed7 ("mtd: rawnand: Use the > ECC framework user input parsing bits") I believe that as well. I've send a response to the original patch mail describing what goes wrong here. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-21 9:40 ` Sascha Hauer @ 2020-09-28 15:20 ` Miquel Raynal 2020-09-29 11:39 ` Fabio Estevam 0 siblings, 1 reply; 19+ messages in thread From: Miquel Raynal @ 2020-09-28 15:20 UTC (permalink / raw) To: Sascha Hauer; +Cc: linux-mtd, Fabio Estevam, Sascha Hauer Hi Fabio, Sascha Hauer <s.hauer@pengutronix.de> wrote on Mon, 21 Sep 2020 11:40:10 +0200: > Hi Fabio, > > On Wed, Sep 16, 2020 at 10:49:03AM -0300, Fabio Estevam wrote: > > Hi Miquel, > > > > On a imx27-phytec-phycard-s-rdk board running linux-next, the > > following warning is seen: > > > > 10:15:26.232069 ------------[ cut here ]------------ > > 10:15:26.232880 WARNING: CPU: 0 PID: 1 at > > drivers/mtd/nand/raw/nand_base.c:5242 nand_scan_with_ids+0x9bc/0x1008 > > 10:15:26.241132 Unsupported ECC algorithm! > > Thanks for spotting this. > > > > > I believe it has been caused by 74a89d7f9ed7 ("mtd: rawnand: Use the > > ECC framework user input parsing bits") > > I believe that as well. I've send a response to the original patch mail > describing what goes wrong here. > > Sascha > > The issue is now addressed thanks to Sascha findings, it is fixed in nand/next and should be in today's linux-next as well. Please tell me if the CI still reports an error! Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-28 15:20 ` Miquel Raynal @ 2020-09-29 11:39 ` Fabio Estevam 2020-09-29 16:31 ` Fabio Estevam 0 siblings, 1 reply; 19+ messages in thread From: Fabio Estevam @ 2020-09-29 11:39 UTC (permalink / raw) To: Miquel Raynal; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer Hi Miquel, On Mon, Sep 28, 2020 at 12:20 PM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > The issue is now addressed thanks to Sascha findings, it is fixed in > nand/next and should be in today's linux-next as well. Please tell me > if the CI still reports an error! linux-next from 20200928 still reports the same error: https://storage.kernelci.org/next/master/next-20200928/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html I checked that the fix pointed out by Sascha is not present in this tree though. I do see it fixed in linux-next from today (20200929), but the report from 20200929 is not yet available. I will let you know after kernelci publishes the boot results from today's next. Thanks, Fabio Estevam ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-29 11:39 ` Fabio Estevam @ 2020-09-29 16:31 ` Fabio Estevam 2020-09-29 17:46 ` Miquel Raynal 2020-09-30 14:58 ` Miquel Raynal 0 siblings, 2 replies; 19+ messages in thread From: Fabio Estevam @ 2020-09-29 16:31 UTC (permalink / raw) To: Miquel Raynal; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer Hi Miquel, On Tue, Sep 29, 2020 at 8:39 AM Fabio Estevam <festevam@gmail.com> wrote: > linux-next from 20200928 still reports the same error: > > https://storage.kernelci.org/next/master/next-20200928/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > I checked that the fix pointed out by Sascha is not present in this tree though. > > I do see it fixed in linux-next from today (20200929), but the report > from 20200929 is not yet available. > > I will let you know after kernelci publishes the boot results from today's next. linux-next 20200929 has an even worse log with a new division by zero: https://storage.kernelci.org/next/master/next-20200929/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-29 16:31 ` Fabio Estevam @ 2020-09-29 17:46 ` Miquel Raynal 2020-09-30 14:58 ` Miquel Raynal 1 sibling, 0 replies; 19+ messages in thread From: Miquel Raynal @ 2020-09-29 17:46 UTC (permalink / raw) To: Fabio Estevam; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer Hi Fabio, Fabio Estevam <festevam@gmail.com> wrote on Tue, 29 Sep 2020 13:31:03 -0300: > Hi Miquel, > > On Tue, Sep 29, 2020 at 8:39 AM Fabio Estevam <festevam@gmail.com> wrote: > > > linux-next from 20200928 still reports the same error: > > > > https://storage.kernelci.org/next/master/next-20200928/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > I checked that the fix pointed out by Sascha is not present in this tree though. > > > > I do see it fixed in linux-next from today (20200929), but the report > > from 20200929 is not yet available. > > > > I will let you know after kernelci publishes the boot results from today's next. > > linux-next 20200929 has an even worse log with a new division by zero: > > https://storage.kernelci.org/next/master/next-20200929/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html Gasp... I'll have a look tomorrow. Unfortunately I don't have access to a board with a raw NAND on it right now so I cannot test easily. Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-29 16:31 ` Fabio Estevam 2020-09-29 17:46 ` Miquel Raynal @ 2020-09-30 14:58 ` Miquel Raynal 2020-10-01 21:39 ` Fabio Estevam 1 sibling, 1 reply; 19+ messages in thread From: Miquel Raynal @ 2020-09-30 14:58 UTC (permalink / raw) To: Fabio Estevam; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer Hello, Fabio Estevam <festevam@gmail.com> wrote on Tue, 29 Sep 2020 13:31:03 -0300: > Hi Miquel, > > On Tue, Sep 29, 2020 at 8:39 AM Fabio Estevam <festevam@gmail.com> wrote: > > > linux-next from 20200928 still reports the same error: > > > > https://storage.kernelci.org/next/master/next-20200928/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > I checked that the fix pointed out by Sascha is not present in this tree though. > > > > I do see it fixed in linux-next from today (20200929), but the report > > from 20200929 is not yet available. > > > > I will let you know after kernelci publishes the boot results from today's next. > > linux-next 20200929 has an even worse log with a new division by zero: > > https://storage.kernelci.org/next/master/next-20200929/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html I found a board (NTC C.H.I.P, sun5i R8 SoC) with a parallel NAND. with the same DT properties but a different NAND controller driver I could not reproduce the issue. However, I did tried with soft BCH and I found a related error which I just fixed. I also got a bug report from Intel's robot which I also fixed. Perhaps what you see is related (still around this ->total entry not being set at the right moment at probe time). If tomorrow's report still returns an error, we will need someone to get physical access to the device to add debugging code and track down the failure. Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-09-30 14:58 ` Miquel Raynal @ 2020-10-01 21:39 ` Fabio Estevam 2020-10-13 17:14 ` Lucas Stach 0 siblings, 1 reply; 19+ messages in thread From: Fabio Estevam @ 2020-10-01 21:39 UTC (permalink / raw) To: Miquel Raynal; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer Hi Miquel, On Wed, Sep 30, 2020 at 11:58 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > I found a board (NTC C.H.I.P, sun5i R8 SoC) with a parallel NAND. > > with the same DT properties but a different NAND controller driver I > could not reproduce the issue. > > However, I did tried with soft BCH and I found a related error which I > just fixed. I also got a bug report from Intel's robot which I also > fixed. Perhaps what you see is related (still around this ->total entry > not being set at the right moment at probe time). > > If tomorrow's report still returns an error, we will need someone > to get physical access to the device to add debugging code and track > down the failure. The linux-next 20201001 results from the kernelci Pengutronix lab did not show up, so let's wait to see the imx27 boot log to be back online again. Thanks, Fabio Estevam ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-01 21:39 ` Fabio Estevam @ 2020-10-13 17:14 ` Lucas Stach 2020-10-13 17:36 ` Miquel Raynal 2020-10-14 12:23 ` Fabio Estevam 0 siblings, 2 replies; 19+ messages in thread From: Lucas Stach @ 2020-10-13 17:14 UTC (permalink / raw) To: Fabio Estevam, Miquel Raynal; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer On Do, 2020-10-01 at 18:39 -0300, Fabio Estevam wrote: > Hi Miquel, > > On Wed, Sep 30, 2020 at 11:58 AM Miquel Raynal > <miquel.raynal@bootlin.com> wrote: > > > I found a board (NTC C.H.I.P, sun5i R8 SoC) with a parallel NAND. > > > > with the same DT properties but a different NAND controller driver I > > could not reproduce the issue. > > > > However, I did tried with soft BCH and I found a related error which I > > just fixed. I also got a bug report from Intel's robot which I also > > fixed. Perhaps what you see is related (still around this ->total entry > > not being set at the right moment at probe time). > > > > If tomorrow's report still returns an error, we will need someone > > to get physical access to the device to add debugging code and track > > down the failure. > > The linux-next 20201001 results from the kernelci Pengutronix lab did > not show up, so let's wait to see the imx27 boot log to be back online > again. Just a heads up: our lab was broken for some time due to a local issue. It's now back to working and the backlog of jobs is being executed on the boards right now, so results should show up in the near furture. Regards, Lucas ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-13 17:14 ` Lucas Stach @ 2020-10-13 17:36 ` Miquel Raynal 2020-10-14 12:23 ` Fabio Estevam 1 sibling, 0 replies; 19+ messages in thread From: Miquel Raynal @ 2020-10-13 17:36 UTC (permalink / raw) To: Lucas Stach; +Cc: linux-mtd, Sascha Hauer, Fabio Estevam, Sascha Hauer Hi Lucas, Lucas Stach <l.stach@pengutronix.de> wrote on Tue, 13 Oct 2020 19:14:21 +0200: > On Do, 2020-10-01 at 18:39 -0300, Fabio Estevam wrote: > > Hi Miquel, > > > > On Wed, Sep 30, 2020 at 11:58 AM Miquel Raynal > > <miquel.raynal@bootlin.com> wrote: > > > > > I found a board (NTC C.H.I.P, sun5i R8 SoC) with a parallel NAND. > > > > > > with the same DT properties but a different NAND controller driver I > > > could not reproduce the issue. > > > > > > However, I did tried with soft BCH and I found a related error which I > > > just fixed. I also got a bug report from Intel's robot which I also > > > fixed. Perhaps what you see is related (still around this ->total entry > > > not being set at the right moment at probe time). > > > > > > If tomorrow's report still returns an error, we will need someone > > > to get physical access to the device to add debugging code and track > > > down the failure. > > > > The linux-next 20201001 results from the kernelci Pengutronix lab did > > not show up, so let's wait to see the imx27 boot log to be back online > > again. > > Just a heads up: our lab was broken for some time due to a local issue. > It's now back to working and the backlog of jobs is being executed on > the boards right now, so results should show up in the near furture. Understood. Based on the previous reports shared by Fabio I tried to forge the html links toward more recent reports but it always failed, now I know why :) Cheers, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-13 17:14 ` Lucas Stach 2020-10-13 17:36 ` Miquel Raynal @ 2020-10-14 12:23 ` Fabio Estevam 2020-10-14 13:26 ` Miquel Raynal 1 sibling, 1 reply; 19+ messages in thread From: Fabio Estevam @ 2020-10-14 12:23 UTC (permalink / raw) To: Lucas Stach; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer, Miquel Raynal Hi Lucas, On Tue, Oct 13, 2020 at 2:14 PM Lucas Stach <l.stach@pengutronix.de> wrote: > Just a heads up: our lab was broken for some time due to a local issue. > It's now back to working and the backlog of jobs is being executed on > the boards right now, so results should show up in the near furture. Thanks. kernelci results are showing up again. Hi Miquel, The ECC issues are still present on imx27-phytec-phycard-s-rdk: https://storage.kernelci.org/next/master/next-20201013/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html BTW, ECC errors are also reported on a Freescale Layerscape 2088A RDB Board: https://www.spinics.net/lists/iommu/msg49772.html Regards, Fabio Estevam ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-14 12:23 ` Fabio Estevam @ 2020-10-14 13:26 ` Miquel Raynal 2020-10-15 19:24 ` Fabio Estevam ` (2 more replies) 0 siblings, 3 replies; 19+ messages in thread From: Miquel Raynal @ 2020-10-14 13:26 UTC (permalink / raw) To: Fabio Estevam; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer, Lucas Stach Hi Fabio, Sascha, Lucas, Fabio Estevam <festevam@gmail.com> wrote on Wed, 14 Oct 2020 09:23:06 -0300: > Hi Lucas, > > On Tue, Oct 13, 2020 at 2:14 PM Lucas Stach <l.stach@pengutronix.de> wrote: > > > Just a heads up: our lab was broken for some time due to a local issue. > > It's now back to working and the backlog of jobs is being executed on > > the boards right now, so results should show up in the near furture. > > Thanks. kernelci results are showing up again. > > Hi Miquel, > > The ECC issues are still present on imx27-phytec-phycard-s-rdk: > https://storage.kernelci.org/next/master/next-20201013/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > BTW, ECC errors are also reported on a Freescale Layerscape 2088A RDB Board: > https://www.spinics.net/lists/iommu/msg49772.html > I supposed the failing division is this one: /* * Set the number of read / write steps for one page depending on ECC * mode. */ ecc->steps = mtd->writesize / ecc->size; if (ecc->steps * ecc->size != mtd->writesize) { WARN(1, "Invalid ECC parameters\n"); ret = -EINVAL; goto err_nand_manuf_cleanup; } Which produces the warning right after. This would mean that ecc->size == 0 But this entry is set before calling nand_scan() in mxcnd_probe() to 512, and that does not make sense. I guess we should be able to reproduce this error with any board using the mxc NAND driver with hardware correction (now also called "on host"). Do you have such a board available to at least verify that the beginning of my logic is right? Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-14 13:26 ` Miquel Raynal @ 2020-10-15 19:24 ` Fabio Estevam 2020-10-16 3:04 ` Han Xu 2020-10-16 6:39 ` Sascha Hauer 2 siblings, 0 replies; 19+ messages in thread From: Fabio Estevam @ 2020-10-15 19:24 UTC (permalink / raw) To: Miquel Raynal; +Cc: Sascha Hauer, linux-mtd, Sascha Hauer, Lucas Stach Hi Miquel, On Wed, Oct 14, 2020 at 10:27 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > I guess we should be able to reproduce this error with any board using > the mxc NAND driver with hardware correction (now also called "on > host"). Do you have such a board available to at least verify that the > beginning of my logic is right? I don't have access to such hardware, sorry. Thanks ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-14 13:26 ` Miquel Raynal 2020-10-15 19:24 ` Fabio Estevam @ 2020-10-16 3:04 ` Han Xu 2020-10-16 6:39 ` Sascha Hauer 2 siblings, 0 replies; 19+ messages in thread From: Han Xu @ 2020-10-16 3:04 UTC (permalink / raw) To: Miquel Raynal Cc: linux-mtd, Sascha Hauer, Fabio Estevam, Sascha Hauer, Lucas Stach On Wed, Oct 14, 2020 at 8:32 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > Hi Fabio, Sascha, Lucas, > > Fabio Estevam <festevam@gmail.com> wrote on Wed, 14 Oct 2020 09:23:06 > -0300: > > > Hi Lucas, > > > > On Tue, Oct 13, 2020 at 2:14 PM Lucas Stach <l.stach@pengutronix.de> wrote: > > > > > Just a heads up: our lab was broken for some time due to a local issue. > > > It's now back to working and the backlog of jobs is being executed on > > > the boards right now, so results should show up in the near furture. > > > > Thanks. kernelci results are showing up again. > > > > Hi Miquel, > > > > The ECC issues are still present on imx27-phytec-phycard-s-rdk: > > https://storage.kernelci.org/next/master/next-20201013/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > BTW, ECC errors are also reported on a Freescale Layerscape 2088A RDB Board: > > https://www.spinics.net/lists/iommu/msg49772.html > > > > I supposed the failing division is this one: > > /* > * Set the number of read / write steps for one page depending on ECC > * mode. > */ > ecc->steps = mtd->writesize / ecc->size; > if (ecc->steps * ecc->size != mtd->writesize) { > WARN(1, "Invalid ECC parameters\n"); > ret = -EINVAL; > goto err_nand_manuf_cleanup; > } > > Which produces the warning right after. This would mean that > > ecc->size == 0 > > But this entry is set before calling nand_scan() in mxcnd_probe() to > 512, and that does not make sense. > > I guess we should be able to reproduce this error with any board using > the mxc NAND driver with hardware correction (now also called "on > host"). Do you have such a board available to at least verify that the > beginning of my logic is right? Hi Miquel, I debugged on one LS1043A platform and found in rawnand_dt_init(), the chip->ecc.strength was set to nand->ecc.user_conf.strength, which may not have been touched before. Same as chip->ecc.size. > > Thanks, > Miquèl > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/ -- Sincerely, Han XU ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-14 13:26 ` Miquel Raynal 2020-10-15 19:24 ` Fabio Estevam 2020-10-16 3:04 ` Han Xu @ 2020-10-16 6:39 ` Sascha Hauer 2020-10-16 7:53 ` Miquel Raynal 2 siblings, 1 reply; 19+ messages in thread From: Sascha Hauer @ 2020-10-16 6:39 UTC (permalink / raw) To: Miquel Raynal; +Cc: linux-mtd, Fabio Estevam, Sascha Hauer, Lucas Stach Hi Miquel, On Wed, Oct 14, 2020 at 03:26:59PM +0200, Miquel Raynal wrote: > Hi Fabio, Sascha, Lucas, > > > > > The ECC issues are still present on imx27-phytec-phycard-s-rdk: > > https://storage.kernelci.org/next/master/next-20201013/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > BTW, ECC errors are also reported on a Freescale Layerscape 2088A RDB Board: > > https://www.spinics.net/lists/iommu/msg49772.html > > > > I supposed the failing division is this one: > > /* > * Set the number of read / write steps for one page depending on ECC > * mode. > */ > ecc->steps = mtd->writesize / ecc->size; > if (ecc->steps * ecc->size != mtd->writesize) { > WARN(1, "Invalid ECC parameters\n"); > ret = -EINVAL; > goto err_nand_manuf_cleanup; > } > > Which produces the warning right after. This would mean that > > ecc->size == 0 > > But this entry is set before calling nand_scan() in mxcnd_probe() to > 512, and that does not make sense. > > I guess we should be able to reproduce this error with any board using > the mxc NAND driver with hardware correction (now also called "on > host"). Do you have such a board available to at least verify that the > beginning of my logic is right? We have a board, I'll give it a test Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: imx27: Unsupported ECC algorithm 2020-10-16 6:39 ` Sascha Hauer @ 2020-10-16 7:53 ` Miquel Raynal 0 siblings, 0 replies; 19+ messages in thread From: Miquel Raynal @ 2020-10-16 7:53 UTC (permalink / raw) To: Sascha Hauer; +Cc: linux-mtd, Fabio Estevam, Sascha Hauer, Lucas Stach Hello, Sascha Hauer <s.hauer@pengutronix.de> wrote on Fri, 16 Oct 2020 08:39:10 +0200: > Hi Miquel, > > On Wed, Oct 14, 2020 at 03:26:59PM +0200, Miquel Raynal wrote: > > Hi Fabio, Sascha, Lucas, > > > > > > > > The ECC issues are still present on imx27-phytec-phycard-s-rdk: > > > https://storage.kernelci.org/next/master/next-20201013/arm/imx_v4_v5_defconfig/gcc-8/lab-pengutronix/baseline-imx27-phytec-phycard-s-rdk.html > > > > > > BTW, ECC errors are also reported on a Freescale Layerscape 2088A RDB Board: > > > https://www.spinics.net/lists/iommu/msg49772.html > > > > > > > I supposed the failing division is this one: > > > > /* > > * Set the number of read / write steps for one page depending on ECC > > * mode. > > */ > > ecc->steps = mtd->writesize / ecc->size; > > if (ecc->steps * ecc->size != mtd->writesize) { > > WARN(1, "Invalid ECC parameters\n"); > > ret = -EINVAL; > > goto err_nand_manuf_cleanup; > > } > > > > Which produces the warning right after. This would mean that > > > > ecc->size == 0 > > > > But this entry is set before calling nand_scan() in mxcnd_probe() to > > 512, and that does not make sense. > > > > I guess we should be able to reproduce this error with any board using > > the mxc NAND driver with hardware correction (now also called "on > > host"). Do you have such a board available to at least verify that the > > beginning of my logic is right? > > We have a board, I'll give it a test Thanks to Han's analysis I may have found the issue. Please have a look at [1]. Sascha, if you reproduce the error you may also want to test this patch. [1] https://patchwork.ozlabs.org/project/linux-mtd/patch/20201016074942.29650-1-miquel.raynal@bootlin.com/ Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2020-10-16 7:54 UTC | newest] Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-09-16 13:49 imx27: Unsupported ECC algorithm Fabio Estevam 2020-09-17 16:13 ` Miquel Raynal 2020-09-17 16:45 ` Miquel Raynal 2020-09-17 17:12 ` Fabio Estevam 2020-09-21 9:40 ` Sascha Hauer 2020-09-28 15:20 ` Miquel Raynal 2020-09-29 11:39 ` Fabio Estevam 2020-09-29 16:31 ` Fabio Estevam 2020-09-29 17:46 ` Miquel Raynal 2020-09-30 14:58 ` Miquel Raynal 2020-10-01 21:39 ` Fabio Estevam 2020-10-13 17:14 ` Lucas Stach 2020-10-13 17:36 ` Miquel Raynal 2020-10-14 12:23 ` Fabio Estevam 2020-10-14 13:26 ` Miquel Raynal 2020-10-15 19:24 ` Fabio Estevam 2020-10-16 3:04 ` Han Xu 2020-10-16 6:39 ` Sascha Hauer 2020-10-16 7:53 ` Miquel Raynal
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.