LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* linux-next: Signed-off-by missing for commits in the drm tree
@ 2018-05-15 23:18 Stephen Rothwell
  2018-05-17 15:27 ` Koo, Anthony
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2018-05-15 23:18 UTC (permalink / raw)
  To: Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Anthony Koo,
	Harry Wentland, Alex Deucher

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

Hi all,

Commits

  f412e8307d0a ("drm/amd/display: Couple bug fixes in stats module")
  e09b6473c605 ("drm/amd/display: Rename encoder_info_packet to dc_info_packet")
  87943159f409 ("drm/amd/display: Only program MSA_TIMING_PARAM if it changed")

are missing a Signed-off-by from their author.

-- 
Cheers,
Stephen Rothwell

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

^ permalink raw reply	[flat|nested] 14+ messages in thread

* RE: linux-next: Signed-off-by missing for commits in the drm tree
  2018-05-15 23:18 linux-next: Signed-off-by missing for commits in the drm tree Stephen Rothwell
@ 2018-05-17 15:27 ` Koo, Anthony
  0 siblings, 0 replies; 14+ messages in thread
From: Koo, Anthony @ 2018-05-17 15:27 UTC (permalink / raw)
  To: Stephen Rothwell, Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Wentland,
	Harry, Deucher, Alexander

Hi Stephen,

Changes are Signed-off-by: Anthony Koo <Anthony.Koo@amd.com>

Thanks,
Anthony

-----Original Message-----
From: Stephen Rothwell [mailto:sfr@canb.auug.org.au] 
Sent: Tuesday, May 15, 2018 7:19 PM
To: Dave Airlie <airlied@linux.ie>; DRI <dri-devel@lists.freedesktop.org>
Cc: Linux-Next Mailing List <linux-next@vger.kernel.org>; Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Koo, Anthony <Anthony.Koo@amd.com>; Wentland, Harry <Harry.Wentland@amd.com>; Deucher, Alexander <Alexander.Deucher@amd.com>
Subject: linux-next: Signed-off-by missing for commits in the drm tree

Hi all,

Commits

  f412e8307d0a ("drm/amd/display: Couple bug fixes in stats module")
  e09b6473c605 ("drm/amd/display: Rename encoder_info_packet to dc_info_packet")
  87943159f409 ("drm/amd/display: Only program MSA_TIMING_PARAM if it changed")

are missing a Signed-off-by from their author.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2018-03-21  9:27 Stephen Rothwell
@ 2018-03-23 11:55 ` Tomi Valkeinen
  0 siblings, 0 replies; 14+ messages in thread
From: Tomi Valkeinen @ 2018-03-23 11:55 UTC (permalink / raw)
  To: Stephen Rothwell, Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Laurent Pinchart

On 21/03/18 11:27, Stephen Rothwell wrote:
> Hi all,
> 
> A series of commits
> 
>   a82f034765fa ("drm: omapdrm: Split init and cleanup from probe and remove functions")
> to
>   663ac57b285d ("drm: omapdrm: venc: Allocate the venc private data structure dynamically")
> 
> are missing a Signed-off-by from their commiter.

Sorry about that again. I now have a script to check this, so I hope
this was the last time...

 Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2018-03-21  9:27 Stephen Rothwell
  2018-03-23 11:55 ` Tomi Valkeinen
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2018-03-21  9:27 UTC (permalink / raw)
  To: Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Tomi Valkeinen, Laurent Pinchart

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

Hi all,

A series of commits

  a82f034765fa ("drm: omapdrm: Split init and cleanup from probe and remove functions")
to
  663ac57b285d ("drm: omapdrm: venc: Allocate the venc private data structure dynamically")

are missing a Signed-off-by from their commiter.

-- 
Cheers,
Stephen Rothwell

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

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2017-12-21 11:40 ` Tomi Valkeinen
@ 2017-12-21 13:06   ` Daniel Vetter
  0 siblings, 0 replies; 14+ messages in thread
From: Daniel Vetter @ 2017-12-21 13:06 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Stephen Rothwell, Dave Airlie, DRI, Linux-Next Mailing List,
	Linux Kernel Mailing List

On Thu, Dec 21, 2017 at 01:40:28PM +0200, Tomi Valkeinen wrote:
> Hi,
> 
> On 21/12/17 07:12, Stephen Rothwell wrote:
> > Hi all,
> > 
> > Commits
> > 
> >    bb5cdf8d1c76 ("drm: omapdrm: Remove filename from header and fix copyright tag")
> >    d66c36a3ee79 ("drm: omapdrm: Simplify platform registration")
> > 
> > are missing a Signed-off-by from their committer.
> 
> Sorry about that, I missed those.
> 
> I think I need to create a script to verify my pull requests for things like
> this...

