diff mbox

mark myself as mainainer for camera on N900

Message ID 20161227205923.GA7859@amd (mailing list archive)
State Accepted, archived
Delegated to: Hans Verkuil
Headers

Commit Message

Pavel Machek Dec. 27, 2016, 8:59 p.m. UTC 
  Mark and Sakari as maintainers for Nokia N900 camera pieces.

Signed-off-by: Pavel Machek <pavel@ucw.cz>

---

Hi!

> Yeah, there was big flamewar about the permissions. In the end Linus
> decided that everyone knows the octal numbers, but the constants are
> tricky. It began with patch series with 1000 patches...
> 
> > Btw. should we update maintainers as well? Would you like to put yourself
> > there? Feel free to add me, too...
> 
> Ok, will do.

Something like this? Actually, I guess we could merge ADP1653 entry
there. Yes, it is random collection of devices, but are usually tested
"together", so I believe one entry makes sense.

(But I have no problem with having multiple entries, too.)

Thanks,
								Pavel

Comments

Sebastian Reichel Dec. 27, 2016, 11:57 p.m. UTC | #1 
Hi,

On Tue, Dec 27, 2016 at 09:59:23PM +0100, Pavel Machek wrote:
> Mark and Sakari as maintainers for Nokia N900 camera pieces.

^^^ missing me after Mark. Otherwise Mark looks like a name :)

> Signed-off-by: Pavel Machek <pavel@ucw.cz>
> 
> ---
> 
> Hi!
> 
> > Yeah, there was big flamewar about the permissions. In the end Linus
> > decided that everyone knows the octal numbers, but the constants are
> > tricky. It began with patch series with 1000 patches...
> > 
> > > Btw. should we update maintainers as well? Would you like to put yourself
> > > there? Feel free to add me, too...
> > 
> > Ok, will do.
> 
> Something like this? Actually, I guess we could merge ADP1653 entry
> there. Yes, it is random collection of devices, but are usually tested
> "together", so I believe one entry makes sense.
> 
> (But I have no problem with having multiple entries, too.)
> 
> Thanks,
> 								Pavel
> 
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 63cefa6..1cb1d97 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8613,6 +8613,14 @@ T:	git git://git.kernel.org/pub/scm/linux/kernel/git/lftan/nios2.git
>  S:	Maintained
>  F:	arch/nios2/
>  
> +NOKIA N900 CAMERA SUPPORT (ET8EK8 SENSOR, AD5820 FOCUS)
> +M:	Pavel Machek <pavel@ucw.cz>
> +M:	Sakari Ailus <sakari.ailus@iki.fi>
> +L:	linux-media@vger.kernel.org
> +S:	Maintained
> +F:	drivers/media/i2c/et8ek8
> +F:	drivers/media/i2c/ad5820.c

Not sure if this is useful information, but I solved it like this
for N900 power supply drivers:

NOKIA N900 POWER SUPPLY DRIVERS
R:	Pali Rohár <pali.rohar@gmail.com>
F:	include/linux/power/bq2415x_charger.h
F:	include/linux/power/bq27xxx_battery.h
F:	include/linux/power/isp1704_charger.h
F:	drivers/power/supply/bq2415x_charger.c
F:	drivers/power/supply/bq27xxx_battery.c
F:	drivers/power/supply/bq27xxx_battery_i2c.c
F:	drivers/power/supply/isp1704_charger.c
F:	drivers/power/supply/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/supply/bq27xxx_battery.c
F:	drivers/power/supply/bq27xxx_battery_i2c.c

POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS
M:	Sebastian Reichel <sre@kernel.org>
L:	linux-pm@vger.kernel.org
T:	git git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git
S:	Maintained
F:	Documentation/devicetree/bindings/power/supply/
F:	include/linux/power_supply.h
F:	drivers/power/supply/

This makes it quite easy to see who applies patches and who should
be Cc'd for what reason:

$ ./scripts/get_maintainer.pl -f drivers/power/supply/bq27xxx_battery.c 
"Pali Rohár" <pali.rohar@gmail.com> (reviewer:NOKIA N900 POWER SUPPLY DRIVERS)
"Andrew F. Davis" <afd@ti.com> (reviewer:TI BQ27XXX POWER SUPPLY DRIVER)
Sebastian Reichel <sre@kernel.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)

-- Sebastian
Sakari Ailus Jan. 25, 2017, 1:48 p.m. UTC | #2 
Hi Sebastian and Pavel,

