| Message ID | 20200921210233.21449-1-krzk@kernel.org |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | MAINTAINERS: move Milo Kim to credits | expand |
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>
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
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.
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
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 --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>
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(-)