Re: [PATCHv5 20/20] MAINTAINERS: add maintainer entry for thermal bindings

Eduardo Valentin
2013-11-13T14:47:50Z

[-- Attachment #1: Type: text/plain, Size: 2985 bytes --]

On 13-11-2013 05:42, Mark Rutland wrote:
> Hi,
>
> On Tue, Nov 12, 2013 at 08:14:41PM +0000, Eduardo Valentin wrote:
>> On 12-11-2013 15:59, Olof Johansson wrote:
>>> On Tue, Nov 12, 2013 at 11:46 AM, Eduardo Valentin
>>> <eduardo.valentin@ti.com> wrote:
>>>> After discussion and agreement of thermal device tree bindings,
>>>> it is desirable to properly maintain thermal bindings for
>>>> existing and upcoming devices.
>>>>
>>>> As original author of thermal device tree bindings, I am
>>>> volunteering to maintain them.
>>>>
>>>> Cc: Rob Herring <rob.herring@calxeda.com>
>>>> Cc: Pawel Moll <pawel.moll@arm.com>
>>>> Cc: Mark Rutland <mark.rutland@arm.com>
>>>> Cc: Stephen Warren <swarren@wwwdotorg.org>
>>>> Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>
>>>> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
>>>> ---
>>>> MAINTAINERS | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index 3438384..94bfc46 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -6178,6 +6178,7 @@ M: Pawel Moll <pawel.moll@arm.com>
>>>> M: Mark Rutland <mark.rutland@arm.com>
>>>> M: Stephen Warren <swarren@wwwdotorg.org>
>>>> M: Ian Campbell <ijc+devicetree@hellion.org.uk>
>>>> +M: Eduardo Valentin <eduardo.valentin@ti.com>
>>>> L: devicetree@vger.kernel.org
>>>> S: Maintained
>>>> F: Documentation/devicetree/
>>>
>>> The number of device-tree maintainers is getting to be somewhat silly.
>>> You don't have to be a maintainer to be a reviewer, and the number is
>>> getting large enough that it might get hard for one maintainer to know
>>> what another is doing.
>>>
>>> So, soft nack, but it's really up to the DT guys.
>>
>> Hello Olof,
>>
>> No issues on my side. As I stated in the patch description, I am
>> volunteering. The motivation comes from the fact that it's been hard to
>> get DT maintainers attention, so I am assuming they are not having
>> enough bandwidth to pay attention to an extra class of bindings.
>>
>> I agree with you, this is up to the DT folks.
>
> When we discussed this in #devicetree, I was under the impression you
> meant adding a line like:
>
> F: Documentation/devicetree/bindings/thermal/

This would work for me too. How about adding this to thermal subsys
entry in MAINTAINERS file?

>
> To your MAINTAINERS entry, rather than adding yourself to the general
> devicetree bindings MAINTAINERS entry. Sorry for the confusion regarding
> this.
>
> Given we have enough difficulty organising ourselves at present, I agree
> with Olof that it doesn't make much sense to throw more people into the
> general devicetree bindings maintainer pile.
>
> However, we would certainly appreciate any review you'd be willing to
> provide.
>
> Thanks,
> Mark.
>
>

--
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]