public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Tim Hockin <thockin@sun.com>
To: David Woodhouse <dwmw2@infradead.org>
Cc: linux-mtd@lists.infradead.org
Subject: Re: cfi_cmdset_0002 and writes
Date: Wed, 18 Jul 2001 15:29:48 -0700	[thread overview]
Message-ID: <3B560DDC.55ECB635@sun.com> (raw)
In-Reply-To: 26325.995493875@redhat.com

David Woodhouse wrote:
> 
> thockin@sun.com said:
> >  The flash can be erased by 64k blocks.  Since it is flash, you can
> > only reprogram '1' bits to '0' bits, and never vice versa.  What I
> > don't find is any hooks in cfi_cmdset_0002 to read a whole block,
> > erase it, make changes to the in-memory block and write it back to
> > flash.

> It doesn't. Doing that kind of thing is scary and dangerous, and certainly
> doesn't belong in the low-level driver. We have real filesystems and
> translation layers which work on flash instead.

Right, but this is our boot ROM we're dealing with :)  Can you suggest a
better alternative?  The device is rarely ever loaded, but when it is
loaded we want to read and re-program the whole thing as safely as
possible.

MTD seems to be geared more towards flash devices that get lots of use. 
How should I go about plugging this into MTD?
 
> However, the primitive read/erase/modify/pray_we_don't_lose_power/writeback
> behaviour you described is available in the 'mtdblock' driver, and it's sane
> enough to use it if you only write to it during setup, then run it normally
> in read-only mode.

We'll be loading the driver for our flash after potentially months of
uptime.  The new image will be flashed in, and the driver will be unloaded
until the next flash update.


-- 
Tim Hockin
Systems Software Engineer
Sun Microsystems, Cobalt Server Appliances
thockin@sun.com

  reply	other threads:[~2001-07-18 22:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-07-18 21:58 cfi_cmdset_0002 and writes Tim Hockin
2001-07-18 22:04 ` David Woodhouse
2001-07-18 22:29   ` Tim Hockin [this message]
2001-07-18 22:42     ` David Woodhouse
2001-07-19 17:44     ` Bjorn Wesen
2001-07-18 22:12 ` David Woodhouse

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=3B560DDC.55ECB635@sun.com \
    --to=thockin@sun.com \
    --cc=dwmw2@infradead.org \
    --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