diff mbox series

[v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

Message ID 20181211190953.13626-1-enric.balletbo@collabora.com
State Awaiting Upstream
Headers show
Series [v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers | expand

Commit Message

Enric Balletbo i Serra Dec. 11, 2018, 7:09 p.m. UTC
There are multiple ChromeOS EC sub-drivers spread in different
subsystems, as all of them are related to the Chrome stuff add
Benson and myself as a maintainers for all these sub-drivers.

Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---

Changes in v2:
- Fix typo in Benson email address.

 MAINTAINERS | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

Comments

Wolfram Sang Dec. 11, 2018, 7:13 p.m. UTC | #1
On Tue, Dec 11, 2018 at 08:09:52PM +0100, Enric Balletbo i Serra wrote:
> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

Great, thanks!

Acked-by: Wolfram Sang <wsa@the-dreams.de>
Sebastian Reichel Dec. 11, 2018, 11:09 p.m. UTC | #2
Hi,

On Tue, Dec 11, 2018 at 08:09:52PM +0100, Enric Balletbo i Serra wrote:
> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/mfd/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> +F:	include/linux/mfd/cros_ec*

Maybe just use the following (+ extra F entry for cros_usbpd, which
does not have the ec part)?

N:	cros[-_]ec

-- Sebastian

>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>  M:	Brian Austin <brian.austin@cirrus.com>
>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
Chanwoo Choi Dec. 12, 2018, 12:55 a.m. UTC | #3
Hi Enric,

On 2018년 12월 12일 04:09, Enric Balletbo i Serra wrote:
> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/mfd/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> +F:	include/linux/mfd/cros_ec*
> +
>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>  M:	Brian Austin <brian.austin@cirrus.com>
>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
> 

For extcon part,
Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
Jonathan Cameron Dec. 12, 2018, 5:06 p.m. UTC | #4
On Tue, 11 Dec 2018 20:09:52 +0100
Enric Balletbo i Serra <enric.balletbo@collabora.com> wrote:

> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> for iio parts.
> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/mfd/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> +F:	include/linux/mfd/cros_ec*
> +
>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>  M:	Brian Austin <brian.austin@cirrus.com>
>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
Benson Leung Dec. 12, 2018, 6:21 p.m. UTC | #5
On Tue, Dec 11, 2018 at 08:09:52PM +0100, Enric Balletbo i Serra wrote:
> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

Signed-off-by: Benson Leung <bleung@chromium.org>

Thanks for sending this, Enric.

> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/mfd/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> +F:	include/linux/mfd/cros_ec*
> +
>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>  M:	Brian Austin <brian.austin@cirrus.com>
>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
> -- 
> 2.19.2
>
Alexandre Belloni Dec. 12, 2018, 10:29 p.m. UTC | #6
On 11/12/2018 20:09:52+0100, Enric Balletbo i Serra wrote:
> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

You definitively don't need my ack to get that applied but anyway:

Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>

However, you should probably indicate which tree do you expect that to
go through ;)

> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/mfd/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> +F:	include/linux/mfd/cros_ec*
> +
>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>  M:	Brian Austin <brian.austin@cirrus.com>
>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
> -- 
> 2.19.2
>
Lee Jones Dec. 13, 2018, 9:22 a.m. UTC | #7
On Tue, 11 Dec 2018, Enric Balletbo i Serra wrote:

> There are multiple ChromeOS EC sub-drivers spread in different
> subsystems, as all of them are related to the Chrome stuff add
> Benson and myself as a maintainers for all these sub-drivers.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> 
> Changes in v2:
> - Fix typo in Benson email address.
> 
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a24129b0b043..bbe7180e2851 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3625,6 +3625,30 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>  F:	drivers/platform/chrome/
>  
> +CHROMEOS EC SUBDRIVERS
> +M:	Benson Leung <bleung@chromium.org>
> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +F:	drivers/iio/accel/cros_ec*
> +F:	drivers/iio/common/cros_ec_sensors/
> +F:	drivers/iio/light/cros_ec*
> +F:	drivers/iio/pressure/cros_ec*
> +F:	drivers/input/keyboard/cros_ec*
> +F:	drivers/power/supply/cros_usbpd-charger.c
> +F:	drivers/pwm/pwm-cros-ec.c
> +F:	drivers/rtc/rtc-cros-ec.c
> +F:	include/linux/iio/common/cros_ec_sensors_core.h


> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> +F:	drivers/mfd/cros_ec*
> +F:	include/linux/mfd/cros_ec*

Acked-by: Lee Jones <lee.jones@linaro.org>
Enric Balletbo i Serra Dec. 13, 2018, 9:30 a.m. UTC | #8
Hi Alexandre,

On 12/12/18 23:29, Alexandre Belloni wrote:
> On 11/12/2018 20:09:52+0100, Enric Balletbo i Serra wrote:
>> There are multiple ChromeOS EC sub-drivers spread in different
>> subsystems, as all of them are related to the Chrome stuff add
>> Benson and myself as a maintainers for all these sub-drivers.
>>
>> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> 
> You definitively don't need my ack to get that applied but anyway:
> > Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> 

Thanks for the ack.

> However, you should probably indicate which tree do you expect that to
> go through ;)
> 

Well, these are sub-drivers for different subsystems but are related to the main
ChromeOS EC. I expect all go through their specific subsystem and their
maintainer. I.e the rtc driver go through your tree, the mfd go through Lee's
tree, iio drivers through Jonathan's tree, etc. This is the reason I did not add
the tree. Main purpose here as that Benson and I are aware when someone sends a
change on one of the sub-drivers and we can review these patches.

Thanks,
 Enric

>> ---
>>
>> Changes in v2:
>> - Fix typo in Benson email address.
>>
>>  MAINTAINERS | 24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index a24129b0b043..bbe7180e2851 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -3625,6 +3625,30 @@ S:	Maintained
>>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>>  F:	drivers/platform/chrome/
>>  
>> +CHROMEOS EC SUBDRIVERS
>> +M:	Benson Leung <bleung@chromium.org>
>> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
>> +S:	Maintained
>> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
>> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
>> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
>> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
>> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
>> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
>> +F:	drivers/extcon/extcon-usbc-cros-ec.c
>> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
>> +F:	drivers/iio/accel/cros_ec*
>> +F:	drivers/iio/common/cros_ec_sensors/
>> +F:	drivers/iio/light/cros_ec*
>> +F:	drivers/iio/pressure/cros_ec*
>> +F:	drivers/input/keyboard/cros_ec*
>> +F:	drivers/mfd/cros_ec*
>> +F:	drivers/power/supply/cros_usbpd-charger.c
>> +F:	drivers/pwm/pwm-cros-ec.c
>> +F:	drivers/rtc/rtc-cros-ec.c
>> +F:	include/linux/iio/common/cros_ec_sensors_core.h
>> +F:	include/linux/mfd/cros_ec*
>> +
>>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>>  M:	Brian Austin <brian.austin@cirrus.com>
>>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
>> -- 
>> 2.19.2
>>
>
Alexandre Belloni Dec. 13, 2018, 9:50 a.m. UTC | #9
On 13/12/2018 10:30:16+0100, Enric Balletbo i Serra wrote:
> > However, you should probably indicate which tree do you expect that to
> > go through ;)
> > 
> 
> Well, these are sub-drivers for different subsystems but are related to the main
> ChromeOS EC. I expect all go through their specific subsystem and their
> maintainer. I.e the rtc driver go through your tree, the mfd go through Lee's
> tree, iio drivers through Jonathan's tree, etc. This is the reason I did not add
> the tree. Main purpose here as that Benson and I are aware when someone sends a
> change on one of the sub-drivers and we can review these patches.
> 

Sure, sorry I was not clear but I was asking about this particular
patch because as it is, you will get acks from all the maintainers but I
think they all think someone else will apply it ;)

