Re: [PATCH] drm/i915: Fix typo in comments and Kconfig.debug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello 
Thanks for your reply.  Exactly , the tools is base on codespell
But it seems not working well > iff
-----Original Message-----
From: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> 
Sent: 2021年7月31日 1:31
To: Cai,Huoqing <caihuoqing@xxxxxxxxx>
Cc: jani.nikula@xxxxxxxxxxxxxxx; joonas.lahtinen@xxxxxxxxxxxxxxx; rodrigo.vivi@xxxxxxxxx; airlied@xxxxxxxx; daniel@xxxxxxxx; intel-gfx@xxxxxxxxxxxxxxxxxxxxx
Subject: Re:  [PATCH] drm/i915: Fix typo in comments and Kconfig.debug

On Fri, Jul 30, 2021 at 05:49:18PM +0800, Cai Huoqing wrote:
>Fix typo:
>*iff  ==> if

this is one that is hard to do automatically because it's also an abbreviation for "if and only if". See https://en.wikipedia.org/wiki/If_and_only_if . In the single place it was changed below, the abbreviation was intended.

I guess these fixes come from some tool. Mind mentioning in the commit message what it is? I would have guessed codespell, but codespell doesn't have this entry, so it must be something else.

Rest looks good. Thanks

Lucas De Marchi

