From: Michal Simek <michal.simek@amd.com>
To: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
Michal Simek <monstr@monstr.eu>
Cc: Ashok Reddy Soma <ashok.reddy.soma@amd.com>,
U-Boot-Denx <u-boot@lists.denx.de>,
Dario Binacchi <dario.binacchi@amarulasolutions.com>,
Simon Glass <sjg@chromium.org>
Subject: Re: [PATCH v2] mtd: nand: Mark reserved blocks
Date: Thu, 2 Feb 2023 15:49:57 +0100 [thread overview]
Message-ID: <4b6fc7ed-86d3-0bf2-bf56-38f86d4aab17@amd.com> (raw)
In-Reply-To: <CAOf5uwnwBEGqXNrhbEbA9MrdR8Np+M75pO375Sg_bBSnX36p_w@mail.gmail.com>
On 2/2/23 14:44, Michael Nazzareno Trimarchi wrote:
> Hi
>
> Il gio 2 feb 2023, 14:21 Michal Simek <monstr@monstr.eu
> <mailto:monstr@monstr.eu>> ha scritto:
>
> Hi Dario,
>
> On 1/5/23 10:46, Ashok Reddy Soma wrote:
> > Reserved blocks are used for storing bad block tables. With "nand bad"
> > command, these reserved blocks are shown as bad blocks. This is leading
> > to confusion when compared with Linux bad blocks. Hence, display
> > "bbt reserved" when printing reserved blocks with "nand bad" command.
> >
> > To acheive this, return 2 which represents reserved from nand_isbad_bbt()
> > instead of 1 in case of reserved blocks and catch it in cmd/nand.c.
> >
> > "nand bad" command display's hexadecimal numbers, so add "0x" prefix.
> >
> > Example log will show up as below.
> >
> > ZynqMP> nand bad
> >
> > Device 0 bad blocks:
> > 0x00400000
> > 0x16800000
> > 0x16c00000
> > 0x17000000
> > 0x3d800000
> > 0x3e400000
> > 0xe8400000
> > 0xff000000 (bbt reserved)
> > 0xff400000 (bbt reserved)
> > 0xff800000 (bbt reserved)
> > 0xffc00000 (bbt reserved)
> > 0x116800000
> > 0x116c00000
> > 0x1ff000000 (bbt reserved)
> > 0x1ff400000 (bbt reserved)
> > 0x1ff800000 (bbt reserved)
> > 0x1ffc00000 (bbt reserved)
> >
> > Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma@amd.com
> <mailto:ashok.reddy.soma@amd.com>>
> > Reviewed-by: Michael Trimarchi <michael@amarulasolutions.com
> <mailto:michael@amarulasolutions.com>>
> > ---
>
> Are you going to apply this patch?
>
>
> Applied already
I looked at u-boot-nand-flash git repo and I can't see it.
Where's the repo where this is merged?
Thanks,
Michal
next prev parent reply other threads:[~2023-02-02 14:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-05 9:46 [PATCH v2] mtd: nand: Mark reserved blocks Ashok Reddy Soma
2023-01-05 10:04 ` Michael Nazzareno Trimarchi
2023-02-02 13:21 ` Michal Simek
2023-02-02 13:44 ` Michael Nazzareno Trimarchi
2023-02-02 14:49 ` Michal Simek [this message]
2023-02-27 16:36 ` Dario Binacchi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4b6fc7ed-86d3-0bf2-bf56-38f86d4aab17@amd.com \
--to=michal.simek@amd.com \
--cc=ashok.reddy.soma@amd.com \
--cc=dario.binacchi@amarulasolutions.com \
--cc=michael@amarulasolutions.com \
--cc=monstr@monstr.eu \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.