MIME-Version: 1.0
In-Reply-To: <20171213081309.22851-1-dhaval.shah@softnautics.com>
References: <20171213081309.22851-1-dhaval.shah@softnautics.com>
From: Andy Shevchenko <andy.shevchenko@gmail.com>
Date: Thu, 14 Dec 2017 19:59:55 +0200
Message-ID: <CAHp75VfTO9iCVFhddb1WHCL8RPKTVT7OhdOwvHJNy60V93wy0A@mail.gmail.com>
Subject: Re: [PATCH 0/3] platform/x86: dell-laptop: All the warnings are resolved.
To: Dhaval Shah <dhaval.shah@softnautics.com>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
        =?UTF-8?Q?Pali_Roh=C3=A1r?= <pali.rohar@gmail.com>,
        Andy Shevchenko <andy@infradead.org>,
        Platform Driver <platform-driver-x86@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 814
Lines: 29

On Wed, Dec 13, 2017 at 10:13 AM, Dhaval Shah
<dhaval.shah@softnautics.com> wrote:
> There are total three types of warnings are resolved.
> First patch  : Block comments use * on subsequent lines
> Second patch : Missing a blank line after declarations
> third patch  : braces {} are not necessary for single statement blocks.
>

Can we survive without this? Does it break build or alike?


> Dhaval Shah (3):
>   platform/x86: dell-laptop: Block comments use * on subsequent lines
>   platform/x86: dell-laptop: Missing a blank line after declarations
>   platform/x86: dell-laptop: braces {} are not necessary for single
>     statement blocks
>
>  drivers/platform/x86/dell-laptop.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
>
> --
> 2.11.0
>


-- 
With Best Regards,
Andy Shevchenko