On 21/11/14 18:38, Mark Brown wrote:

On Fri, Nov 21, 2014 at 02:10:11PM +0200, Jyri Sarha wrote:

quoted

OMAP HDMI audio is fundamentally different to the case on Armada or on BBB.
In omap the whole HDMI IP is integrated to the SoC and there really is no
codec in the ASoC sense. The the cpu-dai transmits the audio directly to
hdmi wire and there is no i2s bus involved. So this case should not be mixed
with the patches Jean-Francois working on. The code is also orthogonal in
that sense that the latest omap-hdmi-audio uses the generic dymmy codec.

The discussion seemed to be about what to do with unplugged connections
which isn't what you're talking about there and does seem like an area
where we should at least be aiming for common behaviour even if not a
common implementation.

There's also all the stuff about parsing EDIDs for capabilities which
would seem to be related to that but seems to have gone off into the
weeds.

I agree all these should be studied and worked further. However, I would
still like to merge this series in the next merge window:

a) The series does fix the OMAP HDMI audio, so that users can actually
use it.

b) The series does make the driver model much better, so that I can
actually be bothered to keep it enabled (earlier, even when it more or
less worked, it was too much hassle when doing development using modules).

 Tomi