alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org, Liam Girdwood <lrg@ti.com>
Subject: Re: [RESEND] ASoC: omap-abe-twl6040: Add device tree support
Date: Thu, 26 Jul 2012 10:52:29 +0300	[thread overview]
Message-ID: <5010F73D.6020904@ti.com> (raw)
In-Reply-To: <20120725214837.GQ3099@opensource.wolfsonmicro.com>

On 07/26/2012 12:48 AM, Mark Brown wrote:
> Please just send the build fix with a followup adding the DT bindings
> and then we can get the DT bindings in for 3.7.

We do not have build breakage as of now since the DT bindings are not in.
The build breakage happened on my local tree when I have rebased it on
mainline yesterday. Stephen can not update the DT part of the omap-abe-twl6040
when he did the snd_soc_dai_link cleanup, obviously.
The patch has been updated to compile cleanly on top of current mainline. I
could have prefixed it as v4 but the change compared to the previous was small.

> It's unfortunate that
> Liam's been so busy and things like this got dropped on the floor but
> I'm not enthusiastic adding something like this after the pull request
> went to Linus.

Yes, I understand your hesitation. On the other hand the patches to enable
audio via DT for the dts files has been merged and present in mainline already
(omap4-sdp/panda/pandaES).

> Though obviously it's not even been compiled in mainline :/

Sure it has not been compiled in mainline. It was/is compiled on top of
mainline during my daily work.

> In general you need to really start paying attention to this stuff
> and start chasing around -rc5 or so.

I know, I should have...

-- 
Péter

  reply	other threads:[~2012-07-26  7:51 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-25 13:09 [RESEND] ASoC: omap-abe-twl6040: Add device tree support Peter Ujfalusi
2012-07-25 13:15 ` Mark Brown
2012-07-25 13:39   ` Peter Ujfalusi
2012-07-25 21:48     ` Mark Brown
2012-07-26  7:52       ` Peter Ujfalusi [this message]
2012-07-26  9:52         ` Mark Brown
2012-07-26 12:00 ` Mark Brown

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=5010F73D.6020904@ti.com \
    --to=peter.ujfalusi@ti.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=lrg@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;
as well as URLs for NNTP newsgroup(s).