Re: [PATCH] MAINTAINERS: Add me on list of Dell laptop drivers

From: Darren Hart
Date: Thu Apr 02 2015 - 01:10:36 EST


On Sun, Mar 29, 2015 at 03:38:50PM +0200, Pali Rohár wrote:

Please include why. No empty commit messages please.

For example, you've written nearly a third of the dell-wmi.c driver, and all the
dell-smo8800.c driver.

Dell Laptop is still pending the keyboard backlight patches though I believe
(have I missed them?), so you don't currently have code ownership there. I agree
you should be on the MAINTAINERS list, but it would be good to get your keyboard
backlight changes in first.

Gosh... sure seems like we should have merged that already... am I missing
something? The last thing I see in master, testing, and next is the revert from
Jan 21....

Also, Matthew, you're the listed maintainer for these, do you agree to adding
Pali?

> Signed-off-by: Pali Rohár <pali.rohar@xxxxxxxxx>
> ---
> MAINTAINERS | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 88c09ca..72a08ef 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3058,10 +3058,16 @@ F: drivers/net/fddi/defxx.*
>
> DELL LAPTOP DRIVER
> M: Matthew Garrett <mjg59@xxxxxxxxxxxxx>
> +M: Pali Rohár <pali.rohar@xxxxxxxxx>
> L: platform-driver-x86@xxxxxxxxxxxxxxx
> S: Maintained
> F: drivers/platform/x86/dell-laptop.c
>
> +DELL LAPTOP FREEFALL DRIVER
> +M: Pali Rohár <pali.rohar@xxxxxxxxx>
> +S: Maintained
> +F: drivers/platform/x86/dell-smo8800.c
> +
> DELL LAPTOP SMM DRIVER
> M: Guenter Roeck <linux@xxxxxxxxxxxx>
> S: Maintained
> @@ -3076,6 +3082,7 @@ F: drivers/firmware/dcdbas.*
>
> DELL WMI EXTRAS DRIVER
> M: Matthew Garrett <mjg59@xxxxxxxxxxxxx>
> +M: Pali Rohár <pali.rohar@xxxxxxxxx>
> S: Maintained
> F: drivers/platform/x86/dell-wmi.c
>
> --
> 1.7.9.5
>
>

--
Darren Hart
Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/