From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753508AbbKBOd1 (ORCPT ); Mon, 2 Nov 2015 09:33:27 -0500 Received: from smtp65.iad3a.emailsrvr.com ([173.203.187.65]:48234 "EHLO smtp65.iad3a.emailsrvr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751024AbbKBOd0 (ORCPT ); Mon, 2 Nov 2015 09:33:26 -0500 X-Sender-Id: abbotti@mev.co.uk Message-ID: <56377433.9080607@mev.co.uk> Date: Mon, 02 Nov 2015 14:33:23 +0000 From: Ian Abbott User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 MIME-Version: 1.0 To: ranjithece24@gmail.com, gregkh@linuxfoundation.org CC: hsweeten@visionengravers.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] comedi: drivers: Fix - BIT macro used coding style issue References: <5637716b.0866420a.ecc6e.ffffe0e5@mx.google.com> In-Reply-To: <5637716b.0866420a.ecc6e.ffffe0e5@mx.google.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/11/15 14:25, ranjithece24@gmail.com wrote: > From: Ranjith > > BIT macro is used for defining bit location instead of shifting > operator - coding style issue > > Signed-off-by: Ranjith T > --- > drivers/staging/comedi/drivers/addi_apci_1032.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/comedi/drivers/addi_apci_1032.c b/drivers/staging/comedi/drivers/addi_apci_1032.c > index b37166d..fd5ce21 100644 > --- a/drivers/staging/comedi/drivers/addi_apci_1032.c > +++ b/drivers/staging/comedi/drivers/addi_apci_1032.c > @@ -85,8 +85,8 @@ > #define APCI1032_STATUS_REG 0x0c > #define APCI1032_CTRL_REG 0x10 > #define APCI1032_CTRL_INT_OR (0 << 1) > -#define APCI1032_CTRL_INT_AND (1 << 1) > -#define APCI1032_CTRL_INT_ENA (1 << 2) > +#define APCI1032_CTRL_INT_AND BIT(1) > +#define APCI1032_CTRL_INT_ENA BIT(2) > > struct apci1032_private { > unsigned long amcc_iobase; /* base of AMCC I/O registers */ > Hartley already fixed this coding style issue. It's in linux-next. -- -=( Ian Abbott @ MEV Ltd. E-mail: )=- -=( Web: http://www.mev.co.uk/ )=-