From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 0/5] drm/i915: Gen2 PLL fixes Date: Wed, 11 Dec 2013 11:12:31 +0100 Message-ID: <20131211101231.GY9804@phenom.ffwll.local> References: <1386608057-20866-1-git-send-email-ville.syrjala@linux.intel.com> <20131210125227.1d2bb70f@neptune.home> <20131210121936.GN10036@intel.com> <20131210133301.5f2788f2@neptune.home> <20131210124707.GO10036@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-wg0-f46.google.com (mail-wg0-f46.google.com [74.125.82.46]) by gabe.freedesktop.org (Postfix) with ESMTP id D170EFAFD8 for ; Wed, 11 Dec 2013 02:11:42 -0800 (PST) Received: by mail-wg0-f46.google.com with SMTP id m15so6053681wgh.1 for ; Wed, 11 Dec 2013 02:11:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <20131210124707.GO10036@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Cc: Bruno =?iso-8859-1?Q?Pr=E9mont?= , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Dec 10, 2013 at 02:47:07PM +0200, Ville Syrj=E4l=E4 wrote: > On Tue, Dec 10, 2013 at 01:33:01PM +0100, Bruno Pr=E9mont wrote: > > Hi Ville, > > = > > On Tue, 10 December 2013 Ville Syrj=E4l=E4 wrote: > > > On Tue, Dec 10, 2013 at 12:52:27PM +0100, Bruno Pr=E9mont wrote: > > > > On Mon, 09 December 2013 Ville Syrj=E4l=E4 wrote: > > > > > There appear to be some gen2 machines that don't really like the = current PLL > > > > > limits we have. We also have some accuracy problems with the PLL = calculations. > > > > > This series aims to eliminate those problems, and at least my 855= machine > > > > > seems happier with these patches. > > > > > = > > > > > Ville Syrj=E4l=E4 (5): > > > > > drm/i915: Extract p2 divider correctly for gen2 LVDS dual c= hannel > > > > > drm/i915: Change N divider minimum from 3 to 2 for gen2 > > > > > drm/i915: Increase gen2 vco frequency limit to 1512 MHz > > > > > drm/i915: Fix 66 MHz LVDS SSC freq for gen2 > > > > > drm/i915: Decrease gen2 vco frequency minimum to 908 MHz > > > > > = > > > > > drivers/gpu/drm/i915/intel_bios.c | 8 ++++---- > > > > > drivers/gpu/drm/i915/intel_display.c | 34 +++++++++++++++++++---= ------------ > > > > > 2 files changed, 23 insertions(+), 19 deletions(-) > > > > = > > > > Here is my dmesg with the 5 patches applied (on top of 3.13-rc3 and > > > > the PLL debug patch). > > > > = > > > > The WARN() are not hit anymore. > > > > = > > > > > > > [ 46.584578] [drm:intel_dump_pipe_config], DPLL =3D 0x0 > > > > [ 46.584581] [drm:intel_dump_pipe_config], DPLL_MD =3D 0x0 > > > > [ 46.584583] [drm:intel_dump_pipe_config], FP0 =3D 0x0 > > > > [ 46.584586] [drm:intel_dump_pipe_config], FP1 =3D 0x0 > > > = > > > OK, so I think the problem got fixed, but my debug patch wasn't > > > all that good since it doesn't dump the values we computed unless > > > the clock is off. > > > = > > > Can you still run this (as root) when the LVDS output is active: > > > # intel_reg_read 0x6014 0x6018 0x6040 0x6044 0x6048 0x604c > > > = > > > intel_reg_read is part of intel-gpu-tools. > > = > > Here are the results (when already running under X since previous > > my mail): > > = > > # intel_reg_read 0x6014 0x6018 0x6040 0x6044 0x6048 0x604c > > 0x6014 : 0x0 > > 0x6018 : 0x90020000 > > 0x6040 : 0x2140E > > 0x6044 : 0x2140E > > 0x6048 : 0x2140E > > 0x604C : 0x2140E > = > Thanks. That's what the BIOS had, so it seems we're good. Now I suppose > we just need to figure out if we dare change the PLL limits for all gen2, > or if we should limit the changes to a more narrow set of platforms. Ok, I've pulled them all into dinq, let's see what happens. -Daniel -- = Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch