From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Deucher Subject: Re: [Intel-gfx] [PATCH] drm/edid: set ELD for firmware and debugfs override EDIDs Date: Fri, 27 Mar 2015 10:25:19 -0400 Message-ID: References: <1427359320-9451-1-git-send-email-jani.nikula@intel.com> <20150326090450.GQ1349@phenom.ffwll.local> <874mp6r6bk.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <874mp6r6bk.fsf@intel.com> Sender: stable-owner@vger.kernel.org To: Jani Nikula Cc: Daniel Vetter , David Airlie , emilsvennesson@gmail.com, jolan@gormsby.com, Intel Graphics Development , "for 3.8" , grenoble@gmail.com, Maling list - DRI developers List-Id: dri-devel@lists.freedesktop.org On Fri, Mar 27, 2015 at 8:08 AM, Jani Nikula wrote: > On Thu, 26 Mar 2015, Daniel Vetter wrote: >> On Thu, Mar 26, 2015 at 10:42:00AM +0200, Jani Nikula wrote: >>> If the user supplies EDID through firmware or debugfs override, the >>> driver callbacks are bypassed and the connector ELD does not get >>> updated, and audio fails. Set ELD for firmware and debugfs EDIDs too. >>> >>> There should be no harm in gratuitously doing this for non HDMI/DP >>> connectors, as it's still up to the driver to use the ELD, if any. >>> >>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82349 >>> Reference: https://bugs.freedesktop.org/show_bug.cgi?id=80691 >>> Reported-by: Emil >>> Reported-by: Rob Engle >>> Tested-by: Jolan Luff >>> Cc: stable@vger.kernel.org >>> Signed-off-by: Jani Nikula >> >> Since it's harmless I wonder whether we shouldn't just do this in >> drm_add_edid_modes unconditionally. But this looks like the right minimal >> patch for -fixes, so Reviewed-by: Daniel Vetter > > While I was hoping to gather review from outside of drm/i915 folks, I > picked this up and pushed to our new topic/drm-fixes branch of [1]. > > I intend to keep picking up (occasional, non-controversial) drm core > fixes aimed at the current development (-rc) kernels, to ensure they're > not dropped, and sending pull requests to Dave as needed. He'll have the > final call whether to pull or not, of course. This is similar to what > Daniel does with the topic/drm-misc branch for drm-next. > > Please let me know if you have any feedback on this. The patch seems fine to me. However, if we are always going to set the ELD for the override cases, why don't we also always set it for the non-override cases rather than making each driver do it. Alex > > Thanks, > Jani. > > > [1] http://cgit.freedesktop.org/drm-intel > > -- > Jani Nikula, Intel Open Source Technology Center