On Wed, Dec 28, 2016 at 12:57:21AM +0100, Sebastian Reichel wrote:
> Hi,
> 
> On Tue, Dec 27, 2016 at 09:59:23PM +0100, Pavel Machek wrote:
> > Mark and Sakari as maintainers for Nokia N900 camera pieces.
> 
> ^^^ missing me after Mark. Otherwise Mark looks like a name :)
> 
> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > 
> > ---
> > 
> > Hi!
> > 
> > > Yeah, there was big flamewar about the permissions. In the end Linus
> > > decided that everyone knows the octal numbers, but the constants are
> > > tricky. It began with patch series with 1000 patches...
> > > 
> > > > Btw. should we update maintainers as well? Would you like to put yourself
> > > > there? Feel free to add me, too...
> > > 
> > > Ok, will do.
> > 
> > Something like this? Actually, I guess we could merge ADP1653 entry
> > there. Yes, it is random collection of devices, but are usually tested
> > "together", so I believe one entry makes sense.
> > 
> > (But I have no problem with having multiple entries, too.)
> > 
> > Thanks,
> > 								Pavel
> > 
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 63cefa6..1cb1d97 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -8613,6 +8613,14 @@ T:	git git://git.kernel.org/pub/scm/linux/kernel/git/lftan/nios2.git
> >  S:	Maintained
> >  F:	arch/nios2/
> >  
> > +NOKIA N900 CAMERA SUPPORT (ET8EK8 SENSOR, AD5820 FOCUS)
> > +M:	Pavel Machek <pavel@ucw.cz>
> > +M:	Sakari Ailus <sakari.ailus@iki.fi>
> > +L:	linux-media@vger.kernel.org
> > +S:	Maintained
> > +F:	drivers/media/i2c/et8ek8
> > +F:	drivers/media/i2c/ad5820.c
> 
> Not sure if this is useful information, but I solved it like this
> for N900 power supply drivers:
> 
> NOKIA N900 POWER SUPPLY DRIVERS
> R:	Pali Rohár <pali.rohar@gmail.com>
> F:	include/linux/power/bq2415x_charger.h
> F:	include/linux/power/bq27xxx_battery.h
> F:	include/linux/power/isp1704_charger.h
> F:	drivers/power/supply/bq2415x_charger.c
> F:	drivers/power/supply/bq27xxx_battery.c
> F:	drivers/power/supply/bq27xxx_battery_i2c.c
> F:	drivers/power/supply/isp1704_charger.c
> F:	drivers/power/supply/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/supply/bq27xxx_battery.c
> F:	drivers/power/supply/bq27xxx_battery_i2c.c
> 
> POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS
> M:	Sebastian Reichel <sre@kernel.org>
> L:	linux-pm@vger.kernel.org
> T:	git git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git
> S:	Maintained
> F:	Documentation/devicetree/bindings/power/supply/
> F:	include/linux/power_supply.h
> F:	drivers/power/supply/
> 
> This makes it quite easy to see who applies patches and who should
> be Cc'd for what reason:
> 
> $ ./scripts/get_maintainer.pl -f drivers/power/supply/bq27xxx_battery.c 
> "Pali Rohár" <pali.rohar@gmail.com> (reviewer:NOKIA N900 POWER SUPPLY DRIVERS)
> "Andrew F. Davis" <afd@ti.com> (reviewer:TI BQ27XXX POWER SUPPLY DRIVER)
> Sebastian Reichel <sre@kernel.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)

I'm adding Pavel's patch to my tree and then send a pull req to Mauro. If
further changes are needed then let's write more patches.
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 63cefa6..1cb1d97 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8613,6 +8613,14 @@  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/lftan/nios2.git
 S:	Maintained
 F:	arch/nios2/
 
+NOKIA N900 CAMERA SUPPORT (ET8EK8 SENSOR, AD5820 FOCUS)
+M:	Pavel Machek <pavel@ucw.cz>
+M:	Sakari Ailus <sakari.ailus@iki.fi>
+L:	linux-media@vger.kernel.org
+S:	Maintained
+F:	drivers/media/i2c/et8ek8
+F:	drivers/media/i2c/ad5820.c
+
 NOKIA N900 POWER SUPPLY DRIVERS
 R:	Pali Rohár <pali.rohar@gmail.com>
 F:	include/linux/power/bq2415x_charger.h