lkml.org 
[lkml]   [2018]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3 1/3] x86: add support for Huawei WMI hotkeys.
Yes, it does.
On Fri, Nov 9, 2018 at 5:52 AM <ayman.bagabas@gmail.com> wrote:
>
> On Thu, 2018-11-08 at 21:58 +0200, Andy Shevchenko wrote:
> > On Thu, Nov 8, 2018 at 7:17 PM Ayman Bagabas <ayman.bagabas@gmail.com
> > > wrote:
> > > This driver adds support for missing hotkeys on some Huawei
> > > laptops.
> > > Currently, only Huawei Matebook X and Matebook X Pro is supported.
> > >
> >
> > Thanks for an update, my comments below.
> >
> >
> > > +config HUAWEI_LAPTOP
> > > + tristate "Huawei WMI hotkeys driver"
> > > + depends on ACPI
> >
> > Do you need an ACPI dependency be explicit here?
>
> Probably don't need it.
>
> >
> > > + depends on ACPI_WMI
> > > + depends on INPUT
> > > + select INPUT_SPARSEKMAP
> > > + help
> > > + This driver provides support for Huawei WMI hotkeys.
> > > + It enables the missing keys and adds support to the
> > > micmute
> > > + LED found on some of these laptops.
> > > +/*
> > > + * Huawei WMI hotkeys
> > > + *
> > > + * Copyright (C) 2018 Ayman Bagabas <
> > > ayman.bagabas@gmail.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
> > > + * the Free Software Foundation, either version 2 of the License,
> > > or
> > > + * (at your option) any later version.
> > > + *
> > > + * This program is distributed in the hope that it will be
> > > useful,
> > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > > + * GNU General Public License for more details.
> > > + *
> > > + * You should have received a copy of the GNU General Public
> > > License
> > > + * along with this program. If not, see <
> > > https://www.gnu.org/licenses/>;.
> > > + *
> >
> > Please, replace this boilerplate text with appropriate SPDX
> > identifier.
>
> Soryy about that. This alright?
>
> // SPDX-License-Identifier: GPL-2.0
> /*
> * Huawei WMI hotkeys
> *
> * Copyright (C) 2018 Ayman Bagabas <ayman.bagabas@gmail.com>
> */
>
>
>


--
With Best Regards,
Andy Shevchenko

\
 
 \ /
  Last update: 2018-11-09 11:42    [W:0.117 / U:1.800 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site