We have one, but it's tied to the entire group maintainership thing ...
But omap is defacto group-maintained between you and Laurent already, so
might be worth to check out.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2017-12-21  5:12 Stephen Rothwell
@ 2017-12-21 11:40 ` Tomi Valkeinen
  2017-12-21 13:06   ` Daniel Vetter
  0 siblings, 1 reply; 14+ messages in thread
From: Tomi Valkeinen @ 2017-12-21 11:40 UTC (permalink / raw)
  To: Stephen Rothwell, Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List

Hi,

On 21/12/17 07:12, Stephen Rothwell wrote:
> Hi all,
> 
> Commits
> 
>    bb5cdf8d1c76 ("drm: omapdrm: Remove filename from header and fix copyright tag")
>    d66c36a3ee79 ("drm: omapdrm: Simplify platform registration")
> 
> are missing a Signed-off-by from their committer.

Sorry about that, I missed those.

I think I need to create a script to verify my pull requests for things 
like this...

  Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. 
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2017-12-21  5:12 Stephen Rothwell
  2017-12-21 11:40 ` Tomi Valkeinen
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2017-12-21  5:12 UTC (permalink / raw)
  To: Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Tomi Valkeinen

Hi all,

Commits

  bb5cdf8d1c76 ("drm: omapdrm: Remove filename from header and fix copyright tag")
  d66c36a3ee79 ("drm: omapdrm: Simplify platform registration")

are missing a Signed-off-by from their committer.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2017-12-05  2:45 Stephen Rothwell
@ 2017-12-05  9:36 ` Laurent Pinchart
  0 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2017-12-05  9:36 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Dave Airlie, DRI, Linux-Next Mailing List,
	Linux Kernel Mailing List, Fabrizio Castro, Laurent Pinchart,
	Biju Das, Kieran Bingham

Hi Stephen,

On Tuesday, 5 December 2017 04:45:50 EET Stephen Rothwell wrote:
> Hi all,
> 
> Commits
> 
>   36a46da90212 ("drm: rcar-du: Add R8A7743 support")
>   cdd907001572 ("drm: rcar-du: Add R8A7745 support")
>   7912dee7775e ("drm: rcar-du: Implement system suspend/resume support")
>   cf05f74ef40e ("drm: rcar-du: Remove unused CRTC suspend/resume functions")
> 
> are missing a Signed-off-by from their committer.

I've sent lots of pull requests to maintainers in the past where I have picked 
patches signed off by the original author without any further modification and 
only added my Acked-by or Reviewed-by line. I thought this was an accepted 
practice as nobody ever complained, but I'll make sure to always include my 
SoB for any patch that goes through my tree in the future. I'm sorry about 
this, it's all my fault.

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2017-12-05  2:45 Stephen Rothwell
  2017-12-05  9:36 ` Laurent Pinchart
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2017-12-05  2:45 UTC (permalink / raw)
  To: Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Fabrizio Castro, Laurent Pinchart, Biju Das, Kieran Bingham

Hi all,

Commits

  36a46da90212 ("drm: rcar-du: Add R8A7743 support")
  cdd907001572 ("drm: rcar-du: Add R8A7745 support")
  7912dee7775e ("drm: rcar-du: Implement system suspend/resume support")
  cf05f74ef40e ("drm: rcar-du: Remove unused CRTC suspend/resume functions")

are missing a Signed-off-by from their committer.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2017-12-03 23:57 Stephen Rothwell
@ 2017-12-04  2:32 ` Zhenyu Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Zhenyu Wang @ 2017-12-04  2:32 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Dave Airlie, DRI, Linux-Next Mailing List,
	Linux Kernel Mailing List, Zhenyu Wang, Daniel Vetter,
	Intel Graphics, Rodrigo Vivi, Zhi Wang

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

On 2017.12.04 10:57:35 +1100, Stephen Rothwell wrote:
> Hi all,
> 
> There is a series of commits
> 
>   54cff6479fd8 - c1802534e5a6
> (not all in that range)
> 
> that are missing a Signed-off-by from their committer.
> 

They were originally committed by Zhi himself, but I had to rebase
onto intel -next before they're sent for pull and missed my signed-off..
Really sorry for that! I think maybe we could still be able to fix that
before next kernel merge.

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2017-12-03 23:57 Stephen Rothwell
  2017-12-04  2:32 ` Zhenyu Wang
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2017-12-03 23:57 UTC (permalink / raw)
  To: Dave Airlie, DRI
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Zhenyu Wang,
	Daniel Vetter, Intel Graphics, Rodrigo Vivi, Zhi Wang

Hi all,

There is a series of commits

  54cff6479fd8 - c1802534e5a6
(not all in that range)

that are missing a Signed-off-by from their committer.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: linux-next: Signed-off-by missing for commits in the drm tree
  2017-08-29  1:44 Stephen Rothwell
@ 2017-08-29  1:51 ` Sinclair Yeh
  0 siblings, 0 replies; 14+ messages in thread
From: Sinclair Yeh @ 2017-08-29  1:51 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Dave Airlie, Linux-Next Mailing List, Linux Kernel Mailing List,
	Andrzej Hajda, Inki Dae, Tobias Jakobi, Thomas Hellstrom

On Tue, Aug 29, 2017 at 11:44:13AM +1000, Stephen Rothwell wrote:
> Hi Dave,
> 
> Commits
> 
>   461e60ea1119 ("drm/exynos/decon5433: use mode info stored in CRTC to detect i80 mode")
>   ac60944ccf23 ("drm/exynos: consistent use of cpp")
>   e300173f0616 ("drm/vmwgfx: Don't use drm_irq_[un]install")

This one is Reviewed-by: Sinclair Yeh <syeh@vmware.com>

>   ef369904aaf7 ("drm/vmwgfx: Move irq bottom half processing to threads")
>   65b97a2bec2f ("drm/vmwgfx: Restart command buffers after errors")
>   5f55be5f306a ("drm/vmwgfx: Support the NOP_ERROR command")
>   1f1a36cc4d49 ("drm/vmwgfx: Fix incorrect command header offset at restart")a

For these 4, I thought SOB the author is enough, but if not, I can add SOB.

Is it easier for you to add those, or would you like me to rebase and send out another
pull request?

thanks,

Sinclair

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2017-08-29  1:44 Stephen Rothwell
  2017-08-29  1:51 ` Sinclair Yeh
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Rothwell @ 2017-08-29  1:44 UTC (permalink / raw)
  To: Dave Airlie
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Andrzej Hajda, Inki Dae, Tobias Jakobi, Thomas Hellstrom,
	Sinclair Yeh

Hi Dave,

Commits

  461e60ea1119 ("drm/exynos/decon5433: use mode info stored in CRTC to detect i80 mode")
  ac60944ccf23 ("drm/exynos: consistent use of cpp")
  e300173f0616 ("drm/vmwgfx: Don't use drm_irq_[un]install")
  ef369904aaf7 ("drm/vmwgfx: Move irq bottom half processing to threads")
  65b97a2bec2f ("drm/vmwgfx: Restart command buffers after errors")
  5f55be5f306a ("drm/vmwgfx: Support the NOP_ERROR command")
  1f1a36cc4d49 ("drm/vmwgfx: Fix incorrect command header offset at restart")

are missing a Signed-off-by from their committers.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

* linux-next: Signed-off-by missing for commits in the drm tree
@ 2017-08-02  6:54 Stephen Rothwell
  0 siblings, 0 replies; 14+ messages in thread
From: Stephen Rothwell @ 2017-08-02  6:54 UTC (permalink / raw)
  To: Dave Airlie
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Alex Deucher

Hi Dave,

Commits

  663ebbf63180 "drm/amdgpu: trace VM flags as 64bits"
  5327dd8acf05 "drm/amdgpu: remove stale TODO comment"
  eabd76cef900 "drm/amd/sched: print sched job id in amd_sched_job trace"
  a69c7e0138fc "drm/amdgpu: update pctl1 ram index/data for mmhub on raven"
  7504938f8e73 "drm/amdgpu: add check when no firmware need to load"
  98a7f88ce9a9 "drm/amdgpu: bind BOs with GTT space allocated directly v2"
  92c60d9cf663 "drm/amdgpu: bind BOs to TTM only once"
  fc9c8f5459f2 "drm/amdgpu: add vm_needs_flush parameter to amdgpu_copy_buffer"
  df264f9e0808 "drm/amdgpu: allow flushing VMID0 before IB execution as well"
  369421cbfabf "drm/amdgpu: fix amdgpu_ring_write_multiple"
  e8110b1c9bdd "drm/amdgpu: move ring helpers to amdgpu_ring.h"

are missing Signed-off-by from their comitter.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2018-05-17 15:27 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-15 23:18 linux-next: Signed-off-by missing for commits in the drm tree Stephen Rothwell
2018-05-17 15:27 ` Koo, Anthony
  -- strict thread matches above, loose matches on Subject: below --
2018-03-21  9:27 Stephen Rothwell
2018-03-23 11:55 ` Tomi Valkeinen
2017-12-21  5:12 Stephen Rothwell
2017-12-21 11:40 ` Tomi Valkeinen
2017-12-21 13:06   ` Daniel Vetter
2017-12-05  2:45 Stephen Rothwell
2017-12-05  9:36 ` Laurent Pinchart
2017-12-03 23:57 Stephen Rothwell
2017-12-04  2:32 ` Zhenyu Wang
2017-08-29  1:44 Stephen Rothwell
2017-08-29  1:51 ` Sinclair Yeh
2017-08-02  6:54 Stephen Rothwell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).