Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1696805 > unrolled thread

linux-next: unsigned commits in the drm-misc tree

Back to article view | Back to linux.kernel

Contents

  linux-next: unsigned commits in the drm-misc tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-07-26 08:20 +0200
    Re: linux-next: unsigned commits in the drm-misc tree Daniel Vetter <daniel.vetter@ffwll.ch> - 2017-07-26 09:10 +0200
      Re: linux-next: unsigned commits in the drm-misc tree Daniel Vetter <daniel.vetter@ffwll.ch> - 2017-07-26 14:30 +0200

#1696805 — 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

[toc] | [next] | [standalone]

#1696833

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 <sfr@canb.auug.org.au> 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

[toc] | [prev] | [next] | [standalone]

#1697047

On Wed, Jul 26, 2017 at 9:09 AM, Daniel Vetter <daniel.vetter@ffwll.ch> 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 <sfr@canb.auug.org.au> 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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web