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

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

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 194 bytes --]

Hi all,

Commit

  ee5596ec15d8 ("dt-bindings: rcc: Add optional external ethernet RX clock properties")

is missing a Signed-off-by from its committer.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

RE: linux-next: Signed-off-by missing for commit in the stm32 tree

[Alexandre TORGUE]

Hi Stephen,


ST Restricted

> -----Original Message-----
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Sent: Tuesday, May 3, 2022 11:28 PM
> To: Alexandre TORGUE <alexandre.torgue@st.com>
> Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Linux 
> Next Mailing List <linux-next@vger.kernel.org>
> Subject: linux-next: Signed-off-by missing for commit in the stm32 
> tree
> 
> Hi all,
> 
> Commit
> 
>   ee5596ec15d8 ("dt-bindings: rcc: Add optional external ethernet RX 
> clock
> properties")
> 
> is missing a Signed-off-by from its committer.
>

Actually this patch has been already merged in Rob tree. I just cherry-pick it in my tree to avoid a merge conflict later for other maintainers. 
So I didn't add my "Signed-off-by".  

Cheers
Alex

> --
> Cheers,
> Stephen Rothwell

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

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 841 bytes --]

Hi Alexandre,

On Wed, 4 May 2022 07:28:53 +0000 Alexandre TORGUE <alexandre.torgue@st.com> wrote:
>
> Actually this patch has been already merged in Rob tree. I just
> cherry-pick it in my tree to avoid a merge conflict later for other
> maintainers. So I didn't add my "Signed-off-by".  

You should add a SOB for every patch you add to your published tree.
That includes cherry-picked commits from other trees.

By the way, most maintainers (and Linus and I) are pretty adept at
sorting out merge conflicts (unless they are really complex, or course,
in which case you should probably have created a branch in one fo the
trees containing the conflicting commits from that tree and then merge
that branch into the both trees - and, of course, noted what is
happening in the merge commits).
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

RE: linux-next: Signed-off-by missing for commit in the stm32 tree

[Alexandre TORGUE]

Stephen


ST Restricted

> -----Original Message-----
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Sent: Wednesday, May 4, 2022 9:42 AM
> To: Alexandre TORGUE <alexandre.torgue@st.com>
> Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Linux 
> Next Mailing List <linux-next@vger.kernel.org>
> Subject: Re: linux-next: Signed-off-by missing for commit in the stm32 
> tree
> 
> Hi Alexandre,
> 
> On Wed, 4 May 2022 07:28:53 +0000 Alexandre TORGUE 
> <alexandre.torgue@st.com> wrote:
> >
> > Actually this patch has been already merged in Rob tree. I just 
> > cherry-pick it in my tree to avoid a merge conflict later for other 
> > maintainers. So I didn't add my "Signed-off-by".
> 
> You should add a SOB for every patch you add to your published tree.
> That includes cherry-picked commits from other trees.
>

Ok I add it now and update the stm32-next branch.

> By the way, most maintainers (and Linus and I) are pretty adept at 
> sorting out merge conflicts (unless they are really complex, or 
> course, in which case you should probably have created a branch in one 
> fo the trees containing the conflicting commits from that tree and 
> then merge that branch into the both trees - and, of course, noted what is happening in the merge commits).
> --

I have no doubt that you could have fixed it 😊. It was also easier for me to add my changes on top of it.

Cheers
Alex 


> Cheers,
> Stephen Rothwell

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

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 200 bytes --]

Hi all,

Commit

  b3c4c7346bc9 ("firmware: arm_scmi: Relax base protocol sanity checks on the protocol list")

is missing a Signed-off-by from its committer.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]