]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/i915: fix typos in drm/i915 files
authorNitin Gote <nitin.r.gote@intel.com>
Mon, 20 Jan 2025 08:15:17 +0000 (13:45 +0530)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 23 Jan 2025 10:48:23 +0000 (05:48 -0500)
Fix all typos in files under drm/i915 reported by codespell tool.

v2: Fix commenting style. <Andi>

v3: "in case" should be capitalized and fix
    comment style. <Krzysztof Niemiec>

Signed-off-by: Nitin Gote <nitin.r.gote@intel.com>
Reviewed-by: Krzysztof Niemiec <krzysztof.niemiec@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250120081517.3237326-9-nitin.r.gote@intel.com
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
16 files changed:
drivers/gpu/drm/i915/i915_driver.c
drivers/gpu/drm/i915/i915_gem.c
drivers/gpu/drm/i915/i915_irq.c
drivers/gpu/drm/i915/i915_module.c
drivers/gpu/drm/i915/i915_perf.c
drivers/gpu/drm/i915/i915_pmu.h
drivers/gpu/drm/i915/i915_reg.h
drivers/gpu/drm/i915/i915_request.c
drivers/gpu/drm/i915/i915_request.h
drivers/gpu/drm/i915/i915_vma.c
drivers/gpu/drm/i915/intel_clock_gating.c
drivers/gpu/drm/i915/intel_gvt.c
drivers/gpu/drm/i915/intel_gvt_mmio_table.c
drivers/gpu/drm/i915/intel_runtime_pm.c
drivers/gpu/drm/i915/intel_runtime_pm.h
drivers/gpu/drm/i915/intel_uncore.c

index c2ae37d6b94d59c5e3ae7004c83db3ff8d00631a..91a7748f4492620f0bd40aafdf3e76e170dc8422 100644 (file)
@@ -1130,7 +1130,7 @@ static int i915_drm_suspend_late(struct drm_device *dev, bool hibernation)
         * leave the device in D0 on those platforms and hope the BIOS will
         * power down the device properly. The issue was seen on multiple old
         * GENs with different BIOS vendors, so having an explicit blacklist
-        * is inpractical; apply the workaround on everything pre GEN6. The
+        * is impractical; apply the workaround on everything pre GEN6. The
         * platforms where the issue was seen:
         * Lenovo Thinkpad X301, X61s, X60, T60, X41
         * Fujitsu FSC S7110
index 070ab654698798feb964e072d39ed3b5bf08516b..8c8d43451f35ecd6855c8b40ca69f359d15246d2 100644 (file)
@@ -1146,11 +1146,11 @@ int i915_gem_init(struct drm_i915_private *dev_priv)
        int ret;
 
        /*
-        * In the proccess of replacing cache_level with pat_index a tricky
+        * In the process of replacing cache_level with pat_index a tricky
         * dependency is created on the definition of the enum i915_cache_level.
-        * in case this enum is changed, PTE encode would be broken.
+        * In case this enum is changed, PTE encode would be broken.
         * Add a WARNING here. And remove when we completely quit using this
-        * enum
+        * enum.
         */
        BUILD_BUG_ON(I915_CACHE_NONE != 0 ||
                     I915_CACHE_LLC != 1 ||
index 202eb1b6ae54d258184bdf8df8294a448519b8de..f98e5cc14724b64fcfdbe65913a30aea478bacfc 100644 (file)
@@ -1233,7 +1233,7 @@ int intel_irq_install(struct drm_i915_private *dev_priv)
 }
 
 /**
- * intel_irq_uninstall - finilizes all irq handling
+ * intel_irq_uninstall - finalizes all irq handling
  * @dev_priv: i915 device instance
  *
  * This stops interrupt and hotplug handling and unregisters and frees all
index 7ed6d70389af97e22b0806684c28519154ebb0dd..2f88970cc0a93d49f1312929b2f8d0f9b526c270 100644 (file)
@@ -24,7 +24,7 @@ static int i915_check_nomodeset(void)
        bool use_kms = true;
 
        /*
-        * Enable KMS by default, unless explicitly overriden by
+        * Enable KMS by default, unless explicitly overridden by
         * either the i915.modeset parameter or by the
         * nomodeset boot option.
         */
index 2406cda75b7b4b26afe2517621ec0b7b290f4bd9..eb7b4e21b124632344b089ba85de8ef1f1c21f21 100644 (file)
@@ -548,7 +548,8 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream)
        bool pollin;
        u32 partial_report_size;
 
-       /* We have to consider the (unlikely) possibility that read() errors
+       /*
+        * We have to consider the (unlikely) possibility that read() errors
         * could result in an OA buffer reset which might reset the head and
         * tail state.
         */
@@ -557,7 +558,8 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream)
        hw_tail = stream->perf->ops.oa_hw_tail_read(stream);
        hw_tail -= gtt_offset;
 
-       /* The tail pointer increases in 64 byte increments, not in report_size
+       /*
+        * The tail pointer increases in 64 byte increments, not in report_size
         * steps. Also the report size may not be a power of 2. Compute
         * potentially partially landed report in the OA buffer
         */
