On Wed, Nov 04, 2015 at 11:20:14PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> Add 'uint32_t offset' to the uncore register access functions. For now
> it's the same as 'reg', but once type safety gets added 'reg' will be
> the type safe register variable and 'offset' the raw offset.

I presume this is patch 29? (I don't see patch 29/29 :(

But when did we start using off_t? Why would we use a uAPI typedef for an
internal type? I can't wait for you to kill it with fire.

We are mixing lots of uint32_t/u32 here. But the premise of the patch is
sound.
-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