All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olof Johansson <olof@lixom.net>
To: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Nicolas Pitre <nico@fluxnic.net>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Arnd Bergmann <arnd@arndb.de>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: How to support SDIO wifi/bt in DT
Date: Sun, 19 Jan 2014 15:30:08 -0800	[thread overview]
Message-ID: <20140119233008.GA4867@quad.lixom.net> (raw)
In-Reply-To: <20140119230924.GB8154@piout.net>

On Mon, Jan 20, 2014 at 12:09:25AM +0100, Alexandre Belloni wrote:
> On Sun, Jan 19, 2014 at 11:29:55AM -0800, Olof Johansson wrote :
> > The "multi-slot" concept I was talking about was one host controller
> > for several physical slots. Seems like at91 implements _and_ uses this
> > but nobody else. However, it seems that they allocate one mmc host per
> > slot, so the same model still works.
> > 
> 
> I think we need to be extra careful about the DT bindings then because
> adding a property in a slot is requiring handling that in the at91
> driver whereas adding a common property for the various mmc hosts can be
> done in the core code.
> 
> From what I remember, it would be really good to put the DT parsing for
> the slots in the core code but I only had a really quick look at that.

The patches I have here do it all in the core per host with common
bindings. I hope to post them tonight. They should fit the at91 model
just fine as well, since they do each slot as a mmc_host, not as a separate
slot with a shared host controller.

(Some drivers don't seem to call mmc_of_parse(), which they will need to do to
take advantage of the common implementation. But that can be addressed driver
by driver as someone has a chance to test it).


-Olof

WARNING: multiple messages have this Message-ID (diff)
From: olof@lixom.net (Olof Johansson)
To: linux-arm-kernel@lists.infradead.org
Subject: How to support SDIO wifi/bt in DT
Date: Sun, 19 Jan 2014 15:30:08 -0800	[thread overview]
Message-ID: <20140119233008.GA4867@quad.lixom.net> (raw)
In-Reply-To: <20140119230924.GB8154@piout.net>

On Mon, Jan 20, 2014 at 12:09:25AM +0100, Alexandre Belloni wrote:
> On Sun, Jan 19, 2014 at 11:29:55AM -0800, Olof Johansson wrote :
> > The "multi-slot" concept I was talking about was one host controller
> > for several physical slots. Seems like at91 implements _and_ uses this
> > but nobody else. However, it seems that they allocate one mmc host per
> > slot, so the same model still works.
> > 
> 
> I think we need to be extra careful about the DT bindings then because
> adding a property in a slot is requiring handling that in the at91
> driver whereas adding a common property for the various mmc hosts can be
> done in the core code.
> 
> From what I remember, it would be really good to put the DT parsing for
> the slots in the core code but I only had a really quick look at that.

The patches I have here do it all in the core per host with common
bindings. I hope to post them tonight. They should fit the at91 model
just fine as well, since they do each slot as a mmc_host, not as a separate
slot with a shared host controller.

(Some drivers don't seem to call mmc_of_parse(), which they will need to do to
take advantage of the common implementation. But that can be addressed driver
by driver as someone has a chance to test it).


-Olof

  reply	other threads:[~2014-01-19 23:32 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-16 13:36 How to support SDIO wifi/bt in DT Russell King - ARM Linux
2014-01-16 13:51 ` Chen-Yu Tsai
2014-01-16 14:02 ` Arnd Bergmann
2014-01-16 17:15   ` Olof Johansson
2014-01-16 17:15     ` Olof Johansson
2014-01-16 19:58     ` Russell King - ARM Linux
2014-01-16 19:58       ` Russell King - ARM Linux
2014-01-16 20:00       ` Olof Johansson
2014-01-16 20:00         ` Olof Johansson
2014-01-16 20:03         ` Russell King - ARM Linux
2014-01-16 20:03           ` Russell King - ARM Linux
2014-01-17  9:39         ` Alexandre Belloni
2014-01-17  9:39           ` Alexandre Belloni
2014-01-17 10:06           ` Chen-Yu Tsai
2014-01-17 10:06             ` Chen-Yu Tsai
2014-01-17 10:14             ` Alexandre Belloni
2014-01-17 10:14               ` Alexandre Belloni
2014-01-17 10:44           ` Andrew Lunn
2014-01-17 10:44             ` Andrew Lunn
2014-02-05 17:11             ` Mark Brown
2014-02-05 17:11               ` Mark Brown
2014-01-16 21:46     ` Arnd Bergmann
2014-01-16 21:46       ` Arnd Bergmann
2014-01-16 21:52       ` Olof Johansson
2014-01-16 21:52         ` Olof Johansson
2014-01-16 22:14       ` Marcel Holtmann
2014-01-16 22:14         ` Marcel Holtmann
2014-01-17  3:08     ` Nicolas Pitre
2014-01-17  3:08       ` Nicolas Pitre
2014-01-17 14:47       ` Arnd Bergmann
2014-01-17 14:47         ` Arnd Bergmann
     [not found]       ` <alpine.LFD.2.10.1401162204560.28907-fMhRO7WWcppj+hNMo8g0rg@public.gmane.org>
2014-01-17 15:14         ` Rob Herring
2014-01-17 15:14           ` Rob Herring
2014-01-17 16:58           ` Nicolas Pitre
2014-01-17 16:58             ` Nicolas Pitre
2014-01-19 19:29             ` Olof Johansson
2014-01-19 19:29               ` Olof Johansson
2014-01-19 20:28               ` Arnd Bergmann
2014-01-19 20:28                 ` Arnd Bergmann
2014-01-19 23:26                 ` Olof Johansson
2014-01-19 23:26                   ` Olof Johansson
2014-01-19 23:09               ` Alexandre Belloni
2014-01-19 23:09                 ` Alexandre Belloni
2014-01-19 23:30                 ` Olof Johansson [this message]
2014-01-19 23:30                   ` Olof Johansson
2014-01-20  3:57                   ` Olof Johansson
2014-01-20  3:57                     ` Olof Johansson
2014-01-17  9:02     ` Alexandre Belloni
2014-01-17  9:02       ` Alexandre Belloni
2014-01-17  9:38       ` Nicolas Ferre
2014-01-17  9:38         ` Nicolas Ferre
2014-01-16 14:08 ` Alexandre Belloni
2014-01-16 14:34   ` Russell King - ARM Linux

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=20140119233008.GA4867@quad.lixom.net \
    --to=olof@lixom.net \
    --cc=alexandre.belloni@free-electrons.com \
    --cc=arnd@arndb.de \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nico@fluxnic.net \
    /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.