From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH] hp-wireless: add Xiaomi's hardware id to the supported list Date: Thu, 13 Apr 2017 11:36:04 -0700 Message-ID: <20170413183604.GK2064@fury> References: <1491496336-19639-1-git-send-email-alex.hung@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([65.50.211.133]:44324 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755133AbdDMSgI (ORCPT ); Thu, 13 Apr 2017 14:36:08 -0400 Content-Disposition: inline In-Reply-To: <1491496336-19639-1-git-send-email-alex.hung@canonical.com> Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: Alex Hung Cc: andy@infradead.org, platform-driver-x86@vger.kernel.org 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 > --- > 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 > + * Copyright (C) 2014-2017 Alex Hung > * > * 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 > > -- Darren Hart VMware Open Source Technology Center