From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: -next trees Date: Thu, 27 Sep 2012 16:42:05 +0200 Message-ID: <20120927144205.GF2098@bremse> References: <505E916F.7000803@gmail.com> <1348497252.15901.46.camel@atropine> <5061FC41.7080009@gmail.com> <1348678836.4244.38.camel@atropine> <50636DEB.8000704@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-bk0-f49.google.com (mail-bk0-f49.google.com [209.85.214.49]) by gabe.freedesktop.org (Postfix) with ESMTP id EAFC89E793 for ; Thu, 27 Sep 2012 07:42:11 -0700 (PDT) Received: by bkwj4 with SMTP id j4so1806003bkw.36 for ; Thu, 27 Sep 2012 07:42:10 -0700 (PDT) Content-Disposition: inline In-Reply-To: <50636DEB.8000704@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Ian Pilcher Cc: "Koskipaa, Antti" , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Wed, Sep 26, 2012 at 04:04:43PM -0500, Ian Pilcher wrote: > On 09/26/2012 12:00 PM, Adam Jackson wrote: > >On Tue, 2012-09-25 at 13:47 -0500, Ian Pilcher wrote: > >> > >>1. The display (LG L246WP) is confused by *any* InfoFrames. > > > >If this is still the case after applying: > > > >commit adf00b26d18e1b3570451296e03bcb20e4798cdd > >Author: Paulo Zanoni > >Date: Tue Sep 25 13:23:34 2012 -0300 > > > > drm/i915: make sure we write all the DIP data bytes > > > >Then I suspect I'm compelled to agree that we need a quirk to forcibly > >disable InfoFrames entirely. I don't like to be difficult about this, > >but the HDMI spec is quite clear that sinks _must_ accept InfoFrames > >defined in either the CEA or HDMI specs, so if we're seeing that class > >of failure I tend to strongly suspect our drivers first. > > I just tested with that patch applied, and the problem still exists, so > consider yourself compelled. > > Unless you object, I will go ahead and create a patch series that does > only: > > - user-defined EDID quirks > - DISABLE_INFOFRAMES quirk > - DISABLE_INFOFRAMES quirk for LG L246WP > > (I will specifically *not* address the audio issue in this series.) > > >>3. drm_detect_monitor_audio is returning true for the LG L246WP, which > >> definitely doesn't have any audio capabilities. This may be a bug > >> in the display's EDID, or it may be a parsing error. > > > >The display is definitely a filthy liar then: > > > > Audio data block > > Linear PCM, max channels 1 > > Supported sample rates (kHz): 48 44.1 32 > > Supported sample sizes (bits): 24 20 16 > > > >Hooray for hardware. Not sure what the logic should be for whether to > >send HDMI audio or not, I'll re-read the appropriate scrolls. > > Well, if hardware didn't suck we wouldn't need quirks. > > If you'd like, I can generate a second patch series that adds some or > all of: > > - NO_AUDIO quirk > - drm_monitor_has_hdmi_audio() function I've volunteered Antti from our team to implement this (and to patch up drm/i915), so this should happen already and with the above no infoframes quirks we shouldn't need any additional quirks. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch