From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Fri, 01 Feb 2013 23:42:04 +0000 Subject: Re: [RFC v2 0/5] Common Display Framework Message-Id: <1699935.hiBjgCN6cp@avalon> List-Id: References: <1353620736-6517-1-git-send-email-laurent.pinchart@ideasonboard.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Rahul Sharma Cc: Rob Clark , Thomas Petazzoni , Linux Fbdev development list , Benjamin Gaignard , Tom Gall , Kyungmin Park , "dri-devel@lists.freedesktop.org" , Ragesh Radhakrishnan , Tomi Valkeinen , Philipp Zabel , Maxime Ripard , Vikas Sajjan , Sumit Semwal , Sebastien Guiriec , "linux-media@vger.kernel.org" Hi Rahul, On Wednesday 09 January 2013 13:53:30 Rahul Sharma wrote: > Hi Laurent, > > CDF will also be helpful in supporting Panels with integrated audio > (HDMI/DP) if we can add audio related control operations to > display_entity_control_ops. Video controls will be called by crtc in DRM/V4L > and audio controls from Alsa. I knew that would come up at some point :-) I agree with you that adding audio support would be a very nice improvement, and I'm totally open to that, but I will concentrate on video, at least to start with. The first reason is that I'm not familiar enough with ALSA, and the second that there's only 24h per day :-) Please feel free, of course, to submit a proposal for audio support. > Secondly, if I need to support get_modes operation in hdmi/dp panel, I need > to implement edid parser inside the panel driver. It will be meaningful to > add get_edid control operation for hdmi/dp. Even if EDID data is parsed in the panel driver, raw EDID will still need to be exported, so a get_edid control operation (or something similar) is definitely needed. There's no disagreement on this, I just haven't included that operation yet because my test hardware is purely panel-based. -- Regards, Laurent Pinchart