Message-ID: <5358C997.50803@amd.com>
Date: Thu, 24 Apr 2014 10:21:43 +0200
From: =?UTF-8?B?Q2hyaXN0aWFuIEvDtm5pZw==?= <christian.koenig@amd.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0
MIME-Version: 1.0
To: Ed Tomlinson <edt@aei.ca>, Dave Airlie <airlied@gmail.com>
CC: Linus Torvalds <torvalds@linux-foundation.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        DRI mailing list <dri-devel@lists.freedesktop.org>
Subject: Re: [git pull] drm fixes
References: <alpine.DEB.2.00.1404190810001.7507@skynet.skynet.ie>
 <CA+55aFwbOQCcd1qsW0rA+bQxN2Vgm9x832ZKmB7fndFX7DBE3w@mail.gmail.com>
 <CAPM=9tyHwYJ4CTFPZD4zLNFQyct8A_UmoyUMgR_bBZR5bJB+Pw@mail.gmail.com>
 <2963840.VTDy2Dfbed@grover>
In-Reply-To: <2963840.VTDy2Dfbed@grover>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org

Am 24.04.2014 00:04, schrieb Ed Tomlinson:
> On Wednesday 23 April 2014 07:54:17 Dave Airlie wrote:
>> On Wed, Apr 23, 2014 at 1:59 AM, Linus Torvalds
>> <torvalds@linux-foundation.org> wrote:
>>> Dave, mind sending me a pull request for drm fixes?
>>>
>>> There's now at least these two:
>>>
>>>   - "drm/radeon/aux: fix hpd assignment for aux bus"
>>>   - "drm/radeon: use fixed PPL ref divider if needed"
>>>
>>> that look like fairly fatal regressions when they affect somebody.
>>>
>>> The fact that we already had *two* independent bugs be reported within
>>> days of that last out-of-merge-window pull request makes me very
>>> unhappy with the state of drm pulls.
>>>
>>> So please make sure that future fixes really are *fixes*. For
>>> regressions only. No more games like this.
>> The pll fallout is fixes for the initial feature that was in the merge window,
>> Tuning plls for monitors is always a pain in the ass, the previous algorithm
>> took a couple of kernels a few years back to get where it was, unfortunately
>> HDMI came along and showed up a bunch of its shortcomings. I'm happy
>> Alex and Christian are on top of things in terms of tracking regressions
>> and making sure they get fixed,
>>
>> the AUX fix yes I'm a bit pissed off about myself, but I missed a pull
>> from a few
>> weeks ago, felt guilty, and maybe should have chosen the other path and let it
>> wait a merge,
>>
>> Christian just sent me a -fixes pull with all of these in it and I'll
>> send it on to you
>> in a few mins.
> Hi
>
> Given the fun I had with rc1 I decided to try this pull before rc2 and its working fine here.
>
> Thanks!

Thanks for testing.

As Dave noted PLLs can be quite painful to get right, so it's 
unfortunately not uncommon that fixing one issue uncovers other 
outstanding issues like in your case.

Please leave me a note if that should ever happen again,
Christian.

>
> Ed Tomlinson

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/