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:17:51 +0300 Message-ID: <20140901131750.GF4193@intel.com> References: <1408483281-28053-1-git-send-email-clinton.a.taylor@intel.com> <20140826081009.GF15520@phenom.ffwll.local> <20140826081107.GG15520@phenom.ffwll.local> <20140901131227.GE4193@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id 494C76E277 for ; Mon, 1 Sep 2014 06:17:55 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140901131227.GE4193@intel.com> 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 Mon, Sep 01, 2014 at 04:12:27PM +0300, Ville Syrj=E4l=E4 wrote: > 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 = 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. Curre= nt > > > > 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 fi= zes > > > > 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. Oh and please use --to --cc etc. also instead of sending the same patch separately to multiple lists. -- = Ville Syrj=E4l=E4 Intel OTC