Re: [PATCH] Fix spelling error for 'parallel'

[Jonathan Corbet <corbet@lwn.net>]

Anish Dabhane <anishdabhane71@gmail.com> writes:

> Signed-off-by: Anish Dabhane <anishdabhane71@gmail.com>
> ---
>  Documentation/gpu/rfc/i915_scheduler.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/gpu/rfc/i915_scheduler.rst b/Documentation/gpu/rfc/i915_scheduler.rst
> index c237ebc024cd..2974525f0ac5 100644
> --- a/Documentation/gpu/rfc/i915_scheduler.rst
> +++ b/Documentation/gpu/rfc/i915_scheduler.rst
> @@ -26,7 +26,7 @@ i915 with the DRM scheduler is:
>  	  which configures a slot with N contexts
>  	* After I915_CONTEXT_ENGINES_EXT_PARALLEL a user can submit N batches to
>  	  a slot in a single execbuf IOCTL and the batches run on the GPU in
> -	  paralllel
> +	  parallel
>  	* Initially only for GuC submission but execlists can be supported if
>  	  needed

The patch itself is fine, but a couple of nits that are worth picking:

- Always use scripts/get_maintainer.pl to see who you should send a
  patch to.  It should have told you to include the DRM maintainers on
  this one, since they generally handle their own docs.

- It is also a good idea to look at patches for the subsystem in
  question and match their subject-line style.  In this case you would
  see examples like:

> 0454a490bdeb drm/doc/rfc: i915 new parallel submission uAPI plan
> f587623b78ff drm/doc/rfc: i915 GuC submission / DRM scheduler

- Some maintainers will never accept patches with empty changelog text.

For a patch like this, none of these are enough to ask you to resend,
but issues like that will get in your way for future work.

I've applied it, thanks for helping to make our documentation better.

jon