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

Mark Rutland
2013-11-13T09:42:38Z

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/

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.