Re: [PATCH 01/12] platform/x86: dell-wmi: label driver as handling notifications

From: Pali RohÃr
Date: Mon Sep 25 2017 - 12:04:15 EST


On Thursday 21 September 2017 08:57:06 Mario Limonciello wrote:
> This driver serves the purpose of responding to WMI based notifications
> from the DELL_EVENT_GUID (9DBB5994-A997-11DA-B012-B622A1EF5492).
> Other GUIDs will be handled by separate drivers.
>
> Update the language used by this driver to avoid future confusion.

Hi! I'm not sure if if "notifications" word is better then "extras".
Basically the most important part of the dell-wmi driver is to deliver
key press events via input device.

Has anybody else better word or description for this?

> Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxxx>
> ---
> MAINTAINERS | 2 +-
> drivers/platform/x86/Kconfig | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2281af4b41b6..5d8ea24a8ee7 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3996,7 +3996,7 @@ S: Maintained
> F: Documentation/dcdbas.txt
> F: drivers/firmware/dcdbas.*
>
> -DELL WMI EXTRAS DRIVER
> +DELL WMI NOTIFICATIONS DRIVER
> M: Matthew Garrett <mjg59@xxxxxxxxxxxxx>
> M: Pali RohÃr <pali.rohar@xxxxxxxxx>
> S: Maintained
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 80b87954f6dd..9e52f05daa2e 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -116,7 +116,7 @@ config DELL_LAPTOP
> laptops (except for some models covered by the Compal driver).
>
> config DELL_WMI
> - tristate "Dell WMI extras"
> + tristate "Dell WMI notifications"
> depends on ACPI_WMI
> depends on DMI
> depends on INPUT

--
Pali RohÃr
pali.rohar@xxxxxxxxx