From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/2] drm/i915: handle sdvo input pixel multiplier correctly again Date: Tue, 3 Sep 2013 21:53:15 +0300 Message-ID: <20130903185315.GJ11428@intel.com> References: <1378233637-24052-1-git-send-email-daniel.vetter@ffwll.ch> 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 C76DFE6DF9 for ; Tue, 3 Sep 2013 11:53:18 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1378233637-24052-1-git-send-email-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Tue, Sep 03, 2013 at 08:40:36PM +0200, Daniel Vetter wrote: > The sdvo input timing needs to be the actual mode, the sdvo > encoder automatically adjusts for the need of pixel doubling or > quadrupling. This was lost in pipe config conversion of the > pixel multiplier in > = > commit 6cc5f341b5830541a1b6945435ca90c69b1b8b21 > Author: Daniel Vetter > Date: Wed Mar 27 00:44:53 2013 +0100 > = > drm/i915: add pipe_config->pixel_multiplier > = > While at it ditch the intel_ prefix from the crtc in > intel_sdvo_mode_set. > = > Cc: Jesse Barnes > Cc: Ville Syrj=E4l=E4 > Signed-off-by: Daniel Vetter For both patches: Reviewed-by: Ville Syrj=E4l=E4 -- = Ville Syrj=E4l=E4 Intel OTC