2018-12-11 19:11:17

by Enric Balletbo i Serra

[permalink] [raw]
Subject: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
---

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 <[email protected]>
+M: Enric Balletbo i Serra <[email protected]>
+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 <[email protected]>
M: Paul Handrigan <[email protected]>
--
2.19.2



2018-12-11 19:16:06

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>

Great, thanks!

Acked-by: Wolfram Sang <[email protected]>


Attachments:
(No filename) (412.00 B)
signature.asc (849.00 B)
Download all attachments

2018-12-11 23:10:21

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
> ---
>
> 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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>
> M: Paul Handrigan <[email protected]>


Attachments:
(No filename) (2.06 kB)
signature.asc (849.00 B)
Download all attachments

2018-12-12 00:56:40

by Chanwoo Choi

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
> ---
>
> 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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>
> M: Paul Handrigan <[email protected]>
>

For extcon part,
Acked-by: Chanwoo Choi <[email protected]>

--
Best Regards,
Chanwoo Choi
Samsung Electronics

2018-12-12 17:08:04

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

On Tue, 11 Dec 2018 20:09:52 +0100
Enric Balletbo i Serra <[email protected]> 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 <[email protected]>
Acked-by: Jonathan Cameron <[email protected]> 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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>
> M: Paul Handrigan <[email protected]>


2018-12-12 18:22:46

by Benson Leung

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>

Signed-off-by: Benson Leung <[email protected]>

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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>
> M: Paul Handrigan <[email protected]>
> --
> 2.19.2
>

--
Benson Leung
Staff Software Engineer
Chrome OS Kernel
Google Inc.
[email protected]
Chromium OS Project
[email protected]


Attachments:
(No filename) (2.17 kB)
signature.asc (849.00 B)
Download all attachments

2018-12-12 22:32:49

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>

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

Acked-by: Alexandre Belloni <[email protected]>

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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>
> M: Paul Handrigan <[email protected]>
> --
> 2.19.2
>

--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

2018-12-13 09:23:55

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
> ---
>
> 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 <[email protected]>
> +M: Enric Balletbo i Serra <[email protected]>
> +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 <[email protected]>

--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

2018-12-13 09:31:31

by Enric Balletbo i Serra

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
>
> You definitively don't need my ack to get that applied but anyway:
> > Acked-by: Alexandre Belloni <[email protected]>
>

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 <[email protected]>
>> +M: Enric Balletbo i Serra <[email protected]>
>> +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 <[email protected]>
>> M: Paul Handrigan <[email protected]>
>> --
>> 2.19.2
>>
>

2018-12-13 09:51:58

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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 <[email protected]>
> >> +M: Enric Balletbo i Serra <[email protected]>
> >> +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 <[email protected]>
> >> M: Paul Handrigan <[email protected]>
> >> --
> >> 2.19.2
> >>
> >

--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

2018-12-13 09:56:45

by Enric Balletbo i Serra

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers



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 <[email protected]>
>>>> +M: Enric Balletbo i Serra <[email protected]>
>>>> +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 <[email protected]>
>>>> M: Paul Handrigan <[email protected]>
>>>> --
>>>> 2.19.2
>>>>
>>>
>

2018-12-13 11:00:35

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers

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.


--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com