Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1263348
| Path | csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod |
|---|---|
| From | Ranjith <ranjithece24@gmail.com> |
| Newsgroups | linux.kernel |
| Subject | [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue |
| Date | Thu, 05 Nov 2015 17:00:03 +0100 |
| Message-ID | <qrv7d-9F-29@gated-at.bofh.it> (permalink) |
| X-Original-To | gregkh@linuxfoundation.org |
| Dkim-Signature | v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id; bh=RYqg2nPViEam3LXmWCdqKFn2t33NYonHgcQWQ/E24Xk=; b=V7sywE9Ve/smfxzX8UdCNv1sSjkUD6Ue04SLPgZkZRuDeGOIbGNGqrKZ+dBfWQYaQl r8HBQFzC5K6r8xqJLh5cqEPJxFl6wK4+dvKWk24DhD/Ski9MUOSAr+g4gaTZSy8b+kza pVI+/+9HwTGhwj+u1CmTSzcZlMhbh3Msc5+LPUgR+YQx8Ssd1xqtlEnzNgg+BFGcRIqc WJDZ9H8HDF2dTC4BaCtpPG5Yx7seIh9kABmEX2s8u+DucLI13rDTiW06QwCyJi9tdo0+ mW1DjqdmzWnu+O2sMgkXAZYeeOXwnDfkSop9+pMJwX+tCBzGH5aWqgYHzQ+/bp2rUXB6 SxnA== |
| X-Received | by 10.68.91.98 with SMTP id cd2mr10365024pbb.9.1446738843360; Thu, 05 Nov 2015 07:54:03 -0800 (PST) |
| X-Mailer | git-send-email 1.7.10.4 |
| Sender | robomod@news.nic.it |
| List-ID | <linux-kernel.vger.kernel.org> |
| X-Mailing-List | linux-kernel@vger.kernel.org |
| Approved | robomod@news.nic.it |
| Lines | 82 |
| Organization | linux.* mail to news gateway |
| X-Original-Cc | abbotti@mev.co.uk, hsweeten@visionengravers.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, ranjithece24@gmail.com |
| X-Original-Date | Thu, 5 Nov 2015 21:27:45 +0530 |
| X-Original-Message-ID | <1446739065-7875-1-git-send-email-ranjithece24@gmail.com> |
| X-Original-Sender | linux-kernel-owner@vger.kernel.org |
| Xref | csiph.com linux.kernel:1263348 |
Show key headers only | View raw
BIT macro is used for defining bit location instead of shifting operator - coding style issue Signed-off-by: Ranjith T <ranjithece24@gmail.com> --- drivers/staging/comedi/drivers/pcmmio.c | 42 +++++++++++++++---------------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/staging/comedi/drivers/pcmmio.c b/drivers/staging/comedi/drivers/pcmmio.c index 10472e6..807795c 100644 --- a/drivers/staging/comedi/drivers/pcmmio.c +++ b/drivers/staging/comedi/drivers/pcmmio.c @@ -84,25 +84,25 @@ #define PCMMIO_AI_LSB_REG 0x00 #define PCMMIO_AI_MSB_REG 0x01 #define PCMMIO_AI_CMD_REG 0x02 -#define PCMMIO_AI_CMD_SE (1 << 7) -#define PCMMIO_AI_CMD_ODD_CHAN (1 << 6) +#define PCMMIO_AI_CMD_SE BIT(7) +#define PCMMIO_AI_CMD_ODD_CHAN BIT(6) #define PCMMIO_AI_CMD_CHAN_SEL(x) (((x) & 0x3) << 4) #define PCMMIO_AI_CMD_RANGE(x) (((x) & 0x3) << 2) #define PCMMIO_RESOURCE_REG 0x02 #define PCMMIO_RESOURCE_IRQ(x) (((x) & 0xf) << 0) #define PCMMIO_AI_STATUS_REG 0x03 -#define PCMMIO_AI_STATUS_DATA_READY (1 << 7) -#define PCMMIO_AI_STATUS_DATA_DMA_PEND (1 << 6) -#define PCMMIO_AI_STATUS_CMD_DMA_PEND (1 << 5) -#define PCMMIO_AI_STATUS_IRQ_PEND (1 << 4) -#define PCMMIO_AI_STATUS_DATA_DRQ_ENA (1 << 2) -#define PCMMIO_AI_STATUS_REG_SEL (1 << 3) -#define PCMMIO_AI_STATUS_CMD_DRQ_ENA (1 << 1) -#define PCMMIO_AI_STATUS_IRQ_ENA (1 << 0) +#define PCMMIO_AI_STATUS_DATA_READY BIT(7) +#define PCMMIO_AI_STATUS_DATA_DMA_PEND BIT(6) +#define PCMMIO_AI_STATUS_CMD_DMA_PEND BIT(5) +#define PCMMIO_AI_STATUS_IRQ_PEND BIT(4) +#define PCMMIO_AI_STATUS_DATA_DRQ_ENA BIT(2) +#define PCMMIO_AI_STATUS_REG_SEL BIT(3) +#define PCMMIO_AI_STATUS_CMD_DRQ_ENA BIT(1) +#define PCMMIO_AI_STATUS_IRQ_ENA BIT(0) #define PCMMIO_AI_RES_ENA_REG 0x03 -#define PCMMIO_AI_RES_ENA_CMD_REG_ACCESS (0 << 3) -#define PCMMIO_AI_RES_ENA_AI_RES_ACCESS (1 << 3) -#define PCMMIO_AI_RES_ENA_DIO_RES_ACCESS (1 << 4) +#define PCMMIO_AI_RES_ENA_CMD_REG_ACCESS 0 +#define PCMMIO_AI_RES_ENA_AI_RES_ACCESS BIT(3) +#define PCMMIO_AI_RES_ENA_DIO_RES_ACCESS BIT(4) #define PCMMIO_AI_2ND_ADC_OFFSET 0x04 #define PCMMIO_AO_LSB_REG 0x08 @@ -125,14 +125,14 @@ #define PCMMIO_AO_CMD_CHAN_SEL(x) (((x) & 0x03) << 1) #define PCMMIO_AO_CMD_CHAN_SEL_ALL (0x0f << 0) #define PCMMIO_AO_STATUS_REG 0x0b -#define PCMMIO_AO_STATUS_DATA_READY (1 << 7) -#define PCMMIO_AO_STATUS_DATA_DMA_PEND (1 << 6) -#define PCMMIO_AO_STATUS_CMD_DMA_PEND (1 << 5) -#define PCMMIO_AO_STATUS_IRQ_PEND (1 << 4) -#define PCMMIO_AO_STATUS_DATA_DRQ_ENA (1 << 2) -#define PCMMIO_AO_STATUS_REG_SEL (1 << 3) -#define PCMMIO_AO_STATUS_CMD_DRQ_ENA (1 << 1) -#define PCMMIO_AO_STATUS_IRQ_ENA (1 << 0) +#define PCMMIO_AO_STATUS_DATA_READY BIT(7) +#define PCMMIO_AO_STATUS_DATA_DMA_PEND BIT(6) +#define PCMMIO_AO_STATUS_CMD_DMA_PEND BIT(5) +#define PCMMIO_AO_STATUS_IRQ_PEND BIT(4) +#define PCMMIO_AO_STATUS_DATA_DRQ_ENA BIT(2) +#define PCMMIO_AO_STATUS_REG_SEL BIT(3) +#define PCMMIO_AO_STATUS_CMD_DRQ_ENA BIT(1) +#define PCMMIO_AO_STATUS_IRQ_ENA BIT(0) #define PCMMIO_AO_RESOURCE_ENA_REG 0x0b #define PCMMIO_AO_2ND_DAC_OFFSET 0x04 -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread
[PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue Ranjith <ranjithece24@gmail.com> - 2015-11-05 17:00 +0100
Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue Ranjith T <ranjithece24@gmail.com> - 2015-11-06 15:00 +0100
Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue Ian Abbott <abbotti@mev.co.uk> - 2015-11-06 16:10 +0100
Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue Greg KH <gregkh@linuxfoundation.org> - 2015-11-06 18:00 +0100
Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-06 21:30 +0100
csiph-web