cmd: nand: Extend nand info to print ecc information

Message ID 20220922133937.277463-1-michael@amarulasolutions.com
State New
Headers show
Series
  • cmd: nand: Extend nand info to print ecc information
Related show

Commit Message

Michael Nazzareno Trimarchi Sept. 22, 2022, 1:39 p.m. UTC
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(-)

Comments

Dario Binacchi Sept. 22, 2022, 1:43 p.m. UTC | #1
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
Michael Nazzareno Trimarchi Oct. 21, 2022, 1:05 p.m. UTC | #2
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
Tom Rini Oct. 21, 2022, 1:14 p.m. UTC | #3
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.

Patch

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);