Re: [PATCH v3 3/4] sta2x11_vip: convert to videobuf2 and control framework

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 05-12-2012 11:27, Federico Vaga escreveu:
>>> Ok, I understand. I will write something like this.
>>>
>>>    * Copyright (C) 2012       ST Microelectronics
>>>    *      author: Federico Vaga <federico.vaga@gmail.com>
>>>    * Copyright (C) 2010       WindRiver Systems, Inc.
>>>    *      authors: Andreas Kies <andreas.kies@windriver.com>
>>>    *               Vlad Lungu <vlad.lungu@windriver.com>
>>
>> Sounds perfect to me.
>
> I will answer to this with a patch

Thanks!

>> As you said, the best place to discuss about it is likely at LKML.
>> [...]
>> Btw, this is why it is called "git blame", and not "git authorship":
>> it is a tool to identify who was the last one that modified the code.
>> Its main usage is to identify who might have introduced a bug on the
>> code.
>
> I know I know, it was just a stupid example to expose the problem that I have
> in my mind. I know that it is very difficult (impossible?) to assign the
> authorship of a single line, and git blame it is not the tool to do this :)
>
> I think you understand what I mean despite the stupid example

Yeah, I hear you.

Not sure if you got my point: the main point of removing MODULE_AUTHOR
and other copyright stuff is that such patch may easily be doing something
that could be considered a copyright violation, being bad not only to
the affected driver, but to the entire Kernel.

So, we need to handle it with due care. Getting other authors's
acks on such patch seems to be the only safe way of doing that.

Regards,
Mauro