All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Liam Girdwood <lrg@slimlogic.co.uk>
Cc: vbarinov <vbarinov@embeddedalley.com>,
	alsa-devel <alsa-devel@alsa-project.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	mano@roarinelk.homelinux.net, "Candelaria Villareal,
	Jorge" <jorge.candelaria@ti.com>,
	Cliff Cai <cliff.cai@analog.com>,
	Haojian Zhuang <haojian.zhuang@gmail.com>,
	dg@emlix.com, Grant Likely <grant.likely@secretlab.ca>,
	Sedji Gaouaou <sedji.gaouaou@atmel.com>,
	"kyungmin.park" <kyungmin.park@samsung.com>,
	ben-linux <ben-linux@fluff.org>,
	Kuninori Morimoto <morimoto.kuninori@renesas.com>,
	Timur Tabi <timur@freescale.com>,
	anemo@mba.ocn.ne.jp
Subject: Re: ASoC - Support for multiple components
Date: Tue, 20 Apr 2010 02:14:19 +0900	[thread overview]
Message-ID: <20100419171418.GD29819@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <1271694314.3208.426.camel@odin>

On Mon, Apr 19, 2010 at 05:25:14PM +0100, Liam Girdwood wrote:
> On Mon, 2010-04-19 at 09:15 -0600, Grant Likely wrote:

> > - Related to that, how are things like i2c codecs handled?  By rights,
> > there will be an i2c device registered under the i2c bus device for
> > the control interface.  How will this relate to the ASoC model?

> I2C/SPI codecs can now register with the driver core as I2C/SPI drivers.
> e.g. the I2C/SPI device probe registers the ASoC device :-

This is very similar to the current code structurally, from the driver
point of view the main change here is that it's now become mandatory to
use device model style stuff.  This means that there's not much to be
scared about if your drivers are using that already, though
soc_of_simple never got converted.

> > - None of the drivers appear to include struct device_driver; which
> > tells me that they don't use the driver core binding model.  How do
> > drivers get bound to devices?  Why does the ASoC code create an driver
> > interface instead of using the core code (what does the core code not
> > provide)?

> ASoC doesn't fully use the driver model for components in this changeset
> (or at all) due to the complex relationships between the individual
> components for probing() and for control of PM operations (i.e. we cant
> have pops and clicks at suspend). 

> This is not to say ASoC wont use the full driver model for components in
> the future. This work is an important step on the way to achieving this
> aim. 

The ordering requirements here mean this is a *very* open question.
Linus just nacked a bunch of stuff for 2.6.34 which would have added
some ability to waiting for components to probe stuff that things like
ASoC would need to use the driver core fully.  At the minute subsystems
like ASoC which are built up of components that don't have much to do
with each other have to do something like ASoC is doing and waiting for
all the components to come online before bringing the subsystem level
devices up.

Again, this isn't much change from a driver point of view from the
situation in 2.6.34, it's more moving forward to making it mandatory
and taking advantage of the features that can be brought than anything
else.

  reply	other threads:[~2010-04-19 17:14 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-19 14:09 ASoC - Support for multiple components Liam Girdwood
2010-04-19 14:14 ` Timur Tabi
2010-04-19 15:15 ` Grant Likely
2010-04-19 16:25   ` Liam Girdwood
2010-04-19 17:14     ` Mark Brown [this message]
2010-04-19 17:23       ` Liam Girdwood
2010-04-19 17:03   ` Mark Brown
2010-04-20  7:17 ` Peter Ujfalusi
2010-04-20 10:46   ` Liam Girdwood
2010-04-21  5:53     ` Peter Ujfalusi
2010-04-21  7:18       ` Liam Girdwood
2010-04-21 17:41         ` Mark Brown
2010-04-26 10:49     ` Mark Brown
2010-04-26 11:17       ` Liam Girdwood
2010-04-26 14:24         ` Mark Brown
2010-04-26 12:05       ` Timur Tabi
2010-04-26 13:13         ` Mark Brown
2010-04-21 21:07 ` Timur Tabi
2010-04-21 22:13   ` Timur Tabi
2010-04-22 11:27     ` Liam Girdwood
2010-04-22 14:52       ` Timur Tabi
2010-04-22 11:03   ` Liam Girdwood

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=20100419171418.GD29819@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=anemo@mba.ocn.ne.jp \
    --cc=ben-linux@fluff.org \
    --cc=cliff.cai@analog.com \
    --cc=dg@emlix.com \
    --cc=grant.likely@secretlab.ca \
    --cc=haojian.zhuang@gmail.com \
    --cc=jorge.candelaria@ti.com \
    --cc=jy0922.shim@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=lrg@slimlogic.co.uk \
    --cc=mano@roarinelk.homelinux.net \
    --cc=morimoto.kuninori@renesas.com \
    --cc=s.hauer@pengutronix.de \
    --cc=sedji.gaouaou@atmel.com \
    --cc=timur@freescale.com \
    --cc=vbarinov@embeddedalley.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 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.