All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] NAND read error on Sequoia
Date: Mon, 18 Jan 2010 05:52:16 +0100	[thread overview]
Message-ID: <201001180552.16999.sr@denx.de> (raw)
In-Reply-To: <20100117230635.6800AC88AE@gemini.denx.de>

Hi Wolfgang,

On Monday 18 January 2010 00:06:35 Wolfgang Denk wrote:
> > I don't think that this is a Sequoia (4xx NDFC) specific problem. My
> > first idea was that this is caused because of bad blocks. Bad blocks are
> > skipped upon read, but "nand read" still wants to read 32MByte in total.
> > Which is not possible when bad blocks are available.
> >
> > But this is not the case here. I just verified this on sequoia (most
> > likely the one you used). No bad blocks here. And it still fails. Reading
> > with one block (block size 16k) less seems to be ok though:
> >
> > => nand read 200000 0 1ffc000
> >
> > NAND read: device 0 offset 0x0, size 0x1ffc000
> >  33538048 bytes read: OK
> >
> > And just reading the last block also doesn't work:
> >
> > => nand read 200000 1ffc000 4000
> >
> > NAND read: device 0 offset 0x1ffc000, size 0x4000
> > Attempt to read outside the flash area
> >  16384 bytes read: ERROR
> >
> > Not sure why this is the case. Perhaps an "off by one" error? Scott do
> > you have an idea?
> 
> Any progress on this?

Yes. And my patch fixing this issue is already in mainline:

f4cfe42758192d09f8375e384cc000aa70d97029

    nand: Fix access to last block in NAND devices
    
    Currently, the last block of NAND devices can't be accessed. This patch
    fixes this issue by correcting the boundary checking (off-by-one error).
    
    Signed-off-by: Stefan Roese <sr@denx.de>
    Cc: Scott Wood <scottwood@freescale.com>
    Cc: Wolfgang Denk <wd@denx.de>

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de

      reply	other threads:[~2010-01-18  4:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-08 22:03 [U-Boot] NAND read error on Sequoia Wolfgang Denk
2009-12-09  6:09 ` Stefan Roese
2010-01-17 23:06   ` Wolfgang Denk
2010-01-18  4:52     ` Stefan Roese [this message]

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=201001180552.16999.sr@denx.de \
    --to=sr@denx.de \
    --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.