drm/xe/uapi: Typo lingo and other small backwards compatible fixes
authorRodrigo Vivi <rodrigo.vivi@intel.com>
Wed, 30 Aug 2023 21:47:14 +0000 (17:47 -0400)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 21 Dec 2023 16:41:06 +0000 (11:41 -0500)
Fix typos, lingo and other small things identified during uapi
review.

v2: Also fix ALIGNMENT typo at xe_query.c
v3: Do not touch property to get/set. (Francois)

Link: https://lore.kernel.org/all/863bebd0c624d6fc2b38c0a06b63e468b4185128.camel@linux.intel.com/
Suggested-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Reviewed-by: Francois Dugast <francois.dugast@intel.com>
drivers/gpu/drm/xe/xe_query.c
include/uapi/drm/xe_drm.h

index 1db77a7c90398fe427a31ba64b293bfeffbab229..c3d396904c7b6085bd3f695364058f4c9cb5aa97 100644 (file)
@@ -195,7 +195,7 @@ static int query_config(struct xe_device *xe, struct drm_xe_device_query *query)
        if (xe_device_get_root_tile(xe)->mem.vram.usable_size)
                config->info[XE_QUERY_CONFIG_FLAGS] =
                        XE_QUERY_CONFIG_FLAGS_HAS_VRAM;
-       config->info[XE_QUERY_CONFIG_MIN_ALIGNEMENT] =
+       config->info[XE_QUERY_CONFIG_MIN_ALIGNMENT] =
                xe->info.vram_flags & XE_VRAM_FLAGS_NEED64K ? SZ_64K : SZ_4K;
        config->info[XE_QUERY_CONFIG_VA_BITS] = xe->info.va_bits;
        config->info[XE_QUERY_CONFIG_GT_COUNT] = xe->info.gt_count;
index 86f16d50e9ccc1144059c7c485985a9ccd06d009..902b5c4f3f5c6afdc5beea989cf3ff3e57b5a3f2 100644 (file)
@@ -256,7 +256,7 @@ struct drm_xe_query_config {
 #define XE_QUERY_CONFIG_REV_AND_DEVICE_ID      0
 #define XE_QUERY_CONFIG_FLAGS                  1
        #define XE_QUERY_CONFIG_FLAGS_HAS_VRAM          (0x1 << 0)
-#define XE_QUERY_CONFIG_MIN_ALIGNEMENT         2
+#define XE_QUERY_CONFIG_MIN_ALIGNMENT          2
 #define XE_QUERY_CONFIG_VA_BITS                        3
 #define XE_QUERY_CONFIG_GT_COUNT               4
 #define XE_QUERY_CONFIG_MEM_REGION_COUNT       5
@@ -449,7 +449,6 @@ struct drm_xe_gem_create {
         * If a VM is specified, this BO must:
         *
         *  1. Only ever be bound to that VM.
-        *
         *  2. Cannot be exported as a PRIME fd.
         */
        __u32 vm_id;
@@ -489,7 +488,7 @@ struct drm_xe_gem_mmap_offset {
  * struct drm_xe_vm_bind_op_error_capture - format of VM bind op error capture
  */
 struct drm_xe_vm_bind_op_error_capture {
-       /** @error: errno that occured */
+       /** @error: errno that occurred */
        __s32 error;
 
        /** @op: operation that encounter an error */
@@ -609,7 +608,7 @@ struct drm_xe_vm_bind_op {
         * caused the error will be captured in drm_xe_vm_bind_op_error_capture.
         * Once the user sees the error (via a ufence +
         * XE_VM_PROPERTY_BIND_OP_ERROR_CAPTURE_ADDRESS), it should free memory
-        * via non-async unbinds, and then restart all queue'd async binds op via
+        * via non-async unbinds, and then restart all queued async binds op via
         * XE_VM_BIND_OP_RESTART. Or alternatively the user should destroy the
         * VM.
         *
@@ -620,7 +619,7 @@ struct drm_xe_vm_bind_op {
 #define XE_VM_BIND_FLAG_ASYNC          (0x1 << 17)
        /*
         * Valid on a faulting VM only, do the MAP operation immediately rather
-        * than differing the MAP to the page fault handler.
+        * than deferring the MAP to the page fault handler.
         */
 #define XE_VM_BIND_FLAG_IMMEDIATE      (0x1 << 18)
        /*
@@ -907,7 +906,7 @@ struct drm_xe_mmio {
 /**
  * struct drm_xe_wait_user_fence - wait user fence
  *
- * Wait on user fence, XE will wakeup on every HW engine interrupt in the
+ * Wait on user fence, XE will wake-up on every HW engine interrupt in the
  * instances list and check if user fence is complete::
  *
  *     (*addr & MASK) OP (VALUE & MASK)
@@ -1039,9 +1038,11 @@ struct drm_xe_vm_madvise {
         */
 #define DRM_XE_VM_MADVISE_PRIORITY             5
 #define                DRM_XE_VMA_PRIORITY_LOW         0
-#define                DRM_XE_VMA_PRIORITY_NORMAL      1       /* Default */
-#define                DRM_XE_VMA_PRIORITY_HIGH        2       /* Must be elevated user */
-       /* Pin the VMA in memory, must be elevated user */
+               /* Default */
+#define                DRM_XE_VMA_PRIORITY_NORMAL      1
+               /* Must be user with elevated privileges */
+#define                DRM_XE_VMA_PRIORITY_HIGH        2
+       /* Pin the VMA in memory, must be user with elevated privileges */
 #define DRM_XE_VM_MADVISE_PIN                  6
        /** @property: property to set */
        __u32 property;