Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-08-19 Thread Stephen Rothwell
Hi Hans, On Sun, 19 Aug 2018 10:21:40 +0200 Hans de Goede wrote: > > On 18-08-18 16:35, Stephen Rothwell wrote: > > Hi all, > > > > Commit > > > >cda5915d15d3 ("platform/x86: touchscreen_dmi: Add info for the Cube > > KNote i1101 tablet") > > For reasons which I do not know youling

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-08-19 Thread Hans de Goede
Hi, On 18-08-18 16:35, Stephen Rothwell wrote: Hi all, Commit cda5915d15d3 ("platform/x86: touchscreen_dmi: Add info for the Cube KNote i1101 tablet") For reasons which I do not know youling does not wish to use his real name. So as with the "platform/x86: touchscreen_dmi: Add info for

linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-08-18 Thread Stephen Rothwell
Hi all, Commit cda5915d15d3 ("platform/x86: touchscreen_dmi: Add info for the Cube KNote i1101 tablet") is missing a Signed-off-by from its author. -- Cheers, Stephen Rothwell pgpo0vHZOCf_2.pgp Description: OpenPGP digital signature

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Darren Hart
On Sat, Jun 02, 2018 at 01:26:46AM +1000, Stephen Rothwell wrote: > Hi, > > On Fri, 01 Jun 2018 07:38:42 -0700 dvh...@infradead.org wrote: > > > > Stephen, are the tests you use available publicly? > > I use the script below when I am fetching trees. You don't want the > "gitk" invocation if

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Stephen Rothwell
Hi, On Fri, 01 Jun 2018 07:38:42 -0700 dvh...@infradead.org wrote: > > Stephen, are the tests you use available publicly? I use the script below when I am fetching trees. You don't want the "gitk" invocation if you are doing this automatically, of course. The script just takes a commit range.

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Andy Shevchenko
On Fri, Jun 1, 2018 at 5:33 PM, wrote: > On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell > wrote: >>On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko >> wrote: >>> >>> Oops. What is the proposed fix for that? It seems we can't rebase >>> published branches. >> >>It's unfixable without a

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Andy Shevchenko
On Fri, Jun 1, 2018 at 3:08 PM, Stephen Rothwell wrote: > Hi Andy, > > On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko > wrote: >> >> Oops. What is the proposed fix for that? It seems we can't rebase >> published branches. > > It's unfixable without a rebase, so you could instead consider it

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread dvhart
On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell wrote: >Hi Andy, > >On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko > wrote: >> >> Oops. What is the proposed fix for that? It seems we can't rebase >> published branches. > >It's unfixable without a rebase, so you could instead consider it an

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread dvhart
On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell wrote: >Hi Andy, > >On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko > wrote: >> >> Oops. What is the proposed fix for that? It seems we can't rebase >> published branches. > >It's unfixable without a rebase, so you could instead consider it an

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Stephen Rothwell
Hi Andy, On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko wrote: > > Oops. What is the proposed fix for that? It seems we can't rebase > published branches. It's unfixable without a rebase, so you could instead consider it an opportunity to improve your processes for the future. :-) --

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Andy Shevchenko
On Fri, Jun 1, 2018 at 2:36 PM, Stephen Rothwell wrote: > Hi all, > > Commit > > bba074f926d1 ("platform/x86: silead_dmi: Add entry for Chuwi Hi8 S806_206 > tablet touchscreen") > > is missing a Signed-off-by from its author. Oops. What is the proposed fix for that? It seems we can't rebase

linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2018-06-01 Thread Stephen Rothwell
Hi all, Commit bba074f926d1 ("platform/x86: silead_dmi: Add entry for Chuwi Hi8 S806_206 tablet touchscreen") is missing a Signed-off-by from its author. -- Cheers, Stephen Rothwell pgpFtHG0Jixuv.pgp Description: OpenPGP digital signature

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-24 Thread Darren Hart
On Sat, Aug 05, 2017 at 02:58:29PM -0700, Darren Hart wrote: > On Thu, Aug 03, 2017 at 08:50:06AM -0700, Darren Hart wrote: > > On Wed, Aug 02, 2017 at 06:06:20PM -0700, Linus Torvalds wrote: > > > On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell > > > wrote: > > > > > > >

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-16 Thread Darren Hart
+Olof and Arnd, I am curious how you handle the situation below as a maintainer team. This problem arose from a new for-next test which triggers on the committer not having a SOB tag in the patch (which can happen when a shared branch is rebased to drop a patch). Do you have a branch that you

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-05 Thread Darren Hart
On Thu, Aug 03, 2017 at 08:50:06AM -0700, Darren Hart wrote: > On Wed, Aug 02, 2017 at 06:06:20PM -0700, Linus Torvalds wrote: > > On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell > > wrote: > > > > > > I would say that if you rebase someone's commit(s), then you are on

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-04 Thread Darren Hart
On Fri, Aug 04, 2017 at 10:44:31AM -0700, Junio C Hamano wrote: > Linus Torvalds writes: > > > On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell > > wrote: > >> > >> I would say that if you rebase someone's commit(s), then you are on the >

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-04 Thread Junio C Hamano
Linus Torvalds writes: > On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell > wrote: >> >> I would say that if you rebase someone's commit(s), then you are on the >> "patch's delivery path" and so should add a Signed-off-by tag. > > Yeah, I

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-03 Thread Darren Hart
On Wed, Aug 02, 2017 at 06:06:20PM -0700, Linus Torvalds wrote: > On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell > wrote: > > > > I would say that if you rebase someone's commit(s), then you are on the > > "patch's delivery path" and so should add a Signed-off-by tag. >

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-03 Thread Stephen Rothwell
Hi Andy, On Thu, 03 Aug 2017 11:17:03 +0300 Andy Shevchenko wrote: > > I just checked what we have in our for-next branch and mentioned commits > have mine SoB. Should they have something else? In Darren's response it became clear that even though you had

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-03 Thread Andy Shevchenko
On Thu, 2017-08-03 at 06:37 +1000, Stephen Rothwell wrote: > Hi Darren, > > Commits > >   890f658c101d ("platform/x86: peaq-wmi: silence a static checker > warning") >   6d8d55626296 ("platform/x86: msi-wmi: remove unnecessary static in > msi_wmi_notify()") >   cd0223c64c60 ("platform/x86:

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-02 Thread Linus Torvalds
On Wed, Aug 2, 2017 at 5:28 PM, Stephen Rothwell wrote: > > I would say that if you rebase someone's commit(s), then you are on the > "patch's delivery path" and so should add a Signed-off-by tag. Yeah, I agree. Rebasing really is pretty much the exact same thing as

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-02 Thread Stephen Rothwell
Hi Darren, On Wed, 2 Aug 2017 16:57:40 -0700 Darren Hart wrote: > > Is this a new check Stephen? Yes :-) > Is there any statement regarding maintainer teams that we must abide by > this? e.g. any time a rebase in a testing branch is made, the > maintainer must also

Re: linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-02 Thread Darren Hart
On Thu, Aug 03, 2017 at 06:37:43AM +1000, Stephen Rothwell wrote: > Hi Darren, > > Commits > > 890f658c101d ("platform/x86: peaq-wmi: silence a static checker warning") > 6d8d55626296 ("platform/x86: msi-wmi: remove unnecessary static in > msi_wmi_notify()") > cd0223c64c60 ("platform/x86:

linux-next: Signed-off-by missing for commit in the drivers-x86 tree

2017-08-02 Thread Stephen Rothwell
Hi Darren, Commits 890f658c101d ("platform/x86: peaq-wmi: silence a static checker warning") 6d8d55626296 ("platform/x86: msi-wmi: remove unnecessary static in msi_wmi_notify()") cd0223c64c60 ("platform/x86: ibm_rtl: remove unnecessary static in ibm_rtl_write()") are missing