From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/edid: Reduce horizontal timings for pixel replicated modes Date: Mon, 1 Sep 2014 16:12:27 +0300 Message-ID: <20140901131227.GE4193@intel.com> References: <1408483281-28053-1-git-send-email-clinton.a.taylor@intel.com> <20140826081009.GF15520@phenom.ffwll.local> <20140826081107.GG15520@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 45ED06E268 for ; Mon, 1 Sep 2014 06:13:29 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140826081107.GG15520@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Aug 26, 2014 at 10:11:07AM +0200, Daniel Vetter wrote: > On Tue, Aug 26, 2014 at 10:10:09AM +0200, Daniel Vetter wrote: > > On Tue, Aug 19, 2014 at 02:21:21PM -0700, clinton.a.taylor@intel.com wr= ote: > > > 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. > > > = > > > V3: 576i@200 htotal fixed. Check min and max pixel clocks. > > > = > > > Signed-off-by: Clint Taylor > > > --- > > > drivers/gpu/drm/drm_edid.c | 96 ++++++++++++++++++---------= ---------- > > > drivers/gpu/drm/i915/intel_hdmi.c | 15 ++++-- > > = > > Still plea to split out the i915 change. > = > Also it's good practice to add all the people who commented on earlier > versions of this patch to the in-patch Cc: list. So Ville should be here > for this one. Also I've totally lost track of these now. Please use --in-reply-to when sending revised patches. -- = Ville Syrj=E4l=E4 Intel OTC