[Intel-gfx] [PATCH 26/29] drm/i915: Add 'offset' to uncore funcs

Chris Wilson chris at chris-wilson.co.uk
Thu Nov 5 02:16:00 PST 2015


On Wed, Nov 04, 2015 at 11:20:14PM +0200, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at 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


More information about the Intel-gfx mailing list