From: "Pali Rohár" <pali.rohar@gmail.com>
To: Sebastian Reichel <sre@kernel.org>
Cc: "Andrew F. Davis" <afd@ti.com>,
linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: Add myself as a reviewer for the bq27xxx driver.
Date: Fri, 15 Jan 2016 15:24:10 +0100 [thread overview]
Message-ID: <20160115142410.GA11560@pali> (raw)
In-Reply-To: <201512312211.32570@pali>
On Thursday 31 December 2015 22:11:32 Pali Rohár wrote:
> On Thursday 31 December 2015 22:06:38 Sebastian Reichel wrote:
> > Hi,
> >
> > I think it would make sense to have it like this:
> >
> > NOKIA N900 POWER SUPPLY DRIVERS
> > +R: Pali Rohár <pali.rohar@gmail.com>
> > -M: Pali Rohár <pali.rohar@gmail.com>
> > -S: Maintained
> > F: include/linux/power/bq2415x_charger.h
> > F: include/linux/power/bq27xxx_battery.h
> > F: include/linux/power/isp1704_charger.h
> > F: drivers/power/bq2415x_charger.c
> > F: drivers/power/bq27xxx_battery.c
> > F: drivers/power/isp1704_charger.c
> > F: drivers/power/rx51_battery.c
> >
> > +TI BQ27XXX POWER SUPPLY DRIVER
> > +R: Andrew F. Davis <afd@ti.com>
> > +F: include/linux/power/bq27xxx_battery.h
> > +F: drivers/power/bq27xxx_battery.c
> > +F: drivers/power/bq27xxx_battery_i2c.c
> > +
> > POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS
> > M: Sebastian Reichel <sre@kernel.org>
> > M: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> > M: David Woodhouse <dwmw2@infradead.org>
> > L: linux-pm@vger.kernel.org
> > T: git git://git.infradead.org/battery-2.6.git
> > S: Maintained
> > F: include/linux/power_supply.h
> > F: drivers/power/
> > X: drivers/power/avs/
> >
> > This would result in the following get_maintainer.pl output
> > for the bq27xxx driver.
> >
> > sre@earth ~/src/linux [n950] % ./scripts/get_maintainer.pl -f
> > drivers/power/bq27xxx_battery.c "Andrew F. Davis" <afd@ti.com>
> > (reviewer:TI BQ27XXX POWER SUPPLY DRIVER) "Pali Rohár"
> > <pali.rohar@gmail.com> (reviewer:NOKIA N900 POWER SUPPLY DRIVERS)
> > Sebastian Reichel <sre@kernel.org> (maintainer:POWER SUPPLY
> > CLASS/SUBSYSTEM and DRIVERS) Dmitry Eremin-Solenikov
> > <dbaryshkov@gmail.com> (maintainer:POWER SUPPLY CLASS/SUBSYSTEM and
> > DRIVERS) David Woodhouse <dwmw2@infradead.org> (maintainer:POWER
> > SUPPLY CLASS/SUBSYSTEM and DRIVERS) linux-pm@vger.kernel.org (open
> > list:POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS)
> > linux-kernel@vger.kernel.org (open list)
> >
> > Pali, what do you think about the reviewer/maintainer change?
> > I think it makes it more clear, that the patches are queued in
> > the battery tree. If you think its a downgrade I don't mind not
> > changing it, though.
> >
> > -- Sebastian
>
> Change is OK for me.
>
Sebastian, one one minor change to your patch:
Add bq27xxx_battery_i2c.c file into NOKIA N900 POWER SUPPLY DRIVERS
section as now bq27200 code is split into two files...
--
Pali Rohár
pali.rohar@gmail.com
prev parent reply other threads:[~2016-01-15 14:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-30 18:35 [PATCH] MAINTAINERS: Add myself as a reviewer for the bq27xxx driver Andrew F. Davis
2015-12-31 18:40 ` Pali Rohár
2015-12-31 21:06 ` Sebastian Reichel
2015-12-31 21:11 ` Pali Rohár
2016-01-15 14:24 ` Pali Rohár [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160115142410.GA11560@pali \
--to=pali.rohar@gmail.com \
--cc=afd@ti.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=sre@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).