> Thanks,
>  Enric
> 
> >> ---
> >>
> >> Changes in v2:
> >> - Fix typo in Benson email address.
> >>
> >>  MAINTAINERS | 24 ++++++++++++++++++++++++
> >>  1 file changed, 24 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index a24129b0b043..bbe7180e2851 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -3625,6 +3625,30 @@ S:	Maintained
> >>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
> >>  F:	drivers/platform/chrome/
> >>  
> >> +CHROMEOS EC SUBDRIVERS
> >> +M:	Benson Leung <bleung@chromium.org>
> >> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
> >> +S:	Maintained
> >> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
> >> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
> >> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
> >> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
> >> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
> >> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
> >> +F:	drivers/extcon/extcon-usbc-cros-ec.c
> >> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
> >> +F:	drivers/iio/accel/cros_ec*
> >> +F:	drivers/iio/common/cros_ec_sensors/
> >> +F:	drivers/iio/light/cros_ec*
> >> +F:	drivers/iio/pressure/cros_ec*
> >> +F:	drivers/input/keyboard/cros_ec*
> >> +F:	drivers/mfd/cros_ec*
> >> +F:	drivers/power/supply/cros_usbpd-charger.c
> >> +F:	drivers/pwm/pwm-cros-ec.c
> >> +F:	drivers/rtc/rtc-cros-ec.c
> >> +F:	include/linux/iio/common/cros_ec_sensors_core.h
> >> +F:	include/linux/mfd/cros_ec*
> >> +
> >>  CIRRUS LOGIC AUDIO CODEC DRIVERS
> >>  M:	Brian Austin <brian.austin@cirrus.com>
> >>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
> >> -- 
> >> 2.19.2
> >>
> >
Enric Balletbo i Serra Dec. 13, 2018, 9:55 a.m. UTC | #10
On 13/12/18 10:50, Alexandre Belloni wrote:
> On 13/12/2018 10:30:16+0100, Enric Balletbo i Serra wrote:
>>> However, you should probably indicate which tree do you expect that to
>>> go through ;)
>>>
>>
>> Well, these are sub-drivers for different subsystems but are related to the main
>> ChromeOS EC. I expect all go through their specific subsystem and their
>> maintainer. I.e the rtc driver go through your tree, the mfd go through Lee's
>> tree, iio drivers through Jonathan's tree, etc. This is the reason I did not add
>> the tree. Main purpose here as that Benson and I are aware when someone sends a
>> change on one of the sub-drivers and we can review these patches.
>>
> 
> Sure, sorry I was not clear but I was asking about this particular
> patch because as it is, you will get acks from all the maintainers but I
> think they all think someone else will apply it ;)
> 

Ah, ok, I didn't got it :)

If nobody complains and we have all the acks I think that this can go through
the Benson's tree (chrome-platform). The reason is that he already has a couple
of patches related to this (remove Olof as a chrome-platform maintainer and
adding me)

