[Intel-gfx] [PATCH 7/8] drm/i915: Re-enable self-refresh

Chris Wilson chris at chris-wilson.co.uk
Fri Mar 18 18:48:10 CET 2011


On Fri, 18 Mar 2011 08:22:16 -0700, Keith Packard <keithp at keithp.com> wrote:
> On Fri, 18 Mar 2011 08:02:09 +0000, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> 
> > A broken implementation of is_pot() prevented the detection of when a
> > singular pipe was enabled.
> 
> > -	return mask && (mask & -mask) == 0;
> > +	return mask && (mask & -mask) == mask;
> 
> I think the form
> 
>         mask && (mask & (mask - 1)) == 0
> 
> is easier to reason about.

I've reassigned authorship to Yuanhan and gone with your preferred form.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list