[PATCH v1 7/8] drm/i915/display: fix typos in i915/display files
Jani Nikula
jani.nikula at linux.intel.com
Tue Jan 7 12:51:58 UTC 2025
On Mon, 06 Jan 2025, Nitin Gote <nitin.r.gote at intel.com> wrote:
> @@ -8440,7 +8440,7 @@ intel_mode_valid_max_plane_size(struct drm_i915_private *dev_priv,
>
> /*
> * Most people will probably want a fullscreen
> - * plane so let's not advertize modes that are
> + * plane so let's not advertise modes that are
> * too big for that.
> */
> if (DISPLAY_VER(dev_priv) >= 30) {
I appreciate the typo fixes, but there's an explicit policy that certain
things are not typos and should not be fixed. This includes British
vs. American spelling.
Can you not tell codespell that?
> @@ -2552,7 +2552,7 @@ intel_dp_compute_config_limits(struct intel_dp *intel_dp,
> * seem to suggest we should do otherwise.
> *
> * Use the maximum clock and number of lanes the eDP panel
> - * advertizes being capable of in case the initial fast
> + * advertises being capable of in case the initial fast
> * optimal params failed us. The panels are generally
> * designed to support only a single clock and lane
> * configuration, and typically on older panels these
Ditto.
> @@ -442,8 +444,10 @@ static int intel_overlay_off(struct intel_overlay *overlay)
> return i915_active_wait(&overlay->last_flip);
> }
>
> -/* recover from an interruption due to a signal
> - * We have to be careful not to repeat work forever an make forward progess. */
> +/*
> + * recover from an interruption due to a signal
> + * We have to be careful not to repeat work forever an make forward progress.
> + */
If you're changing the comment format while at it, might as well
capitalize the first sentence and add a period.
BR,
Jani.
--
Jani Nikula, Intel
More information about the Intel-gfx
mailing list