2019-05-20 14:56:30

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: Fixes tag needs some work in the drm-intel tree

Hi all,

In commit

0d90ccb70211 ("drm/i915: Delay semaphore submission until the start of the signaler")

Fixes tag

Fixes: e88619646971 ("drm/i915: Use HW semaphores for inter-engine synchroni

has these problem(s):

- Subject has leading but no trailing parentheses
- Subject has leading but no trailing quotes

Please don't split Fixes tags across more than one line.



--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2019-05-21 13:04:17

by Joonas Lahtinen

[permalink] [raw]
Subject: Re: linux-next: Fixes tag needs some work in the drm-intel tree

Quoting Stephen Rothwell (2019-05-20 15:15:38)
> Hi all,
>
> In commit
>
> 0d90ccb70211 ("drm/i915: Delay semaphore submission until the start of the signaler")
>
> Fixes tag
>
> Fixes: e88619646971 ("drm/i915: Use HW semaphores for inter-engine synchroni
>
> has these problem(s):
>
> - Subject has leading but no trailing parentheses
> - Subject has leading but no trailing quotes
>
> Please don't split Fixes tags across more than one line.

Thanks for the report.

This was a copy'n paste mishap, detected by our tooling (and fixed by
me) at the time of creating a PR. Unfortunately the check was not being
enforced by tooling at commit time. We'll fix that.

Regards, Joonas

2019-05-21 13:06:33

by Joonas Lahtinen

[permalink] [raw]
Subject: Comments in Fixes: line (Was: Re: linux-next: Fixes tag needs some work in the drm-intel tree)

We also have an incoming patch where the Fixes: line has a comment in
it. Does your tooling account for this when checking the Fixes: line?

Regards, Joonas

2019-05-21 21:44:37

by Stephen Rothwell

[permalink] [raw]
Subject: Re: Comments in Fixes: line (Was: Re: linux-next: Fixes tag needs some work in the drm-intel tree)

Hi Joonas,

On Tue, 21 May 2019 16:04:16 +0300 Joonas Lahtinen <[email protected]> wrote:
>
> We also have an incoming patch where the Fixes: line has a comment in
> it. Does your tooling account for this when checking the Fixes: line?

I will make sure mine does.

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature