platform/x86: dell-wmi: Add a better description for "stealth mode"

This is based on Mario's explanation and observation of my laptop.

Suggested-by: "Pali Rohár" <pali.rohar@gmail.com>
Signed-off-by: Andy Lutomirski <luto@kernel.org>
Cc: Mario Limonciello <mario_limonciello@dell.com>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>

diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c
index 24467b1f0f21ed38a9cdc0d6f0c74d986ff77680..73aee0d1aea467a2056b1f0c6c36827e690d385b 100644 (file)
--- a/drivers/platform/x86/dell-wmi.c
+++ b/drivers/platform/x86/dell-wmi.c
@@ -272,7 +272,12 @@ static const struct key_entry dell_wmi_keymap_type_0010[] __initconst = {
        /* RGB keyboard backlight control */
        { KE_IGNORE, 0x154, { KEY_RESERVED } },
 
-       /* Stealth mode toggle */
+       /*
+        * Stealth mode toggle. This will "disable all lights and sounds".
+        * The action is performed by the BIOS and EC; the WMI event is just
+        * a notification. On the XPS 13 9350, this is Fn+F7, and there's
+        * a BIOS setting to enable and disable the hotkey.
+        */
        { KE_IGNORE, 0x155, { KEY_RESERVED } },
 
        /* Rugged magnetic dock attach/detach events */