>*specifc  ==> specific
>*adjustement  ==> adjustment
>*preferrably  ==> preferably
>*forseeable  ==> foreseeable
>*overwritting  ==> overwriting
>*sempahores  ==> semaphores
>*additonally  ==> additionally
>*allcoated  ==> allocated
>*contigious  ==> contiguous
>*priorty  ==> priority
>*priviliged  ==> privileged
>*fullfilling  ==> fulfilling
>*Timestmap  ==> Timestamp
>*accesible  ==> accessible
>*becaue  ==> because
>*overriden  ==> overridden
>*prarameter  ==> parameter
>*the the  ==> the
>
>Signed-off-by: Cai Huoqing <caihuoqing@xxxxxxxxx>
>---
> drivers/gpu/drm/i915/Kconfig.debug      |  2 +-
> drivers/gpu/drm/i915/i915_drv.h         |  6 +++---
> drivers/gpu/drm/i915/i915_gpu_error.c   |  2 +-
> drivers/gpu/drm/i915/i915_irq.c         |  2 +-
> drivers/gpu/drm/i915/i915_pci.c         |  4 ++--
> drivers/gpu/drm/i915/i915_perf.c        |  4 ++--
> drivers/gpu/drm/i915/i915_pmu.h         |  2 +-
> drivers/gpu/drm/i915/i915_reg.h         |  2 +-
> drivers/gpu/drm/i915/i915_request.c     |  4 ++--
> drivers/gpu/drm/i915/i915_vma.c         |  4 ++--
> drivers/gpu/drm/i915/intel_pm.c         | 10 +++++-----
> drivers/gpu/drm/i915/intel_region_ttm.c |  2 +-  
>drivers/gpu/drm/i915/intel_runtime_pm.c |  2 +-
> 13 files changed, 23 insertions(+), 23 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/Kconfig.debug 
>b/drivers/gpu/drm/i915/Kconfig.debug
>index 72a38f28393f..c444d97b7c9c 100644
>--- a/drivers/gpu/drm/i915/Kconfig.debug
>+++ b/drivers/gpu/drm/i915/Kconfig.debug
>@@ -2,7 +2,7 @@
> config DRM_I915_WERROR
> 	bool "Force GCC to throw an error instead of a warning when compiling"
> 	# As this may inadvertently break the build, only allow the user
>-	# to shoot oneself in the foot iff they aim really hard
>+	# to shoot oneself in the foot if they aim really hard
> 	depends on EXPERT
> 	# We use the dependency on !COMPILE_TEST to not be enabled in
> 	# allmodconfig or allyesconfig configurations diff --git 
>a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h 
>index 734a2fce1efe..3ae32edf367a 100644
>--- a/drivers/gpu/drm/i915/i915_drv.h
>+++ b/drivers/gpu/drm/i915/i915_drv.h
>@@ -335,7 +335,7 @@ struct drm_i915_display_funcs {  enum 
>i915_cache_level {
> 	I915_CACHE_NONE = 0,
> 	I915_CACHE_LLC, /* also used for snoopable memory on non-LLC */
>-	I915_CACHE_L3_LLC, /* gen7+, L3 sits between the domain specifc
>+	I915_CACHE_L3_LLC, /* gen7+, L3 sits between the domain specific
> 			      caches, eg sampler/render caches, and the
> 			      large Last-Level-Cache. LLC is coherent with
> 			      the CPU, but L3 is only visible to the GPU. */ @@ -383,7 
>+383,7 @@ struct intel_fbc {
> 			int src_h;
> 			bool visible;
> 			/*
>-			 * Display surface base address adjustement for
>+			 * Display surface base address adjustment for
> 			 * pageflips. Note that on gen4+ this only adjusts up
> 			 * to a tile, offsets within a tile are handled in
> 			 * the hw itself (with the TILEOFF register).
>@@ -795,7 +795,7 @@ struct drm_i915_private {
> 	 */
> 	struct resource dsm;
> 	/**
>-	 * Reseved portion of Data Stolen Memory
>+	 * Reserved portion of Data Stolen Memory
> 	 */
> 	struct resource dsm_reserved;
>
>diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c 
>b/drivers/gpu/drm/i915/i915_gpu_error.c
>index 35c97c39f125..601401a510c5 100644
>--- a/drivers/gpu/drm/i915/i915_gpu_error.c
>+++ b/drivers/gpu/drm/i915/i915_gpu_error.c
>@@ -1498,7 +1498,7 @@ gt_record_uc(struct intel_gt_coredump *gt,
> 	memcpy(&error_uc->huc_fw, &uc->huc.fw, sizeof(uc->huc.fw));
>
> 	/* Non-default firmware paths will be specified by the modparam.
>-	 * As modparams are generally accesible from the userspace make
>+	 * As modparams are generally accessible from the userspace make
> 	 * explicit copies of the firmware paths.
> 	 */
> 	error_uc->guc_fw.path = kstrdup(uc->guc.fw.path, ALLOW_FAIL); diff 
>--git a/drivers/gpu/drm/i915/i915_irq.c 
>b/drivers/gpu/drm/i915/i915_irq.c index e2171bd2820e..b426236e16e9 
>100644
>--- a/drivers/gpu/drm/i915/i915_irq.c
>+++ b/drivers/gpu/drm/i915/i915_irq.c
>@@ -62,7 +62,7 @@
>
> /*
>  * Interrupt statistic for PMU. Increments the counter only if the
>- * interrupt originated from the the GPU so interrupts from a device 
>which
>+ * interrupt originated from the GPU so interrupts from a device which
>  * shares the interrupt line are not accounted.
>  */
> static inline void pmu_irq_stats(struct drm_i915_private *i915, diff 
>--git a/drivers/gpu/drm/i915/i915_pci.c 
>b/drivers/gpu/drm/i915/i915_pci.c index ec80cd1cd00c..7be7087e9433 
>100644
>--- a/drivers/gpu/drm/i915/i915_pci.c
>+++ b/drivers/gpu/drm/i915/i915_pci.c
>@@ -1264,8 +1264,8 @@ static int __init i915_init(void)
> 		return err > 0 ? 0 : err;
>
> 	/*
>-	 * Enable KMS by default, unless explicitly overriden by
>-	 * either the i915.modeset prarameter or by the
>+	 * Enable KMS by default, unless explicitly overridden by
>+	 * either the i915.modeset parameter or by the
> 	 * vga_text_mode_force boot option.
> 	 */
>
>diff --git a/drivers/gpu/drm/i915/i915_perf.c 
>b/drivers/gpu/drm/i915/i915_perf.c
>index 9f94914958c3..9e6043c1b65f 100644
>--- a/drivers/gpu/drm/i915/i915_perf.c
>+++ b/drivers/gpu/drm/i915/i915_perf.c
>@@ -3457,7 +3457,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;
>@@ -3939,7 +3939,7 @@ static u32 mask_reg_value(u32 reg, u32 val)
> 	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.
> 	 */
>diff --git a/drivers/gpu/drm/i915/i915_pmu.h 
>b/drivers/gpu/drm/i915/i915_pmu.h index 60f9595f902c..4f315a6bc10f 
>100644
>--- a/drivers/gpu/drm/i915/i915_pmu.h
>+++ b/drivers/gpu/drm/i915/i915_pmu.h
>@@ -95,7 +95,7 @@ struct i915_pmu {
> 	/**
> 	 * @timer_last:
> 	 *
>-	 * Timestmap of the previous timer invocation.
>+	 * Timestamp of the previous timer invocation.
> 	 */
> 	ktime_t timer_last;
>
>diff --git a/drivers/gpu/drm/i915/i915_reg.h 
>b/drivers/gpu/drm/i915/i915_reg.h index 730afa341f8a..a7ba32e66de2 
>100644
>--- a/drivers/gpu/drm/i915/i915_reg.h
>+++ b/drivers/gpu/drm/i915/i915_reg.h
>@@ -2955,7 +2955,7 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
> #define   HDPORT_DDI_USED(phy)		REG_BIT(2 * (phy) + 1)
> #define   HDPORT_ENABLED		REG_BIT(0)
>
>-/* Make render/texture TLB fetches lower priorty than associated data
>+/* 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)
>diff --git a/drivers/gpu/drm/i915/i915_request.c 
>b/drivers/gpu/drm/i915/i915_request.c
>index 37aef1308573..865aaf8378e4 100644
>--- a/drivers/gpu/drm/i915/i915_request.c
>+++ b/drivers/gpu/drm/i915/i915_request.c
>@@ -489,7 +489,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 @@ -1239,7 +1239,7 @@ emit_semaphore_wait(struct i915_request *to,
> 	/*
> 	 * If this or its dependents are waiting on an external fence
> 	 * that may fail catastrophically, then we want to avoid using
>-	 * sempahores as they bypass the fence signaling metadata, and we
>+	 * semaphores as they bypass the fence signaling metadata, and we
> 	 * lose the fence->error propagation.
> 	 */
> 	if (from->sched.flags & I915_SCHED_HAS_EXTERNAL_CHAIN) diff --git 
>a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c 
>index 0f227f28b280..1ecbf94370dd 100644
>--- a/drivers/gpu/drm/i915/i915_vma.c
>+++ b/drivers/gpu/drm/i915/i915_vma.c
>@@ -626,7 +626,7 @@ bool i915_gem_valid_gtt_space(struct i915_vma *vma, 
>unsigned long color)
>  *
>  * 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.
>+ * preferably the oldest idle entry to make room for the new VMA.
>  *
>  * Returns:
>  * 0 on success, negative error code otherwise.
>@@ -696,7 +696,7 @@ i915_vma_insert(struct i915_vma *vma, u64 size, u64 alignment, u64 flags)
> 		 * 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) { diff --git 
>a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c 
>index 10a39a7d8150..740f0c8c1d30 100644
>--- a/drivers/gpu/drm/i915/intel_pm.c
>+++ b/drivers/gpu/drm/i915/intel_pm.c
>@@ -660,7 +660,7 @@ static const struct intel_watermark_params 
>i845_wm_info = {
>  * @latency: Memory wakeup latency in 0.1us units
>  *
>  * Compute the watermark using the method 1 or "small buffer"
>- * formula. The caller may additonally add extra cachelines
>+ * formula. The caller may additionally add extra cachelines
>  * to account for TLB misses and clock crossings.
>  *
>  * This method is concerned with the short term drain rate @@ -707,7 
>+707,7 @@ static unsigned int intel_wm_method1(unsigned int pixel_rate,
>  * @latency: Memory wakeup latency in 0.1us units
>  *
>  * Compute the watermark using the method 2 or "large buffer"
>- * formula. The caller may additonally add extra cachelines
>+ * formula. The caller may additionally add extra cachelines
>  * to account for TLB misses and clock crossings.
>  *
>  * This method is concerned with the long term drain rate @@ -1719,7 
>+1719,7 @@ static int vlv_compute_fifo(struct intel_crtc_state *crtc_state)
> 	/*
> 	 * When enabling sprite0 after sprite1 has already been enabled
> 	 * we tend to get an underrun unless sprite0 already has some
>-	 * FIFO space allcoated. Hence we always allocate at least one
>+	 * FIFO space allocated. Hence we always allocate at least one
> 	 * cacheline for sprite0 whenever sprite1 is enabled.
> 	 *
> 	 * All other plane enable sequences appear immune to this problem.
>@@ -2919,7 +2919,7 @@ static void intel_read_wm_latency(struct drm_i915_private *dev_priv,
> 		 * WaWmMemoryReadLatency
> 		 *
> 		 * punit doesn't take into account the read latency so we need
>-		 * to add proper adjustement to each valid level we retrieve
>+		 * to add proper adjustment to each valid level we retrieve
> 		 * from the punit when level 0 response data is 0us.
> 		 */
> 		if (wm[0] == 0) {
>@@ -4091,7 +4091,7 @@ u32 skl_ddb_dbuf_slice_mask(struct 
>drm_i915_private *dev_priv,
>
> 	/*
> 	 * Per plane DDB entry can in a really worst case be on multiple slices
>-	 * but single entry is anyway contigious.
>+	 * but single entry is anyway contiguous.
> 	 */
> 	while (start_slice <= end_slice) {
> 		slice_mask |= BIT(start_slice);
>diff --git a/drivers/gpu/drm/i915/intel_region_ttm.c 
>b/drivers/gpu/drm/i915/intel_region_ttm.c
>index 27fe0668d094..d0d1972343b7 100644
>--- a/drivers/gpu/drm/i915/intel_region_ttm.c
>+++ b/drivers/gpu/drm/i915/intel_region_ttm.c
>@@ -204,7 +204,7 @@ intel_region_ttm_node_alloc(struct 
>intel_memory_region *mem,
>
> 	/*
> 	 * We ignore the flags for now since we're using the range
>-	 * manager and contigous and min page size would be fulfilled
>+	 * manager and contiguous and min page size would be fulfilled
> 	 * by default if size is min page size aligned.
> 	 */
> 	mock_bo.base.size = size;
>diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c 
>b/drivers/gpu/drm/i915/intel_runtime_pm.c
>index eaf7688f517d..dfcf43e55fc0 100644
>--- a/drivers/gpu/drm/i915/intel_runtime_pm.c
>+++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
>@@ -575,7 +575,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);
>--
>2.25.1
>




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux