linux-next: Signed-off-by missing for commits in the drm tree

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

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

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Sinclair Yeh]

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

RE: linux-next: Signed-off-by missing for commits in the drm tree

[Koo, Anthony]

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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

[-- 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 --]

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Tomi Valkeinen]

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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

[-- Attachment #1.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 #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Daniel Vetter]

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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Tomi Valkeinen]

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

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

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

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Laurent Pinchart]

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

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

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

Re: linux-next: Signed-off-by missing for commits in the drm tree

[Zhenyu Wang]

[-- Attachment #1.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 #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

linux-next: Signed-off-by missing for commits in the drm tree

[Stephen Rothwell]

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