Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753240AbdCMQB1 (ORCPT ); Mon, 13 Mar 2017 12:01:27 -0400 Received: from mail-qk0-f193.google.com ([209.85.220.193]:35173 "EHLO mail-qk0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751089AbdCMQBT (ORCPT ); Mon, 13 Mar 2017 12:01:19 -0400 MIME-Version: 1.0 In-Reply-To: <20170305225721.GF28473@marvin.atrad.com.au> References: <20170301064254.1311-1-kernel@kempniu.pl> <20170305225721.GF28473@marvin.atrad.com.au> From: Andy Shevchenko Date: Mon, 13 Mar 2017 18:01:17 +0200 Message-ID: Subject: Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup To: Jonathan Woithe Cc: "Micha?? K??pie??" , Darren Hart , Andy Shevchenko , Platform Driver , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 969 Lines: 26 On Mon, Mar 6, 2017 at 12:57 AM, Jonathan Woithe wrote: > On Wed, Mar 01, 2017 at 07:42:52AM +0100, Micha?? K??pie?? wrote: >> Here are two minor cleanups for acpi_fujitsu_bl_notify() that I came up >> with while preparing the sparse keymap migration. >> >> Changes from v1: >> >> - Rebase on top of reworked Alan Jenkins' cleanup patch series. >> >> - Join integer variable declarations into a single line in patch 2/2. >> >> drivers/platform/x86/fujitsu-laptop.c | 64 +++++++++++++++-------------------- >> 1 file changed, 28 insertions(+), 36 deletions(-) > > These two clean ups, as their descriptions indicate, improve the clarity of > the driver code and permit some minor optimisations. No regressions are > evident when tested on S7020 hardware. Please apply. > > Tested-by: Jonathan Woithe > Reviewed-by: Jonathan Woithe Pushed to testing, thanks! -- With Best Regards, Andy Shevchenko