path: root/include/uapi/drm/i915_drm.h
diff options
authorVille Syrjälä <>2015-11-06 21:43:41 +0200
committerVille Syrjälä <>2015-11-18 14:35:16 +0200
commit8697600b4046f26e497b200aff020f10ae6968ae (patch)
treee9002e9eece1e767c08c3a12d8476464a092790b /include/uapi/drm/i915_drm.h
parent3613cf1d8854e64186edb8514f4f5c5b1a1ef679 (diff)
drm/i915: Make the high dword offset more explicit in i915_reg_read_ioctl
Store the upper dword of the register offset in the whitelist as well. This would allow it to read register where the two halves aren't sitting right next to each other, and it'll make it easier to make register access type safe. While at it change the register offsets to u32 from u64. Our register space isn't quite that big, yet :) v2: Use ldw/udw as the suffixes, and add a note about 64bit wide split regs (Chris) Cc: Chris Wilson <> Signed-off-by: Ville Syrjälä <> Link: Reviewed-by: Chris Wilson <>
Diffstat (limited to 'include/uapi/drm/i915_drm.h')
1 files changed, 6 insertions, 0 deletions
diff --git a/include/uapi/drm/i915_drm.h b/include/uapi/drm/i915_drm.h
index 67cebe6d978f..67ef73a5d6eb 100644
--- a/include/uapi/drm/i915_drm.h
+++ b/include/uapi/drm/i915_drm.h
@@ -1079,6 +1079,12 @@ struct drm_i915_gem_context_destroy {
struct drm_i915_reg_read {
+ /*
+ * Register offset.
+ * For 64bit wide registers where the upper 32bits don't immediately
+ * follow the lower 32bits, the offset of the lower 32bits must
+ * be specified
+ */
__u64 offset;
__u64 val; /* Return value */