From mboxrd@z Thu Jan 1 00:00:00 1970 From: Clint Taylor Subject: Re: [PATCH] drm/edid: Reduce horizontal timings for pixel replicated modes Date: Mon, 25 Aug 2014 08:57:50 -0700 Message-ID: <53FB5CFE.7070809@intel.com> References: <1408395734-21955-1-git-send-email-clinton.a.taylor@intel.com> <20140825130258.GF15520@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id CD33B89FC9 for ; Mon, 25 Aug 2014 08:59:09 -0700 (PDT) In-Reply-To: <20140825130258.GF15520@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On 08/25/2014 06:02 AM, Daniel Vetter wrote: > On Mon, Aug 18, 2014 at 02:02:14PM -0700, clinton.a.taylor@intel.com wrote: >> From: Clint Taylor >> >> Pixel replicated modes should be 720 horizontal pixel and pixel >> replicated by the HW across the HDMI cable at 2X pixel clock. Current >> horizontal resolution of 1440 does not allow pixel duplication to >> occur and scaling artifacts occur on the TV. HDMI certification >> 7-26 currently fails for all pixel replicated modes. This change fizes >> the HDMI certification issues with 480i/576i. >> >> V2: Removed interlace flag from VICs 44 and 45. Will be submitted in >> another patch. Various other formatting fixes. >> >> Signed-off-by: Clint Taylor >> --- >> drivers/gpu/drm/drm_edid.c | 96 ++++++++++++++++++------------------- >> drivers/gpu/drm/i915/intel_hdmi.c | 10 +++- > > I think the i915 side of this double-clock support should be split out > into a separate patch. Makes it easier for people to spot that there's > some driver work to enable this properly. > -Daniel > Two patches will work. The HDMI driver will just reject the SD interlaced modes without the i915 changes. Clint