In the past, I was told off for patches that update this file without following the procedure described in include/drm/README. Tbh, that procedure causes some annoyances.

Anyway, it's definitely useful to have the patch out on the mailing list in any case.

Cheers,
Nicolai

On 21.03.2017 20:56, Marek Olšák wrote:
From: Marek Olšák <marek.ol...@amd.com>

---
 include/drm/amdgpu_drm.h | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h
index 5797283..d702a95 100644
--- a/include/drm/amdgpu_drm.h
+++ b/include/drm/amdgpu_drm.h
@@ -202,42 +202,47 @@ union drm_amdgpu_ctx {

 struct drm_amdgpu_gem_userptr {
        __u64           addr;
        __u64           size;
        /* AMDGPU_GEM_USERPTR_* */
        __u32           flags;
        /* Resulting GEM handle */
        __u32           handle;
 };

+/* SI-CI-VI: */
 /* same meaning as the GB_TILE_MODE and GL_MACRO_TILE_MODE fields */
 #define AMDGPU_TILING_ARRAY_MODE_SHIFT                 0
 #define AMDGPU_TILING_ARRAY_MODE_MASK                  0xf
 #define AMDGPU_TILING_PIPE_CONFIG_SHIFT                        4
 #define AMDGPU_TILING_PIPE_CONFIG_MASK                 0x1f
 #define AMDGPU_TILING_TILE_SPLIT_SHIFT                 9
 #define AMDGPU_TILING_TILE_SPLIT_MASK                  0x7
 #define AMDGPU_TILING_MICRO_TILE_MODE_SHIFT            12
 #define AMDGPU_TILING_MICRO_TILE_MODE_MASK             0x7
 #define AMDGPU_TILING_BANK_WIDTH_SHIFT                 15
 #define AMDGPU_TILING_BANK_WIDTH_MASK                  0x3
 #define AMDGPU_TILING_BANK_HEIGHT_SHIFT                        17
 #define AMDGPU_TILING_BANK_HEIGHT_MASK                 0x3
 #define AMDGPU_TILING_MACRO_TILE_ASPECT_SHIFT          19
 #define AMDGPU_TILING_MACRO_TILE_ASPECT_MASK           0x3
 #define AMDGPU_TILING_NUM_BANKS_SHIFT                  21
 #define AMDGPU_TILING_NUM_BANKS_MASK                   0x3

+/* GFX9 and later: */
+#define AMDGPU_TILING_SWIZZLE_MODE_SHIFT               0
+#define AMDGPU_TILING_SWIZZLE_MODE_MASK                        0x1f
+
 #define AMDGPU_TILING_SET(field, value) \
-       (((value) & AMDGPU_TILING_##field##_MASK) << 
AMDGPU_TILING_##field##_SHIFT)
+       (((uint64_t)(value) & AMDGPU_TILING_##field##_MASK) << 
AMDGPU_TILING_##field##_SHIFT)
 #define AMDGPU_TILING_GET(value, field) \
-       (((value) >> AMDGPU_TILING_##field##_SHIFT) & 
AMDGPU_TILING_##field##_MASK)
+       (((uint64_t)(value) >> AMDGPU_TILING_##field##_SHIFT) & 
AMDGPU_TILING_##field##_MASK)

 #define AMDGPU_GEM_METADATA_OP_SET_METADATA                  1
 #define AMDGPU_GEM_METADATA_OP_GET_METADATA                  2

 /** The same structure is shared for input/output */
 struct drm_amdgpu_gem_metadata {
        /** GEM Object handle */
        __u32   handle;
        /** Do we want get or set metadata */
        __u32   op;
@@ -748,16 +753,17 @@ struct drm_amdgpu_info_vce_clock_table {

 /*
  * Supported GPU families
  */
 #define AMDGPU_FAMILY_UNKNOWN                  0
 #define AMDGPU_FAMILY_SI                       110 /* Hainan, Oland, Verde, 
Pitcairn, Tahiti */
 #define AMDGPU_FAMILY_CI                       120 /* Bonaire, Hawaii */
 #define AMDGPU_FAMILY_KV                       125 /* Kaveri, Kabini, Mullins 
*/
 #define AMDGPU_FAMILY_VI                       130 /* Iceland, Tonga */
 #define AMDGPU_FAMILY_CZ                       135 /* Carrizo, Stoney */
+#define AMDGPU_FAMILY_AI                       141 /* Vega10 */

 #if defined(__cplusplus)
 }
 #endif

 #endif


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to