[Intel-gfx] [PATCH] drm/i915: remove render reclock support

Eric Anholt eric at anholt.net
Mon Dec 21 18:30:24 CET 2009


On Fri, 18 Dec 2009 15:36:16 -0800, Eric Anholt <eric at anholt.net> wrote:
> On Thu, 17 Dec 2009 11:11:13 -0800, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> > This code generally fails to adjust the render clock, and when it does,
> > it conflicts with some other register settings and can cause problems.
> > 
> > So remove this code altogether.  I'm reworking it now to do the right
> > thing, but the only bit it will share is the VBT check for whether
> > reclocking is supported, so I'm leaving that bit.
> > 
> > Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> 
> So, you're looking to get this patch as-is into .33, right? not just
> RFC?  And shouldn't it go to stable, since I'm assuming it's the fix for
> all the x200s flickering regressions we've seen?

OK, went ahead and applied it anyway.  Added to commit message:

    Reverts most of 652c393a3368af84359da37c45afc35a91144960 ("add dynamic
    clock frequency control"), though for many the regressions showed up
    in the later 181a5336d6cc836f05507410d66988c483ad0154 ("Fix render
    reclock availability detection").
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20091221/f56d8d01/attachment.sig>


More information about the Intel-gfx mailing list