Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752579AbdLMJ2I (ORCPT ); Wed, 13 Dec 2017 04:28:08 -0500 Received: from mail-wr0-f196.google.com ([209.85.128.196]:34009 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752408AbdLMJZj (ORCPT ); Wed, 13 Dec 2017 04:25:39 -0500 X-Google-Smtp-Source: ACJfBou2V9MtPSsy+pSZUtYfdgSYuPcOwAM7mEEsPoK8i0YJwfZlI6DLMrdtQ99uhWqiqWPZ+rNkoO2+Rk9ixDKXR/4= MIME-Version: 1.0 In-Reply-To: <20171213082301.23209-1-dhaval.shah@softnautics.com> References: <20171213082301.23209-1-dhaval.shah@softnautics.com> From: Philippe Ombredanne Date: Wed, 13 Dec 2017 10:24:57 +0100 Message-ID: Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines To: Dhaval Shah Cc: pali.rohar@gmail.com, Darren Hart , Andy Shevchenko , platform-driver-x86@vger.kernel.org, LKML 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-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id vBD9SG8Q017922 Content-Length: 2202 Lines: 53 Dhaval, On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah wrote: > Resolved Block comments use * on subsequent lines checkpatch warnings. > Issue found by checkpatch. > > Signed-off-by: Dhaval Shah > --- > drivers/platform/x86/dell-rbtn.c | 26 +++++++++++++------------- > drivers/platform/x86/dell-rbtn.h | 26 +++++++++++++------------- > 2 files changed, 26 insertions(+), 26 deletions(-) > > diff --git a/drivers/platform/x86/dell-rbtn.c b/drivers/platform/x86/dell-rbtn.c > index f3afe778001e..cab93c6dcd34 100644 > --- a/drivers/platform/x86/dell-rbtn.c > +++ b/drivers/platform/x86/dell-rbtn.c > @@ -1,17 +1,17 @@ > /* > - Dell Airplane Mode Switch driver > - Copyright (C) 2014-2015 Pali Rohár > - > - 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. > -*/ > + * Dell Airplane Mode Switch driver > + * Copyright (C) 2014-2015 Pali Rohár > + * > + * 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. > + */ Do you mind using the new SPDX ids instead of this long legalese? You can check the doc patches from Thomas for details [1] [1] https://lkml.org/lkml/2017/12/4/934 -- Cordially Philippe Ombredanne