public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: tglx@linutronix.de (Thomas Gleixner)
To: linux-mtd@lists.infradead.org
Subject: copy filesystem to nand flash without ecc
Date: Mon, 17 Feb 2003 16:37:42 +0100	[thread overview]
Message-ID: <200302171637.42445.tglx@linutronix.de> (raw)
In-Reply-To: <3E50D76A.40300@netscape.net>

On Monday 17 February 2003 13:36, kokseng choo wrote:
> I also notice that nand.h also define the ecc location for
> oobsel=NAND_NONE_OOB, which in fact exactly the same as that of JFFS2.
> Why don't we just add those definitions in  oobconfigs[0] instead of
> zeroing it? Since a nand driver without ECC is really a bad idea which
> makes nand_write thus pwrite becomes useless. Can we just have a default
> "oobsel" during compile time?; which means explicitly
> define oobconfigs[0] to either JFFS2 or YAFFS or zero through
> configuration. By doing that, the mtd user does not have to use different
> API or ioctl to write to the flash. The "cp" utils will just work.
Yep, but you still do not write fs-specific information in the OOB-area, 
except the ecc.
This still solves not the problem to write a fs-image to the chip. Therefor we
need a function, which includes writing of OOB data and filesystem dependent 
oob-layout selection.

> As a mtd user at userspace, ideally I would like to use the same pwrite()
> function for the flash, and I wish I do not have to know what is the
> flash type I am talking to. All I need to know is I am talking to a
> mtd device, and I do not have to know anything about ECC, just let
> the driver handle is implicitly.
That's true if you use NAND without a filesystem.

> I attach a patch which add the default entry for oobconfig[0], I think
> it will help me a bit as a mtd and JFFS2 user at userspace.
> Please comment if it does not make sense, thank you.
I'm not happy with that, as you force it to JFFS2 layout and therefor YAFFS 
users _CANNOT_ use it. I will think about a more flexible solution.

-- 
Thomas
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx at linutronix.de

  reply	other threads:[~2003-02-17 15:37 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-13 11:16 copy filesystem to nand flash without ecc Kschoo70C at netscape.net
2003-02-13 16:52 ` Thomas Gleixner
2003-02-14  8:59   ` David Woodhouse
2003-02-14  9:03     ` Thomas Gleixner
2003-02-14 17:10   ` Alex Samoutin
2003-02-14 22:37     ` Thomas Gleixner
2003-02-15  0:13       ` David Woodhouse
2003-02-17 12:36   ` kokseng choo
2003-02-17 15:37     ` Thomas Gleixner [this message]
2003-02-17 15:02       ` kokseng choo
2003-02-17 16:02       ` kokseng choo
2003-02-17 22:30         ` Charles Manning

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=200302171637.42445.tglx@linutronix.de \
    --to=tglx@linutronix.de \
    --cc=linux-mtd@lists.infradead.org \
    /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