All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] drm: Fix typo in comments
@ 2021-07-30 13:27 Cai Huoqing
  2021-08-02  8:19 ` Daniel Vetter
  0 siblings, 1 reply; 3+ messages in thread
From: Cai Huoqing @ 2021-07-30 13:27 UTC (permalink / raw)
  To: maarten.lankhorst, mripard, tzimmermann, airlied, daniel
  Cc: Cai Huoqing, dri-devel

fix typo for drm

v1->v2:
respin with the change "iff ==> implies that"

Reviewed-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
---
 drivers/gpu/drm/drm_aperture.c          |  2 +-
 drivers/gpu/drm/drm_atomic.c            |  2 +-
 drivers/gpu/drm/drm_atomic_helper.c     | 10 +++++-----
 drivers/gpu/drm/drm_atomic_uapi.c       |  6 +++---
 drivers/gpu/drm/drm_auth.c              |  2 +-
 drivers/gpu/drm/drm_bridge.c            |  2 +-
 drivers/gpu/drm/drm_bufs.c              |  2 +-
 drivers/gpu/drm/drm_cache.c             |  2 +-
 drivers/gpu/drm/drm_damage_helper.c     |  2 +-
 drivers/gpu/drm/drm_dp_helper.c         |  8 ++++----
 drivers/gpu/drm/drm_drv.c               |  4 ++--
 drivers/gpu/drm/drm_dsc.c               |  2 +-
 drivers/gpu/drm/drm_edid.c              |  4 ++--
 drivers/gpu/drm/drm_fb_helper.c         |  2 +-
 drivers/gpu/drm/drm_file.c              |  6 +++---
 drivers/gpu/drm/drm_format_helper.c     |  2 +-
 drivers/gpu/drm/drm_framebuffer.c       |  2 +-
 drivers/gpu/drm/drm_gem.c               |  4 ++--
 drivers/gpu/drm/drm_gem_atomic_helper.c |  4 ++--
 drivers/gpu/drm/drm_gem_shmem_helper.c  |  2 +-
 drivers/gpu/drm/drm_gem_vram_helper.c   |  2 +-
 drivers/gpu/drm/drm_hdcp.c              |  2 +-
 drivers/gpu/drm/drm_ioctl.c             |  4 ++--
 drivers/gpu/drm/drm_irq.c               |  2 +-
 drivers/gpu/drm/drm_mm.c                |  2 +-
 drivers/gpu/drm/drm_mode_object.c       |  2 +-
 drivers/gpu/drm/drm_modes.c             |  4 ++--
 drivers/gpu/drm/drm_plane.c             |  2 +-
 drivers/gpu/drm/drm_plane_helper.c      |  2 +-
 drivers/gpu/drm/drm_prime.c             |  2 +-
 drivers/gpu/drm/drm_probe_helper.c      |  2 +-
 drivers/gpu/drm/drm_property.c          |  2 +-
 drivers/gpu/drm/drm_scdc_helper.c       |  2 +-
 drivers/gpu/drm/drm_syncobj.c           |  2 +-
 drivers/gpu/drm/drm_vblank.c            | 12 ++++++------
 drivers/gpu/drm/drm_vma_manager.c       |  2 +-
 36 files changed, 58 insertions(+), 58 deletions(-)

diff --git a/drivers/gpu/drm/drm_aperture.c b/drivers/gpu/drm/drm_aperture.c
index 9ac39cf11694..74bd4a76b253 100644
--- a/drivers/gpu/drm/drm_aperture.c
+++ b/drivers/gpu/drm/drm_aperture.c
@@ -78,7 +78,7 @@
  *
  * Drivers that are susceptible to being removed by other drivers, such as
  * generic EFI or VESA drivers, have to register themselves as owners of their
- * given framebuffer memory. Ownership of the framebuffer memory is achived
+ * given framebuffer memory. Ownership of the framebuffer memory is achieved
  * by calling devm_aperture_acquire_from_firmware(). On success, the driver
  * is the owner of the framebuffer range. The function fails if the
  * framebuffer is already by another driver. See below for an example.
diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index d820423fac32..b127e30082ba 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu/drm/drm_atomic.c
@@ -723,7 +723,7 @@ static void drm_atomic_plane_print_state(struct drm_printer *p,
  * clocks, scaler units, bandwidth and fifo limits shared among a group of
  * planes or CRTCs, and so on) it makes sense to model these as independent
  * objects. Drivers then need to do similar state tracking and commit ordering for
- * such private (since not exposed to userpace) objects as the atomic core and
+ * such private (since not exposed to userspace) objects as the atomic core and
  * helpers already provide for connectors, planes and CRTCs.
  *
  * To make this easier on drivers the atomic core provides some support to track
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
index f7bf1ea62d58..7ee480b6efde 100644
--- a/drivers/gpu/drm/drm_atomic_helper.c
+++ b/drivers/gpu/drm/drm_atomic_helper.c
@@ -634,7 +634,7 @@ drm_atomic_helper_check_modeset(struct drm_device *dev,
 			 * connectors and a NULL mode.
 			 *
 			 * The other way around is true as well. enable != 0
-			 * iff connectors are attached and a mode is set.
+			 * implies that connectors are attached and a mode is set.
 			 */
 			new_crtc_state->mode_changed = true;
 			new_crtc_state->connectors_changed = true;
@@ -1686,7 +1686,7 @@ static void commit_work(struct work_struct *work)
 }
 
 /**
- * drm_atomic_helper_async_check - check if state can be commited asynchronously
+ * drm_atomic_helper_async_check - check if state can be committed asynchronously
  * @dev: DRM device
  * @state: the driver state object
  *
@@ -1695,7 +1695,7 @@ static void commit_work(struct work_struct *work)
  * but just do in-place changes on the current state.
  *
  * It will return 0 if the commit can happen in an asynchronous fashion or error
- * if not. Note that error just mean it can't be commited asynchronously, if it
+ * if not. Note that error just mean it can't be committed asynchronously, if it
  * fails the commit should be treated like a normal synchronous commit.
  */
 int drm_atomic_helper_async_check(struct drm_device *dev,
@@ -2583,7 +2583,7 @@ EXPORT_SYMBOL(drm_atomic_helper_commit_planes);
  *
  * This function can only be savely used when planes are not allowed to move
  * between different CRTCs because this function doesn't handle inter-CRTC
- * depencies. Callers need to ensure that either no such depencies exist,
+ * dependencies. Callers need to ensure that either no such dependencies exist,
  * resolve them through ordering of commit calls or through some other means.
  */
 void
@@ -2720,7 +2720,7 @@ EXPORT_SYMBOL(drm_atomic_helper_cleanup_planes);
 /**
  * drm_atomic_helper_swap_state - store atomic state into current sw state
  * @state: atomic state
- * @stall: stall for preceeding commits
+ * @stall: stall for preceding commits
  *
  * This function stores the atomic state into the current state pointers in all
  * driver objects. It should be called after all failing steps have been done
diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
index 7e48d40600ff..909f31833181 100644
--- a/drivers/gpu/drm/drm_atomic_uapi.c
+++ b/drivers/gpu/drm/drm_atomic_uapi.c
@@ -48,7 +48,7 @@
  * in all its forms: The monster ATOMIC IOCTL itself, code for GET_PROPERTY and
  * SET_PROPERTY IOCTLs. Plus interface functions for compatibility helpers and
  * drivers which have special needs to construct their own atomic updates, e.g.
- * for load detect or similiar.
+ * for load detect or similar.
  */
 
 /**
@@ -753,7 +753,7 @@ static int drm_atomic_connector_set_property(struct drm_connector *connector,
 		 * restore the state it wants on VT switch. So if the userspace
 		 * tries to change the link_status from GOOD to BAD, driver
 		 * silently rejects it and returns a 0. This prevents userspace
-		 * from accidently breaking  the display when it restores the
+		 * from accidentally breaking  the display when it restores the
 		 * state.
 		 */
 		if (state->link_status != DRM_LINK_STATUS_GOOD)
@@ -1064,7 +1064,7 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
  * DOC: explicit fencing properties
  *
  * Explicit fencing allows userspace to control the buffer synchronization
- * between devices. A Fence or a group of fences are transfered to/from
+ * between devices. A Fence or a group of fences are transferred to/from
  * userspace using Sync File fds and there are two DRM properties for that.
  * IN_FENCE_FD on each DRM Plane to send fences to the kernel and
  * OUT_FENCE_PTR on each DRM CRTC to receive fences from the kernel.
diff --git a/drivers/gpu/drm/drm_auth.c b/drivers/gpu/drm/drm_auth.c
index 3a64a6a79ade..eba438027539 100644
--- a/drivers/gpu/drm/drm_auth.c
+++ b/drivers/gpu/drm/drm_auth.c
@@ -52,7 +52,7 @@
  *
  * In addition only one &drm_master can be the current master for a &drm_device.
  * It can be switched through the DROP_MASTER and SET_MASTER IOCTL, or
- * implicitly through closing/openeing the primary device node. See also
+ * implicitly through closing/opening the primary device node. See also
  * drm_is_current_master().
  *
  * Clients can authenticate against the current master (if it matches their own)
diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c
index 044acd07c153..d4d447a8725a 100644
--- a/drivers/gpu/drm/drm_bridge.c
+++ b/drivers/gpu/drm/drm_bridge.c
@@ -972,7 +972,7 @@ drm_atomic_bridge_propagate_bus_flags(struct drm_bridge *bridge,
 	bridge_state->output_bus_cfg.flags = output_flags;
 
 	/*
-	 * Propage the output flags to the input end of the bridge. Again, it's
+	 * Propagate the output flags to the input end of the bridge. Again, it's
 	 * not necessarily what all bridges want, but that's what most of them
 	 * do, and by doing that by default we avoid forcing drivers to
 	 * duplicate the "dummy propagation" logic.
diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c
index ae8e4d76209c..fcca21e8efac 100644
--- a/drivers/gpu/drm/drm_bufs.c
+++ b/drivers/gpu/drm/drm_bufs.c
@@ -1502,7 +1502,7 @@ int drm_legacy_freebufs(struct drm_device *dev, void *data,
  *
  * Maps the AGP, SG or PCI buffer region with vm_mmap(), and copies information
  * about each buffer into user space. For PCI buffers, it calls vm_mmap() with
- * offset equal to 0, which drm_mmap() interpretes as PCI buffers and calls
+ * offset equal to 0, which drm_mmap() interprets as PCI buffers and calls
  * drm_mmap_dma().
  */
 int __drm_legacy_mapbufs(struct drm_device *dev, void *data, int *p,
diff --git a/drivers/gpu/drm/drm_cache.c b/drivers/gpu/drm/drm_cache.c
index 546599f19a93..30cc59fe6ef7 100644
--- a/drivers/gpu/drm/drm_cache.c
+++ b/drivers/gpu/drm/drm_cache.c
@@ -170,7 +170,7 @@ drm_clflush_virt_range(void *addr, unsigned long length)
 		for (; addr < end; addr += size)
 			clflushopt(addr);
 		clflushopt(end - 1); /* force serialisation */
-		mb(); /*Ensure that evry data cache line entry is flushed*/
+		mb(); /*Ensure that every data cache line entry is flushed*/
 		return;
 	}
 
diff --git a/drivers/gpu/drm/drm_damage_helper.c b/drivers/gpu/drm/drm_damage_helper.c
index 3a4126dc2520..dbc4312160a5 100644
--- a/drivers/gpu/drm/drm_damage_helper.c
+++ b/drivers/gpu/drm/drm_damage_helper.c
@@ -170,7 +170,7 @@ int drm_atomic_helper_dirtyfb(struct drm_framebuffer *fb,
 	int ret = 0;
 
 	/*
-	 * When called from ioctl, we are interruptable, but not when called
+	 * When called from ioctl, we are interruptible, but not when called
 	 * internally (ie. defio worker)
 	 */
 	drm_modeset_acquire_init(&ctx,
diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c
index b5f75ca05774..ed3ed6d3da25 100644
--- a/drivers/gpu/drm/drm_dp_helper.c
+++ b/drivers/gpu/drm/drm_dp_helper.c
@@ -772,7 +772,7 @@ int drm_dp_downstream_max_tmds_clock(const u8 dpcd[DP_RECEIVER_CAP_SIZE],
 		 * It's left up to the driver to check the
 		 * DP dual mode adapter's max TMDS clock.
 		 *
-		 * Unfortunatley it looks like branch devices
+		 * Unfortunately it looks like branch devices
 		 * may not fordward that the DP dual mode i2c
 		 * access so we just usually get i2c nak :(
 		 */
@@ -1365,7 +1365,7 @@ static int drm_dp_i2c_msg_duration(const struct drm_dp_aux_msg *msg,
 }
 
 /*
- * Deterine how many retries should be attempted to successfully transfer
+ * Determine how many retries should be attempted to successfully transfer
  * the specified message, based on the estimated durations of the
  * i2c and AUX transfers.
  */
@@ -1418,7 +1418,7 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg)
 			/*
 			 * While timeouts can be errors, they're usually normal
 			 * behavior (for instance, when a driver tries to
-			 * communicate with a non-existant DisplayPort device).
+			 * communicate with a non-existent DisplayPort device).
 			 * Avoid spamming the kernel log with timeout errors.
 			 */
 			if (ret == -ETIMEDOUT)
@@ -3327,7 +3327,7 @@ drm_edp_backlight_probe_max(struct drm_dp_aux *aux, struct drm_edp_backlight_inf
 	fxp = DIV_ROUND_CLOSEST(1000 * DP_EDP_BACKLIGHT_FREQ_BASE_KHZ, driver_pwm_freq_hz);
 
 	/* Use highest possible value of Pn for more granularity of brightness adjustment while
-	 * satifying the conditions below.
+	 * satisfying the conditions below.
 	 * - Pn is in the range of Pn_min and Pn_max
 	 * - F is in the range of 1 and 255
 	 * - FxP is within 25% of desired value.
diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index 8804ec7d3215..7a5097467ba5 100644
--- a/drivers/gpu/drm/drm_drv.c
+++ b/drivers/gpu/drm/drm_drv.c
@@ -249,7 +249,7 @@ void drm_minor_release(struct drm_minor *minor)
  * Finally when everything is up and running and ready for userspace the device
  * instance can be published using drm_dev_register().
  *
- * There is also deprecated support for initalizing device instances using
+ * There is also deprecated support for initializing device instances using
  * bus-specific helpers and the &drm_driver.load callback. But due to
  * backwards-compatibility needs the device instance have to be published too
  * early, which requires unpretty global locking to make safe and is therefore
@@ -379,7 +379,7 @@ void drm_minor_release(struct drm_minor *minor)
  * shortcoming however, drm_dev_unplug() marks the drm_device as unplugged before
  * drm_atomic_helper_shutdown() is called. This means that if the disable code
  * paths are protected, they will not run on regular driver module unload,
- * possibily leaving the hardware enabled.
+ * possibly leaving the hardware enabled.
  */
 
 /**
diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c
index ff602f7ec65b..46a3c1b62463 100644
--- a/drivers/gpu/drm/drm_dsc.c
+++ b/drivers/gpu/drm/drm_dsc.c
@@ -98,7 +98,7 @@ void drm_dsc_pps_payload_pack(struct drm_dsc_picture_parameter_set *pps_payload,
 {
 	int i;
 
-	/* Protect against someone accidently changing struct size */
+	/* Protect against someone accidentally changing struct size */
 	BUILD_BUG_ON(sizeof(*pps_payload) !=
 		     DP_SDP_PPS_HEADER_PAYLOAD_BYTES_MINUS_1 + 1);
 
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 81d5f2524246..6325877c5fd6 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -1919,7 +1919,7 @@ EXPORT_SYMBOL(drm_add_override_edid_modes);
  * level, drivers must make all reasonable efforts to expose it as an I2C
  * adapter and use drm_get_edid() instead of abusing this function.
  *
- * The EDID may be overridden using debugfs override_edid or firmare EDID
+ * The EDID may be overridden using debugfs override_edid or firmware EDID
  * (drm_load_edid_firmware() and drm.edid_firmware parameter), in this priority
  * order. Having either of them bypasses actual EDID reads.
  *
@@ -5906,7 +5906,7 @@ drm_hdmi_vendor_infoframe_from_display_mode(struct hdmi_vendor_infoframe *frame,
 	 * (ie.vic==0 and s3d_struct==0) we will still send it if we
 	 * know that the sink can handle it. This is based on a
 	 * suggestion in HDMI 2.0 Appendix F. Apparently some sinks
-	 * have trouble realizing that they shuld switch from 3D to 2D
+	 * have trouble realizing that they should switch from 3D to 2D
 	 * mode if the source simply stops sending the infoframe when
 	 * it wants to switch from 3D to 2D.
 	 */
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index d77a24507d30..3ab078321045 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -565,7 +565,7 @@ struct fb_info *drm_fb_helper_alloc_fbi(struct drm_fb_helper *fb_helper)
 		goto err_release;
 
 	/*
-	 * TODO: We really should be smarter here and alloc an apperture
+	 * TODO: We really should be smarter here and alloc an aperture
 	 * for each IORESOURCE_MEM resource helper->dev->dev has and also
 	 * init the ranges of the appertures based on the resources.
 	 * Note some drivers currently count on there being only 1 empty
diff --git a/drivers/gpu/drm/drm_file.c b/drivers/gpu/drm/drm_file.c
index ceb1a9723855..ed25168619fc 100644
--- a/drivers/gpu/drm/drm_file.c
+++ b/drivers/gpu/drm/drm_file.c
@@ -405,7 +405,7 @@ static int drm_open_helper(struct file *filp, struct drm_minor *minor)
  *
  * RETURNS:
  *
- * 0 on success or negative errno value on falure.
+ * 0 on success or negative errno value on failure.
  */
 int drm_open(struct inode *inode, struct file *filp)
 {
@@ -548,7 +548,7 @@ EXPORT_SYMBOL(drm_release_noglobal);
  * @offset: offset to read
  *
  * This function must be used by drivers as their &file_operations.read
- * method iff they use DRM events for asynchronous signalling to userspace.
+ * method if they use DRM events for asynchronous signalling to userspace.
  * Since events are used by the KMS API for vblank and page flip completion this
  * means all modern display drivers must use it.
  *
@@ -641,7 +641,7 @@ EXPORT_SYMBOL(drm_read);
  * @wait: poll waiter table
  *
  * This function must be used by drivers as their &file_operations.read method
- * iff they use DRM events for asynchronous signalling to userspace.  Since
+ * if they use DRM events for asynchronous signalling to userspace.  Since
  * events are used by the KMS API for vblank and page flip completion this means
  * all modern display drivers must use it.
  *
diff --git a/drivers/gpu/drm/drm_format_helper.c b/drivers/gpu/drm/drm_format_helper.c
index 0e885cd34107..5231104b1498 100644
--- a/drivers/gpu/drm/drm_format_helper.c
+++ b/drivers/gpu/drm/drm_format_helper.c
@@ -412,7 +412,7 @@ EXPORT_SYMBOL(drm_fb_blit_rect_dstclip);
  * of the display and the framebuffer mismatch, the copy function will
  * attempt to convert between them.
  *
- * See drm_fb_blit_rect_dstclip() for more inforamtion.
+ * See drm_fb_blit_rect_dstclip() for more information.
  *
  * Returns:
  * 0 on success, or a negative error code otherwise.
diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c
index 4d01464b6f95..a8ad0da225a7 100644
--- a/drivers/gpu/drm/drm_framebuffer.c
+++ b/drivers/gpu/drm/drm_framebuffer.c
@@ -1090,7 +1090,7 @@ void drm_framebuffer_remove(struct drm_framebuffer *fb)
 
 	/*
 	 * drm ABI mandates that we remove any deleted framebuffers from active
-	 * useage. But since most sane clients only remove framebuffers they no
+	 * usage. But since most sane clients only remove framebuffers they no
 	 * longer need, try to optimize this away.
 	 *
 	 * Since we're holding a reference ourselves, observing a refcount of 1
diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
index 68deb1de8235..6c102ba2d9ac 100644
--- a/drivers/gpu/drm/drm_gem.c
+++ b/drivers/gpu/drm/drm_gem.c
@@ -901,7 +901,7 @@ drm_gem_open_ioctl(struct drm_device *dev, void *data,
 }
 
 /**
- * drm_gem_open - initalizes GEM file-private structures at devnode open time
+ * drm_gem_open - initializes GEM file-private structures at devnode open time
  * @dev: drm_device which is being opened by userspace
  * @file_private: drm file-private structure to set up
  *
@@ -936,7 +936,7 @@ drm_gem_release(struct drm_device *dev, struct drm_file *file_private)
  * drm_gem_object_release - release GEM buffer object resources
  * @obj: GEM buffer object
  *
- * This releases any structures and resources used by @obj and is the invers of
+ * This releases any structures and resources used by @obj and is the inverse of
  * drm_gem_object_init().
  */
 void
diff --git a/drivers/gpu/drm/drm_gem_atomic_helper.c b/drivers/gpu/drm/drm_gem_atomic_helper.c
index 26af09b959d4..86b3c676e0b1 100644
--- a/drivers/gpu/drm/drm_gem_atomic_helper.c
+++ b/drivers/gpu/drm/drm_gem_atomic_helper.c
@@ -52,7 +52,7 @@
  *
  * The helpers for shadow-buffered planes establish and release mappings,
  * and provide struct drm_shadow_plane_state, which stores the plane's mapping
- * for commit-tail functons.
+ * for commit-tail functions.
  *
  * Shadow-buffered planes can easily be enabled by using the provided macros
  * %DRM_GEM_SHADOW_PLANE_FUNCS and %DRM_GEM_SHADOW_PLANE_HELPER_FUNCS.
@@ -374,7 +374,7 @@ EXPORT_SYMBOL(drm_gem_prepare_shadow_fb);
  * This function implements struct &drm_plane_helper_funcs.cleanup_fb.
  * This function unmaps all buffer objects of the plane's framebuffer.
  *
- * See drm_gem_prepare_shadow_fb() for more inforamtion.
+ * See drm_gem_prepare_shadow_fb() for more information.
  */
 void drm_gem_cleanup_shadow_fb(struct drm_plane *plane, struct drm_plane_state *plane_state)
 {
diff --git a/drivers/gpu/drm/drm_gem_shmem_helper.c b/drivers/gpu/drm/drm_gem_shmem_helper.c
index d5e6d4568f99..a61946374c82 100644
--- a/drivers/gpu/drm/drm_gem_shmem_helper.c
+++ b/drivers/gpu/drm/drm_gem_shmem_helper.c
@@ -368,7 +368,7 @@ static void drm_gem_shmem_vunmap_locked(struct drm_gem_shmem_object *shmem,
 }
 
 /*
- * drm_gem_shmem_vunmap - Unmap a virtual mapping fo a shmem GEM object
+ * drm_gem_shmem_vunmap - Unmap a virtual mapping for a shmem GEM object
  * @shmem: shmem GEM object
  * @map: Kernel virtual address where the SHMEM GEM object was mapped
  *
diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c b/drivers/gpu/drm/drm_gem_vram_helper.c
index 1e9b82e51a07..43cf7e887d1a 100644
--- a/drivers/gpu/drm/drm_gem_vram_helper.c
+++ b/drivers/gpu/drm/drm_gem_vram_helper.c
@@ -96,7 +96,7 @@ static const struct drm_gem_object_funcs drm_gem_vram_object_funcs;
  * memory region. Call drm_gem_vram_offset() to retrieve this value. Typically
  * it's used to program the hardware's scanout engine for framebuffers, set
  * the cursor overlay's image for a mouse cursor, or use it as input to the
- * hardware's draing engine.
+ * hardware's drawing engine.
  *
  * To access a buffer object's memory from the DRM driver, call
  * drm_gem_vram_vmap(). It maps the buffer into kernel address
diff --git a/drivers/gpu/drm/drm_hdcp.c b/drivers/gpu/drm/drm_hdcp.c
index 910108ccaae1..ca9b8f697202 100644
--- a/drivers/gpu/drm/drm_hdcp.c
+++ b/drivers/gpu/drm/drm_hdcp.c
@@ -280,7 +280,7 @@ static int drm_hdcp_request_srm(struct drm_device *drm_dev,
  * https://www.digital-cp.com/sites/default/files/specifications/HDCP%20on%20HDMI%20Specification%20Rev2_2_Final1.pdf
  *
  * Returns:
- * Count of the revoked KSVs or -ve error number incase of the failure.
+ * Count of the revoked KSVs or -ve error number in case of the failure.
  */
 int drm_hdcp_check_ksvs_revoked(struct drm_device *drm_dev, u8 *ksvs,
 				u32 ksv_count)
diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/drm_ioctl.c
index f454e0424086..be4a52dc4d6f 100644
--- a/drivers/gpu/drm/drm_ioctl.c
+++ b/drivers/gpu/drm/drm_ioctl.c
@@ -426,7 +426,7 @@ static int drm_setversion(struct drm_device *dev, void *data, struct drm_file *f
 }
 
 /**
- * drm_noop - DRM no-op ioctl implemntation
+ * drm_noop - DRM no-op ioctl implementation
  * @dev: DRM device for the ioctl
  * @data: data pointer for the ioctl
  * @file_priv: DRM file for the ioctl call
@@ -446,7 +446,7 @@ int drm_noop(struct drm_device *dev, void *data,
 EXPORT_SYMBOL(drm_noop);
 
 /**
- * drm_invalid_op - DRM invalid ioctl implemntation
+ * drm_invalid_op - DRM invalid ioctl implementation
  * @dev: DRM device for the ioctl
  * @data: data pointer for the ioctl
  * @file_priv: DRM file for the ioctl call
diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
index 201eae4bba6c..4a853011549a 100644
--- a/drivers/gpu/drm/drm_irq.c
+++ b/drivers/gpu/drm/drm_irq.c
@@ -72,7 +72,7 @@
  * The DRM core provides very simple support helpers to enable IRQ handling on a
  * device through the drm_irq_install() and drm_irq_uninstall() functions. This
  * only supports devices with a single interrupt on the main device stored in
- * &drm_device.dev and set as the device paramter in drm_dev_alloc().
+ * &drm_device.dev and set as the device parameter in drm_dev_alloc().
  *
  * These IRQ helpers are strictly optional. Since these helpers don't automatically
  * clean up the requested interrupt like e.g. devm_request_irq() they're not really
diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c
index a4a04d246135..93d48a6f04ab 100644
--- a/drivers/gpu/drm/drm_mm.c
+++ b/drivers/gpu/drm/drm_mm.c
@@ -700,7 +700,7 @@ EXPORT_SYMBOL(drm_mm_replace_node);
  * interfaces. First a scan operation needs to be initialized with
  * drm_mm_scan_init() or drm_mm_scan_init_with_range(). The driver adds
  * objects to the roster, probably by walking an LRU list, but this can be
- * freely implemented. Eviction candiates are added using
+ * freely implemented. Eviction candidates are added using
  * drm_mm_scan_add_block() until a suitable hole is found or there are no
  * further evictable objects. Eviction roster metadata is tracked in &struct
  * drm_mm_scan.
diff --git a/drivers/gpu/drm/drm_mode_object.c b/drivers/gpu/drm/drm_mode_object.c
index b26588b52795..86d9e907c0b2 100644
--- a/drivers/gpu/drm/drm_mode_object.c
+++ b/drivers/gpu/drm/drm_mode_object.c
@@ -91,7 +91,7 @@ void drm_mode_object_register(struct drm_device *dev,
 }
 
 /**
- * drm_mode_object_unregister - free a modeset identifer
+ * drm_mode_object_unregister - free a modeset identifier
  * @dev: DRM device
  * @object: object to free
  *
diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c
index ae53ea624c73..1c72208d8133 100644
--- a/drivers/gpu/drm/drm_modes.c
+++ b/drivers/gpu/drm/drm_modes.c
@@ -1542,7 +1542,7 @@ static int drm_mode_parse_cmdline_int(const char *delim, unsigned int *int_ret)
 
 	/*
 	 * delim must point to the '=', otherwise it is a syntax error and
-	 * if delim points to the terminating zero, then delim + 1 wil point
+	 * if delim points to the terminating zero, then delim + 1 will point
 	 * past the end of the string.
 	 */
 	if (*delim != '=')
@@ -1972,7 +1972,7 @@ int drm_mode_convert_umode(struct drm_device *dev,
 	out->flags = in->flags;
 	/*
 	 * Old xf86-video-vmware (possibly others too) used to
-	 * leave 'type' unititialized. Just ignore any bits we
+	 * leave 'type' uninitialized. Just ignore any bits we
 	 * don't like. It's a just hint after all, and more
 	 * useful for the kernel->userspace direction anyway.
 	 */
diff --git a/drivers/gpu/drm/drm_plane.c b/drivers/gpu/drm/drm_plane.c
index b373958ecb30..98ca1801253b 100644
--- a/drivers/gpu/drm/drm_plane.c
+++ b/drivers/gpu/drm/drm_plane.c
@@ -38,7 +38,7 @@
 /**
  * DOC: overview
  *
- * A plane represents an image source that can be blended with or overlayed on
+ * A plane represents an image source that can be blended with or overlaid on
  * top of a CRTC during the scanout process. Planes take their input data from a
  * &drm_framebuffer object. The plane itself specifies the cropping and scaling
  * of that image, and where it is placed on the visible area of a display
diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c
index 3aae7ea522f2..5b2d0ca03705 100644
--- a/drivers/gpu/drm/drm_plane_helper.c
+++ b/drivers/gpu/drm/drm_plane_helper.c
@@ -210,7 +210,7 @@ static int drm_primary_helper_update(struct drm_plane *plane, struct drm_crtc *c
 	 * We call set_config() directly here rather than using
 	 * drm_mode_set_config_internal.  We're reprogramming the same
 	 * connectors that were already in use, so we shouldn't need the extra
-	 * cross-CRTC fb refcounting to accomodate stealing connectors.
+	 * cross-CRTC fb refcounting to accommodate stealing connectors.
 	 * drm_mode_setplane() already handles the basic refcounting for the
 	 * framebuffers involved in this operation.
 	 */
diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
index 2a54f86856af..3085858272c6 100644
--- a/drivers/gpu/drm/drm_prime.c
+++ b/drivers/gpu/drm/drm_prime.c
@@ -73,7 +73,7 @@
  * Thus the chain of references always flows in one direction, avoiding loops:
  * importing GEM object -> dma-buf -> exported GEM bo. A further complication
  * are the lookup caches for import and export. These are required to guarantee
- * that any given object will always have only one uniqe userspace handle. This
+ * that any given object will always have only one unique userspace handle. This
  * is required to allow userspace to detect duplicated imports, since some GEM
  * drivers do fail command submissions if a given buffer object is listed more
  * than once. These import and export caches in &drm_prime_file_private only
diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c
index e7e1ee2aa352..5606bca3caa8 100644
--- a/drivers/gpu/drm/drm_probe_helper.c
+++ b/drivers/gpu/drm/drm_probe_helper.c
@@ -757,7 +757,7 @@ EXPORT_SYMBOL(drm_kms_helper_poll_disable);
  * drm_kms_helper_poll_init - initialize and enable output polling
  * @dev: drm_device
  *
- * This function intializes and then also enables output polling support for
+ * This function initializes and then also enables output polling support for
  * @dev. Drivers which do not have reliable hotplug support in hardware can use
  * this helper infrastructure to regularly poll such connectors for changes in
  * their connection state.
diff --git a/drivers/gpu/drm/drm_property.c b/drivers/gpu/drm/drm_property.c
index 27c824a6eb60..6c353c9dc772 100644
--- a/drivers/gpu/drm/drm_property.c
+++ b/drivers/gpu/drm/drm_property.c
@@ -434,7 +434,7 @@ EXPORT_SYMBOL(drm_property_add_enum);
 /**
  * drm_property_destroy - destroy a drm property
  * @dev: drm device
- * @property: property to destry
+ * @property: property to destroy
  *
  * This function frees a property including any attached resources like
  * enumeration values.
diff --git a/drivers/gpu/drm/drm_scdc_helper.c b/drivers/gpu/drm/drm_scdc_helper.c
index 991b8c86d78d..48a382464d54 100644
--- a/drivers/gpu/drm/drm_scdc_helper.c
+++ b/drivers/gpu/drm/drm_scdc_helper.c
@@ -241,7 +241,7 @@ bool drm_scdc_set_high_tmds_clock_ratio(struct i2c_adapter *adapter, bool set)
 	/*
 	 * The spec says that a source should wait minimum 1ms and maximum
 	 * 100ms after writing the TMDS config for clock ratio. Lets allow a
-	 * wait of upto 2ms here.
+	 * wait of up to 2ms here.
 	 */
 	usleep_range(1000, 2000);
 	return true;
diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
index 1c5b9ef6da37..c9a9d74f338c 100644
--- a/drivers/gpu/drm/drm_syncobj.c
+++ b/drivers/gpu/drm/drm_syncobj.c
@@ -725,7 +725,7 @@ static int drm_syncobj_export_sync_file(struct drm_file *file_private,
 	return ret;
 }
 /**
- * drm_syncobj_open - initalizes syncobj file-private structures at devnode open time
+ * drm_syncobj_open - initializes syncobj file-private structures at devnode open time
  * @file_private: drm file-private structure to set up
  *
  * Called at device open time, sets up the structure for handling refcounting
diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c
index bba6781cc48f..977b6509bb4b 100644
--- a/drivers/gpu/drm/drm_vblank.c
+++ b/drivers/gpu/drm/drm_vblank.c
@@ -191,7 +191,7 @@ static u32 drm_max_vblank_count(struct drm_device *dev, unsigned int pipe)
 
 /*
  * "No hw counter" fallback implementation of .get_vblank_counter() hook,
- * if there is no useable hardware frame counter available.
+ * if there is no usable hardware frame counter available.
  */
 static u32 drm_vblank_no_hw_counter(struct drm_device *dev, unsigned int pipe)
 {
@@ -905,7 +905,7 @@ drm_get_last_vbltimestamp(struct drm_device *dev, unsigned int pipe,
  * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
  * provide a barrier: Any writes done before calling
  * drm_crtc_handle_vblank() will be visible to callers of the later
- * functions, iff the vblank count is the same or a later one.
+ * functions, if the vblank count is the same or a later one.
  *
  * See also &drm_vblank_crtc.count.
  *
@@ -968,7 +968,7 @@ static u64 drm_vblank_count_and_time(struct drm_device *dev, unsigned int pipe,
  * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
  * provide a barrier: Any writes done before calling
  * drm_crtc_handle_vblank() will be visible to callers of the later
- * functions, iff the vblank count is the same or a later one.
+ * functions, if the vblank count is the same or a later one.
  *
  * See also &drm_vblank_crtc.count.
  */
@@ -1997,7 +1997,7 @@ EXPORT_SYMBOL(drm_handle_vblank);
  * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
  * provide a barrier: Any writes done before calling
  * drm_crtc_handle_vblank() will be visible to callers of the later
- * functions, iff the vblank count is the same or a later one.
+ * functions, if the vblank count is the same or a later one.
  *
  * See also &drm_vblank_crtc.count.
  *
@@ -2014,7 +2014,7 @@ EXPORT_SYMBOL(drm_crtc_handle_vblank);
  * Get crtc VBLANK count.
  *
  * \param dev DRM device
- * \param data user arguement, pointing to a drm_crtc_get_sequence structure.
+ * \param data user argument, pointing to a drm_crtc_get_sequence structure.
  * \param file_priv drm file private for the user's open file descriptor
  */
 
@@ -2070,7 +2070,7 @@ int drm_crtc_get_sequence_ioctl(struct drm_device *dev, void *data,
  * Queue a event for VBLANK sequence
  *
  * \param dev DRM device
- * \param data user arguement, pointing to a drm_crtc_queue_sequence structure.
+ * \param data user argument, pointing to a drm_crtc_queue_sequence structure.
  * \param file_priv drm file private for the user's open file descriptor
  */
 
diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c
index 4565319fa6b3..7de37f8c68fd 100644
--- a/drivers/gpu/drm/drm_vma_manager.c
+++ b/drivers/gpu/drm/drm_vma_manager.c
@@ -361,7 +361,7 @@ EXPORT_SYMBOL(drm_vma_node_revoke);
  * This is locked against concurrent access internally.
  *
  * RETURNS:
- * true iff @filp is on the list
+ * true if @filp is on the list
  */
 bool drm_vma_node_is_allowed(struct drm_vma_offset_node *node,
 			     struct drm_file *tag)
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] drm: Fix typo in comments
  2021-07-30 13:27 [PATCH v2] drm: Fix typo in comments Cai Huoqing
@ 2021-08-02  8:19 ` Daniel Vetter
  2021-08-02  8:34   ` Cai,Huoqing
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Vetter @ 2021-08-02  8:19 UTC (permalink / raw)
  To: Cai Huoqing
  Cc: maarten.lankhorst, mripard, tzimmermann, airlied, daniel, dri-devel

On Fri, Jul 30, 2021 at 09:27:29PM +0800, Cai Huoqing wrote:
> fix typo for drm
> 
> v1->v2:
> respin with the change "iff ==> implies that"
> 
> Reviewed-by: Daniel Vetter <daniel@ffwll.ch>

I did not give you a Reviewed-by: tag, please dont forge them. At least
in the linux kernel an r-b tag is a fairly formal statement:

https://dri.freedesktop.org/docs/drm/process/submitting-patches.html?highlight=statement%20oversight#reviewer-s-statement-of-oversight

The more informal Acked-by is generally ok to add if you get something
like "looks good to me", but better to ask for confirmation too in that
case.

I've removed it before pushing.

> Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>

Pushed to drm-misc-next, thanks for your patch.
-Daniel

> ---
>  drivers/gpu/drm/drm_aperture.c          |  2 +-
>  drivers/gpu/drm/drm_atomic.c            |  2 +-
>  drivers/gpu/drm/drm_atomic_helper.c     | 10 +++++-----
>  drivers/gpu/drm/drm_atomic_uapi.c       |  6 +++---
>  drivers/gpu/drm/drm_auth.c              |  2 +-
>  drivers/gpu/drm/drm_bridge.c            |  2 +-
>  drivers/gpu/drm/drm_bufs.c              |  2 +-
>  drivers/gpu/drm/drm_cache.c             |  2 +-
>  drivers/gpu/drm/drm_damage_helper.c     |  2 +-
>  drivers/gpu/drm/drm_dp_helper.c         |  8 ++++----
>  drivers/gpu/drm/drm_drv.c               |  4 ++--
>  drivers/gpu/drm/drm_dsc.c               |  2 +-
>  drivers/gpu/drm/drm_edid.c              |  4 ++--
>  drivers/gpu/drm/drm_fb_helper.c         |  2 +-
>  drivers/gpu/drm/drm_file.c              |  6 +++---
>  drivers/gpu/drm/drm_format_helper.c     |  2 +-
>  drivers/gpu/drm/drm_framebuffer.c       |  2 +-
>  drivers/gpu/drm/drm_gem.c               |  4 ++--
>  drivers/gpu/drm/drm_gem_atomic_helper.c |  4 ++--
>  drivers/gpu/drm/drm_gem_shmem_helper.c  |  2 +-
>  drivers/gpu/drm/drm_gem_vram_helper.c   |  2 +-
>  drivers/gpu/drm/drm_hdcp.c              |  2 +-
>  drivers/gpu/drm/drm_ioctl.c             |  4 ++--
>  drivers/gpu/drm/drm_irq.c               |  2 +-
>  drivers/gpu/drm/drm_mm.c                |  2 +-
>  drivers/gpu/drm/drm_mode_object.c       |  2 +-
>  drivers/gpu/drm/drm_modes.c             |  4 ++--
>  drivers/gpu/drm/drm_plane.c             |  2 +-
>  drivers/gpu/drm/drm_plane_helper.c      |  2 +-
>  drivers/gpu/drm/drm_prime.c             |  2 +-
>  drivers/gpu/drm/drm_probe_helper.c      |  2 +-
>  drivers/gpu/drm/drm_property.c          |  2 +-
>  drivers/gpu/drm/drm_scdc_helper.c       |  2 +-
>  drivers/gpu/drm/drm_syncobj.c           |  2 +-
>  drivers/gpu/drm/drm_vblank.c            | 12 ++++++------
>  drivers/gpu/drm/drm_vma_manager.c       |  2 +-
>  36 files changed, 58 insertions(+), 58 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_aperture.c b/drivers/gpu/drm/drm_aperture.c
> index 9ac39cf11694..74bd4a76b253 100644
> --- a/drivers/gpu/drm/drm_aperture.c
> +++ b/drivers/gpu/drm/drm_aperture.c
> @@ -78,7 +78,7 @@
>   *
>   * Drivers that are susceptible to being removed by other drivers, such as
>   * generic EFI or VESA drivers, have to register themselves as owners of their
> - * given framebuffer memory. Ownership of the framebuffer memory is achived
> + * given framebuffer memory. Ownership of the framebuffer memory is achieved
>   * by calling devm_aperture_acquire_from_firmware(). On success, the driver
>   * is the owner of the framebuffer range. The function fails if the
>   * framebuffer is already by another driver. See below for an example.
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index d820423fac32..b127e30082ba 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -723,7 +723,7 @@ static void drm_atomic_plane_print_state(struct drm_printer *p,
>   * clocks, scaler units, bandwidth and fifo limits shared among a group of
>   * planes or CRTCs, and so on) it makes sense to model these as independent
>   * objects. Drivers then need to do similar state tracking and commit ordering for
> - * such private (since not exposed to userpace) objects as the atomic core and
> + * such private (since not exposed to userspace) objects as the atomic core and
>   * helpers already provide for connectors, planes and CRTCs.
>   *
>   * To make this easier on drivers the atomic core provides some support to track
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index f7bf1ea62d58..7ee480b6efde 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -634,7 +634,7 @@ drm_atomic_helper_check_modeset(struct drm_device *dev,
>  			 * connectors and a NULL mode.
>  			 *
>  			 * The other way around is true as well. enable != 0
> -			 * iff connectors are attached and a mode is set.
> +			 * implies that connectors are attached and a mode is set.
>  			 */
>  			new_crtc_state->mode_changed = true;
>  			new_crtc_state->connectors_changed = true;
> @@ -1686,7 +1686,7 @@ static void commit_work(struct work_struct *work)
>  }
>  
>  /**
> - * drm_atomic_helper_async_check - check if state can be commited asynchronously
> + * drm_atomic_helper_async_check - check if state can be committed asynchronously
>   * @dev: DRM device
>   * @state: the driver state object
>   *
> @@ -1695,7 +1695,7 @@ static void commit_work(struct work_struct *work)
>   * but just do in-place changes on the current state.
>   *
>   * It will return 0 if the commit can happen in an asynchronous fashion or error
> - * if not. Note that error just mean it can't be commited asynchronously, if it
> + * if not. Note that error just mean it can't be committed asynchronously, if it
>   * fails the commit should be treated like a normal synchronous commit.
>   */
>  int drm_atomic_helper_async_check(struct drm_device *dev,
> @@ -2583,7 +2583,7 @@ EXPORT_SYMBOL(drm_atomic_helper_commit_planes);
>   *
>   * This function can only be savely used when planes are not allowed to move
>   * between different CRTCs because this function doesn't handle inter-CRTC
> - * depencies. Callers need to ensure that either no such depencies exist,
> + * dependencies. Callers need to ensure that either no such dependencies exist,
>   * resolve them through ordering of commit calls or through some other means.
>   */
>  void
> @@ -2720,7 +2720,7 @@ EXPORT_SYMBOL(drm_atomic_helper_cleanup_planes);
>  /**
>   * drm_atomic_helper_swap_state - store atomic state into current sw state
>   * @state: atomic state
> - * @stall: stall for preceeding commits
> + * @stall: stall for preceding commits
>   *
>   * This function stores the atomic state into the current state pointers in all
>   * driver objects. It should be called after all failing steps have been done
> diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
> index 7e48d40600ff..909f31833181 100644
> --- a/drivers/gpu/drm/drm_atomic_uapi.c
> +++ b/drivers/gpu/drm/drm_atomic_uapi.c
> @@ -48,7 +48,7 @@
>   * in all its forms: The monster ATOMIC IOCTL itself, code for GET_PROPERTY and
>   * SET_PROPERTY IOCTLs. Plus interface functions for compatibility helpers and
>   * drivers which have special needs to construct their own atomic updates, e.g.
> - * for load detect or similiar.
> + * for load detect or similar.
>   */
>  
>  /**
> @@ -753,7 +753,7 @@ static int drm_atomic_connector_set_property(struct drm_connector *connector,
>  		 * restore the state it wants on VT switch. So if the userspace
>  		 * tries to change the link_status from GOOD to BAD, driver
>  		 * silently rejects it and returns a 0. This prevents userspace
> -		 * from accidently breaking  the display when it restores the
> +		 * from accidentally breaking  the display when it restores the
>  		 * state.
>  		 */
>  		if (state->link_status != DRM_LINK_STATUS_GOOD)
> @@ -1064,7 +1064,7 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
>   * DOC: explicit fencing properties
>   *
>   * Explicit fencing allows userspace to control the buffer synchronization
> - * between devices. A Fence or a group of fences are transfered to/from
> + * between devices. A Fence or a group of fences are transferred to/from
>   * userspace using Sync File fds and there are two DRM properties for that.
>   * IN_FENCE_FD on each DRM Plane to send fences to the kernel and
>   * OUT_FENCE_PTR on each DRM CRTC to receive fences from the kernel.
> diff --git a/drivers/gpu/drm/drm_auth.c b/drivers/gpu/drm/drm_auth.c
> index 3a64a6a79ade..eba438027539 100644
> --- a/drivers/gpu/drm/drm_auth.c
> +++ b/drivers/gpu/drm/drm_auth.c
> @@ -52,7 +52,7 @@
>   *
>   * In addition only one &drm_master can be the current master for a &drm_device.
>   * It can be switched through the DROP_MASTER and SET_MASTER IOCTL, or
> - * implicitly through closing/openeing the primary device node. See also
> + * implicitly through closing/opening the primary device node. See also
>   * drm_is_current_master().
>   *
>   * Clients can authenticate against the current master (if it matches their own)
> diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c
> index 044acd07c153..d4d447a8725a 100644
> --- a/drivers/gpu/drm/drm_bridge.c
> +++ b/drivers/gpu/drm/drm_bridge.c
> @@ -972,7 +972,7 @@ drm_atomic_bridge_propagate_bus_flags(struct drm_bridge *bridge,
>  	bridge_state->output_bus_cfg.flags = output_flags;
>  
>  	/*
> -	 * Propage the output flags to the input end of the bridge. Again, it's
> +	 * Propagate the output flags to the input end of the bridge. Again, it's
>  	 * not necessarily what all bridges want, but that's what most of them
>  	 * do, and by doing that by default we avoid forcing drivers to
>  	 * duplicate the "dummy propagation" logic.
> diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c
> index ae8e4d76209c..fcca21e8efac 100644
> --- a/drivers/gpu/drm/drm_bufs.c
> +++ b/drivers/gpu/drm/drm_bufs.c
> @@ -1502,7 +1502,7 @@ int drm_legacy_freebufs(struct drm_device *dev, void *data,
>   *
>   * Maps the AGP, SG or PCI buffer region with vm_mmap(), and copies information
>   * about each buffer into user space. For PCI buffers, it calls vm_mmap() with
> - * offset equal to 0, which drm_mmap() interpretes as PCI buffers and calls
> + * offset equal to 0, which drm_mmap() interprets as PCI buffers and calls
>   * drm_mmap_dma().
>   */
>  int __drm_legacy_mapbufs(struct drm_device *dev, void *data, int *p,
> diff --git a/drivers/gpu/drm/drm_cache.c b/drivers/gpu/drm/drm_cache.c
> index 546599f19a93..30cc59fe6ef7 100644
> --- a/drivers/gpu/drm/drm_cache.c
> +++ b/drivers/gpu/drm/drm_cache.c
> @@ -170,7 +170,7 @@ drm_clflush_virt_range(void *addr, unsigned long length)
>  		for (; addr < end; addr += size)
>  			clflushopt(addr);
>  		clflushopt(end - 1); /* force serialisation */
> -		mb(); /*Ensure that evry data cache line entry is flushed*/
> +		mb(); /*Ensure that every data cache line entry is flushed*/
>  		return;
>  	}
>  
> diff --git a/drivers/gpu/drm/drm_damage_helper.c b/drivers/gpu/drm/drm_damage_helper.c
> index 3a4126dc2520..dbc4312160a5 100644
> --- a/drivers/gpu/drm/drm_damage_helper.c
> +++ b/drivers/gpu/drm/drm_damage_helper.c
> @@ -170,7 +170,7 @@ int drm_atomic_helper_dirtyfb(struct drm_framebuffer *fb,
>  	int ret = 0;
>  
>  	/*
> -	 * When called from ioctl, we are interruptable, but not when called
> +	 * When called from ioctl, we are interruptible, but not when called
>  	 * internally (ie. defio worker)
>  	 */
>  	drm_modeset_acquire_init(&ctx,
> diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c
> index b5f75ca05774..ed3ed6d3da25 100644
> --- a/drivers/gpu/drm/drm_dp_helper.c
> +++ b/drivers/gpu/drm/drm_dp_helper.c
> @@ -772,7 +772,7 @@ int drm_dp_downstream_max_tmds_clock(const u8 dpcd[DP_RECEIVER_CAP_SIZE],
>  		 * It's left up to the driver to check the
>  		 * DP dual mode adapter's max TMDS clock.
>  		 *
> -		 * Unfortunatley it looks like branch devices
> +		 * Unfortunately it looks like branch devices
>  		 * may not fordward that the DP dual mode i2c
>  		 * access so we just usually get i2c nak :(
>  		 */
> @@ -1365,7 +1365,7 @@ static int drm_dp_i2c_msg_duration(const struct drm_dp_aux_msg *msg,
>  }
>  
>  /*
> - * Deterine how many retries should be attempted to successfully transfer
> + * Determine how many retries should be attempted to successfully transfer
>   * the specified message, based on the estimated durations of the
>   * i2c and AUX transfers.
>   */
> @@ -1418,7 +1418,7 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg)
>  			/*
>  			 * While timeouts can be errors, they're usually normal
>  			 * behavior (for instance, when a driver tries to
> -			 * communicate with a non-existant DisplayPort device).
> +			 * communicate with a non-existent DisplayPort device).
>  			 * Avoid spamming the kernel log with timeout errors.
>  			 */
>  			if (ret == -ETIMEDOUT)
> @@ -3327,7 +3327,7 @@ drm_edp_backlight_probe_max(struct drm_dp_aux *aux, struct drm_edp_backlight_inf
>  	fxp = DIV_ROUND_CLOSEST(1000 * DP_EDP_BACKLIGHT_FREQ_BASE_KHZ, driver_pwm_freq_hz);
>  
>  	/* Use highest possible value of Pn for more granularity of brightness adjustment while
> -	 * satifying the conditions below.
> +	 * satisfying the conditions below.
>  	 * - Pn is in the range of Pn_min and Pn_max
>  	 * - F is in the range of 1 and 255
>  	 * - FxP is within 25% of desired value.
> diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> index 8804ec7d3215..7a5097467ba5 100644
> --- a/drivers/gpu/drm/drm_drv.c
> +++ b/drivers/gpu/drm/drm_drv.c
> @@ -249,7 +249,7 @@ void drm_minor_release(struct drm_minor *minor)
>   * Finally when everything is up and running and ready for userspace the device
>   * instance can be published using drm_dev_register().
>   *
> - * There is also deprecated support for initalizing device instances using
> + * There is also deprecated support for initializing device instances using
>   * bus-specific helpers and the &drm_driver.load callback. But due to
>   * backwards-compatibility needs the device instance have to be published too
>   * early, which requires unpretty global locking to make safe and is therefore
> @@ -379,7 +379,7 @@ void drm_minor_release(struct drm_minor *minor)
>   * shortcoming however, drm_dev_unplug() marks the drm_device as unplugged before
>   * drm_atomic_helper_shutdown() is called. This means that if the disable code
>   * paths are protected, they will not run on regular driver module unload,
> - * possibily leaving the hardware enabled.
> + * possibly leaving the hardware enabled.
>   */
>  
>  /**
> diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c
> index ff602f7ec65b..46a3c1b62463 100644
> --- a/drivers/gpu/drm/drm_dsc.c
> +++ b/drivers/gpu/drm/drm_dsc.c
> @@ -98,7 +98,7 @@ void drm_dsc_pps_payload_pack(struct drm_dsc_picture_parameter_set *pps_payload,
>  {
>  	int i;
>  
> -	/* Protect against someone accidently changing struct size */
> +	/* Protect against someone accidentally changing struct size */
>  	BUILD_BUG_ON(sizeof(*pps_payload) !=
>  		     DP_SDP_PPS_HEADER_PAYLOAD_BYTES_MINUS_1 + 1);
>  
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 81d5f2524246..6325877c5fd6 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -1919,7 +1919,7 @@ EXPORT_SYMBOL(drm_add_override_edid_modes);
>   * level, drivers must make all reasonable efforts to expose it as an I2C
>   * adapter and use drm_get_edid() instead of abusing this function.
>   *
> - * The EDID may be overridden using debugfs override_edid or firmare EDID
> + * The EDID may be overridden using debugfs override_edid or firmware EDID
>   * (drm_load_edid_firmware() and drm.edid_firmware parameter), in this priority
>   * order. Having either of them bypasses actual EDID reads.
>   *
> @@ -5906,7 +5906,7 @@ drm_hdmi_vendor_infoframe_from_display_mode(struct hdmi_vendor_infoframe *frame,
>  	 * (ie.vic==0 and s3d_struct==0) we will still send it if we
>  	 * know that the sink can handle it. This is based on a
>  	 * suggestion in HDMI 2.0 Appendix F. Apparently some sinks
> -	 * have trouble realizing that they shuld switch from 3D to 2D
> +	 * have trouble realizing that they should switch from 3D to 2D
>  	 * mode if the source simply stops sending the infoframe when
>  	 * it wants to switch from 3D to 2D.
>  	 */
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index d77a24507d30..3ab078321045 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -565,7 +565,7 @@ struct fb_info *drm_fb_helper_alloc_fbi(struct drm_fb_helper *fb_helper)
>  		goto err_release;
>  
>  	/*
> -	 * TODO: We really should be smarter here and alloc an apperture
> +	 * TODO: We really should be smarter here and alloc an aperture
>  	 * for each IORESOURCE_MEM resource helper->dev->dev has and also
>  	 * init the ranges of the appertures based on the resources.
>  	 * Note some drivers currently count on there being only 1 empty
> diff --git a/drivers/gpu/drm/drm_file.c b/drivers/gpu/drm/drm_file.c
> index ceb1a9723855..ed25168619fc 100644
> --- a/drivers/gpu/drm/drm_file.c
> +++ b/drivers/gpu/drm/drm_file.c
> @@ -405,7 +405,7 @@ static int drm_open_helper(struct file *filp, struct drm_minor *minor)
>   *
>   * RETURNS:
>   *
> - * 0 on success or negative errno value on falure.
> + * 0 on success or negative errno value on failure.
>   */
>  int drm_open(struct inode *inode, struct file *filp)
>  {
> @@ -548,7 +548,7 @@ EXPORT_SYMBOL(drm_release_noglobal);
>   * @offset: offset to read
>   *
>   * This function must be used by drivers as their &file_operations.read
> - * method iff they use DRM events for asynchronous signalling to userspace.
> + * method if they use DRM events for asynchronous signalling to userspace.
>   * Since events are used by the KMS API for vblank and page flip completion this
>   * means all modern display drivers must use it.
>   *
> @@ -641,7 +641,7 @@ EXPORT_SYMBOL(drm_read);
>   * @wait: poll waiter table
>   *
>   * This function must be used by drivers as their &file_operations.read method
> - * iff they use DRM events for asynchronous signalling to userspace.  Since
> + * if they use DRM events for asynchronous signalling to userspace.  Since
>   * events are used by the KMS API for vblank and page flip completion this means
>   * all modern display drivers must use it.
>   *
> diff --git a/drivers/gpu/drm/drm_format_helper.c b/drivers/gpu/drm/drm_format_helper.c
> index 0e885cd34107..5231104b1498 100644
> --- a/drivers/gpu/drm/drm_format_helper.c
> +++ b/drivers/gpu/drm/drm_format_helper.c
> @@ -412,7 +412,7 @@ EXPORT_SYMBOL(drm_fb_blit_rect_dstclip);
>   * of the display and the framebuffer mismatch, the copy function will
>   * attempt to convert between them.
>   *
> - * See drm_fb_blit_rect_dstclip() for more inforamtion.
> + * See drm_fb_blit_rect_dstclip() for more information.
>   *
>   * Returns:
>   * 0 on success, or a negative error code otherwise.
> diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c
> index 4d01464b6f95..a8ad0da225a7 100644
> --- a/drivers/gpu/drm/drm_framebuffer.c
> +++ b/drivers/gpu/drm/drm_framebuffer.c
> @@ -1090,7 +1090,7 @@ void drm_framebuffer_remove(struct drm_framebuffer *fb)
>  
>  	/*
>  	 * drm ABI mandates that we remove any deleted framebuffers from active
> -	 * useage. But since most sane clients only remove framebuffers they no
> +	 * usage. But since most sane clients only remove framebuffers they no
>  	 * longer need, try to optimize this away.
>  	 *
>  	 * Since we're holding a reference ourselves, observing a refcount of 1
> diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
> index 68deb1de8235..6c102ba2d9ac 100644
> --- a/drivers/gpu/drm/drm_gem.c
> +++ b/drivers/gpu/drm/drm_gem.c
> @@ -901,7 +901,7 @@ drm_gem_open_ioctl(struct drm_device *dev, void *data,
>  }
>  
>  /**
> - * drm_gem_open - initalizes GEM file-private structures at devnode open time
> + * drm_gem_open - initializes GEM file-private structures at devnode open time
>   * @dev: drm_device which is being opened by userspace
>   * @file_private: drm file-private structure to set up
>   *
> @@ -936,7 +936,7 @@ drm_gem_release(struct drm_device *dev, struct drm_file *file_private)
>   * drm_gem_object_release - release GEM buffer object resources
>   * @obj: GEM buffer object
>   *
> - * This releases any structures and resources used by @obj and is the invers of
> + * This releases any structures and resources used by @obj and is the inverse of
>   * drm_gem_object_init().
>   */
>  void
> diff --git a/drivers/gpu/drm/drm_gem_atomic_helper.c b/drivers/gpu/drm/drm_gem_atomic_helper.c
> index 26af09b959d4..86b3c676e0b1 100644
> --- a/drivers/gpu/drm/drm_gem_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_gem_atomic_helper.c
> @@ -52,7 +52,7 @@
>   *
>   * The helpers for shadow-buffered planes establish and release mappings,
>   * and provide struct drm_shadow_plane_state, which stores the plane's mapping
> - * for commit-tail functons.
> + * for commit-tail functions.
>   *
>   * Shadow-buffered planes can easily be enabled by using the provided macros
>   * %DRM_GEM_SHADOW_PLANE_FUNCS and %DRM_GEM_SHADOW_PLANE_HELPER_FUNCS.
> @@ -374,7 +374,7 @@ EXPORT_SYMBOL(drm_gem_prepare_shadow_fb);
>   * This function implements struct &drm_plane_helper_funcs.cleanup_fb.
>   * This function unmaps all buffer objects of the plane's framebuffer.
>   *
> - * See drm_gem_prepare_shadow_fb() for more inforamtion.
> + * See drm_gem_prepare_shadow_fb() for more information.
>   */
>  void drm_gem_cleanup_shadow_fb(struct drm_plane *plane, struct drm_plane_state *plane_state)
>  {
> diff --git a/drivers/gpu/drm/drm_gem_shmem_helper.c b/drivers/gpu/drm/drm_gem_shmem_helper.c
> index d5e6d4568f99..a61946374c82 100644
> --- a/drivers/gpu/drm/drm_gem_shmem_helper.c
> +++ b/drivers/gpu/drm/drm_gem_shmem_helper.c
> @@ -368,7 +368,7 @@ static void drm_gem_shmem_vunmap_locked(struct drm_gem_shmem_object *shmem,
>  }
>  
>  /*
> - * drm_gem_shmem_vunmap - Unmap a virtual mapping fo a shmem GEM object
> + * drm_gem_shmem_vunmap - Unmap a virtual mapping for a shmem GEM object
>   * @shmem: shmem GEM object
>   * @map: Kernel virtual address where the SHMEM GEM object was mapped
>   *
> diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c b/drivers/gpu/drm/drm_gem_vram_helper.c
> index 1e9b82e51a07..43cf7e887d1a 100644
> --- a/drivers/gpu/drm/drm_gem_vram_helper.c
> +++ b/drivers/gpu/drm/drm_gem_vram_helper.c
> @@ -96,7 +96,7 @@ static const struct drm_gem_object_funcs drm_gem_vram_object_funcs;
>   * memory region. Call drm_gem_vram_offset() to retrieve this value. Typically
>   * it's used to program the hardware's scanout engine for framebuffers, set
>   * the cursor overlay's image for a mouse cursor, or use it as input to the
> - * hardware's draing engine.
> + * hardware's drawing engine.
>   *
>   * To access a buffer object's memory from the DRM driver, call
>   * drm_gem_vram_vmap(). It maps the buffer into kernel address
> diff --git a/drivers/gpu/drm/drm_hdcp.c b/drivers/gpu/drm/drm_hdcp.c
> index 910108ccaae1..ca9b8f697202 100644
> --- a/drivers/gpu/drm/drm_hdcp.c
> +++ b/drivers/gpu/drm/drm_hdcp.c
> @@ -280,7 +280,7 @@ static int drm_hdcp_request_srm(struct drm_device *drm_dev,
>   * https://www.digital-cp.com/sites/default/files/specifications/HDCP%20on%20HDMI%20Specification%20Rev2_2_Final1.pdf
>   *
>   * Returns:
> - * Count of the revoked KSVs or -ve error number incase of the failure.
> + * Count of the revoked KSVs or -ve error number in case of the failure.
>   */
>  int drm_hdcp_check_ksvs_revoked(struct drm_device *drm_dev, u8 *ksvs,
>  				u32 ksv_count)
> diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/drm_ioctl.c
> index f454e0424086..be4a52dc4d6f 100644
> --- a/drivers/gpu/drm/drm_ioctl.c
> +++ b/drivers/gpu/drm/drm_ioctl.c
> @@ -426,7 +426,7 @@ static int drm_setversion(struct drm_device *dev, void *data, struct drm_file *f
>  }
>  
>  /**
> - * drm_noop - DRM no-op ioctl implemntation
> + * drm_noop - DRM no-op ioctl implementation
>   * @dev: DRM device for the ioctl
>   * @data: data pointer for the ioctl
>   * @file_priv: DRM file for the ioctl call
> @@ -446,7 +446,7 @@ int drm_noop(struct drm_device *dev, void *data,
>  EXPORT_SYMBOL(drm_noop);
>  
>  /**
> - * drm_invalid_op - DRM invalid ioctl implemntation
> + * drm_invalid_op - DRM invalid ioctl implementation
>   * @dev: DRM device for the ioctl
>   * @data: data pointer for the ioctl
>   * @file_priv: DRM file for the ioctl call
> diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
> index 201eae4bba6c..4a853011549a 100644
> --- a/drivers/gpu/drm/drm_irq.c
> +++ b/drivers/gpu/drm/drm_irq.c
> @@ -72,7 +72,7 @@
>   * The DRM core provides very simple support helpers to enable IRQ handling on a
>   * device through the drm_irq_install() and drm_irq_uninstall() functions. This
>   * only supports devices with a single interrupt on the main device stored in
> - * &drm_device.dev and set as the device paramter in drm_dev_alloc().
> + * &drm_device.dev and set as the device parameter in drm_dev_alloc().
>   *
>   * These IRQ helpers are strictly optional. Since these helpers don't automatically
>   * clean up the requested interrupt like e.g. devm_request_irq() they're not really
> diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c
> index a4a04d246135..93d48a6f04ab 100644
> --- a/drivers/gpu/drm/drm_mm.c
> +++ b/drivers/gpu/drm/drm_mm.c
> @@ -700,7 +700,7 @@ EXPORT_SYMBOL(drm_mm_replace_node);
>   * interfaces. First a scan operation needs to be initialized with
>   * drm_mm_scan_init() or drm_mm_scan_init_with_range(). The driver adds
>   * objects to the roster, probably by walking an LRU list, but this can be
> - * freely implemented. Eviction candiates are added using
> + * freely implemented. Eviction candidates are added using
>   * drm_mm_scan_add_block() until a suitable hole is found or there are no
>   * further evictable objects. Eviction roster metadata is tracked in &struct
>   * drm_mm_scan.
> diff --git a/drivers/gpu/drm/drm_mode_object.c b/drivers/gpu/drm/drm_mode_object.c
> index b26588b52795..86d9e907c0b2 100644
> --- a/drivers/gpu/drm/drm_mode_object.c
> +++ b/drivers/gpu/drm/drm_mode_object.c
> @@ -91,7 +91,7 @@ void drm_mode_object_register(struct drm_device *dev,
>  }
>  
>  /**
> - * drm_mode_object_unregister - free a modeset identifer
> + * drm_mode_object_unregister - free a modeset identifier
>   * @dev: DRM device
>   * @object: object to free
>   *
> diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c
> index ae53ea624c73..1c72208d8133 100644
> --- a/drivers/gpu/drm/drm_modes.c
> +++ b/drivers/gpu/drm/drm_modes.c
> @@ -1542,7 +1542,7 @@ static int drm_mode_parse_cmdline_int(const char *delim, unsigned int *int_ret)
>  
>  	/*
>  	 * delim must point to the '=', otherwise it is a syntax error and
> -	 * if delim points to the terminating zero, then delim + 1 wil point
> +	 * if delim points to the terminating zero, then delim + 1 will point
>  	 * past the end of the string.
>  	 */
>  	if (*delim != '=')
> @@ -1972,7 +1972,7 @@ int drm_mode_convert_umode(struct drm_device *dev,
>  	out->flags = in->flags;
>  	/*
>  	 * Old xf86-video-vmware (possibly others too) used to
> -	 * leave 'type' unititialized. Just ignore any bits we
> +	 * leave 'type' uninitialized. Just ignore any bits we
>  	 * don't like. It's a just hint after all, and more
>  	 * useful for the kernel->userspace direction anyway.
>  	 */
> diff --git a/drivers/gpu/drm/drm_plane.c b/drivers/gpu/drm/drm_plane.c
> index b373958ecb30..98ca1801253b 100644
> --- a/drivers/gpu/drm/drm_plane.c
> +++ b/drivers/gpu/drm/drm_plane.c
> @@ -38,7 +38,7 @@
>  /**
>   * DOC: overview
>   *
> - * A plane represents an image source that can be blended with or overlayed on
> + * A plane represents an image source that can be blended with or overlaid on
>   * top of a CRTC during the scanout process. Planes take their input data from a
>   * &drm_framebuffer object. The plane itself specifies the cropping and scaling
>   * of that image, and where it is placed on the visible area of a display
> diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c
> index 3aae7ea522f2..5b2d0ca03705 100644
> --- a/drivers/gpu/drm/drm_plane_helper.c
> +++ b/drivers/gpu/drm/drm_plane_helper.c
> @@ -210,7 +210,7 @@ static int drm_primary_helper_update(struct drm_plane *plane, struct drm_crtc *c
>  	 * We call set_config() directly here rather than using
>  	 * drm_mode_set_config_internal.  We're reprogramming the same
>  	 * connectors that were already in use, so we shouldn't need the extra
> -	 * cross-CRTC fb refcounting to accomodate stealing connectors.
> +	 * cross-CRTC fb refcounting to accommodate stealing connectors.
>  	 * drm_mode_setplane() already handles the basic refcounting for the
>  	 * framebuffers involved in this operation.
>  	 */
> diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
> index 2a54f86856af..3085858272c6 100644
> --- a/drivers/gpu/drm/drm_prime.c
> +++ b/drivers/gpu/drm/drm_prime.c
> @@ -73,7 +73,7 @@
>   * Thus the chain of references always flows in one direction, avoiding loops:
>   * importing GEM object -> dma-buf -> exported GEM bo. A further complication
>   * are the lookup caches for import and export. These are required to guarantee
> - * that any given object will always have only one uniqe userspace handle. This
> + * that any given object will always have only one unique userspace handle. This
>   * is required to allow userspace to detect duplicated imports, since some GEM
>   * drivers do fail command submissions if a given buffer object is listed more
>   * than once. These import and export caches in &drm_prime_file_private only
> diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c
> index e7e1ee2aa352..5606bca3caa8 100644
> --- a/drivers/gpu/drm/drm_probe_helper.c
> +++ b/drivers/gpu/drm/drm_probe_helper.c
> @@ -757,7 +757,7 @@ EXPORT_SYMBOL(drm_kms_helper_poll_disable);
>   * drm_kms_helper_poll_init - initialize and enable output polling
>   * @dev: drm_device
>   *
> - * This function intializes and then also enables output polling support for
> + * This function initializes and then also enables output polling support for
>   * @dev. Drivers which do not have reliable hotplug support in hardware can use
>   * this helper infrastructure to regularly poll such connectors for changes in
>   * their connection state.
> diff --git a/drivers/gpu/drm/drm_property.c b/drivers/gpu/drm/drm_property.c
> index 27c824a6eb60..6c353c9dc772 100644
> --- a/drivers/gpu/drm/drm_property.c
> +++ b/drivers/gpu/drm/drm_property.c
> @@ -434,7 +434,7 @@ EXPORT_SYMBOL(drm_property_add_enum);
>  /**
>   * drm_property_destroy - destroy a drm property
>   * @dev: drm device
> - * @property: property to destry
> + * @property: property to destroy
>   *
>   * This function frees a property including any attached resources like
>   * enumeration values.
> diff --git a/drivers/gpu/drm/drm_scdc_helper.c b/drivers/gpu/drm/drm_scdc_helper.c
> index 991b8c86d78d..48a382464d54 100644
> --- a/drivers/gpu/drm/drm_scdc_helper.c
> +++ b/drivers/gpu/drm/drm_scdc_helper.c
> @@ -241,7 +241,7 @@ bool drm_scdc_set_high_tmds_clock_ratio(struct i2c_adapter *adapter, bool set)
>  	/*
>  	 * The spec says that a source should wait minimum 1ms and maximum
>  	 * 100ms after writing the TMDS config for clock ratio. Lets allow a
> -	 * wait of upto 2ms here.
> +	 * wait of up to 2ms here.
>  	 */
>  	usleep_range(1000, 2000);
>  	return true;
> diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
> index 1c5b9ef6da37..c9a9d74f338c 100644
> --- a/drivers/gpu/drm/drm_syncobj.c
> +++ b/drivers/gpu/drm/drm_syncobj.c
> @@ -725,7 +725,7 @@ static int drm_syncobj_export_sync_file(struct drm_file *file_private,
>  	return ret;
>  }
>  /**
> - * drm_syncobj_open - initalizes syncobj file-private structures at devnode open time
> + * drm_syncobj_open - initializes syncobj file-private structures at devnode open time
>   * @file_private: drm file-private structure to set up
>   *
>   * Called at device open time, sets up the structure for handling refcounting
> diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c
> index bba6781cc48f..977b6509bb4b 100644
> --- a/drivers/gpu/drm/drm_vblank.c
> +++ b/drivers/gpu/drm/drm_vblank.c
> @@ -191,7 +191,7 @@ static u32 drm_max_vblank_count(struct drm_device *dev, unsigned int pipe)
>  
>  /*
>   * "No hw counter" fallback implementation of .get_vblank_counter() hook,
> - * if there is no useable hardware frame counter available.
> + * if there is no usable hardware frame counter available.
>   */
>  static u32 drm_vblank_no_hw_counter(struct drm_device *dev, unsigned int pipe)
>  {
> @@ -905,7 +905,7 @@ drm_get_last_vbltimestamp(struct drm_device *dev, unsigned int pipe,
>   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
>   * provide a barrier: Any writes done before calling
>   * drm_crtc_handle_vblank() will be visible to callers of the later
> - * functions, iff the vblank count is the same or a later one.
> + * functions, if the vblank count is the same or a later one.
>   *
>   * See also &drm_vblank_crtc.count.
>   *
> @@ -968,7 +968,7 @@ static u64 drm_vblank_count_and_time(struct drm_device *dev, unsigned int pipe,
>   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
>   * provide a barrier: Any writes done before calling
>   * drm_crtc_handle_vblank() will be visible to callers of the later
> - * functions, iff the vblank count is the same or a later one.
> + * functions, if the vblank count is the same or a later one.
>   *
>   * See also &drm_vblank_crtc.count.
>   */
> @@ -1997,7 +1997,7 @@ EXPORT_SYMBOL(drm_handle_vblank);
>   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
>   * provide a barrier: Any writes done before calling
>   * drm_crtc_handle_vblank() will be visible to callers of the later
> - * functions, iff the vblank count is the same or a later one.
> + * functions, if the vblank count is the same or a later one.
>   *
>   * See also &drm_vblank_crtc.count.
>   *
> @@ -2014,7 +2014,7 @@ EXPORT_SYMBOL(drm_crtc_handle_vblank);
>   * Get crtc VBLANK count.
>   *
>   * \param dev DRM device
> - * \param data user arguement, pointing to a drm_crtc_get_sequence structure.
> + * \param data user argument, pointing to a drm_crtc_get_sequence structure.
>   * \param file_priv drm file private for the user's open file descriptor
>   */
>  
> @@ -2070,7 +2070,7 @@ int drm_crtc_get_sequence_ioctl(struct drm_device *dev, void *data,
>   * Queue a event for VBLANK sequence
>   *
>   * \param dev DRM device
> - * \param data user arguement, pointing to a drm_crtc_queue_sequence structure.
> + * \param data user argument, pointing to a drm_crtc_queue_sequence structure.
>   * \param file_priv drm file private for the user's open file descriptor
>   */
>  
> diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c
> index 4565319fa6b3..7de37f8c68fd 100644
> --- a/drivers/gpu/drm/drm_vma_manager.c
> +++ b/drivers/gpu/drm/drm_vma_manager.c
> @@ -361,7 +361,7 @@ EXPORT_SYMBOL(drm_vma_node_revoke);
>   * This is locked against concurrent access internally.
>   *
>   * RETURNS:
> - * true iff @filp is on the list
> + * true if @filp is on the list
>   */
>  bool drm_vma_node_is_allowed(struct drm_vma_offset_node *node,
>  			     struct drm_file *tag)
> -- 
> 2.25.1
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH v2] drm: Fix typo in comments
  2021-08-02  8:19 ` Daniel Vetter
