On Thu, Mar 2, 2017 at 7:03 AM, Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Fri, Feb 24, 2017 at 12:33 PM, Michał Kępień <kernel@xxxxxxxxxx> wrote: >> Commit 2cfa4b311acc ("platform/x86: intel-hid: Support 5 button array") >> contains some coding style cleanups compared to the code it was based >> on. This patch series "backports" these cleanups to the rest of the >> driver in order to make it more self-consistent and removes a redundant >> set_bit() call as a bonus. >> >> This series was not tested on hardware supported by intel-hid. > > Alex, do you have an access to hardware to test? I wouldn't going to > apply this until I get your tag(s) (in the best case > Reviewed-and-tested-by). The patches look good and tested on Latitude 7480 Reviewed-and-tested-by: Alex Hung <alex.hung@xxxxxxxxxxxxx> > > -- > With Best Regards, > Andy Shevchenko