public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Rohit <h.rohit@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Flex-OneNAND driver
Date: Mon, 03 Nov 2008 13:05:48 +0530	[thread overview]
Message-ID: <490EA9D4.8080801@samsung.com> (raw)
In-Reply-To: <9c9fda240810272153i39a4a830h6389ee63afbaeb0a@mail.gmail.com>

Hi,

Kyungmin Park wrote:
>> Why?  If it's dynamic changes to the boundary that concern you, you could
>> tear down the device(s) and re-initialize.
> 
> Okay consider that add reset or reprobe command to do this
> 
We can expose Flex-OneNAND as a SLC device and a MLC device by having
two mtd_info structures. The difference between the two devices is block size only.

 1. This makes user aware of SLC and MLC areas.
 2. We expose two devices where as there is a single physical device.
    Also, MTD provides erase regions capability for such devices ie which
    have many erase regions.

Exposing as two devices reduces code by removing code for erase regions, but only little.
Also other support for Flex-OneNAND is still needed ie
- all 4KB dataram is used during read/write.
- read/write oob-only commands are not present.
- LSB recovery for MLC area read failure
- 4 ecc registers

>> Of course, I may simply be missing something about how this hardware works.
>>
>>> Also, comments from MTD mailing list have been included.
>> If this is going to go into Linux, perhaps we should wait until it is merged
>> there and then import the result?
> 
> It's almost done and wait to merge into MTD tree.
> As I mentioned. First merge MTD tree and then U-Boot tree.
> 
>> At least, I'd like Kyungmin Park's ack for OneNAND stuff.
> 
> Basically I acked this patch.

I'll repost after MTD merge.

Thanks,
Rohit

  parent reply	other threads:[~2008-11-03  7:35 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-22  6:28 [U-Boot] [PATCH] Flex-OneNAND driver apgmoorthy
2008-10-14 11:30 ` Wolfgang Denk
2008-10-14 15:37   ` Scott Wood
2008-10-22 23:09 ` Scott Wood
2008-10-24 13:50   ` Rohit
2008-10-27 18:04     ` Scott Wood
2008-10-28  4:53       ` Kyungmin Park
2008-10-28 15:51         ` Scott Wood
2008-11-03  7:35         ` Rohit [this message]
2008-11-06  3:29   ` Amit Kumar Sharma
2008-11-06 17:11     ` Scott Wood
  -- strict thread matches above, loose matches on Subject: below --
2008-09-25 13:38 apgmoorthy
2008-09-26  1:27 ` Kyungmin Park
2008-10-16  7:43   ` Rohit

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=490EA9D4.8080801@samsung.com \
    --to=h.rohit@samsung.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox