[PATCH] cmd: nand: Extend nand info to print ecc information

[Michael Nazzareno Trimarchi]

HI Tom

On Thu, Sep 22, 2022 at 3:43 PM Dario Binacchi
<dario.binacchi at amarulasolutions.com> wrote:
>
> Hi Michael,
>
> On Thu, Sep 22, 2022 at 3:39 PM Michael Trimarchi
> <michael at 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 at 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 at amarulasolutions.com>
>

Can I pick it up?

Michael

> Thanks and regards,
> Dario
> --
>
> Dario Binacchi
>
> Embedded Linux Developer
>
> dario.binacchi at amarulasolutions.com
>
> __________________________________
>
>
> Amarula Solutions SRL
>
> Via Le Canevare 30, 31100 Treviso, Veneto, IT
>
> T. +39 042 243 5310
> info at amarulasolutions.com
>
> www.amarulasolutions.com



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com