From: Todd Poynor <tpoynor@mvista.com>
To: tglx@linutronix.de
Cc: linux-mtd@lists.infradead.org
Subject: Re: NAND bbt build reads wrong oob data
Date: Fri, 15 Jul 2005 00:58:48 -0700 [thread overview]
Message-ID: <42D76CB8.4050606@mvista.com> (raw)
In-Reply-To: <1121412168.24449.98.camel@tglx.tec.linutronix.de>
Thomas Gleixner wrote:
> On Thu, 2005-07-14 at 20:19 -0700, Todd Poynor wrote:
>
>>The value read seems to be a different byte, perhaps offset 14 instead
>>of offset 5 as specified by the placement scheme. The hack that seems
>>to fix it for me, namely to read the whole OOB area instead of the
>>individual byte, is below.
>
>
> That's pretty strange. Which chip type are you using ?
NAND device: Manufacturer ID: 0xec, Chip ID: 0x45 (Samsung NAND 32MiB
1,8V 16-bit)
using omap-nand-flash.c driver from OMAP community tree
linux-omap-2.6.git at source.mvista.com (which is slated for sending to
linux-mtd after conversion of board specifics to LDM platform device stuff).
OK, this is pretty unexpected, so I'll try harder to figure out where it
goes wrong or whether I've got another board that it affects (a
colleague reported some new bad blocks messages after upgrading to the
same software on a different board, will see if there's a similar
situation).
>>and
>>NAND_BBT_SCAN2NDPAGE is triggering a BUG I haven't investigated yet.)
>
>
> You introduced it by setting readlen = mtd->ooblock. I know that the
> name of this is confusing. oobblock is the page size. So readlen is >
> buffersize.
d'oh! Thanks,
--
Todd
next prev parent reply other threads:[~2005-07-15 7:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-07-15 3:19 NAND bbt build reads wrong oob data Todd Poynor
2005-07-15 7:22 ` Thomas Gleixner
2005-07-15 7:58 ` Todd Poynor [this message]
2005-07-15 13:57 ` Thomas Gleixner
-- strict thread matches above, loose matches on Subject: below --
2005-07-15 16:51 Martin Fouts
2005-07-15 18:57 ` Todd Poynor
[not found] <DE88BDF02F4319469812588C7950A97E931035@ussunex1.palmsource.com>
2005-07-15 20:32 ` Thomas Gleixner
2005-07-16 11:33 ` David Woodhouse
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=42D76CB8.4050606@mvista.com \
--to=tpoynor@mvista.com \
--cc=linux-mtd@lists.infradead.org \
--cc=tglx@linutronix.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox