2021-02-10 17:34:52

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

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 add Dan Murphy from TI to look after:
- TI LP855x backlight driver,
- TI LP8727 charger driver,
- TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
Cc: Dan Murphy <[email protected]>
Acked-by: Dan Murphy <[email protected]>
Acked-by: Jonathan Cameron <[email protected]>
Acked-by: Sebastian Reichel <[email protected]>
Acked-by: Daniel Thompson <[email protected]>
Acked-by: Mark Brown <[email protected]>
Acked-by: Jingoo Han <[email protected]>

---

Dear Lee,

Could you take care about this patch?

Best regards,
Krzysztof

Changes since v2:
1. Fix subject (TP -> TI)

Changes since v1:
1. Add Dan Murphy, do not remove the entries.
---
CREDITS | 3 +++
MAINTAINERS | 6 +++---
2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/CREDITS b/CREDITS
index be097156bd71..71552790774d 100644
--- a/CREDITS
+++ b/CREDITS
@@ -1933,6 +1933,9 @@ N: Kukjin Kim
E: [email protected]
D: Samsung S3C, S5P and Exynos ARM architectures

+N: Milo Kim
+D: TI LP855x, LP8727 and LP8788 drivers
+
N: Sangbeom Kim
E: [email protected]
D: Samsung SoC Audio (ASoC) drivers
diff --git a/MAINTAINERS b/MAINTAINERS
index 00bca3e220cc..8f3887813d3b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17881,20 +17881,20 @@ F: sound/soc/codecs/isabelle*
F: sound/soc/codecs/lm49453*

TI LP855x BACKLIGHT DRIVER
-M: Milo Kim <[email protected]>
+M: Dan Murphy <[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]>
+M: Dan Murphy <[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]>
+M: Dan Murphy <[email protected]>
S: Maintained
F: drivers/iio/adc/lp8788_adc.c
F: drivers/leds/leds-lp8788.c
--
2.25.1


2021-02-10 17:40:06

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

On Wed, Feb 10, 2021 at 11:33:49AM -0600, Dan Murphy wrote:
> Krzysztof
>
> On 2/10/21 11:29 AM, Krzysztof Kozlowski wrote:
> > 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 add Dan Murphy from TI to look after:
> > - TI LP855x backlight driver,
> > - TI LP8727 charger driver,
> > - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> >
> > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > Cc: Dan Murphy <[email protected]>
>
> Unfortunately I need to remove my ACK.  My TI email will be disabled this
> Friday.
>
> No replacement has been identified

Ah, then I propose to mark entries as orphaned - without any maintainer.
Other option is to remove them entirely so they will be covered by
regular power supply entry.

Any preferences here?

Best regards,
Krzysztof

2021-02-10 17:41:09

by Dan Murphy

[permalink] [raw]
Subject: Re: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

Krzysztof

On 2/10/21 11:29 AM, Krzysztof Kozlowski wrote:
> 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 add Dan Murphy from TI to look after:
> - TI LP855x backlight driver,
> - TI LP8727 charger driver,
> - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> Cc: Dan Murphy <[email protected]>

Unfortunately I need to remove my ACK.  My TI email will be disabled
this Friday.

No replacement has been identified

Dan

2021-02-11 09:16:08

by Lee Jones

[permalink] [raw]
Subject: Re: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

On Wed, 10 Feb 2021, Krzysztof Kozlowski wrote:

> On Wed, Feb 10, 2021 at 11:33:49AM -0600, Dan Murphy wrote:
> > Krzysztof
> >
> > On 2/10/21 11:29 AM, Krzysztof Kozlowski wrote:
> > > 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 add Dan Murphy from TI to look after:
> > > - TI LP855x backlight driver,
> > > - TI LP8727 charger driver,
> > > - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> > >
> > > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > > Cc: Dan Murphy <[email protected]>
> >
> > Unfortunately I need to remove my ACK.  My TI email will be disabled this
> > Friday.
> >
> > No replacement has been identified
>
> Ah, then I propose to mark entries as orphaned - without any maintainer.
> Other option is to remove them entirely so they will be covered by
> regular power supply entry.
>
> Any preferences here?

Counter-proposal: Since they are driver-level entries, just remove
them altogether. They are not orphaned as they will still come under
the subsystem umbrella.

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

2021-02-11 09:16:20

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

On Thu, Feb 11, 2021 at 09:03:27AM +0000, Lee Jones wrote:
> On Wed, 10 Feb 2021, Krzysztof Kozlowski wrote:
>
> > On Wed, Feb 10, 2021 at 11:33:49AM -0600, Dan Murphy wrote:
> > > Krzysztof
> > >
> > > On 2/10/21 11:29 AM, Krzysztof Kozlowski wrote:
> > > > 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 add Dan Murphy from TI to look after:
> > > > - TI LP855x backlight driver,
> > > > - TI LP8727 charger driver,
> > > > - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> > > >
> > > > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > > > Cc: Dan Murphy <[email protected]>
> > >
> > > Unfortunately I need to remove my ACK.  My TI email will be disabled this
> > > Friday.
> > >
> > > No replacement has been identified
> >
> > Ah, then I propose to mark entries as orphaned - without any maintainer.
> > Other option is to remove them entirely so they will be covered by
> > regular power supply entry.
> >
> > Any preferences here?
>
> Counter-proposal: Since they are driver-level entries, just remove
> them altogether. They are not orphaned as they will still come under
> the subsystem umbrella.

I also vote for this (that was actually my v1 of this patch).

Best regards,
Krzysztof

2021-02-11 12:29:07

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [RESEND PATCH v3] MAINTAINERS: add Dan Murphy as TI LP8xxx drivers maintainer

On Thu, 11 Feb 2021 10:05:16 +0100
Krzysztof Kozlowski <[email protected]> wrote:

> On Thu, Feb 11, 2021 at 09:03:27AM +0000, Lee Jones wrote:
> > On Wed, 10 Feb 2021, Krzysztof Kozlowski wrote:
> >
> > > On Wed, Feb 10, 2021 at 11:33:49AM -0600, Dan Murphy wrote:
> > > > Krzysztof
> > > >
> > > > On 2/10/21 11:29 AM, Krzysztof Kozlowski wrote:
> > > > > 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 add Dan Murphy from TI to look after:
> > > > > - TI LP855x backlight driver,
> > > > > - TI LP8727 charger driver,
> > > > > - TI LP8788 MFD (ADC, LEDs, charger and regulator) drivers.
> > > > >
> > > > > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > > > > Cc: Dan Murphy <[email protected]>
> > > >
> > > > Unfortunately I need to remove my ACK.? My TI email will be disabled this
> > > > Friday.
> > > >
> > > > No replacement has been identified
> > >
> > > Ah, then I propose to mark entries as orphaned - without any maintainer.
> > > Other option is to remove them entirely so they will be covered by
> > > regular power supply entry.
> > >
> > > Any preferences here?
> >
> > Counter-proposal: Since they are driver-level entries, just remove
> > them altogether. They are not orphaned as they will still come under
> > the subsystem umbrella.
>
> I also vote for this (that was actually my v1 of this patch).
>
> Best regards,
> Krzysztof
>

Likewise, fine with just dropping the entries.

Jonathan