| Message ID | 1491496336-19639-1-git-send-email-alex.hung@canonical.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Darren Hart |
| Headers | show |
On Thu, Apr 06, 2017 at 05:32:16PM +0100, Alex Hung wrote: > The airplane mode button on Xiaomi's new laptops are the same as HP > laptops. > > This is tested on Xiaomi Notebook Air 13. > > Signed-off-by: Alex Hung <alex.hung@canonical.com> > --- > drivers/platform/x86/hp-wireless.c | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/platform/x86/hp-wireless.c b/drivers/platform/x86/hp-wireless.c > index 988eedb..1d4df3d 100644 > --- a/drivers/platform/x86/hp-wireless.c > +++ b/drivers/platform/x86/hp-wireless.c > @@ -1,7 +1,7 @@ > /* > - * hp-wireless button for Windows 8 > + * Airplane mode button for HP & Xiaomi laptops > * > - * Copyright (C) 2014 Alex Hung <alex.hung@canonical.com> > + * Copyright (C) 2014-2017 Alex Hung <alex.hung@canonical.com> > * > * This program is free software; you can redistribute it and/or modify > * it under the terms of the GNU General Public License as published by > @@ -29,11 +29,13 @@ > MODULE_LICENSE("GPL"); > MODULE_AUTHOR("Alex Hung"); > MODULE_ALIAS("acpi*:HPQ6001:*"); > +MODULE_ALIAS("acpi*:WSTADEF:*"); > > static struct input_dev *hpwl_input_dev; > > static const struct acpi_device_id hpwl_ids[] = { > {"HPQ6001", 0}, > + {"WSTADEF", 0}, > {"", 0}, > }; > > @@ -112,7 +114,7 @@ static int __init hpwl_init(void) > { > int err; > > - pr_info("Initializing HPQ6001 module\n"); > + pr_info("Initializing HPQ6001/WSTADEF module\n"); It's just a module, no need to keep the initializing/exiting messages. Rather than start adding every HID to the message, let's just drop the messages now. Rather than resend, I've just dropped them and updated the commit, and pushed to testing. Let me know if you have any concerns. > err = acpi_bus_register_driver(&hpwl_driver); > if (err) > pr_err("Unable to register HP wireless control driver.\n"); > @@ -122,7 +124,7 @@ static int __init hpwl_init(void) > > static void __exit hpwl_exit(void) > { > - pr_info("Exiting HPQ6001 module\n"); > + pr_info("Exiting HPQ6001/WSTADEF module\n"); > acpi_bus_unregister_driver(&hpwl_driver); > } > > -- > 2.7.4 > >
diff --git a/drivers/platform/x86/hp-wireless.c b/drivers/platform/x86/hp-wireless.c index 988eedb..1d4df3d 100644 --- a/drivers/platform/x86/hp-wireless.c +++ b/drivers/platform/x86/hp-wireless.c @@ -1,7 +1,7 @@ /* - * hp-wireless button for Windows 8 + * Airplane mode button for HP & Xiaomi laptops * - * Copyright (C) 2014 Alex Hung <alex.hung@canonical.com> + * Copyright (C) 2014-2017 Alex Hung <alex.hung@canonical.com> * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -29,11 +29,13 @@ MODULE_LICENSE("GPL"); MODULE_AUTHOR("Alex Hung"); MODULE_ALIAS("acpi*:HPQ6001:*"); +MODULE_ALIAS("acpi*:WSTADEF:*"); static struct input_dev *hpwl_input_dev; static const struct acpi_device_id hpwl_ids[] = { {"HPQ6001", 0}, + {"WSTADEF", 0}, {"", 0}, }; @@ -112,7 +114,7 @@ static int __init hpwl_init(void) { int err; - pr_info("Initializing HPQ6001 module\n"); + pr_info("Initializing HPQ6001/WSTADEF module\n"); err = acpi_bus_register_driver(&hpwl_driver); if (err) pr_err("Unable to register HP wireless control driver.\n"); @@ -122,7 +124,7 @@ static int __init hpwl_init(void) static void __exit hpwl_exit(void) { - pr_info("Exiting HPQ6001 module\n"); + pr_info("Exiting HPQ6001/WSTADEF module\n"); acpi_bus_unregister_driver(&hpwl_driver); }
The airplane mode button on Xiaomi's new laptops are the same as HP laptops. This is tested on Xiaomi Notebook Air 13. Signed-off-by: Alex Hung <alex.hung@canonical.com> --- drivers/platform/x86/hp-wireless.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)