diff mbox series

MAINTAINERS: move Milo Kim to credits

Message ID 20200921210233.21449-1-krzk@kernel.org (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: move Milo Kim to credits | expand

Commit Message

Krzysztof Kozlowski Sept. 21, 2020, 9:02 p.m. UTC
Milo Kim's email in TI bounces with permanent error (550: Invalid
recipient).  Last email from him on LKML was in 2017.  Move Milo Kim to
credits and remove the separate driver entries for:

 - TI LP855x backlight driver,
 - TI LP8727 charger driver,
 - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 CREDITS     |  3 +++
 MAINTAINERS | 23 -----------------------
 2 files changed, 3 insertions(+), 23 deletions(-)

Comments

Pavel Machek Sept. 21, 2020, 9:06 p.m. UTC | #1
Hi!

> Milo Kim's email in TI bounces with permanent error (550: Invalid
> recipient).  Last email from him on LKML was in 2017.  Move Milo Kim to
> credits and remove the separate driver entries for:
> 
>  - TI LP855x backlight driver,
>  - TI LP8727 charger driver,
>  - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

I believe normal way would be to mark the entries "orphaned", not to
drop them altogether. Plus, I believe someone from TI is likely to
step up.

Best regards,
								Pavel



> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5b9621ca2b31..031adeff29cc 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17471,29 +17471,6 @@ S:	Maintained
>  F:	sound/soc/codecs/isabelle*
>  F:	sound/soc/codecs/lm49453*
>  
> -TI LP855x BACKLIGHT DRIVER
> -M:	Milo Kim <milo.kim@ti.com>
> -S:	Maintained
> -F:	Documentation/driver-api/backlight/lp855x-driver.rst
> -F:	drivers/video/backlight/lp855x_bl.c
> -F:	include/linux/platform_data/lp855x.h
> -
> -TI LP8727 CHARGER DRIVER
> -M:	Milo Kim <milo.kim@ti.com>
> -S:	Maintained
> -F:	drivers/power/supply/lp8727_charger.c
> -F:	include/linux/platform_data/lp8727.h
> -
> -TI LP8788 MFD DRIVER
> -M:	Milo Kim <milo.kim@ti.com>
> -S:	Maintained
> -F:	drivers/iio/adc/lp8788_adc.c
> -F:	drivers/leds/leds-lp8788.c
> -F:	drivers/mfd/lp8788*.c
> -F:	drivers/power/supply/lp8788-charger.c
> -F:	drivers/regulator/lp8788-*.c
> -F:	include/linux/mfd/lp8788*.h
> -
>  TI NETCP ETHERNET DRIVER
>  M:	Wingman Kwok <w-kwok2@ti.com>
>  M:	Murali Karicheri <m-karicheri2@ti.com>
Krzysztof Kozlowski Sept. 22, 2020, 7:08 a.m. UTC | #2
On Mon, 21 Sep 2020 at 23:06, Pavel Machek <pavel@ucw.cz> wrote:
>
> Hi!
>
> > Milo Kim's email in TI bounces with permanent error (550: Invalid
> > recipient).  Last email from him on LKML was in 2017.  Move Milo Kim to
> > credits and remove the separate driver entries for:
> >
> >  - TI LP855x backlight driver,
> >  - TI LP8727 charger driver,
> >  - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> I believe normal way would be to mark the entries "orphaned", not to
> drop them altogether. Plus, I believe someone from TI is likely to
> step up.

These are entries for specific drivers so they are covered by the
subsystem maintainers. You believe someone will step up, I believe if
these were important for TI, they would find the person some time ago,
so the emails won't bounce... This was similar with BQ chargers where
Andrew's email was bouncing and after a few weeks it was fixed. To me
it looks like TI does not have any priority in maintaining separate
driver entries so what is the point to keep orphaned driver-entry? It
is not the case where we have an orphaned subsystem and we look for a
person to handle it...

Best regards,
Krzysztof
Mark Brown Sept. 22, 2020, 9:36 a.m. UTC | #3
On Tue, Sep 22, 2020 at 09:08:37AM +0200, Krzysztof Kozlowski wrote:
> On Mon, 21 Sep 2020 at 23:06, Pavel Machek <pavel@ucw.cz> wrote:

> > I believe normal way would be to mark the entries "orphaned", not to
> > drop them altogether. Plus, I believe someone from TI is likely to
> > step up.

> These are entries for specific drivers so they are covered by the
> subsystem maintainers. You believe someone will step up, I believe if
> these were important for TI, they would find the person some time ago,
> so the emails won't bounce... This was similar with BQ chargers where

It's fairly common for mobile parts to get dropped relatively quickly as
the technology moves fairly quickly in that market, I think a lot of teh
parts that Milo was working on were mobile ones.
Dan Murphy Sept. 22, 2020, 11:36 a.m. UTC | #4
All