@ 2021-08-02  8:34   ` Cai,Huoqing
  0 siblings, 0 replies; 3+ messages in thread
From: Cai,Huoqing @ 2021-08-02  8:34 UTC (permalink / raw)
  To: Daniel Vetter; +Cc: maarten.lankhorst, mripard, tzimmermann, airlied, dri-devel

Hello,

> -----Original Message-----
> From: Daniel Vetter <daniel@ffwll.ch>
> Sent: 2021年8月2日 16:20
> To: Cai,Huoqing <caihuoqing@baidu.com>
> Cc: maarten.lankhorst@linux.intel.com; mripard@kernel.org;
> tzimmermann@suse.de; airlied@linux.ie; daniel@ffwll.ch; dri-
> devel@lists.freedesktop.org
> Subject: Re: [PATCH v2] drm: Fix typo in comments
> 
> On Fri, Jul 30, 2021 at 09:27:29PM +0800, Cai Huoqing wrote:
> > fix typo for drm
> >
> > v1->v2:
> > respin with the change "iff ==> implies that"
> >
> > Reviewed-by: Daniel Vetter <daniel@ffwll.ch>
> 
> I did not give you a Reviewed-by: tag, please dont forge them. At least in the linux
> kernel an r-b tag is a fairly formal statement:
> 
> https://dri.freedesktop.org/docs/drm/process/submitting-
> patches.html?highlight=statement%20oversight#reviewer-s-statement-of-
> oversight
> 
> The more informal Acked-by is generally ok to add if you get something like "looks
> good to me", but better to ask for confirmation too in that case.
I'm very sorry,  this is my mistake
I will remove it. 
> 
> I've removed it before pushing.
It's ok
> 
> > Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
> 
> Pushed to drm-misc-next, thanks for your patch.
> -Daniel
> 
> > ---
> >  drivers/gpu/drm/drm_aperture.c          |  2 +-
> >  drivers/gpu/drm/drm_atomic.c            |  2 +-
> >  drivers/gpu/drm/drm_atomic_helper.c     | 10 +++++-----
> >  drivers/gpu/drm/drm_atomic_uapi.c       |  6 +++---
> >  drivers/gpu/drm/drm_auth.c              |  2 +-
> >  drivers/gpu/drm/drm_bridge.c            |  2 +-
> >  drivers/gpu/drm/drm_bufs.c              |  2 +-
> >  drivers/gpu/drm/drm_cache.c             |  2 +-
> >  drivers/gpu/drm/drm_damage_helper.c     |  2 +-
> >  drivers/gpu/drm/drm_dp_helper.c         |  8 ++++----
> >  drivers/gpu/drm/drm_drv.c               |  4 ++--
> >  drivers/gpu/drm/drm_dsc.c               |  2 +-
> >  drivers/gpu/drm/drm_edid.c              |  4 ++--
> >  drivers/gpu/drm/drm_fb_helper.c         |  2 +-
> >  drivers/gpu/drm/drm_file.c              |  6 +++---
> >  drivers/gpu/drm/drm_format_helper.c     |  2 +-
> >  drivers/gpu/drm/drm_framebuffer.c       |  2 +-
> >  drivers/gpu/drm/drm_gem.c               |  4 ++--
> >  drivers/gpu/drm/drm_gem_atomic_helper.c |  4 ++--
> > drivers/gpu/drm/drm_gem_shmem_helper.c  |  2 +-
> >  drivers/gpu/drm/drm_gem_vram_helper.c   |  2 +-
> >  drivers/gpu/drm/drm_hdcp.c              |  2 +-
> >  drivers/gpu/drm/drm_ioctl.c             |  4 ++--
> >  drivers/gpu/drm/drm_irq.c               |  2 +-
> >  drivers/gpu/drm/drm_mm.c                |  2 +-
> >  drivers/gpu/drm/drm_mode_object.c       |  2 +-
> >  drivers/gpu/drm/drm_modes.c             |  4 ++--
> >  drivers/gpu/drm/drm_plane.c             |  2 +-
> >  drivers/gpu/drm/drm_plane_helper.c      |  2 +-
> >  drivers/gpu/drm/drm_prime.c             |  2 +-
> >  drivers/gpu/drm/drm_probe_helper.c      |  2 +-
> >  drivers/gpu/drm/drm_property.c          |  2 +-
> >  drivers/gpu/drm/drm_scdc_helper.c       |  2 +-
> >  drivers/gpu/drm/drm_syncobj.c           |  2 +-
> >  drivers/gpu/drm/drm_vblank.c            | 12 ++++++------
> >  drivers/gpu/drm/drm_vma_manager.c       |  2 +-
> >  36 files changed, 58 insertions(+), 58 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_aperture.c
> > b/drivers/gpu/drm/drm_aperture.c index 9ac39cf11694..74bd4a76b253
> > 100644
> > --- a/drivers/gpu/drm/drm_aperture.c
> > +++ b/drivers/gpu/drm/drm_aperture.c
> > @@ -78,7 +78,7 @@
> >   *
> >   * Drivers that are susceptible to being removed by other drivers, such as
> >   * generic EFI or VESA drivers, have to register themselves as owners
> > of their
> > - * given framebuffer memory. Ownership of the framebuffer memory is
> > achived
> > + * given framebuffer memory. Ownership of the framebuffer memory is
> > + achieved
> >   * by calling devm_aperture_acquire_from_firmware(). On success, the driver
> >   * is the owner of the framebuffer range. The function fails if the
> >   * framebuffer is already by another driver. See below for an example.
> > diff --git a/drivers/gpu/drm/drm_atomic.c
> > b/drivers/gpu/drm/drm_atomic.c index d820423fac32..b127e30082ba 100644
> > --- a/drivers/gpu/drm/drm_atomic.c
> > +++ b/drivers/gpu/drm/drm_atomic.c
> > @@ -723,7 +723,7 @@ static void drm_atomic_plane_print_state(struct
> drm_printer *p,
> >   * clocks, scaler units, bandwidth and fifo limits shared among a group of
> >   * planes or CRTCs, and so on) it makes sense to model these as independent
> >   * objects. Drivers then need to do similar state tracking and commit
> > ordering for
> > - * such private (since not exposed to userpace) objects as the atomic
> > core and
> > + * such private (since not exposed to userspace) objects as the
> > + atomic core and
> >   * helpers already provide for connectors, planes and CRTCs.
> >   *
> >   * To make this easier on drivers the atomic core provides some
> > support to track diff --git a/drivers/gpu/drm/drm_atomic_helper.c
> > b/drivers/gpu/drm/drm_atomic_helper.c
> > index f7bf1ea62d58..7ee480b6efde 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -634,7 +634,7 @@ drm_atomic_helper_check_modeset(struct
> drm_device *dev,
> >  			 * connectors and a NULL mode.
> >  			 *
> >  			 * The other way around is true as well. enable != 0
> > -			 * iff connectors are attached and a mode is set.
> > +			 * implies that connectors are attached and a mode is
> set.
> >  			 */
> >  			new_crtc_state->mode_changed = true;
> >  			new_crtc_state->connectors_changed = true; @@ -
> 1686,7 +1686,7 @@
> > static void commit_work(struct work_struct *work)  }
> >
> >  /**
> > - * drm_atomic_helper_async_check - check if state can be commited
> > asynchronously
> > + * drm_atomic_helper_async_check - check if state can be committed
> > + asynchronously
> >   * @dev: DRM device
> >   * @state: the driver state object
> >   *
> > @@ -1695,7 +1695,7 @@ static void commit_work(struct work_struct *work)
> >   * but just do in-place changes on the current state.
> >   *
> >   * It will return 0 if the commit can happen in an asynchronous
> > fashion or error
> > - * if not. Note that error just mean it can't be commited
> > asynchronously, if it
> > + * if not. Note that error just mean it can't be committed
> > + asynchronously, if it
> >   * fails the commit should be treated like a normal synchronous commit.
> >   */
> >  int drm_atomic_helper_async_check(struct drm_device *dev, @@ -2583,7
> > +2583,7 @@ EXPORT_SYMBOL(drm_atomic_helper_commit_planes);
> >   *
> >   * This function can only be savely used when planes are not allowed to move
> >   * between different CRTCs because this function doesn't handle
> > inter-CRTC
> > - * depencies. Callers need to ensure that either no such depencies
> > exist,
> > + * dependencies. Callers need to ensure that either no such
> > + dependencies exist,
> >   * resolve them through ordering of commit calls or through some other means.
> >   */
> >  void
> > @@ -2720,7 +2720,7 @@
> EXPORT_SYMBOL(drm_atomic_helper_cleanup_planes);
> >  /**
> >   * drm_atomic_helper_swap_state - store atomic state into current sw state
> >   * @state: atomic state
> > - * @stall: stall for preceeding commits
> > + * @stall: stall for preceding commits
> >   *
> >   * This function stores the atomic state into the current state pointers in all
> >   * driver objects. It should be called after all failing steps have
> > been done diff --git a/drivers/gpu/drm/drm_atomic_uapi.c
> > b/drivers/gpu/drm/drm_atomic_uapi.c
> > index 7e48d40600ff..909f31833181 100644
> > --- a/drivers/gpu/drm/drm_atomic_uapi.c
> > +++ b/drivers/gpu/drm/drm_atomic_uapi.c
> > @@ -48,7 +48,7 @@
> >   * in all its forms: The monster ATOMIC IOCTL itself, code for GET_PROPERTY
> and
> >   * SET_PROPERTY IOCTLs. Plus interface functions for compatibility helpers and
> >   * drivers which have special needs to construct their own atomic updates, e.g.
> > - * for load detect or similiar.
> > + * for load detect or similar.
> >   */
> >
> >  /**
> > @@ -753,7 +753,7 @@ static int drm_atomic_connector_set_property(struct
> drm_connector *connector,
> >  		 * restore the state it wants on VT switch. So if the userspace
> >  		 * tries to change the link_status from GOOD to BAD, driver
> >  		 * silently rejects it and returns a 0. This prevents userspace
> > -		 * from accidently breaking  the display when it restores the
> > +		 * from accidentally breaking  the display when it restores the
> >  		 * state.
> >  		 */
> >  		if (state->link_status != DRM_LINK_STATUS_GOOD) @@ -
> 1064,7 +1064,7
> > @@ int drm_atomic_set_property(struct drm_atomic_state *state,
> >   * DOC: explicit fencing properties
> >   *
> >   * Explicit fencing allows userspace to control the buffer
> > synchronization
> > - * between devices. A Fence or a group of fences are transfered
> > to/from
> > + * between devices. A Fence or a group of fences are transferred
> > + to/from
> >   * userspace using Sync File fds and there are two DRM properties for that.
> >   * IN_FENCE_FD on each DRM Plane to send fences to the kernel and
> >   * OUT_FENCE_PTR on each DRM CRTC to receive fences from the kernel.
> > diff --git a/drivers/gpu/drm/drm_auth.c b/drivers/gpu/drm/drm_auth.c
> > index 3a64a6a79ade..eba438027539 100644
> > --- a/drivers/gpu/drm/drm_auth.c
> > +++ b/drivers/gpu/drm/drm_auth.c
> > @@ -52,7 +52,7 @@
> >   *
> >   * In addition only one &drm_master can be the current master for a
> &drm_device.
> >   * It can be switched through the DROP_MASTER and SET_MASTER IOCTL,
> > or
> > - * implicitly through closing/openeing the primary device node. See
> > also
> > + * implicitly through closing/opening the primary device node. See
> > + also
> >   * drm_is_current_master().
> >   *
> >   * Clients can authenticate against the current master (if it matches
> > their own) diff --git a/drivers/gpu/drm/drm_bridge.c
> > b/drivers/gpu/drm/drm_bridge.c index 044acd07c153..d4d447a8725a 100644
> > --- a/drivers/gpu/drm/drm_bridge.c
> > +++ b/drivers/gpu/drm/drm_bridge.c
> > @@ -972,7 +972,7 @@ drm_atomic_bridge_propagate_bus_flags(struct
> drm_bridge *bridge,
> >  	bridge_state->output_bus_cfg.flags = output_flags;
> >
> >  	/*
> > -	 * Propage the output flags to the input end of the bridge. Again, it's
> > +	 * Propagate the output flags to the input end of the bridge. Again,
> > +it's
> >  	 * not necessarily what all bridges want, but that's what most of them
> >  	 * do, and by doing that by default we avoid forcing drivers to
> >  	 * duplicate the "dummy propagation" logic.
> > diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c
> > index ae8e4d76209c..fcca21e8efac 100644
> > --- a/drivers/gpu/drm/drm_bufs.c
> > +++ b/drivers/gpu/drm/drm_bufs.c
> > @@ -1502,7 +1502,7 @@ int drm_legacy_freebufs(struct drm_device *dev,
> void *data,
> >   *
> >   * Maps the AGP, SG or PCI buffer region with vm_mmap(), and copies
> information
> >   * about each buffer into user space. For PCI buffers, it calls
> > vm_mmap() with
> > - * offset equal to 0, which drm_mmap() interpretes as PCI buffers and
> > calls
> > + * offset equal to 0, which drm_mmap() interprets as PCI buffers and
> > + calls
> >   * drm_mmap_dma().
> >   */
> >  int __drm_legacy_mapbufs(struct drm_device *dev, void *data, int *p,
> > diff --git a/drivers/gpu/drm/drm_cache.c b/drivers/gpu/drm/drm_cache.c
> > index 546599f19a93..30cc59fe6ef7 100644
> > --- a/drivers/gpu/drm/drm_cache.c
> > +++ b/drivers/gpu/drm/drm_cache.c
> > @@ -170,7 +170,7 @@ drm_clflush_virt_range(void *addr, unsigned long
> length)
> >  		for (; addr < end; addr += size)
> >  			clflushopt(addr);
> >  		clflushopt(end - 1); /* force serialisation */
> > -		mb(); /*Ensure that evry data cache line entry is flushed*/
> > +		mb(); /*Ensure that every data cache line entry is flushed*/
> >  		return;
> >  	}
> >
> > diff --git a/drivers/gpu/drm/drm_damage_helper.c
> > b/drivers/gpu/drm/drm_damage_helper.c
> > index 3a4126dc2520..dbc4312160a5 100644
> > --- a/drivers/gpu/drm/drm_damage_helper.c
> > +++ b/drivers/gpu/drm/drm_damage_helper.c
> > @@ -170,7 +170,7 @@ int drm_atomic_helper_dirtyfb(struct
> drm_framebuffer *fb,
> >  	int ret = 0;
> >
> >  	/*
> > -	 * When called from ioctl, we are interruptable, but not when called
> > +	 * When called from ioctl, we are interruptible, but not when called
> >  	 * internally (ie. defio worker)
> >  	 */
> >  	drm_modeset_acquire_init(&ctx,
> > diff --git a/drivers/gpu/drm/drm_dp_helper.c
> > b/drivers/gpu/drm/drm_dp_helper.c index b5f75ca05774..ed3ed6d3da25
> > 100644
> > --- a/drivers/gpu/drm/drm_dp_helper.c
> > +++ b/drivers/gpu/drm/drm_dp_helper.c
> > @@ -772,7 +772,7 @@ int drm_dp_downstream_max_tmds_clock(const u8
> dpcd[DP_RECEIVER_CAP_SIZE],
> >  		 * It's left up to the driver to check the
> >  		 * DP dual mode adapter's max TMDS clock.
> >  		 *
> > -		 * Unfortunatley it looks like branch devices
> > +		 * Unfortunately it looks like branch devices
> >  		 * may not fordward that the DP dual mode i2c
> >  		 * access so we just usually get i2c nak :(
> >  		 */
> > @@ -1365,7 +1365,7 @@ static int drm_dp_i2c_msg_duration(const struct
> > drm_dp_aux_msg *msg,  }
> >
> >  /*
> > - * Deterine how many retries should be attempted to successfully
> > transfer
> > + * Determine how many retries should be attempted to successfully
> > + transfer
> >   * the specified message, based on the estimated durations of the
> >   * i2c and AUX transfers.
> >   */
> > @@ -1418,7 +1418,7 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux
> *aux, struct drm_dp_aux_msg *msg)
> >  			/*
> >  			 * While timeouts can be errors, they're usually normal
> >  			 * behavior (for instance, when a driver tries to
> > -			 * communicate with a non-existant DisplayPort device).
> > +			 * communicate with a non-existent DisplayPort device).
> >  			 * Avoid spamming the kernel log with timeout errors.
> >  			 */
> >  			if (ret == -ETIMEDOUT)
> > @@ -3327,7 +3327,7 @@ drm_edp_backlight_probe_max(struct drm_dp_aux
> *aux, struct drm_edp_backlight_inf
> >  	fxp = DIV_ROUND_CLOSEST(1000 *
> DP_EDP_BACKLIGHT_FREQ_BASE_KHZ,
> > driver_pwm_freq_hz);
> >
> >  	/* Use highest possible value of Pn for more granularity of brightness
> adjustment while
> > -	 * satifying the conditions below.
> > +	 * satisfying the conditions below.
> >  	 * - Pn is in the range of Pn_min and Pn_max
> >  	 * - F is in the range of 1 and 255
> >  	 * - FxP is within 25% of desired value.
> > diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> > index 8804ec7d3215..7a5097467ba5 100644
> > --- a/drivers/gpu/drm/drm_drv.c
> > +++ b/drivers/gpu/drm/drm_drv.c
> > @@ -249,7 +249,7 @@ void drm_minor_release(struct drm_minor *minor)
> >   * Finally when everything is up and running and ready for userspace the device
> >   * instance can be published using drm_dev_register().
> >   *
> > - * There is also deprecated support for initalizing device instances
> > using
> > + * There is also deprecated support for initializing device instances
> > + using
> >   * bus-specific helpers and the &drm_driver.load callback. But due to
> >   * backwards-compatibility needs the device instance have to be published too
> >   * early, which requires unpretty global locking to make safe and is
> > therefore @@ -379,7 +379,7 @@ void drm_minor_release(struct drm_minor
> *minor)
> >   * shortcoming however, drm_dev_unplug() marks the drm_device as
> unplugged before
> >   * drm_atomic_helper_shutdown() is called. This means that if the disable code
> >   * paths are protected, they will not run on regular driver module
> > unload,
> > - * possibily leaving the hardware enabled.
> > + * possibly leaving the hardware enabled.
> >   */
> >
> >  /**
> > diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c
> > index ff602f7ec65b..46a3c1b62463 100644
> > --- a/drivers/gpu/drm/drm_dsc.c
> > +++ b/drivers/gpu/drm/drm_dsc.c
> > @@ -98,7 +98,7 @@ void drm_dsc_pps_payload_pack(struct
> > drm_dsc_picture_parameter_set *pps_payload,  {
> >  	int i;
> >
> > -	/* Protect against someone accidently changing struct size */
> > +	/* Protect against someone accidentally changing struct size */
> >  	BUILD_BUG_ON(sizeof(*pps_payload) !=
> >  		     DP_SDP_PPS_HEADER_PAYLOAD_BYTES_MINUS_1 + 1);
> >
> > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> > index 81d5f2524246..6325877c5fd6 100644
> > --- a/drivers/gpu/drm/drm_edid.c
> > +++ b/drivers/gpu/drm/drm_edid.c
> > @@ -1919,7 +1919,7 @@ EXPORT_SYMBOL(drm_add_override_edid_modes);
> >   * level, drivers must make all reasonable efforts to expose it as an I2C
> >   * adapter and use drm_get_edid() instead of abusing this function.
> >   *
> > - * The EDID may be overridden using debugfs override_edid or firmare
> > EDID
> > + * The EDID may be overridden using debugfs override_edid or firmware
> > + EDID
> >   * (drm_load_edid_firmware() and drm.edid_firmware parameter), in this
> priority
> >   * order. Having either of them bypasses actual EDID reads.
> >   *
> > @@ -5906,7 +5906,7 @@
> drm_hdmi_vendor_infoframe_from_display_mode(struct
> hdmi_vendor_infoframe *frame,
> >  	 * (ie.vic==0 and s3d_struct==0) we will still send it if we
> >  	 * know that the sink can handle it. This is based on a
> >  	 * suggestion in HDMI 2.0 Appendix F. Apparently some sinks
> > -	 * have trouble realizing that they shuld switch from 3D to 2D
> > +	 * have trouble realizing that they should switch from 3D to 2D
> >  	 * mode if the source simply stops sending the infoframe when
> >  	 * it wants to switch from 3D to 2D.
> >  	 */
> > diff --git a/drivers/gpu/drm/drm_fb_helper.c
> > b/drivers/gpu/drm/drm_fb_helper.c index d77a24507d30..3ab078321045
> > 100644
> > --- a/drivers/gpu/drm/drm_fb_helper.c
> > +++ b/drivers/gpu/drm/drm_fb_helper.c
> > @@ -565,7 +565,7 @@ struct fb_info *drm_fb_helper_alloc_fbi(struct
> drm_fb_helper *fb_helper)
> >  		goto err_release;
> >
> >  	/*
> > -	 * TODO: We really should be smarter here and alloc an apperture
> > +	 * TODO: We really should be smarter here and alloc an aperture
> >  	 * for each IORESOURCE_MEM resource helper->dev->dev has and also
> >  	 * init the ranges of the appertures based on the resources.
> >  	 * Note some drivers currently count on there being only 1 empty
> > diff --git a/drivers/gpu/drm/drm_file.c b/drivers/gpu/drm/drm_file.c
> > index ceb1a9723855..ed25168619fc 100644
> > --- a/drivers/gpu/drm/drm_file.c
> > +++ b/drivers/gpu/drm/drm_file.c
> > @@ -405,7 +405,7 @@ static int drm_open_helper(struct file *filp, struct
> drm_minor *minor)
> >   *
> >   * RETURNS:
> >   *
> > - * 0 on success or negative errno value on falure.
> > + * 0 on success or negative errno value on failure.
> >   */
> >  int drm_open(struct inode *inode, struct file *filp)  { @@ -548,7
> > +548,7 @@ EXPORT_SYMBOL(drm_release_noglobal);
> >   * @offset: offset to read
> >   *
> >   * This function must be used by drivers as their
> > &file_operations.read
> > - * method iff they use DRM events for asynchronous signalling to userspace.
> > + * method if they use DRM events for asynchronous signalling to userspace.
> >   * Since events are used by the KMS API for vblank and page flip completion this
> >   * means all modern display drivers must use it.
> >   *
> > @@ -641,7 +641,7 @@ EXPORT_SYMBOL(drm_read);
> >   * @wait: poll waiter table
> >   *
> >   * This function must be used by drivers as their
> > &file_operations.read method
> > - * iff they use DRM events for asynchronous signalling to userspace.
> > Since
> > + * if they use DRM events for asynchronous signalling to userspace.
> > + Since
> >   * events are used by the KMS API for vblank and page flip completion this
> means
> >   * all modern display drivers must use it.
> >   *
> > diff --git a/drivers/gpu/drm/drm_format_helper.c
> > b/drivers/gpu/drm/drm_format_helper.c
> > index 0e885cd34107..5231104b1498 100644
> > --- a/drivers/gpu/drm/drm_format_helper.c
> > +++ b/drivers/gpu/drm/drm_format_helper.c
> > @@ -412,7 +412,7 @@ EXPORT_SYMBOL(drm_fb_blit_rect_dstclip);
> >   * of the display and the framebuffer mismatch, the copy function will
> >   * attempt to convert between them.
> >   *
> > - * See drm_fb_blit_rect_dstclip() for more inforamtion.
> > + * See drm_fb_blit_rect_dstclip() for more information.
> >   *
> >   * Returns:
> >   * 0 on success, or a negative error code otherwise.
> > diff --git a/drivers/gpu/drm/drm_framebuffer.c
> > b/drivers/gpu/drm/drm_framebuffer.c
> > index 4d01464b6f95..a8ad0da225a7 100644
> > --- a/drivers/gpu/drm/drm_framebuffer.c
> > +++ b/drivers/gpu/drm/drm_framebuffer.c
> > @@ -1090,7 +1090,7 @@ void drm_framebuffer_remove(struct
> > drm_framebuffer *fb)
> >
> >  	/*
> >  	 * drm ABI mandates that we remove any deleted framebuffers from
> active
> > -	 * useage. But since most sane clients only remove framebuffers they no
> > +	 * usage. But since most sane clients only remove framebuffers they
> > +no
> >  	 * longer need, try to optimize this away.
> >  	 *
> >  	 * Since we're holding a reference ourselves, observing a refcount
> > of 1 diff --git a/drivers/gpu/drm/drm_gem.c
> > b/drivers/gpu/drm/drm_gem.c index 68deb1de8235..6c102ba2d9ac 100644
> > --- a/drivers/gpu/drm/drm_gem.c
> > +++ b/drivers/gpu/drm/drm_gem.c
> > @@ -901,7 +901,7 @@ drm_gem_open_ioctl(struct drm_device *dev, void
> > *data,  }
> >
> >  /**
> > - * drm_gem_open - initalizes GEM file-private structures at devnode
> > open time
> > + * drm_gem_open - initializes GEM file-private structures at devnode
> > + open time
> >   * @dev: drm_device which is being opened by userspace
> >   * @file_private: drm file-private structure to set up
> >   *
> > @@ -936,7 +936,7 @@ drm_gem_release(struct drm_device *dev, struct
> drm_file *file_private)
> >   * drm_gem_object_release - release GEM buffer object resources
> >   * @obj: GEM buffer object
> >   *
> > - * This releases any structures and resources used by @obj and is the
> > invers of
> > + * This releases any structures and resources used by @obj and is the
> > + inverse of
> >   * drm_gem_object_init().
> >   */
> >  void
> > diff --git a/drivers/gpu/drm/drm_gem_atomic_helper.c
> > b/drivers/gpu/drm/drm_gem_atomic_helper.c
> > index 26af09b959d4..86b3c676e0b1 100644
> > --- a/drivers/gpu/drm/drm_gem_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_gem_atomic_helper.c
> > @@ -52,7 +52,7 @@
> >   *
> >   * The helpers for shadow-buffered planes establish and release mappings,
> >   * and provide struct drm_shadow_plane_state, which stores the
> > plane's mapping
> > - * for commit-tail functons.
> > + * for commit-tail functions.
> >   *
> >   * Shadow-buffered planes can easily be enabled by using the provided macros
> >   * %DRM_GEM_SHADOW_PLANE_FUNCS
> and %DRM_GEM_SHADOW_PLANE_HELPER_FUNCS.
> > @@ -374,7 +374,7 @@ EXPORT_SYMBOL(drm_gem_prepare_shadow_fb);
> >   * This function implements struct &drm_plane_helper_funcs.cleanup_fb.
> >   * This function unmaps all buffer objects of the plane's framebuffer.
> >   *
> > - * See drm_gem_prepare_shadow_fb() for more inforamtion.
> > + * See drm_gem_prepare_shadow_fb() for more information.
> >   */
> >  void drm_gem_cleanup_shadow_fb(struct drm_plane *plane, struct
> > drm_plane_state *plane_state)  { diff --git
> > a/drivers/gpu/drm/drm_gem_shmem_helper.c
> > b/drivers/gpu/drm/drm_gem_shmem_helper.c
> > index d5e6d4568f99..a61946374c82 100644
> > --- a/drivers/gpu/drm/drm_gem_shmem_helper.c
> > +++ b/drivers/gpu/drm/drm_gem_shmem_helper.c
> > @@ -368,7 +368,7 @@ static void drm_gem_shmem_vunmap_locked(struct
> > drm_gem_shmem_object *shmem,  }
> >
> >  /*
> > - * drm_gem_shmem_vunmap - Unmap a virtual mapping fo a shmem GEM
> > object
> > + * drm_gem_shmem_vunmap - Unmap a virtual mapping for a shmem GEM
> > + object
> >   * @shmem: shmem GEM object
> >   * @map: Kernel virtual address where the SHMEM GEM object was mapped
> >   *
> > diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c
> > b/drivers/gpu/drm/drm_gem_vram_helper.c
> > index 1e9b82e51a07..43cf7e887d1a 100644
> > --- a/drivers/gpu/drm/drm_gem_vram_helper.c
> > +++ b/drivers/gpu/drm/drm_gem_vram_helper.c
> > @@ -96,7 +96,7 @@ static const struct drm_gem_object_funcs
> drm_gem_vram_object_funcs;
> >   * memory region. Call drm_gem_vram_offset() to retrieve this value. Typically
> >   * it's used to program the hardware's scanout engine for framebuffers, set
> >   * the cursor overlay's image for a mouse cursor, or use it as input
> > to the
> > - * hardware's draing engine.
> > + * hardware's drawing engine.
> >   *
> >   * To access a buffer object's memory from the DRM driver, call
> >   * drm_gem_vram_vmap(). It maps the buffer into kernel address diff
> > --git a/drivers/gpu/drm/drm_hdcp.c b/drivers/gpu/drm/drm_hdcp.c index
> > 910108ccaae1..ca9b8f697202 100644
> > --- a/drivers/gpu/drm/drm_hdcp.c
> > +++ b/drivers/gpu/drm/drm_hdcp.c
> > @@ -280,7 +280,7 @@ static int drm_hdcp_request_srm(struct drm_device
> *drm_dev,
> >   * https://www.digital-
> cp.com/sites/default/files/specifications/HDCP%20on%20HDMI%20Specification
> %20Rev2_2_Final1.pdf
> >   *
> >   * Returns:
> > - * Count of the revoked KSVs or -ve error number incase of the failure.
> > + * Count of the revoked KSVs or -ve error number in case of the failure.
> >   */
> >  int drm_hdcp_check_ksvs_revoked(struct drm_device *drm_dev, u8 *ksvs,
> >  				u32 ksv_count)
> > diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/drm_ioctl.c
> > index f454e0424086..be4a52dc4d6f 100644
> > --- a/drivers/gpu/drm/drm_ioctl.c
> > +++ b/drivers/gpu/drm/drm_ioctl.c
> > @@ -426,7 +426,7 @@ static int drm_setversion(struct drm_device *dev,
> > void *data, struct drm_file *f  }
> >
> >  /**
> > - * drm_noop - DRM no-op ioctl implemntation
> > + * drm_noop - DRM no-op ioctl implementation
> >   * @dev: DRM device for the ioctl
> >   * @data: data pointer for the ioctl
> >   * @file_priv: DRM file for the ioctl call @@ -446,7 +446,7 @@ int
> > drm_noop(struct drm_device *dev, void *data,  EXPORT_SYMBOL(drm_noop);
> >
> >  /**
> > - * drm_invalid_op - DRM invalid ioctl implemntation
> > + * drm_invalid_op - DRM invalid ioctl implementation
> >   * @dev: DRM device for the ioctl
> >   * @data: data pointer for the ioctl
> >   * @file_priv: DRM file for the ioctl call diff --git
> > a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index
> > 201eae4bba6c..4a853011549a 100644
> > --- a/drivers/gpu/drm/drm_irq.c
> > +++ b/drivers/gpu/drm/drm_irq.c
> > @@ -72,7 +72,7 @@
> >   * The DRM core provides very simple support helpers to enable IRQ handling
> on a
> >   * device through the drm_irq_install() and drm_irq_uninstall() functions. This
> >   * only supports devices with a single interrupt on the main device
> > stored in
> > - * &drm_device.dev and set as the device paramter in drm_dev_alloc().
> > + * &drm_device.dev and set as the device parameter in drm_dev_alloc().
> >   *
> >   * These IRQ helpers are strictly optional. Since these helpers don't
> automatically
> >   * clean up the requested interrupt like e.g. devm_request_irq()
> > they're not really diff --git a/drivers/gpu/drm/drm_mm.c
> > b/drivers/gpu/drm/drm_mm.c index a4a04d246135..93d48a6f04ab 100644
> > --- a/drivers/gpu/drm/drm_mm.c
> > +++ b/drivers/gpu/drm/drm_mm.c
> > @@ -700,7 +700,7 @@ EXPORT_SYMBOL(drm_mm_replace_node);
> >   * interfaces. First a scan operation needs to be initialized with
> >   * drm_mm_scan_init() or drm_mm_scan_init_with_range(). The driver adds
> >   * objects to the roster, probably by walking an LRU list, but this
> > can be
> > - * freely implemented. Eviction candiates are added using
> > + * freely implemented. Eviction candidates are added using
> >   * drm_mm_scan_add_block() until a suitable hole is found or there are no
> >   * further evictable objects. Eviction roster metadata is tracked in &struct
> >   * drm_mm_scan.
> > diff --git a/drivers/gpu/drm/drm_mode_object.c
> > b/drivers/gpu/drm/drm_mode_object.c
> > index b26588b52795..86d9e907c0b2 100644
> > --- a/drivers/gpu/drm/drm_mode_object.c
> > +++ b/drivers/gpu/drm/drm_mode_object.c
> > @@ -91,7 +91,7 @@ void drm_mode_object_register(struct drm_device
> > *dev,  }
> >
> >  /**
> > - * drm_mode_object_unregister - free a modeset identifer
> > + * drm_mode_object_unregister - free a modeset identifier
> >   * @dev: DRM device
> >   * @object: object to free
> >   *
> > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c
> > index ae53ea624c73..1c72208d8133 100644
> > --- a/drivers/gpu/drm/drm_modes.c
> > +++ b/drivers/gpu/drm/drm_modes.c
> > @@ -1542,7 +1542,7 @@ static int drm_mode_parse_cmdline_int(const char
> > *delim, unsigned int *int_ret)
> >
> >  	/*
> >  	 * delim must point to the '=', otherwise it is a syntax error and
> > -	 * if delim points to the terminating zero, then delim + 1 wil point
> > +	 * if delim points to the terminating zero, then delim + 1 will
> > +point
> >  	 * past the end of the string.
> >  	 */
> >  	if (*delim != '=')
> > @@ -1972,7 +1972,7 @@ int drm_mode_convert_umode(struct drm_device
> *dev,
> >  	out->flags = in->flags;
> >  	/*
> >  	 * Old xf86-video-vmware (possibly others too) used to
> > -	 * leave 'type' unititialized. Just ignore any bits we
> > +	 * leave 'type' uninitialized. Just ignore any bits we
> >  	 * don't like. It's a just hint after all, and more
> >  	 * useful for the kernel->userspace direction anyway.
> >  	 */
> > diff --git a/drivers/gpu/drm/drm_plane.c b/drivers/gpu/drm/drm_plane.c
> > index b373958ecb30..98ca1801253b 100644
> > --- a/drivers/gpu/drm/drm_plane.c
> > +++ b/drivers/gpu/drm/drm_plane.c
> > @@ -38,7 +38,7 @@
> >  /**
> >   * DOC: overview
> >   *
> > - * A plane represents an image source that can be blended with or
> > overlayed on
> > + * A plane represents an image source that can be blended with or
> > + overlaid on
> >   * top of a CRTC during the scanout process. Planes take their input data from
> a
> >   * &drm_framebuffer object. The plane itself specifies the cropping and scaling
> >   * of that image, and where it is placed on the visible area of a
> > display diff --git a/drivers/gpu/drm/drm_plane_helper.c
> > b/drivers/gpu/drm/drm_plane_helper.c
> > index 3aae7ea522f2..5b2d0ca03705 100644
> > --- a/drivers/gpu/drm/drm_plane_helper.c
> > +++ b/drivers/gpu/drm/drm_plane_helper.c
> > @@ -210,7 +210,7 @@ static int drm_primary_helper_update(struct
> drm_plane *plane, struct drm_crtc *c
> >  	 * We call set_config() directly here rather than using
> >  	 * drm_mode_set_config_internal.  We're reprogramming the same
> >  	 * connectors that were already in use, so we shouldn't need the extra
> > -	 * cross-CRTC fb refcounting to accomodate stealing connectors.
> > +	 * cross-CRTC fb refcounting to accommodate stealing connectors.
> >  	 * drm_mode_setplane() already handles the basic refcounting for the
> >  	 * framebuffers involved in this operation.
> >  	 */
> > diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
> > index 2a54f86856af..3085858272c6 100644
> > --- a/drivers/gpu/drm/drm_prime.c
> > +++ b/drivers/gpu/drm/drm_prime.c
> > @@ -73,7 +73,7 @@
> >   * Thus the chain of references always flows in one direction, avoiding loops:
> >   * importing GEM object -> dma-buf -> exported GEM bo. A further
> complication
> >   * are the lookup caches for import and export. These are required to
> > guarantee
> > - * that any given object will always have only one uniqe userspace
> > handle. This
> > + * that any given object will always have only one unique userspace
> > + handle. This
> >   * is required to allow userspace to detect duplicated imports, since some GEM
> >   * drivers do fail command submissions if a given buffer object is listed more
> >   * than once. These import and export caches in
> > &drm_prime_file_private only diff --git
> > a/drivers/gpu/drm/drm_probe_helper.c
> > b/drivers/gpu/drm/drm_probe_helper.c
> > index e7e1ee2aa352..5606bca3caa8 100644
> > --- a/drivers/gpu/drm/drm_probe_helper.c
> > +++ b/drivers/gpu/drm/drm_probe_helper.c
> > @@ -757,7 +757,7 @@ EXPORT_SYMBOL(drm_kms_helper_poll_disable);
> >   * drm_kms_helper_poll_init - initialize and enable output polling
> >   * @dev: drm_device
> >   *
> > - * This function intializes and then also enables output polling
> > support for
> > + * This function initializes and then also enables output polling
> > + support for
> >   * @dev. Drivers which do not have reliable hotplug support in hardware can
> use
> >   * this helper infrastructure to regularly poll such connectors for changes in
> >   * their connection state.
> > diff --git a/drivers/gpu/drm/drm_property.c
> > b/drivers/gpu/drm/drm_property.c index 27c824a6eb60..6c353c9dc772
> > 100644
> > --- a/drivers/gpu/drm/drm_property.c
> > +++ b/drivers/gpu/drm/drm_property.c
> > @@ -434,7 +434,7 @@ EXPORT_SYMBOL(drm_property_add_enum);
> >  /**
> >   * drm_property_destroy - destroy a drm property
> >   * @dev: drm device
> > - * @property: property to destry
> > + * @property: property to destroy
> >   *
> >   * This function frees a property including any attached resources like
> >   * enumeration values.
> > diff --git a/drivers/gpu/drm/drm_scdc_helper.c
> > b/drivers/gpu/drm/drm_scdc_helper.c
> > index 991b8c86d78d..48a382464d54 100644
> > --- a/drivers/gpu/drm/drm_scdc_helper.c
> > +++ b/drivers/gpu/drm/drm_scdc_helper.c
> > @@ -241,7 +241,7 @@ bool drm_scdc_set_high_tmds_clock_ratio(struct
> i2c_adapter *adapter, bool set)
> >  	/*
> >  	 * The spec says that a source should wait minimum 1ms and maximum
> >  	 * 100ms after writing the TMDS config for clock ratio. Lets allow a
> > -	 * wait of upto 2ms here.
> > +	 * wait of up to 2ms here.
> >  	 */
> >  	usleep_range(1000, 2000);
> >  	return true;
> > diff --git a/drivers/gpu/drm/drm_syncobj.c
> > b/drivers/gpu/drm/drm_syncobj.c index 1c5b9ef6da37..c9a9d74f338c
> > 100644
> > --- a/drivers/gpu/drm/drm_syncobj.c
> > +++ b/drivers/gpu/drm/drm_syncobj.c
> > @@ -725,7 +725,7 @@ static int drm_syncobj_export_sync_file(struct
> drm_file *file_private,
> >  	return ret;
> >  }
> >  /**
> > - * drm_syncobj_open - initalizes syncobj file-private structures at
> > devnode open time
> > + * drm_syncobj_open - initializes syncobj file-private structures at
> > + devnode open time
> >   * @file_private: drm file-private structure to set up
> >   *
> >   * Called at device open time, sets up the structure for handling
> > refcounting diff --git a/drivers/gpu/drm/drm_vblank.c
> > b/drivers/gpu/drm/drm_vblank.c index bba6781cc48f..977b6509bb4b 100644
> > --- a/drivers/gpu/drm/drm_vblank.c
> > +++ b/drivers/gpu/drm/drm_vblank.c
> > @@ -191,7 +191,7 @@ static u32 drm_max_vblank_count(struct drm_device
> > *dev, unsigned int pipe)
> >
> >  /*
> >   * "No hw counter" fallback implementation of .get_vblank_counter()
> > hook,
> > - * if there is no useable hardware frame counter available.
> > + * if there is no usable hardware frame counter available.
> >   */
> >  static u32 drm_vblank_no_hw_counter(struct drm_device *dev, unsigned
> > int pipe)  { @@ -905,7 +905,7 @@ drm_get_last_vbltimestamp(struct
> > drm_device *dev, unsigned int pipe,
> >   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
> >   * provide a barrier: Any writes done before calling
> >   * drm_crtc_handle_vblank() will be visible to callers of the later
> > - * functions, iff the vblank count is the same or a later one.
> > + * functions, if the vblank count is the same or a later one.
> >   *
> >   * See also &drm_vblank_crtc.count.
> >   *
> > @@ -968,7 +968,7 @@ static u64 drm_vblank_count_and_time(struct
> drm_device *dev, unsigned int pipe,
> >   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
> >   * provide a barrier: Any writes done before calling
> >   * drm_crtc_handle_vblank() will be visible to callers of the later
> > - * functions, iff the vblank count is the same or a later one.
> > + * functions, if the vblank count is the same or a later one.
> >   *
> >   * See also &drm_vblank_crtc.count.
> >   */
> > @@ -1997,7 +1997,7 @@ EXPORT_SYMBOL(drm_handle_vblank);
> >   * and drm_crtc_vblank_count() or drm_crtc_vblank_count_and_time()
> >   * provide a barrier: Any writes done before calling
> >   * drm_crtc_handle_vblank() will be visible to callers of the later
> > - * functions, iff the vblank count is the same or a later one.
> > + * functions, if the vblank count is the same or a later one.
> >   *
> >   * See also &drm_vblank_crtc.count.
> >   *
> > @@ -2014,7 +2014,7 @@ EXPORT_SYMBOL(drm_crtc_handle_vblank);
> >   * Get crtc VBLANK count.
> >   *
> >   * \param dev DRM device
> > - * \param data user arguement, pointing to a drm_crtc_get_sequence
> structure.
> > + * \param data user argument, pointing to a drm_crtc_get_sequence
> structure.
> >   * \param file_priv drm file private for the user's open file descriptor
> >   */
> >
> > @@ -2070,7 +2070,7 @@ int drm_crtc_get_sequence_ioctl(struct drm_device
> *dev, void *data,
> >   * Queue a event for VBLANK sequence
> >   *
> >   * \param dev DRM device
> > - * \param data user arguement, pointing to a drm_crtc_queue_sequence
> structure.
> > + * \param data user argument, pointing to a drm_crtc_queue_sequence
> structure.
> >   * \param file_priv drm file private for the user's open file descriptor
> >   */
> >
> > diff --git a/drivers/gpu/drm/drm_vma_manager.c
> > b/drivers/gpu/drm/drm_vma_manager.c
> > index 4565319fa6b3..7de37f8c68fd 100644
> > --- a/drivers/gpu/drm/drm_vma_manager.c
> > +++ b/drivers/gpu/drm/drm_vma_manager.c
> > @@ -361,7 +361,7 @@ EXPORT_SYMBOL(drm_vma_node_revoke);
> >   * This is locked against concurrent access internally.
> >   *
> >   * RETURNS:
> > - * true iff @filp is on the list
> > + * true if @filp is on the list
> >   */
> >  bool drm_vma_node_is_allowed(struct drm_vma_offset_node *node,
> >  			     struct drm_file *tag)
> > --
> > 2.25.1
> >
> 
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-02  8:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-30 13:27 [PATCH v2] drm: Fix typo in comments Cai Huoqing
2021-08-02  8:19 ` Daniel Vetter
2021-08-02  8:34   ` Cai,Huoqing

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.