2017-07-26 06:11:29

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: unsigned commits in the drm-misc tree

Hi all,

I noticed a set of commits that have no Signed-off-by from their
committer:

d9864a1d2dfc ("drm/stm: drv: Rename platform driver name")

to

ed34d261a12a ("drm/stm: dsi: Constify phy ops structure")

--
Cheers,
Stephen Rothwell


2017-07-26 07:09:50

by Daniel Vetter

[permalink] [raw]
Subject: Re: linux-next: unsigned commits in the drm-misc tree

Oops, that shouldn't have happened. Actually, our maintainer tooling
ensures this doesn't happen, by auto-adding the committer sob line.
But these patches (and a bunch of others pushed by Benjamin) haven't
been pushed by our tooling it seems (the Link: tag is missing at
least).

Benjamin, what happened there?

Thanks, Daniel


On Wed, Jul 26, 2017 at 8:11 AM, Stephen Rothwell <[email protected]> wrote:
> Hi all,
>
> I noticed a set of commits that have no Signed-off-by from their
> committer:
>
> d9864a1d2dfc ("drm/stm: drv: Rename platform driver name")
>
> to
>
> ed34d261a12a ("drm/stm: dsi: Constify phy ops structure")
>
> --
> Cheers,
> Stephen Rothwell



--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

2017-07-26 12:20:32

by Daniel Vetter

[permalink] [raw]
Subject: Re: linux-next: unsigned commits in the drm-misc tree

On Wed, Jul 26, 2017 at 9:09 AM, Daniel Vetter <[email protected]> wrote:
> Oops, that shouldn't have happened. Actually, our maintainer tooling
> ensures this doesn't happen, by auto-adding the committer sob line.
> But these patches (and a bunch of others pushed by Benjamin) haven't
> been pushed by our tooling it seems (the Link: tag is missing at
> least).
>
> Benjamin, what happened there?

Ok, figured it out, added another safety check to the scripting, and
hard-reset the tree. Unfortunately some of the patches already landed
in drm-next, so that needed a hard-reset too, plus in drm-intel-next,
where I still need to do the hard-reset. Ugh.

Benjamin: As part of the hard-reset I've thrown out all the patches
you've committed. That was simpler than digging out the right patches
from the rebase push. Please re-apply and push the right ones again.

My apologies for the hiccup, we maintainers (Dave, Sean & me) should
have caught this earlier.

Thanks, Daniel

>
> Thanks, Daniel
>
>
> On Wed, Jul 26, 2017 at 8:11 AM, Stephen Rothwell <[email protected]> wrote:
>> Hi all,
>>
>> I noticed a set of commits that have no Signed-off-by from their
>> committer:
>>
>> d9864a1d2dfc ("drm/stm: drv: Rename platform driver name")
>>
>> to
>>
>> ed34d261a12a ("drm/stm: dsi: Constify phy ops structure")
>>
>> --
>> Cheers,
>> Stephen Rothwell
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch



--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch