public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
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

  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