[PATCH 1/8] arm: omap4: enable CEC pin for Pandaboard A4 and ES

[Jyri Sarha]

On 06/27/17 12:27, Hans Verkuil wrote:
> On 27/06/17 11:14, Tony Lindgren wrote:
>> * Hans Verkuil <hverkuil at xs4all.nl> [170627 01:39]:
>>> On 26/06/17 13:07, Tony Lindgren wrote:
>>>> Tomi,
>>>>
>>>> * Tomi Valkeinen <tomi.valkeinen at ti.com> [170428 04:15]:
>>>>> On 14/04/17 13:25, Hans Verkuil wrote:
>>>>>> From: Hans Verkuil <hans.verkuil at cisco.com>
>>>>>>
>>>>>> The CEC pin was always pulled up, making it impossible to use it.
>>>> ...
>>>>
>>>>> Tony, can you queue this? It's safe to apply separately from the rest of
>>>>> the HDMI CEC work.
>>>>
>>>> So the dts changes are merged now but what's the status of the CEC driver
>>>> changes? Were there some issues as I don't see them in next?
>>>
>>> Tomi advised me to wait until a 'hotplug-interrupt-handling series' for the
>>> omap driver is merged to prevent conflicts. Last I heard (about 3 weeks ago)
>>> this was still pending review.
>>
>> OK thanks for the update.
>>
>> Adding Jyri to Cc, hopefully the CEC support allows also setting the
>> HDMI audio volume level on devices implementing it? Or am I too
>> optimistic? :)
> 
> I'm not quite sure what you mean. Do you want CEC to change the volume on the
> TV, or use the TV's remote to change the volume of the HDMI audio output of the
> omap4?
> 

There is no real volume on HDMI audio output as it is a digital
interface, but it should be possible to provide some volume control
using TV's volume trough CEC.

> Anyway, either is supported, but it requires a userspace implementation.
> 

A module to pulseaudio or some extra features to alsa-lib should be
generic enough (who knows, maybe there is already something).

Just an idea. If someone really needs this, the pieces to put it
together should be there.

Best regards,
Jyri

> Although TV remote control messages will be mapped to an input device, and if
> those are hooked up to the alsa audio volume, then this already works.
>> Regards,
> 
> 	Hans
> 
>>
>>> Tomi, any updates on this? It would be nice to get this in for 4.14.
>>
>> Yeah seems like we have real mainline kernel user needs for this one.
>>
>> Regards,
>>
>> Tony
>>
>