Re: [PATCH v7.1 00/19] Rework OMAP4+ HDMI audio support

From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Mark Brown <broonie@kernel.org>, Jyri Sarha <jsarha@ti.com>
Cc: alsa-devel@alsa-project.org, linux-fbdev@vger.kernel.org,
	linux-omap@vger.kernel.org, peter.ujfalusi@ti.com,
	liam.r.girdwood@linux.intel.com
Subject: Re: [PATCH v7.1 00/19] Rework OMAP4+ HDMI audio support
Date: Thu, 20 Nov 2014 10:59:44 +0000	[thread overview]
Message-ID: <546DC9A0.8060700@ti.com> (raw)
In-Reply-To: <54646648.5030807@ti.com>

[-- Attachment #1: Type: text/plain, Size: 1731 bytes --]

Hi Mark,

On 13/11/14 10:05, Tomi Valkeinen wrote:
> Hi Mark,
> 
> On 13/11/14 00:23, Mark Brown wrote:
>> On Wed, Nov 12, 2014 at 04:40:51PM +0200, Jyri Sarha wrote:
>>
>>> It would make the most sense to get these in trough fbdev tree. So it
>>> would be nice to get acked-bys (if the patches are Ok) for ASoC side
>>> changes from appropriate maintainers.
>>
>> So, this is a very large series which gets reposted every so often to no
>> apparent interest from the video side, there's been no response at all
> 
> Sorry for the lack of communication. We've been discussing this series
> on irc. It's been mostly about how to manage the device/driver split
> between drivers/video/ and sound/ sides.
> 
>> that I can remember and even the earlier bits of the series before it
>> starts touching audio don't seem to be getting merged.  What's going on
>> here?
> 
> The series is all audio in terms of functionality. The first few patches
> could probably be merged independently, but I've wanted this whole OMAP
> HDMI audio rewrite to be in one series.
> 
> I'll start testing this latest series, and I hope we can merge it for
> the next merge window so that we'll finally get the OMAP HDMI audio working.
> 
> I don't have much knowledge of the asoc architecture, so I probably
> can't comment much on the sound/ side design. For me the most important
> things are that 1) it works 2) I can easily unload/load the modules
> (which was broken in some of the earlier versions).

The series looks good to me, and works for me.

Do you have any comments for the sound/ parts? If not, I can merge this
series via fbdev tree, and for that I'd like your ack on the sound/ patches.

 Tomi

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]