Re: [Xen-devel] [PATCH] drmif: add ABI for para-virtual DRM/KMS

[Lars Kurth <lars.kurth@xxxxxxxxxx>]

Oleksandr,


On 24/11/2016 11:30, "Oleksandr Andrushchenko" <andr2000@xxxxxxxxx> wrote:

>This is the ABI for the two halves of a para-virtualized
>DRM/KMS driver.
>
>Signed-off-by: Oleksandr Andrushchenko <andr2000@xxxxxxxxx>
>Signed-off-by: Oleksandr Grytsov <al1img@xxxxxxxxx>

...

>+ *
>+ * Copyright (C) 2016 EPAM Systems Inc.
>+ *
>+ * Authors: Oleksandr Andrushchenko <andr2000@xxxxxxxxx>
>+ *          Oleksandr Grytsov <al1img@xxxxxxxxx>
>+ */

I have a slight concern here about the use of private e-mail addresses in
the Signed-off-by tags and authors field in the (c) notice. It contradicts
"Copyright (C) 2016 EPAM Systems Inc." and will make it very hard to
execute licenses changes, if that needs to be done in future.

The Signed-off-by tag is a way to identify the (c) holder of a patch. I am
assuming that in this case the (c) holder would be EPAM. Is this correct?

If yes, please change the Signed-off-by: and author fields to an EPAM
e-mail address. You can still send from gmail addresses though.
If not, please change the "Copyright (C) 2016 EPAM Systems Inc."
accordingly

I am assuming there is a similar issue in the other patches.
Please make sure you fix all of them.

Best Regards
Lars

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel