[PATCH] cmd: nand: Extend nand info to print ecc information
Extract the information about ecc strength and ecc step size from mtd controller. This information is usefull to check if what we think as ecc is what we really configured.
Signed-off-by: Michael Trimarchi michael@amarulasolutions.com --- cmd/nand.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/cmd/nand.c b/cmd/nand.c index e730484d0b..9a9794684b 100644 --- a/cmd/nand.c +++ b/cmd/nand.c @@ -417,12 +417,14 @@ static void nand_print_and_set_info(int idx) printf("%dx ", chip->numchips); printf("%s, sector size %u KiB\n", mtd->name, mtd->erasesize >> 10); - printf(" Page size %8d b\n", mtd->writesize); - printf(" OOB size %8d b\n", mtd->oobsize); - printf(" Erase size %8d b\n", mtd->erasesize); - printf(" subpagesize %8d b\n", chip->subpagesize); - printf(" options 0x%08x\n", chip->options); - printf(" bbt options 0x%08x\n", chip->bbt_options); + printf(" Page size %8d b\n", mtd->writesize); + printf(" OOB size %8d b\n", mtd->oobsize); + printf(" Erase size %8d b\n", mtd->erasesize); + printf(" ecc strength %8d bits\n", mtd->ecc_strength); + printf(" ecc step size %8d b\n", mtd->ecc_step_size); + printf(" subpagesize %8d b\n", chip->subpagesize); + printf(" options 0x%08x\n", chip->options); + printf(" bbt options 0x%08x\n", chip->bbt_options);
/* Set geometry info */ env_set_hex("nand_writesize", mtd->writesize);
Hi Michael,
On Thu, Sep 22, 2022 at 3:39 PM Michael Trimarchi michael@amarulasolutions.com wrote:
Extract the information about ecc strength and ecc step size from mtd controller. This information is usefull to check if what we think as ecc is what we really configured.
Signed-off-by: Michael Trimarchi michael@amarulasolutions.com
cmd/nand.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/cmd/nand.c b/cmd/nand.c index e730484d0b..9a9794684b 100644 --- a/cmd/nand.c +++ b/cmd/nand.c @@ -417,12 +417,14 @@ static void nand_print_and_set_info(int idx) printf("%dx ", chip->numchips); printf("%s, sector size %u KiB\n", mtd->name, mtd->erasesize >> 10);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" ecc strength %8d bits\n", mtd->ecc_strength);printf(" ecc step size %8d b\n", mtd->ecc_step_size);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options); /* Set geometry info */ env_set_hex("nand_writesize", mtd->writesize);-- 2.34.1
Reviewed-by: Dario Binacchi dario.binacchi@amarulasolutions.com
Thanks and regards, Dario
HI Tom
On Thu, Sep 22, 2022 at 3:43 PM Dario Binacchi dario.binacchi@amarulasolutions.com wrote:
Hi Michael,
On Thu, Sep 22, 2022 at 3:39 PM Michael Trimarchi michael@amarulasolutions.com wrote:
Extract the information about ecc strength and ecc step size from mtd controller. This information is usefull to check if what we think as ecc is what we really configured.
Signed-off-by: Michael Trimarchi michael@amarulasolutions.com
cmd/nand.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/cmd/nand.c b/cmd/nand.c index e730484d0b..9a9794684b 100644 --- a/cmd/nand.c +++ b/cmd/nand.c @@ -417,12 +417,14 @@ static void nand_print_and_set_info(int idx) printf("%dx ", chip->numchips); printf("%s, sector size %u KiB\n", mtd->name, mtd->erasesize >> 10);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" ecc strength %8d bits\n", mtd->ecc_strength);printf(" ecc step size %8d b\n", mtd->ecc_step_size);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options); /* Set geometry info */ env_set_hex("nand_writesize", mtd->writesize);-- 2.34.1
Reviewed-by: Dario Binacchi dario.binacchi@amarulasolutions.com
Can I pick it up?
Michael
Thanks and regards, Dario --
Dario Binacchi
Embedded Linux Developer
dario.binacchi@amarulasolutions.com
Amarula Solutions SRL
Via Le Canevare 30, 31100 Treviso, Veneto, IT
T. +39 042 243 5310 info@amarulasolutions.com
www.amarulasolutions.com
On Fri, Oct 21, 2022 at 03:05:48PM +0200, Michael Nazzareno Trimarchi wrote:
HI Tom
On Thu, Sep 22, 2022 at 3:43 PM Dario Binacchi dario.binacchi@amarulasolutions.com wrote:
Hi Michael,
On Thu, Sep 22, 2022 at 3:39 PM Michael Trimarchi michael@amarulasolutions.com wrote:
Extract the information about ecc strength and ecc step size from mtd controller. This information is usefull to check if what we think as ecc is what we really configured.
Signed-off-by: Michael Trimarchi michael@amarulasolutions.com
cmd/nand.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/cmd/nand.c b/cmd/nand.c index e730484d0b..9a9794684b 100644 --- a/cmd/nand.c +++ b/cmd/nand.c @@ -417,12 +417,14 @@ static void nand_print_and_set_info(int idx) printf("%dx ", chip->numchips); printf("%s, sector size %u KiB\n", mtd->name, mtd->erasesize >> 10);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options);
printf(" Page size %8d b\n", mtd->writesize);printf(" OOB size %8d b\n", mtd->oobsize);printf(" Erase size %8d b\n", mtd->erasesize);printf(" ecc strength %8d bits\n", mtd->ecc_strength);printf(" ecc step size %8d b\n", mtd->ecc_step_size);printf(" subpagesize %8d b\n", chip->subpagesize);printf(" options 0x%08x\n", chip->options);printf(" bbt options 0x%08x\n", chip->bbt_options); /* Set geometry info */ env_set_hex("nand_writesize", mtd->writesize);-- 2.34.1
Reviewed-by: Dario Binacchi dario.binacchi@amarulasolutions.com
Can I pick it up?
Yes.
participants (4)
-
Dario Binacchi -
Michael Nazzareno Trimarchi -
Michael Trimarchi
-
Tom Rini