On 9/22/20 4:36 AM, Mark Brown wrote:
> On Tue, Sep 22, 2020 at 09:08:37AM +0200, Krzysztof Kozlowski wrote:
>> On Mon, 21 Sep 2020 at 23:06, Pavel Machek <pavel@ucw.cz> wrote:
>>> I believe normal way would be to mark the entries "orphaned", not to
>>> drop them altogether. Plus, I believe someone from TI is likely to
>>> step up.
>> These are entries for specific drivers so they are covered by the
>> subsystem maintainers. You believe someone will step up, I believe if
>> these were important for TI, they would find the person some time ago,
>> so the emails won't bounce... This was similar with BQ chargers where
> It's fairly common for mobile parts to get dropped relatively quickly as
> the technology moves fairly quickly in that market, I think a lot of teh
> parts that Milo was working on were mobile ones.

These specific drivers don't see many patches applied to them. These 
drivers did have a few patches this year to fix random bugs.

Since I have worked in these other subsystems if replacing the 
Maintainer is desired over removal then my name and email can be added 
like I did with Andrews.

Dan Murphy <dmurphy@ti.com>

Dan
Krzysztof Kozlowski Sept. 22, 2020, 11:45 a.m. UTC | #5
On Tue, 22 Sep 2020 at 13:36, Dan Murphy <dmurphy@ti.com> wrote:
>
> All
>
> On 9/22/20 4:36 AM, Mark Brown wrote:
> > On Tue, Sep 22, 2020 at 09:08:37AM +0200, Krzysztof Kozlowski wrote:
> >> On Mon, 21 Sep 2020 at 23:06, Pavel Machek <pavel@ucw.cz> wrote:
> >>> I believe normal way would be to mark the entries "orphaned", not to
> >>> drop them altogether. Plus, I believe someone from TI is likely to
> >>> step up.
> >> These are entries for specific drivers so they are covered by the
> >> subsystem maintainers. You believe someone will step up, I believe if
> >> these were important for TI, they would find the person some time ago,
> >> so the emails won't bounce... This was similar with BQ chargers where
> > It's fairly common for mobile parts to get dropped relatively quickly as
> > the technology moves fairly quickly in that market, I think a lot of teh
> > parts that Milo was working on were mobile ones.
>
> These specific drivers don't see many patches applied to them. These
> drivers did have a few patches this year to fix random bugs.
>
> Since I have worked in these other subsystems if replacing the
> Maintainer is desired over removal then my name and email can be added
> like I did with Andrews.
>
> Dan Murphy <dmurphy@ti.com>

I will add then yours name in v2 of this patch. Thanks for taking care of these.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/CREDITS b/CREDITS
index 1df63cdf71df..46384b11f258 100644
--- a/CREDITS
+++ b/CREDITS
@@ -1910,6 +1910,9 @@  S: 660 Harvard Ave. #7
 S: Santa Clara, CA 95051
 S: USA
 
+N: Milo Kim
+D: TI LP855x, LP8727 and LP8788 drivers
+
 N: Russell King
 E: rmk@arm.linux.org.uk
 D: Linux/arm integrator, maintainer & hacker
diff --git a/MAINTAINERS b/MAINTAINERS
index 5b9621ca2b31..031adeff29cc 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17471,29 +17471,6 @@  S:	Maintained
 F:	sound/soc/codecs/isabelle*
 F:	sound/soc/codecs/lm49453*
 
-TI LP855x BACKLIGHT DRIVER
-M:	Milo Kim <milo.kim@ti.com>
-S:	Maintained
-F:	Documentation/driver-api/backlight/lp855x-driver.rst
-F:	drivers/video/backlight/lp855x_bl.c
-F:	include/linux/platform_data/lp855x.h
-
-TI LP8727 CHARGER DRIVER
-M:	Milo Kim <milo.kim@ti.com>
-S:	Maintained
-F:	drivers/power/supply/lp8727_charger.c
-F:	include/linux/platform_data/lp8727.h
-
-TI LP8788 MFD DRIVER
-M:	Milo Kim <milo.kim@ti.com>
-S:	Maintained
-F:	drivers/iio/adc/lp8788_adc.c
-F:	drivers/leds/leds-lp8788.c
-F:	drivers/mfd/lp8788*.c
-F:	drivers/power/supply/lp8788-charger.c
-F:	drivers/regulator/lp8788-*.c
-F:	include/linux/mfd/lp8788*.h
-
 TI NETCP ETHERNET DRIVER
 M:	Wingman Kwok <w-kwok2@ti.com>
 M:	Murali Karicheri <m-karicheri2@ti.com>