From: matthieu castet <castet.matthieu@free.fr>
To: Linux Kernel list <linux-kernel@vger.kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>, Alex Dubov <oakad@yahoo.com>
Subject: Re: Smartmedia/xd card support - request for comments
Date: Sat, 19 Apr 2008 10:31:51 +0200 [thread overview]
Message-ID: <4809ADF7.9040604@free.fr> (raw)
Hi,
> On Fri, 18 Apr 2008, Alex Dubov wrote:
>
> First off, can you please use a mail client which does line breaks at
> around 78 chars ?
>
>> As you understand, both smartmedia media and hosts follow certain
>> spec (publicly available here:
>> http://www.ssfdc.or.jp/spec/index_e.htm, basic registration
>> required). xd card spec is almost identical, adding some cosmetic
>> features and restrictions (such as prohibition of single page
>> programming). This means, that smartmedia access protocol can be
>> abstracted out in the reader-independent way. You can seen in the
>> xd_card_blk.c that it can operate both completely dumb controllers,
>> while taking protocol shortcuts (using host->caps) to accommodate
>> smarter readers (the only feature currently not there is
>> adapter-side page copy - it can be easily added under FBD_COPY
>> clause in xd_card_trans_req). The backend itself (jmb38x_xd.c)
BTW some bare nand flash have support to page copy (there is some issue
because of no ecc checking during the copy). So the support of this
could be usefull in mtd.
> Having a separate driver infrastructure for clever controllers, which
> provide only semi raw access to the FLASH chip is fine simply because
> such controllers do not fit into the MTD layer.
I wonder if mtd shouldn't modified to handler more clever controller :
I saw some raw nand controller where you can program a batch of
command/read/write/ecc checking (that work on a internal RAM of the
controller (and may be use DMA)) and all you have to do is to wait the
controller do the job.
These controllers are very hard to integrate in current nand layer
because it is very low level controller oriented.
>> > Correct. Erase() is the only callback-based function and every single
>> > driver implements it synchronous anyway. Improving this situation is
>> > needed for mtd's own sake anyway. Either that or replacing mtd with
>> > something else, likely the block device code.
>>
Current mtd have other limitations (4GB limit fix, dma buffer, ...),
but in the end it should be fixed.
Matthieu
PS : by using mtd layer you :
- will make improvement that will be useful for everybody
- will use a stack that have been and will be well tested
- will have some controller drivers for free
- will have nand fs for free (for people that don't trust FTL).
- may be provide interesting FTL for raw nand controller
next reply other threads:[~2008-04-19 8:32 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-19 8:31 matthieu castet [this message]
2008-04-20 16:01 ` Smartmedia/xd card support - request for comments Jörn Engel
2008-04-21 1:33 ` Alex Dubov
-- strict thread matches above, loose matches on Subject: below --
2008-04-17 8:50 Alex Dubov
2008-04-17 9:01 ` Ben Dooks
2008-04-17 9:11 ` Alex Dubov
2008-04-17 13:10 ` Andy Lutomirski
2008-04-17 13:36 ` Arnd Bergmann
2008-04-17 13:38 ` Thomas Gleixner
2008-04-17 14:07 ` Jörn Engel
2008-04-17 19:11 ` Thomas Gleixner
2008-04-17 21:21 ` Jörn Engel
2008-04-18 8:47 ` Alex Dubov
2008-04-18 9:35 ` Thomas Gleixner
2008-04-19 2:49 ` Alex Dubov
2008-04-19 5:56 ` Thomas Gleixner
2008-04-18 14:00 ` Jörn Engel
2008-04-19 3:05 ` Alex Dubov
2008-04-19 6:37 ` Thomas Gleixner
2008-04-19 16:35 ` Jörn Engel
2008-04-20 2:25 ` Alex Dubov
2008-04-17 13:35 ` Thomas Gleixner
2008-04-17 14:19 ` Jörn Engel
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=4809ADF7.9040604@free.fr \
--to=castet.matthieu@free.fr \
--cc=linux-kernel@vger.kernel.org \
--cc=oakad@yahoo.com \
--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 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.