On Wed, 27 Apr 2011 14:01:50 -0700, Jesse Barnes <jbar...@virtuousgeek.org> 
wrote:
> On Wed, 27 Apr 2011 07:59:17 +0100
> Chris Wilson <ch...@chris-wilson.co.uk> wrote:
> > So I think we just want IS_GEN7() for IVB code.
> 
> I'd rather keep them separate since we know we'll have gen7 chips with
> different display engines in the future.  I've been trying to use
> IS_GEN for render related checks, IS_IVB for specific display checks,
> and HAS_PCH for the ilk+ split (though for the most part that should go
> away with proper function call backs).

If there is method to the madness, fine. I like the explanation and so
perhaps we should include that in the code somewhere. And we just need to
make sure we are consistent in our usage.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to