On Thu, Feb 14, 2013 at 10:45 PM, Dave Airlie <airlied at gmail.com> wrote: > On Fri, Feb 15, 2013 at 6:59 AM, Daniel Vetter <daniel at ffwll.ch> wrote: >> On Thu, Feb 14, 2013 at 08:50:25PM +0000, Chris Wilson wrote: >>> On Wed, Feb 13, 2013 at 10:20:22PM +0100, Patrik Jakobsson wrote: >>> > The Intel PRM says the M1 and M2 divisors must be in the range of 10-20 and 5-9. >>> > Since we do all calculations based on them being register values (which are >>> > subtracted by 2) we need to specify them accordingly. >>> > >>> > Signed-off-by: Patrik Jakobsson <patrik.r.jakobsson at gmail.com> >>> >>> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> >> >> Queued both for -next with a cc: stable tag to increase the odds of >> detecting bugs (or fixing some). Thanks for the patches. >> -Daniel > > yeah no stable for this sorta thing, this could go horribly wrong, if > it fxies something stable it later. > > stable isn't meant for testing patches that could break things. Ok, unstabled again. But wouldn't really have been the first time we'd throw a revert to stable ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
