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

From: Enric Balletbo i Serra <enric.balletbo@collabora.com>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: linux-kernel@vger.kernel.org,
	Alessandro Zummo <a.zummo@towertech.it>,
	linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org,
	linux-i2c@vger.kernel.org, groeck@chromium.org,
	kernel@collabora.com, linux-rtc@vger.kernel.org,
	Chanwoo Choi <cw00.choi@samsung.com>,
	Benson Leung <bleung@chromium.org>,
	Sebastian Reichel <sre@kernel.org>,
	linux-input@vger.kernel.org,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Jonathan Cameron <jic23@kernel.org>,
	Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
	devicetree@vger.kernel.org,
	Thierry Reding <thierry.reding@gmail.com>,
	linux-pm@vger.kernel.org, MyungJoo Ham <myungjoo.ham@samsung.com>,
	Lee Jones <lee.jones@linaro.org>,
	Hartmut Knaack <knaack.h@gmx.de>,
	Rob Herring <robh+dt@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Mark Rutland <mark.rutland@arm.com>,
	Arnd Bergmann <arnd@arndb.de>,
	"David S. Miller" <davem@davemloft.net>,
	Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
	Nicolas Ferre <nicolas.ferre@microchip.com>,
	Greg Kroah-Hartman <"gre gkh"@linuxfoundation.org>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH v2] MAINTAINERS: add maintainers for ChromeOS EC sub-drivers
Date: Thu, 13 Dec 2018 10:30:16 +0100	[thread overview]
Message-ID: <6f688bfe-2ebe-1934-0375-016ae859dcfb@collabora.com> (raw)
In-Reply-To: <20181212222937.GL19816@piout.net>

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
>>
> 