2020-09-21 21:04:38

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: move Milo Kim to credits

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 <[email protected]>
---
CREDITS | 3 +++
MAINTAINERS | 23 -----------------------
2 files changed, 3 insertions(+), 23 deletions(-)

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: [email protected]
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 <[email protected]>
-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 <[email protected]>
-S: Maintained
-F: drivers/power/supply/lp8727_charger.c
-F: include/linux/platform_data/lp8727.h
-
-TI LP8788 MFD DRIVER
-M: Milo Kim <[email protected]>
-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 <[email protected]>
M: Murali Karicheri <[email protected]>
--
2.17.1


2020-09-21 21:07:44

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: move Milo Kim to credits

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

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 <[email protected]>
> -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 <[email protected]>
> -S: Maintained
> -F: drivers/power/supply/lp8727_charger.c
> -F: include/linux/platform_data/lp8727.h
> -
> -TI LP8788 MFD DRIVER
> -M: Milo Kim <[email protected]>
> -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 <[email protected]>
> M: Murali Karicheri <[email protected]>

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html


Attachments:
(No filename) (1.75 kB)
signature.asc (188.00 B)
Digital signature
Download all attachments

2020-09-22 08:33:40

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: move Milo Kim to credits

On Mon, 21 Sep 2020 at 23:06, Pavel Machek <[email protected]> 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 <[email protected]>
>
> 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

2020-09-22 09:38:55

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: move Milo Kim to credits

On Tue, Sep 22, 2020 at 09:08:37AM +0200, Krzysztof Kozlowski wrote:
> On Mon, 21 Sep 2020 at 23:06, Pavel Machek <[email protected]> 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.


Attachments:
(No filename) (791.00 B)
signature.asc (499.00 B)
Download all attachments

2020-09-22 11:39:43

by Dan Murphy

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: move Milo Kim to credits

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

Dan

2020-09-22 11:50:39

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: move Milo Kim to credits

On Tue, 22 Sep 2020 at 13:36, Dan Murphy <[email protected]> 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 <[email protected]> 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 <[email protected]>

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

Best regards,
Krzysztof