From: Huang Shijie <b32955@freescale.com>
To: <dedekind1@gmail.com>
Cc: Fabio Estevam <fabio.estevam@freescale.com>,
Huang Shijie <shijie8@gmail.com>,
linux-mtd@lists.infradead.org, marex.vasut@denx.de,
kernel@pengutronix.de, shawn.guo@linaro.org,
Fabio Estevam <festevam@gmail.com>,
dwmw2@infradead.org
Subject: Re: [PATCH 1/2] nand: nand_bbt: Export nand_update_bbt
Date: Tue, 28 Aug 2012 10:19:33 +0800 [thread overview]
Message-ID: <503C2AB5.30108@freescale.com> (raw)
In-Reply-To: <1346081158.2848.363.camel@sauron.fi.intel.com>
于 2012年08月27日 23:25, Artem Bityutskiy 写道:
> On Thu, 2012-08-23 at 11:36 -0400, Huang Shijie wrote:
>>> Why this driver redefined ->block_markbad() at all, it is not supposed
>> For hardware reason, in mx23, the bad block mark is stored in the
>> first byte of the nand page;
>> in mx28/mx50/mx6q, the bad block mark is stored in the first byte of the OOB.
>>
>> That's why the driver redefined the ->block_markbad().
> OK. Would you please tell about the driver some more, I am particularly
> interested how th mx23 case works.
>
> * So the bad block marker is the first byte of the eraseblock set to 0?
yes.
> * What if the user data stars with zero? Or you hide the first NAND page
> from the user?
Please see the picture in the common_nfc_set_geometry().
It's a real nand page layout.
The `M` is metadata, it's about 10 byte len.
The bad block marker is stored in the metadata, not the the
`data` section which save the user's data.
We do not hide the first NAND page.
> * Can you point me to the code where you check if the eraseblock is bad
> or not at the initialization time (sorry, I could find myself,
> by trying to save time).
>
Please see the mx23_boot_init().
When mx23 reads a new nand chip in the first time, it will scan all
the nand chip. If it finds a bad block, it will call the
nand->block_markbad() which
is just the gpmi_block_markbad(). the gpmi_block_markbad() will mark the
first byte
of the nand page to 0 (the mx23 does not support the `swap` feature). So
NAND boot mode,
the ROM of mx23 will check the first byte of the NAND page, if it finds
a 0, it knows that
this is a bad block.
thanks
Huang Shijie
next prev parent reply other threads:[~2012-08-28 2:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-28 22:29 [PATCH] nand: Kconfig: Allow MTD_NAND_GPMI_NAND to be built as module Fabio Estevam
2012-07-28 22:29 ` [PATCH 1/2] nand: nand_bbt: Export nand_update_bbt Fabio Estevam
2012-08-23 15:08 ` Artem Bityutskiy
2012-08-23 15:36 ` Huang Shijie
2012-08-24 6:41 ` Artem Bityutskiy
2012-08-24 10:35 ` Huang Shijie
2012-08-24 10:35 ` Huang Shijie
2012-08-24 22:57 ` Brian Norris
2012-08-26 13:05 ` Huang Shijie
2012-08-27 15:25 ` Artem Bityutskiy
2012-08-28 2:19 ` Huang Shijie [this message]
2012-08-28 13:02 ` Artem Bityutskiy
-- strict thread matches above, loose matches on Subject: below --
2012-07-28 22:30 [PATCH 2/2] nand: Kconfig: Allow MTD_NAND_GPMI_NAND to be built as module Fabio Estevam
2012-07-28 22:30 ` [PATCH 1/2] nand: nand_bbt: Export nand_update_bbt Fabio Estevam
2012-08-14 22:50 ` Fabio Estevam
2012-08-15 12:59 ` Artem Bityutskiy
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=503C2AB5.30108@freescale.com \
--to=b32955@freescale.com \
--cc=dedekind1@gmail.com \
--cc=dwmw2@infradead.org \
--cc=fabio.estevam@freescale.com \
--cc=festevam@gmail.com \
--cc=kernel@pengutronix.de \
--cc=linux-mtd@lists.infradead.org \
--cc=marex.vasut@denx.de \
--cc=shawn.guo@linaro.org \
--cc=shijie8@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).