]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915/uapi: Add comment clarifying purpose of I915_TILING_* values
authorMatt Roper <matthew.d.roper@intel.com>
Tue, 12 Oct 2021 22:12:45 +0000 (15:12 -0700)
committerMatt Roper <matthew.d.roper@intel.com>
Fri, 15 Oct 2021 00:43:27 +0000 (17:43 -0700)
commitf4453687a2139073fa9e9ca8b223c069180fc2b2
tree4947c52b844cfe835b3b20efb7b10e401ebc36fc
parent20efd30248249c6931df83cbd196833f77f85101
drm/i915/uapi: Add comment clarifying purpose of I915_TILING_* values

The I915_TILING_* values in our uapi header are intended solely for use
with the old get_tiling/set_tiling ioctls that operate on hardware
de-tiling fences; all other uapi communication about tiling types is
done via framebuffer modifiers rather than with these old values.

On newer Intel platforms detiling fences no longer exist so the old
get_tiling/set_tiling ioctls are no longer usable and will always return
-EOPNOTSUPP.  This means there's no reason to add new tiling types (such
as the Tile4 format introduced by Xe_HP) to the uapi header here.  Any
kernel-internal code that needs to represent tiling format should either
rely on framebuffer modifiers (as the display code does) or use some
kind of non-uapi enum (as the GEM blt selftest now does).

References: https://patchwork.freedesktop.org/patch/456656/?series=95308
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Caz Yokoyama <caz.yokoyama@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211012221245.2609670-1-matthew.d.roper@intel.com
include/uapi/drm/i915_drm.h