>> Thanks,
>>  Enric
>>
>>>> ---
>>>>
>>>> Changes in v2:
>>>> - Fix typo in Benson email address.
>>>>
>>>>  MAINTAINERS | 24 ++++++++++++++++++++++++
>>>>  1 file changed, 24 insertions(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index a24129b0b043..bbe7180e2851 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -3625,6 +3625,30 @@ S:	Maintained
>>>>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
>>>>  F:	drivers/platform/chrome/
>>>>  
>>>> +CHROMEOS EC SUBDRIVERS
>>>> +M:	Benson Leung <bleung@chromium.org>
>>>> +M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
>>>> +S:	Maintained
>>>> +F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
>>>> +F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
>>>> +F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
>>>> +F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
>>>> +F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
>>>> +F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
>>>> +F:	drivers/extcon/extcon-usbc-cros-ec.c
>>>> +F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
>>>> +F:	drivers/iio/accel/cros_ec*
>>>> +F:	drivers/iio/common/cros_ec_sensors/
>>>> +F:	drivers/iio/light/cros_ec*
>>>> +F:	drivers/iio/pressure/cros_ec*
>>>> +F:	drivers/input/keyboard/cros_ec*
>>>> +F:	drivers/mfd/cros_ec*
>>>> +F:	drivers/power/supply/cros_usbpd-charger.c
>>>> +F:	drivers/pwm/pwm-cros-ec.c
>>>> +F:	drivers/rtc/rtc-cros-ec.c
>>>> +F:	include/linux/iio/common/cros_ec_sensors_core.h
>>>> +F:	include/linux/mfd/cros_ec*
>>>> +
>>>>  CIRRUS LOGIC AUDIO CODEC DRIVERS
>>>>  M:	Brian Austin <brian.austin@cirrus.com>
>>>>  M:	Paul Handrigan <Paul.Handrigan@cirrus.com>
>>>> -- 
>>>> 2.19.2
>>>>
>>>
>
Alexandre Belloni Dec. 13, 2018, 10:59 a.m. UTC | #11
On 13/12/2018 10:55:29+0100, Enric Balletbo i Serra wrote:
> 
> 
> On 13/12/18 10:50, Alexandre Belloni wrote:
> > On 13/12/2018 10:30:16+0100, Enric Balletbo i Serra wrote:
> >>> However, you should probably indicate which tree do you expect that to
> >>> go through ;)
> >>>
> >>
> >> Well, these are sub-drivers for different subsystems but are related to the main
> >> ChromeOS EC. I expect all go through their specific subsystem and their
> >> maintainer. I.e the rtc driver go through your tree, the mfd go through Lee's
> >> tree, iio drivers through Jonathan's tree, etc. This is the reason I did not add
> >> the tree. Main purpose here as that Benson and I are aware when someone sends a
> >> change on one of the sub-drivers and we can review these patches.
> >>
> > 
> > Sure, sorry I was not clear but I was asking about this particular
> > patch because as it is, you will get acks from all the maintainers but I
> > think they all think someone else will apply it ;)
> > 
> 
> Ah, ok, I didn't got it :)
> 
> If nobody complains and we have all the acks I think that this can go through
> the Benson's tree (chrome-platform). The reason is that he already has a couple
> of patches related to this (remove Olof as a chrome-platform maintainer and
> adding me)
> 

Ok, great.
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index a24129b0b043..bbe7180e2851 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3625,6 +3625,30 @@  S:	Maintained
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bleung/chrome-platform.git
 F:	drivers/platform/chrome/
 
+CHROMEOS EC SUBDRIVERS
+M:	Benson Leung <bleung@chromium.org>
+M:	Enric Balletbo i Serra <enric.balletbo@collabora.com>
+S:	Maintained
+F:	Documentation/devicetree/bindings/extcon/extcon-usbc-cros-ec.txt
+F:	Documentation/devicetree/bindings/input/cros-ec-keyb.txt
+F:	Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.txt
+F:	Documentation/devicetree/bindings/i2c/i2c-cros-ec-tunnel.txt
+F:	Documentation/devicetree/bindings/mfd/cros-ec.txt
+F:	Documentation/ABI/testing/sysfs-bus-iio-cros-ec
+F:	drivers/extcon/extcon-usbc-cros-ec.c
+F:	drivers/i2c/busses/i2c-cros-ec-tunnel.c
+F:	drivers/iio/accel/cros_ec*
+F:	drivers/iio/common/cros_ec_sensors/
+F:	drivers/iio/light/cros_ec*
+F:	drivers/iio/pressure/cros_ec*
+F:	drivers/input/keyboard/cros_ec*
+F:	drivers/mfd/cros_ec*
+F:	drivers/power/supply/cros_usbpd-charger.c
+F:	drivers/pwm/pwm-cros-ec.c
+F:	drivers/rtc/rtc-cros-ec.c
+F:	include/linux/iio/common/cros_ec_sensors_core.h
+F:	include/linux/mfd/cros_ec*
+
 CIRRUS LOGIC AUDIO CODEC DRIVERS
 M:	Brian Austin <brian.austin@cirrus.com>
 M:	Paul Handrigan <Paul.Handrigan@cirrus.com>