@@ -569,8 +571,9 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream)
 
        tail = hw_tail;
 
-       /* Walk the stream backward until we find a report with report
-        * id and timestmap not at 0. Since the circular buffer pointers
+       /*
+        * Walk the stream backward until we find a report with report
+        * id and timestamp not at 0. Since the circular buffer pointers
         * progress by increments of 64 bytes and that reports can be up
         * to 256 bytes long, we can't tell whether a report has fully
         * landed in memory before the report id and timestamp of the
@@ -3849,7 +3852,7 @@ i915_perf_open_ioctl_locked(struct i915_perf *perf,
        }
 
        /*
-        * Asking for SSEU configuration is a priviliged operation.
+        * Asking for SSEU configuration is a privileged operation.
         */
        if (props->has_sseu)
                privileged_op = true;
@@ -4478,14 +4481,16 @@ static bool gen12_is_valid_mux_addr(struct i915_perf *perf, u32 addr)
 
 static u32 mask_reg_value(u32 reg, u32 val)
 {
-       /* HALF_SLICE_CHICKEN2 is programmed with a the
+       /*
+        * HALF_SLICE_CHICKEN2 is programmed with a the
         * WaDisableSTUnitPowerOptimization workaround. Make sure the value
         * programmed by userspace doesn't change this.
         */
        if (REG_EQUAL(reg, HALF_SLICE_CHICKEN2))
                val = val & ~_MASKED_BIT_ENABLE(GEN8_ST_PO_DISABLE);
 
-       /* WAIT_FOR_RC6_EXIT has only one bit fullfilling the function
+       /*
+        * WAIT_FOR_RC6_EXIT has only one bit fulfilling the function
         * indicated by its name and a bunch of selection fields used by OA
         * configs.
         */
index 8e66d63d0c9f98c061c6ad6a31ae36303d1f9b8a..0ec78c2b4f20323e728bfa7b0af4ed55f6021659 100644 (file)
@@ -103,7 +103,7 @@ struct i915_pmu {
        /**
         * @timer_last:
         *
-        * Timestmap of the previous timer invocation.
+        * Timestamp of the previous timer invocation.
         */
        ktime_t timer_last;
 
index 765e6c0528fb0b5a894395b77a5edbf0b0c80009..b9e2aa1c6f8afefade761b8d291bb62efb96e53c 100644 (file)
@@ -84,7 +84,7 @@
  * Try to name registers according to the specs. If the register name changes in
  * the specs from platform to another, stick to the original name.
  *
- * Try to re-use existing register macro definitions. Only add new macros for
+ * Try to reuse existing register macro definitions. Only add new macros for
  * new register offsets, or when the register contents have changed enough to
  * warrant a full redefinition.
  *
 #define MBUS_ABOX_BT_CREDIT_POOL1_MASK (0x1F << 0)
 #define MBUS_ABOX_BT_CREDIT_POOL1(x)   ((x) << 0)
 
-/* Make render/texture TLB fetches lower priorty than associated data
- *   fetches. This is not turned on by default
+/*
+ * Make render/texture TLB fetches lower priority than associated data
+ * fetches. This is not turned on by default.
  */
 #define   MI_ARB_RENDER_TLB_LOW_PRIORITY       (1 << 15)
 
index 8f62cfa23fb768bbaac589551868a2749f8c7e79..f8c584ce3295e472a9d4591738554008c3c7d6ec 100644 (file)
@@ -473,7 +473,7 @@ static bool __request_in_flight(const struct i915_request *signal)
         * to avoid tearing.]
         *
         * Note that the read of *execlists->active may race with the promotion
-        * of execlists->pending[] to execlists->inflight[], overwritting
+        * of execlists->pending[] to execlists->inflight[], overwriting
         * the value at *execlists->active. This is fine. The promotion implies
         * that we received an ACK from the HW, and so the context is not
         * stuck -- if we do not see ourselves in *active, the inflight status
index 0ac55b2e4223e5165eeb6131f26d423bcff47a28..5f7e8138ec1468d3f649e0a8c63d2d3a18554534 100644 (file)
@@ -161,7 +161,7 @@ enum {
         * parent-child relationship (parallel submission, multi-lrc) that
         * hit an error while generating requests in the execbuf IOCTL.
         * Indicates this request should be skipped as another request in
-        * submission / relationship encoutered an error.
+        * submission / relationship encountered an error.
         */
        I915_FENCE_FLAG_SKIP_PARALLEL,
 
@@ -187,7 +187,7 @@ enum {
  * RCU lookup of it that may race against reallocation of the struct
  * from the slab freelist. We intentionally do not zero the structure on
  * allocation so that the lookup can use the dangling pointers (and is
- * cogniscent that those pointers may be wrong). Instead, everything that
+ * cognisant that those pointers may be wrong). Instead, everything that
  * needs to be initialised must be done so explicitly.
  *
  * The requests are reference counted.
index 776f8cc51b2fc0d9cd9242a60d746f902e62c0d5..61b49007ecd48a584dfe93a9043bcce295cbe372 100644 (file)
@@ -778,8 +778,8 @@ bool i915_gem_valid_gtt_space(struct i915_vma *vma, unsigned long color)
  * @flags: mask of PIN_* flags to use
  *
  * First we try to allocate some free space that meets the requirements for
- * the VMA. Failiing that, if the flags permit, it will evict an old VMA,
- * preferrably the oldest idle entry to make room for the new VMA.
+ * the VMA. Failing that, if the flags permit, it will evict an old VMA,
+ * preferably the oldest idle entry to make room for the new VMA.
  *
  * Returns:
  * 0 on success, negative error code otherwise.
@@ -877,7 +877,7 @@ i915_vma_insert(struct i915_vma *vma, struct i915_gem_ww_ctx *ww,
                 * objects which need to be tightly packed into the low 32bits.
                 *
                 * Note that we assume that GGTT are limited to 4GiB for the
-                * forseeable future. See also i915_ggtt_offset().
+                * foreseeable future. See also i915_ggtt_offset().
                 */
                if (upper_32_bits(end - 1) &&
                    vma->page_sizes.sg > I915_GTT_PAGE_SIZE &&
@@ -1001,7 +1001,7 @@ rotate_pages(struct drm_i915_gem_object *obj, unsigned int offset,
 
                /*
                 * The DE ignores the PTEs for the padding tiles, the sg entry
-                * here is just a conenience to indicate how many padding PTEs
+                * here is just a convenience to indicate how many padding PTEs
                 * to insert at this spot.
                 */
                sg_set_page(sg, NULL, left, 0);
index f76642886569ebb7a492c3d4e82a2dd57a9016b1..387b2640016901b5081e0cf43c90fa7b360bb65e 100644 (file)
@@ -682,7 +682,7 @@ static void i85x_init_clock_gating(struct drm_i915_private *i915)
         * Have FBC ignore 3D activity since we use software
         * render tracking, and otherwise a pure 3D workload
         * (even if it just renders a single frame and then does
-        * abosultely nothing) would not allow FBC to recompress
+        * absolutely nothing) would not allow FBC to recompress
         * until a 2D blit occurs.
         */
        intel_uncore_write(&i915->uncore, SCPD0,
index a5383a2bc64b9d186a965e36423a547592d3d69e..dae9dce7d1b3d8da4354eb796390da4071d0b325 100644 (file)
@@ -265,7 +265,7 @@ void intel_gvt_driver_remove(struct drm_i915_private *dev_priv)
 }
 
 /**
- * intel_gvt_resume - GVT resume routine wapper
+ * intel_gvt_resume - GVT resume routine wrapper
  *
  * @dev_priv: drm i915 private *
  *
index ee1cd2126f97b474aa7c381c74b5922c5b6be1a8..04076316e1397c2f6adb3cd3761e6a193d465211 100644 (file)
@@ -1260,7 +1260,7 @@ static int iterate_bxt_mmio(struct intel_gvt_mmio_table_iter *iter)
 
 /**
  * intel_gvt_iterate_mmio_table - Iterate the GVT MMIO table
- * @iter: the interator
+ * @iter: the iterator
  *
  * This function is called for iterating the GVT MMIO table when i915 is
  * taking the snapshot of the HW and GVT is building MMIO tracking table.
index 1a47ecfd3fd875eac03629dba5aba95cbc5f773c..8d9f4c410546e4144d4bc8bbc6696f3bd9498848 100644 (file)
@@ -375,7 +375,7 @@ void intel_runtime_pm_enable(struct intel_runtime_pm *rpm)
         * leave the device suspended skipping the driver's suspend handlers
         * if the device was already runtime suspended. This is needed due to
         * the difference in our runtime and system suspend sequence and
-        * becaue the HDA driver may require us to enable the audio power
+        * because the HDA driver may require us to enable the audio power
         * domain during system suspend.
         */
        dev_pm_set_driver_flags(kdev, DPM_FLAG_NO_DIRECT_COMPLETE);
index e22669d61e954a9bc22f007d06aaf46c268f3410..7428bd8fa67f4e9c2524a6487b91ca837a9dac55 100644 (file)
@@ -31,7 +31,7 @@ struct drm_printer;
  * it can be changed with the standard runtime PM files from sysfs.
  *
  * The irqs_disabled variable becomes true exactly after we disable the IRQs and
- * goes back to false exactly before we reenable the IRQs. We use this variable
+ * goes back to false exactly before we re-enable the IRQs. We use this variable
  * to check if someone is trying to enable/disable IRQs while they're supposed
  * to be disabled. This shouldn't happen and we'll print some error messages in
  * case it happens.
index eed4937c3ff30974fd3205ed6125e66e311e0f99..04ef628e208b3b997d1fbda0c73c46f46f21cfc3 100644 (file)
@@ -2477,7 +2477,7 @@ static int sanity_check_mmio_access(struct intel_uncore *uncore)
 
        /*
         * Sanitycheck that MMIO access to the device is working properly.  If
-        * the CPU is unable to communcate with a PCI device, BAR reads will
+        * the CPU is unable to communicate with a PCI device, BAR reads will
         * return 0xFFFFFFFF.  Let's make sure the device isn't in this state
         * before we start trying to access registers.
         *