Re: [PATCH] drm/uapi: drm_mode.h: fix spellos and grammar

[Jani Nikula <jani.nikula@linux.intel.com>]

On Tue, 12 Dec 2023, Randy Dunlap <rdunlap@infradead.org> wrote:
> Correct spellos reported by codespell.
> Fix some grammar (as 's' to a few words).
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Thomas Zimmermann <tzimmermann@suse.de>

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

> ---
>  include/uapi/drm/drm_mode.h |   18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff -- a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
> --- a/include/uapi/drm/drm_mode.h
> +++ b/include/uapi/drm/drm_mode.h
> @@ -36,10 +36,10 @@ extern "C" {
>  /**
>   * DOC: overview
>   *
> - * DRM exposes many UAPI and structure definition to have a consistent
> - * and standardized interface with user.
> + * DRM exposes many UAPI and structure definitions to have a consistent
> + * and standardized interface with users.
>   * Userspace can refer to these structure definitions and UAPI formats
> - * to communicate to driver
> + * to communicate to drivers.
>   */
>  
>  #define DRM_CONNECTOR_NAME_LEN	32
> @@ -540,7 +540,7 @@ struct drm_mode_get_connector {
>  /* the PROP_ATOMIC flag is used to hide properties from userspace that
>   * is not aware of atomic properties.  This is mostly to work around
>   * older userspace (DDX drivers) that read/write each prop they find,
> - * witout being aware that this could be triggering a lengthy modeset.
> + * without being aware that this could be triggering a lengthy modeset.
>   */
>  #define DRM_MODE_PROP_ATOMIC        0x80000000
>  
> @@ -664,7 +664,7 @@ struct drm_mode_fb_cmd {
>  };
>  
>  #define DRM_MODE_FB_INTERLACED	(1<<0) /* for interlaced framebuffers */
> -#define DRM_MODE_FB_MODIFIERS	(1<<1) /* enables ->modifer[] */
> +#define DRM_MODE_FB_MODIFIERS	(1<<1) /* enables ->modifier[] */
>  
>  /**
>   * struct drm_mode_fb_cmd2 - Frame-buffer metadata.
> @@ -881,8 +881,8 @@ struct hdr_metadata_infoframe {
>  	 * These are coded as unsigned 16-bit values in units of
>  	 * 0.00002, where 0x0000 represents zero and 0xC350
>  	 * represents 1.0000.
> -	 * @display_primaries.x: X cordinate of color primary.
> -	 * @display_primaries.y: Y cordinate of color primary.
> +	 * @display_primaries.x: X coordinate of color primary.
> +	 * @display_primaries.y: Y coordinate of color primary.
>  	 */
>  	struct {
>  		__u16 x, y;
> @@ -892,8 +892,8 @@ struct hdr_metadata_infoframe {
>  	 * These are coded as unsigned 16-bit values in units of
>  	 * 0.00002, where 0x0000 represents zero and 0xC350
>  	 * represents 1.0000.
> -	 * @white_point.x: X cordinate of whitepoint of color primary.
> -	 * @white_point.y: Y cordinate of whitepoint of color primary.
> +	 * @white_point.x: X coordinate of whitepoint of color primary.
> +	 * @white_point.y: Y coordinate of whitepoint of color primary.
>  	 */
>  	struct {
>  		__u16 x, y;

-- 
Jani Nikula, Intel