mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Jörn Engel" <joern@logfs.org>
To: Alex Dubov <oakad@yahoo.com>
Cc: Linux kernel mailing list <linux-kernel@vger.kernel.org>
Subject: Re: Smartmedia/xd card support - request for comments
Date: Thu, 17 Apr 2008 16:19:11 +0200	[thread overview]
Message-ID: <20080417141911.GB30001@logfs.org> (raw)
In-Reply-To: <466372.69780.qm@web36702.mail.mud.yahoo.com>

On Thu, 17 April 2008 01:50:33 -0700, Alex Dubov wrote:
> 
> Current version will always issue whole block writes (no individual page programming). I,in fact,
> implemented a version which tracks partially filled blocks, but experiments with some Olympus xd
> cards triggered a lot of funny hardware problems with this, and then, it appeared that latest
> version of xd card spec specifically disallows partial block writes.

drivers/mtd/nand/alauda.c already does page writes.  You didn't know the
existing wheel and invented new one.  That happens and there is nothing
wrong with it.

Your chances of merging your new code as-is are roughly zero.  But I
would still like to see the code and see where yours is better and ours
should be improved.  Or you could take a look yourself and send patches.

> MemoryStick spec does allow individual page writes, but so did an older version of xd card spec.
> Given this, I decided to start with simpler version of FTL for now and replace it with a more
> complex one if need arises.

Are Memory Sticks covered by your code?  That would be a worthwhile
improvement.

Jörn

-- 
Joern's library part 3:
http://inst.eecs.berkeley.edu/~cs152/fa05/handouts/clark-test.pdf

  parent reply	other threads:[~2008-04-17 14:19 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2008-04-19  8:31 matthieu castet
2008-04-20 16:01 ` Jörn Engel
2008-04-21  1:33   ` Alex Dubov

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=20080417141911.GB30001@logfs.org \
    --to=joern@logfs.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oakad@yahoo.com \
    /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

all inboxes | Powered by JetHome®