mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Michal Suchanek <hramrach@gmail.com>,
	"R, Vignesh" <vigneshr@ti.com>,
	devicetree <devicetree@vger.kernel.org>,
	Brian Norris <computersforpeace@gmail.com>,
	Tony Lindgren <tony@atomide.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-spi <linux-spi@vger.kernel.org>,
	Huang Shijie <b32955@freescale.com>,
	MTD Maling List <linux-mtd@lists.infradead.org>,
	linux-omap@vger.kernel.org, David Woodhouse <dwmw2@infradead.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [RFC PATCH 1/5] spi: introduce flag for memory mapped read
Date: Thu, 6 Aug 2015 12:00:18 +0100	[thread overview]
Message-ID: <20150806110018.GS20873@sirena.org.uk> (raw)
In-Reply-To: <20150806102225.GI7576@n2100.arm.linux.org.uk>

[-- Attachment #1: Type: text/plain, Size: 1519 bytes --]

On Thu, Aug 06, 2015 at 11:22:25AM +0100, Russell King - ARM Linux wrote:

> If that's the case, then maybe you should consider whether using the SPI
> bus infrastructure is really the best way forward.  Would it make more
> sense instead to adopt a different software structure, something more
> high-level like:

>      +-------------------------------------------+
>      |          m25p80 high-level driver         |
>      +----------------------+--------------------+
>      |   SPI m25p80 driver  |                    |
>      +----------------------+                    |
>      |      SPI layer       |  Special driver    |
>      +----------------------+                    |
>      |    SPI bus driver    |                    |
>      +----------------------+--------------------+
>      |     SPI hardware     |  Special hardware  |
>      +----------------------+--------------------+

Yes, that's what's been talked about before - for some of these devices
they're sufficiently flash specialist that we just don't bother exposing
a SPI interface at all though AIUI they could be persuaded to do it.  It
isn't entirely clear that we want exactly that split, if the devices are
reasonable SPI controllers we will want to handle the case where they
have flash and non-flash devices on the same bus.  For that there is
going to be some generalisable work possible for managing switching
between memory mapped and SPI modes where those are mutually exclusive,
especially if the switch between them isn't free.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

  reply	other threads:[~2015-08-06 11:00 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-28  8:41 [RFC PATCH 0/5] Add memory mapped read support for TI QSPI Vignesh R
2015-07-28  8:41 ` [RFC PATCH 1/5] spi: introduce flag for memory mapped read Vignesh R
2015-07-31 18:17   ` Mark Brown
2015-08-03  4:57     ` Vignesh R
2015-08-04 15:51       ` Mark Brown
2015-08-04 17:59         ` R, Vignesh
2015-08-05  5:21           ` Michal Suchanek
2015-08-05  5:35             ` Vignesh R
2015-08-05  5:57               ` Michal Suchanek
2015-08-05 11:50           ` Mark Brown
2015-08-05 12:40             ` Michal Suchanek
2015-08-05 12:44               ` Mark Brown
2015-08-05 12:56                 ` Michal Suchanek
2015-08-06  9:02                   ` Mark Brown
2015-08-06 10:01                     ` Michal Suchanek
2015-08-06 10:22                       ` Russell King - ARM Linux
2015-08-06 11:00                         ` Mark Brown [this message]
2015-08-06 11:02                         ` Michal Suchanek
2015-08-06 12:25                         ` Vignesh R
2015-08-06 13:51                           ` Russell King - ARM Linux
2015-08-06 16:14                             ` Geert Uytterhoeven
2015-08-06 18:20                               ` Michal Suchanek
2015-08-06 21:33                               ` Russell King - ARM Linux
2015-08-07  7:38                                 ` Michal Suchanek
2015-08-07  8:35                                   ` Vignesh R
2015-08-07  8:25                                 ` Martin Sperl
2015-08-07 10:16                                   ` Michal Suchanek
2015-08-12  9:27                                     ` Vignesh R
2015-08-06 16:46                             ` Mark Brown
2015-08-06 18:20                           ` Mark Brown
2015-08-06 11:23                       ` Mark Brown
2015-08-06 11:42                         ` Michal Suchanek
2015-08-06 16:03                           ` Mark Brown
2015-07-28  8:41 ` [RFC PATCH 2/5] spi: spi-ti-qspi: Add memory mapped read support Vignesh R
2015-07-28  8:41 ` [RFC PATCH 3/5] mtd: devices: m25p80: set flag to request memory mapped read Vignesh R
2015-07-28  8:41 ` [RFC PATCH 4/5] ARM: dts: DRA7: Add memory map region entries for qspi Vignesh R
2015-07-31 13:48   ` Sekhar Nori
2015-08-03  5:09     ` Vignesh R
2015-07-31 18:19   ` Mark Brown
2015-08-03  5:02     ` Vignesh R
2015-08-04 15:52       ` Mark Brown
2015-07-31 21:28   ` Brian Norris
2015-08-03  5:06     ` Vignesh R
2015-07-28  8:41 ` [RFC PATCH 5/5] ARM: dts: AM4372: " Vignesh R

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=20150806110018.GS20873@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=b32955@freescale.com \
    --cc=computersforpeace@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dwmw2@infradead.org \
    --cc=hramrach@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=tony@atomide.com \
    --cc=vigneshr@ti.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®