Re: [PATCH] drm/i915: add immutable zpos plane properties

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday, April 2, 2019 3:35 PM, Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> wrote:
> Quoting Simon Ser (2019-03-30 00:19:25)
>
> > From: emersion contact@xxxxxxxxxxx
>
> Please fix your From: field.

Gah.

> > This adds basic immutable support for the zpos property. The zpos increases
> > from bottom to top: primary, sprites, cursor.
> > Signed-off-by: Simon Ser contact@xxxxxxxxxxx
>
> This is just Ville's patch rebased, so it's incorrect to strip the S-o-b,
> please read:
>
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin

Honestly I wasn't sure what to do, especially considering that I've
edited most of the lines. I tried to ask on IRC but I got no reply.

Thanks for clearing that up! So just to be sure I don't mess up v2:

- This is Ville's patch so he definitely should be the author.
- Ville's S-o-b should definitely be kept as-is.
- Now, should I add my own S-o-b? It seems like I should.
- Co-Authored-By probably doesn't make sense here.
- I originally wanted to add my Reviewed-by tag, but it probably
  wouldn't make sense if I have a S-o-b tag.

Is this correct?
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux