From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lelnx194.ext.ti.com ([198.47.27.80]:49277 "EHLO lelnx194.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751498AbdF0JXl (ORCPT ); Tue, 27 Jun 2017 05:23:41 -0400 Subject: Re: [PATCH 1/8] arm: omap4: enable CEC pin for Pandaboard A4 and ES To: Tony Lindgren , Hans Verkuil CC: Tomi Valkeinen , , , Hans Verkuil , "linux-omap@vger.kernel.org" References: <20170414102512.48834-1-hverkuil@xs4all.nl> <20170414102512.48834-2-hverkuil@xs4all.nl> <4355dab4-9c70-77f7-f89b-9a1cf24976cf@ti.com> <20170626110711.GW3730@atomide.com> <701dbbfa-000a-2b93-405b-246aa90b6dd6@xs4all.nl> <20170627091421.GZ3730@atomide.com> From: Jyri Sarha Message-ID: <0b04a7bc-a969-7cdf-9f98-45d70337cf40@ti.com> Date: Tue, 27 Jun 2017 12:23:34 +0300 MIME-Version: 1.0 In-Reply-To: <20170627091421.GZ3730@atomide.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-GB Content-Transfer-Encoding: 8bit Sender: linux-media-owner@vger.kernel.org List-ID: On 06/27/17 12:14, Tony Lindgren wrote: > * Hans Verkuil [170627 01:39]: >> On 26/06/17 13:07, Tony Lindgren wrote: >>> Tomi, >>> >>> * Tomi Valkeinen [170428 04:15]: >>>> On 14/04/17 13:25, Hans Verkuil wrote: >>>>> From: Hans Verkuil >>>>> >>>>> 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? :) > As long as you do not expect a regular ALSA-volume to work.. But I don't see why some CEC application would not work. However, I guess one could implement this as feature to ALSA too but AFAIK no such thing exists at the moment. Best regards, Jyri >> 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 > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jyri Sarha Subject: Re: [PATCH 1/8] arm: omap4: enable CEC pin for Pandaboard A4 and ES Date: Tue, 27 Jun 2017 12:23:34 +0300 Message-ID: <0b04a7bc-a969-7cdf-9f98-45d70337cf40@ti.com> References: <20170414102512.48834-1-hverkuil@xs4all.nl> <20170414102512.48834-2-hverkuil@xs4all.nl> <4355dab4-9c70-77f7-f89b-9a1cf24976cf@ti.com> <20170626110711.GW3730@atomide.com> <701dbbfa-000a-2b93-405b-246aa90b6dd6@xs4all.nl> <20170627091421.GZ3730@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <20170627091421.GZ3730@atomide.com> Content-Language: en-GB Sender: linux-media-owner@vger.kernel.org To: Tony Lindgren , Hans Verkuil Cc: Tomi Valkeinen , linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org, Hans Verkuil , "linux-omap@vger.kernel.org" List-Id: linux-omap@vger.kernel.org On 06/27/17 12:14, Tony Lindgren wrote: > * Hans Verkuil [170627 01:39]: >> On 26/06/17 13:07, Tony Lindgren wrote: >>> Tomi, >>> >>> * Tomi Valkeinen [170428 04:15]: >>>> On 14/04/17 13:25, Hans Verkuil wrote: >>>>> From: Hans Verkuil >>>>> >>>>> 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? :) > As long as you do not expect a regular ALSA-volume to work.. But I don't see why some CEC application would not work. However, I guess one could implement this as feature to ALSA too but AFAIK no such thing exists at the moment. Best regards, Jyri >> 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 >