Re: [PATCHv3 7/7] MAINTAINERS: add recently created files to dw_dmac section

[viresh kumar]

On Thu, Sep 27, 2012 at 1:21 PM, Felipe Balbi <balbi@xxxxxx> wrote:
> On Thu, Sep 27, 2012 at 10:32:01AM +0300, Andy Shevchenko wrote:
>> Append myself to the mail entry of the section as well.
>>
>> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>> ---
>>  MAINTAINERS |    4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7dfd0eb..b87cbb1d 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -5999,10 +5999,10 @@ F:    drivers/tty/serial
>>
>>  SYNOPSYS DESIGNWARE DMAC DRIVER
>>  M:   Viresh Kumar <viresh.linux@xxxxxxxxx>
>> +M:   Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>
> this needs to be agreed with Viresh first, I guess. Not sure if it was
> done or not. If it was, sorry for the noise ;-)

:)

@Andy: It's not that i don't want anybody else to be a Maintainer of
this module, but i am
a bit confused with what is written in description of MAINTAINERS file...

M: is not considered for Mail normally, but as Maintainer - who is
bound to look and review
at all the patches, Has right to Ack or Nack something (Only if he is
on the logical side :) )

Suppose, if there are 10-20 people from different companies, who want
to have themselves
in cc for patches to a specific driver/subsystem, Will all of them add
themselves to
MAINTAINERS file? I guess NO.

If you only want yourself to be in cc, then ideally after these
patches you should be there
without these changes.

People must cc after running scripts/get_maintainers.pl and it should
return your name too,
if i am not incorrect.

But if you want to be a MAINTAINER explicitly, which i believe you are
not (looking at the commit
log), then this change is fine.

--
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/