All of lore.kernel.org
 help / color / mirror / Atom feed
From: Todd Poynor <tpoynor@mvista.com>
To: linux-mtd@lists.infradead.org
Subject: Re: xip changes between 2.6.11 and 20050530?
Date: Fri, 29 Jul 2005 13:02:07 -0700	[thread overview]
Message-ID: <42EA8B3F.3040403@mvista.com> (raw)
In-Reply-To: <20050729065012.GA8186@ku-gbr.de>

Konstantin Kletschke wrote:

> To be sure:
> Any xip_* function only calls functions marked __xipram (which are
> located in RAM then hopefully also) and only __xipram marked functions
> call functions located still in flash?

Nico's comments in cfi_cmdset_0001.c describe it better than I did:

* No interrupt what so ever can be serviced while the flash isn't in array
  * mode.  This is ensured by the xip_disable() and xip_enable() functions
  * enclosing any code path where the flash is known not to be in array 
mode.
  * And within a XIP disabled code path, only functions marked with __xipram
  * may be called and nothing else (it's a good thing to inspect generated
  * assembly to make sure inline functions were actually inlined and 
that gcc
  * didn't emit calls to its own support functions). Also configuring 
MTD CFI
  * support to a single buswidth and a single interleave is also 
recommended.

The reason for "single buswidth and a single interleave" is to avoid 
division operations to compute various things driven by those two 
parameters, which will probably call a library function placed by 
default in flash to do the divide; if a single buswidth and interleave 
is configured then the derived values are compiled as constants and need 
not be computed at runtime.

Yes, it's all a little delicate. ;)

-- 
Todd

      reply	other threads:[~2005-07-29 20:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-06 14:45 xip changes between 2.6.11 and 20050530? Konstantin Kletschke
2005-07-06 16:44 ` Todd Poynor
2005-07-07  8:22   ` Konstantin Kletschke
2005-07-07 12:33   ` Konstantin Kletschke
2005-07-28 12:17 ` Konstantin Kletschke
2005-07-28 12:39   ` Konstantin Kletschke
2005-07-28 15:01   ` Nicolas Pitre
2005-07-28 18:07     ` Konstantin Kletschke
2005-07-28 18:15       ` Todd Poynor
2005-07-29  6:50         ` Konstantin Kletschke
2005-07-29 20:02           ` Todd Poynor [this message]

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=42EA8B3F.3040403@mvista.com \
    --to=tpoynor@mvista.com \
    --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 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.