<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8"></head><body ><div><br></div><div>Yeah, haven't looked too hard at the changes, hopefully the fixed numbers can work across all out BYT SKUs. I'll test more when I get my new platform next week.</div><div><br></div>Jesse Barnes, Intel Open Source Technology Center<br><br><br>-------- Original message --------<br>From: Daniel Vetter <daniel@ffwll.ch> <br>Date: 14/09/2013  4:28 AM  (GMT-08:00) <br>To: Jesse Barnes <jbarnes@virtuousgeek.org> <br>Cc: intel-gfx <intel-gfx@lists.freedesktop.org> <br>Subject: Re: [Intel-gfx] [PATCH] Revert "drm/i915: Move Valleyview DP DPLL   divisor calc to intel_dp_set_clock v2" <br> <br><br>On Sat, Sep 14, 2013 at 2:29 AM, Jesse Barnes <jbarnes@virtuousgeek.org> wrote:<br>> On Fri, 13 Sep 2013 17:27:54 -0700<br>> Jesse Barnes <jbarnes@virtuousgeek.org> wrote:<br>><br>>> This reverts commit 65ce4bf5a15fcd4d15898be47795d0550eb2325c.<br>><br>> Found this was breaking eDP for me with -nightly as of today on Baley<br>> Bay boards.<br>><br>> Chris, can you verify this on your system too?  I'm working on getting<br>> a new one with eDP support to test more.<br><br>Checking with my calculator the pll settings for the 1.62 clock are<br>wrong. Can you try to just fix them?<br>-Daniel<br>-- <br>Daniel Vetter<br>Software Engineer, Intel Corporation<br>+41 (0) 79 365 57 48 - http://blog.ffwll.ch<br><br></body>