* [Fwd: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6, 5]
@ 2013-05-14 23:44 Benjamin Herrenschmidt
2013-05-15 5:31 ` Takashi Iwai
0 siblings, 1 reply; 2+ messages in thread
From: Benjamin Herrenschmidt @ 2013-05-14 23:44 UTC (permalink / raw)
To: Takashi Iwai; +Cc: Michael Ellerman, alsa-devel
[-- Attachment #1: Type: text/plain, Size: 61 bytes --]
Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
[-- Attachment #2: Forwarded message — [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6,5 --]
[-- Type: message/rfc822, Size: 4040 bytes --]
From: Michael Ellerman <michael@ellerman.id.au>
To: <linuxppc-dev@ozlabs.org>
Cc: newchief@king.net.pl
Subject: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6,5
Date: Mon, 6 May 2013 11:01:05 +1000
Message-ID: <1367802065-32150-1-git-send-email-michael@ellerman.id.au>
Either one or a combination of commits 81e5d86
"Register i2c devices from device-tree" and 3a3dd01
"Improve detection of devices from device-tree" broke sound on
PowerBook6,5 machines.
Fix it by adding an entry to the new driver to match PowerBook6,5
machines.
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
---
sound/aoa/fabrics/layout.c | 8 ++++++++
sound/aoa/soundbus/i2sbus/core.c | 3 ++-
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/sound/aoa/fabrics/layout.c b/sound/aoa/fabrics/layout.c
index 552b97a..61ab640 100644
--- a/sound/aoa/fabrics/layout.c
+++ b/sound/aoa/fabrics/layout.c
@@ -113,6 +113,7 @@ MODULE_ALIAS("sound-layout-100");
MODULE_ALIAS("aoa-device-id-14");
MODULE_ALIAS("aoa-device-id-22");
MODULE_ALIAS("aoa-device-id-35");
+MODULE_ALIAS("aoa-device-id-44");
/* onyx with all but microphone connected */
static struct codec_connection onyx_connections_nomic[] = {
@@ -361,6 +362,13 @@ static struct layout layouts[] = {
.connections = tas_connections_nolineout,
},
},
+ /* PowerBook6,5 */
+ { .device_id = 44,
+ .codecs[0] = {
+ .name = "tas",
+ .connections = tas_connections_all,
+ },
+ },
/* PowerBook6,7 */
{ .layout_id = 80,
.codecs[0] = {
diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus/core.c
index 0106583..15e7613 100644
--- a/sound/aoa/soundbus/i2sbus/core.c
+++ b/sound/aoa/soundbus/i2sbus/core.c
@@ -200,7 +200,8 @@ static int i2sbus_add_dev(struct macio_dev *macio,
* We probably cannot handle all device-id machines,
* so restrict to those we do handle for now.
*/
- if (id && (*id == 22 || *id == 14 || *id == 35)) {
+ if (id && (*id == 22 || *id == 14 || *id == 35 ||
+ *id == 44)) {
snprintf(dev->sound.modalias, 32,
"aoa-device-id-%d", *id);
ok = 1;
--
1.7.10.4
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [Fwd: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6, 5]
2013-05-14 23:44 [Fwd: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6, 5] Benjamin Herrenschmidt
@ 2013-05-15 5:31 ` Takashi Iwai
0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2013-05-15 5:31 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: Michael Ellerman, alsa-devel
At Wed, 15 May 2013 09:44:47 +1000,
Benjamin Herrenschmidt wrote:
>
> [1 <text/plain (7bit)>]
> Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Thanks, I applied the patch now to sound git tree.
Takashi
> [2 Forwarded message — [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6,5 <message/rfc822 (7bit)>]
> To: <linuxppc-dev@ozlabs.org>
> Cc: newchief@king.net.pl
> Subject: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6,5
> From: Michael Ellerman <michael@ellerman.id.au>
> Delivered-To: linuxppc-dev@ozlabs.org
> Date: Mon, 6 May 2013 11:01:05 +1000
> Message-Id: <1367802065-32150-1-git-send-email-michael@ellerman.id.au>
> List-Unsubscribe: <https://lists.ozlabs.org/options/linuxppc-dev>, <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>
> List-Archive: <http://lists.ozlabs.org/pipermail/linuxppc-dev/>
> List-Post: <mailto:linuxppc-dev@lists.ozlabs.org>
> List-Help: <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>
> List-Subscribe: <https://lists.ozlabs.org/listinfo/linuxppc-dev>, <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>
> MIME-Version: 1.0
> Content-Transfer-Encoding: 7bit
>
> Either one or a combination of commits 81e5d86
> "Register i2c devices from device-tree" and 3a3dd01
> "Improve detection of devices from device-tree" broke sound on
> PowerBook6,5 machines.
>
> Fix it by adding an entry to the new driver to match PowerBook6,5
> machines.
>
> Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
> ---
> sound/aoa/fabrics/layout.c | 8 ++++++++
> sound/aoa/soundbus/i2sbus/core.c | 3 ++-
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/sound/aoa/fabrics/layout.c b/sound/aoa/fabrics/layout.c
> index 552b97a..61ab640 100644
> --- a/sound/aoa/fabrics/layout.c
> +++ b/sound/aoa/fabrics/layout.c
> @@ -113,6 +113,7 @@ MODULE_ALIAS("sound-layout-100");
> MODULE_ALIAS("aoa-device-id-14");
> MODULE_ALIAS("aoa-device-id-22");
> MODULE_ALIAS("aoa-device-id-35");
> +MODULE_ALIAS("aoa-device-id-44");
>
> /* onyx with all but microphone connected */
> static struct codec_connection onyx_connections_nomic[] = {
> @@ -361,6 +362,13 @@ static struct layout layouts[] = {
> .connections = tas_connections_nolineout,
> },
> },
> + /* PowerBook6,5 */
> + { .device_id = 44,
> + .codecs[0] = {
> + .name = "tas",
> + .connections = tas_connections_all,
> + },
> + },
> /* PowerBook6,7 */
> { .layout_id = 80,
> .codecs[0] = {
> diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus/core.c
> index 0106583..15e7613 100644
> --- a/sound/aoa/soundbus/i2sbus/core.c
> +++ b/sound/aoa/soundbus/i2sbus/core.c
> @@ -200,7 +200,8 @@ static int i2sbus_add_dev(struct macio_dev *macio,
> * We probably cannot handle all device-id machines,
> * so restrict to those we do handle for now.
> */
> - if (id && (*id == 22 || *id == 14 || *id == 35)) {
> + if (id && (*id == 22 || *id == 14 || *id == 35 ||
> + *id == 44)) {
> snprintf(dev->sound.modalias, 32,
> "aoa-device-id-%d", *id);
> ok = 1;
> --
> 1.7.10.4
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-05-15 5:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-14 23:44 [Fwd: [PATCH] ALSA: snd-aoa: Add a layout entry for PowerBook6, 5] Benjamin Herrenschmidt
2013-05-15 5:31 ` Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox