All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Timur Tabi <timur@freescale.com>
Cc: alsa-devel@alsa-project.org,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	kumar.gala@freescale.com,
	Grant Likely <grant.likely@secretlab.ca>,
	linuxppc-dev@ozlabs.org, Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: [PATCH 1/2] powerpc: add platform registration	for ALSA SoC drivers
Date: Tue, 27 Apr 2010 21:59:24 +0100	[thread overview]
Message-ID: <20100427205924.GC15083@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <4BD74D0C.40303@freescale.com>

On Tue, Apr 27, 2010 at 03:46:04PM -0500, Timur Tabi wrote:

[Reflowed into 80 columns; please fix your mail client.]

> > (I've omitted the DMA nodes and some irrelevant details)  This is
> > enough information for a simplistic driver registration that probably
> > makes a lot of assumptions.  Such as the ssi represents a single
> > logical sound device.  It won't handle complex representations, but in

> Why would I ever represent the SSI as anything but a single logical
> sound device?  Let ALSA handle synchronizing multiple streams together
> if it wants to.

...

> > 		dai-links = <&ssi1 0 &codec 0
> >                            &ssi1 1 &codec 1>;
> > 		[...]

> I don't know when I would ever do this.  The two SSI devices are
> completely independent.  Why would I bind them together into one
> "device"?

It's entirely possible that if the board designer intended the verious
SSIs to be used in concert they've done something like cross wire the
clocks which creates a board-specific interrelationship that needs to be
dealt with.

WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Timur Tabi <timur@freescale.com>
Cc: alsa-devel@alsa-project.org, kumar.gala@freescale.com,
	linuxppc-dev@ozlabs.org, Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: [alsa-devel] [PATCH 1/2] powerpc: add platform registration for ALSA SoC drivers
Date: Tue, 27 Apr 2010 21:59:24 +0100	[thread overview]
Message-ID: <20100427205924.GC15083@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <4BD74D0C.40303@freescale.com>

On Tue, Apr 27, 2010 at 03:46:04PM -0500, Timur Tabi wrote:

[Reflowed into 80 columns; please fix your mail client.]

> > (I've omitted the DMA nodes and some irrelevant details)  This is
> > enough information for a simplistic driver registration that probably
> > makes a lot of assumptions.  Such as the ssi represents a single
> > logical sound device.  It won't handle complex representations, but in

> Why would I ever represent the SSI as anything but a single logical
> sound device?  Let ALSA handle synchronizing multiple streams together
> if it wants to.

...

> > 		dai-links = <&ssi1 0 &codec 0
> >                            &ssi1 1 &codec 1>;
> > 		[...]

> I don't know when I would ever do this.  The two SSI devices are
> completely independent.  Why would I bind them together into one
> "device"?

It's entirely possible that if the board designer intended the verious
SSIs to be used in concert they've done something like cross wire the
clocks which creates a board-specific interrelationship that needs to be
dealt with.

  reply	other threads:[~2010-04-27 20:59 UTC|newest]

Thread overview: 108+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-26 20:49 [PATCH 1/2] powerpc: add platform registration for ALSA SoC drivers Timur Tabi
2010-04-27  6:36 ` Benjamin Herrenschmidt
2010-04-27  6:36   ` Benjamin Herrenschmidt
2010-04-27  8:07   ` Liam Girdwood
2010-04-27  8:07     ` [alsa-devel] " Liam Girdwood
2010-04-27 14:52     ` Timur Tabi
2010-04-27 14:52       ` [alsa-devel] " Timur Tabi
2010-04-27 15:20       ` Liam Girdwood
2010-04-27 15:20         ` [alsa-devel] " Liam Girdwood
2010-04-27 15:28         ` Timur Tabi
2010-04-27 15:28           ` [alsa-devel] " Timur Tabi
2010-04-27 15:56           ` Timur Tabi
2010-04-27 15:56             ` [alsa-devel] " Timur Tabi
2010-04-27 16:41           ` Liam Girdwood
2010-04-27 16:41             ` [alsa-devel] " Liam Girdwood
2010-04-27 18:32             ` Timur Tabi
2010-04-27 18:32               ` [alsa-devel] " Timur Tabi
2010-04-27 19:15               ` Grant Likely
2010-04-27 19:15                 ` [alsa-devel] " Grant Likely
2010-04-27 20:04                 ` Timur Tabi
2010-04-27 20:04                   ` [alsa-devel] " Timur Tabi
2010-04-27 20:38                   ` Mark Brown
2010-04-27 20:38                     ` [alsa-devel] " Mark Brown
2010-04-28  4:19                   ` Benjamin Herrenschmidt
2010-04-28  4:19                     ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-28  4:18                 ` Benjamin Herrenschmidt
2010-04-28  4:18                   ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-30 21:46         ` Timur Tabi
2010-04-30 21:46           ` [alsa-devel] " Timur Tabi
2010-04-30 22:04           ` Timur Tabi
2010-04-30 22:04             ` [alsa-devel] " Timur Tabi
2010-04-27 20:24     ` Grant Likely
2010-04-27 20:24       ` [alsa-devel] " Grant Likely
2010-04-27 20:46       ` Timur Tabi
2010-04-27 20:46         ` [alsa-devel] " Timur Tabi
2010-04-27 20:59         ` Mark Brown [this message]
2010-04-27 20:59           ` Mark Brown
2010-04-27 21:03           ` Timur Tabi
2010-04-27 21:03             ` [alsa-devel] " Timur Tabi
2010-04-27 21:11             ` Mark Brown
2010-04-27 21:11               ` [alsa-devel] " Mark Brown
2010-04-28  4:25           ` Benjamin Herrenschmidt
2010-04-28  4:25             ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-28 13:00             ` Mark Brown
2010-04-28 13:00               ` [alsa-devel] " Mark Brown
2010-04-29  0:42               ` Benjamin Herrenschmidt
2010-04-29  0:42                 ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-28  5:37         ` Grant Likely
2010-04-28  5:37           ` [alsa-devel] " Grant Likely
2010-04-28 13:35           ` Timur Tabi
2010-04-28 13:35             ` [alsa-devel] " Timur Tabi
2010-04-28 13:57             ` Grant Likely
2010-04-28 13:57               ` [alsa-devel] " Grant Likely
2010-04-28 16:20               ` Timur Tabi
2010-04-28 16:20                 ` [alsa-devel] " Timur Tabi
2010-04-28 16:47                 ` Grant Likely
2010-04-28 16:47                   ` [alsa-devel] " Grant Likely
2010-04-28 17:27                   ` Timur Tabi
2010-04-28 17:27                     ` [alsa-devel] " Timur Tabi
2010-04-27 22:29       ` Mark Brown
2010-04-27 22:29         ` [alsa-devel] " Mark Brown
2010-04-28  2:31         ` Grant Likely
2010-04-28  2:31           ` [alsa-devel] " Grant Likely
2010-04-28  9:16           ` Mark Brown
2010-04-28  9:16             ` [alsa-devel] " Mark Brown
2010-04-28  4:10         ` Benjamin Herrenschmidt
2010-04-28  4:10           ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-28 12:07           ` Mark Brown
2010-04-28 12:07             ` [alsa-devel] " Mark Brown
2010-04-29  0:36             ` Benjamin Herrenschmidt
2010-04-29  0:36               ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-29  3:43               ` Grant Likely
2010-04-29  3:43                 ` [alsa-devel] " Grant Likely
2010-04-28 13:19         ` Timur Tabi
2010-04-28 13:19           ` [alsa-devel] " Timur Tabi
2010-04-28 13:39           ` Mark Brown
2010-04-28 13:39             ` [alsa-devel] " Mark Brown
2010-04-27  9:54   ` Mark Brown
2010-04-27  9:54     ` Mark Brown
2010-04-27 10:09     ` Benjamin Herrenschmidt
2010-04-27 10:09       ` Benjamin Herrenschmidt
2010-04-27 10:41       ` Mark Brown
2010-04-27 10:41         ` Mark Brown
2010-04-27 20:27       ` Grant Likely
2010-04-27 20:27         ` Grant Likely
2010-04-27 20:50         ` Mark Brown
2010-04-27 20:50           ` Mark Brown
2010-04-27 20:53           ` Timur Tabi
2010-04-27 20:53             ` Timur Tabi
2010-04-28 12:49         ` Liam Girdwood
2010-04-28 12:49           ` [alsa-devel] " Liam Girdwood
2010-04-28 20:35       ` Timur Tabi
2010-04-28 20:35         ` [alsa-devel] " Timur Tabi
2010-04-28 21:58         ` Grant Likely
2010-04-28 21:58           ` [alsa-devel] " Grant Likely
2010-04-28 22:13           ` Timur Tabi
2010-04-28 22:13             ` [alsa-devel] " Timur Tabi
     [not found]             ` <r2oed82fe3e1004281513k23b54b56v7904a4a34750c90b-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-04-28 22:23               ` Grant Likely
2010-04-28 22:23                 ` Grant Likely
2010-04-29  0:52               ` Benjamin Herrenschmidt
2010-04-29  0:52                 ` Benjamin Herrenschmidt
2010-04-29  3:44                 ` Grant Likely
2010-04-29  3:44                   ` [alsa-devel] " Grant Likely
2010-04-29  0:50         ` Benjamin Herrenschmidt
2010-04-29  0:50           ` [alsa-devel] " Benjamin Herrenschmidt
2010-04-27 19:21 ` Grant Likely
2010-04-27 19:21   ` Grant Likely
2010-04-27 20:05   ` Timur Tabi

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=20100427205924.GC15083@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=benh@kernel.crashing.org \
    --cc=grant.likely@secretlab.ca \
    --cc=kumar.gala@freescale.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=lrg@slimlogic.co.uk \
    --cc=timur@freescale.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.