Re: [PATCH] drivers/x86: add thinkpad-wmi

From: Andy Shevchenko
Date: Mon Sep 04 2017 - 13:15:38 EST


On Mon, Sep 4, 2017 at 11:21 AM, Corentin Chary
<corentin.chary@xxxxxxxxx> wrote:
> This driver has been available on https://github.com/iksaif/thinkpad-wmi for
> a few year and is already deployed on large fleets of thinkpad laptops.
>
> The WMI interface is documented here: http://download.lenovo.com/ibmdl/pub/pc/pccbbs/thinkcentre_pdf/hrdeploy_en.pdf
> It mostly focused on changing BIOS/Firmware settings.

I will do full review later, few comments right now though.
Thanks for doing this btw.

> +Date: Aug 2017
> +KernelVersion: 4.14

v4.15 apparently

> + * Thinkpad WMI hotkey driver

Solely for hot keys?

> + *
> + * Copyright(C) 2012 Corentin Chary <corentin.chary@xxxxxxxxx>

2012,2017?


> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA

Remove this. It had been changed once, no guarantee it will not again.


> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/types.h>
> +#include <linux/debugfs.h>
> +#include <linux/uaccess.h>
> +#include <linux/seq_file.h>
> +#include <linux/platform_device.h>
> +#include <linux/device.h>
> +#include <linux/acpi.h>

Alphabetical.

> +static int __init thinkpad_wmi_init(void)
> +{
> + platform_device = platform_create_bundle(&platform_driver,
> + thinkpad_wmi_probe,
> + NULL, 0, NULL, 0);
> + if (IS_ERR(platform_device))
> + return PTR_ERR(platform_device);
> + return 0;
> +}
> +
> +static void __exit thinkpad_wmi_exit(void)
> +{
> + platform_device_unregister(platform_device);
> + platform_driver_unregister(&platform_driver);
> +}
> +
> +module_init(thinkpad_wmi_init);
> +module_exit(thinkpad_wmi_exit);

I didn't read the code, does it use WMI bus which Andy L. introduced recently?

--
With Best Regards,
Andy Shevchenko