Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754766AbdGVUhc (ORCPT ); Sat, 22 Jul 2017 16:37:32 -0400 Received: from mail.kernel.org ([198.145.29.99]:34836 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750865AbdGVUhb (ORCPT ); Sat, 22 Jul 2017 16:37:31 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C2373219A9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jic23@kernel.org Date: Sat, 22 Jul 2017 21:37:26 +0100 From: Jonathan Cameron To: Jaya Durga Cc: gregkh@linuxfoundation.org, lars@metafoo.de, Michael.Hennerich@analog.com, knaack.h@gmx.de, pmeerw@pmeerw.net, linux-iio@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 3/3] Staging: iio: adc: ad7280a.c: Fixed Macro argument reuse Message-ID: <20170722213726.2d88252e@kernel.org> In-Reply-To: <1500467157-14739-3-git-send-email-rjdurga@gmail.com> References: <1500467157-14739-1-git-send-email-rjdurga@gmail.com> <1500467157-14739-3-git-send-email-rjdurga@gmail.com> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3456 Lines: 94 On Wed, 19 Jul 2017 17:55:57 +0530 Jaya Durga wrote: > CHECK: Macro argument reuse 'addr' - possible side-effects? > > convert AD7280A_DEVADDR to ad7280a_devaddr static function > to fix checkpath check > > v3: small style changes > > Signed-off-by: Jaya Durga This patch is fine so I've applied it to the togreg branch of iio.git which will be initially pushed out as testing for the autobuilders to play with it. Just as an aside there was no connection between the 3 patches in this series. As such, what was the point in making them a series? If you aren't either working on one driver, or applying the same change to a set of drivers, please don't combine patches into a series of random unconnected patches. Jonathan > --- > drivers/staging/iio/adc/ad7280a.c | 21 +++++++++++++-------- > 1 file changed, 13 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c > index d5ab83f..f85dde9 100644 > --- a/drivers/staging/iio/adc/ad7280a.c > +++ b/drivers/staging/iio/adc/ad7280a.c > @@ -99,9 +99,14 @@ > #define AD7280A_DEVADDR_MASTER 0 > #define AD7280A_DEVADDR_ALL 0x1F > /* 5-bit device address is sent LSB first */ > -#define AD7280A_DEVADDR(addr) (((addr & 0x1) << 4) | ((addr & 0x2) << 3) | \ > - (addr & 0x4) | ((addr & 0x8) >> 3) | \ > - ((addr & 0x10) >> 4)) > +static unsigned int ad7280a_devaddr(unsigned int addr) > +{ > + return ((addr & 0x1) << 4) | > + ((addr & 0x2) << 3) | > + (addr & 0x4) | > + ((addr & 0x8) >> 3) | > + ((addr & 0x10) >> 4); > +} > > /* During a read a valid write is mandatory. > * So writing to the highest available address (Address 0x1F) > @@ -372,7 +377,7 @@ static int ad7280_chain_setup(struct ad7280_state *st) > if (ad7280_check_crc(st, val)) > return -EIO; > > - if (n != AD7280A_DEVADDR(val >> 27)) > + if (n != ad7280a_devaddr(val >> 27)) > return -EIO; > } > > @@ -511,7 +516,7 @@ static int ad7280_channel_init(struct ad7280_state *st) > st->channels[cnt].info_mask_shared_by_type = > BIT(IIO_CHAN_INFO_SCALE); > st->channels[cnt].address = > - AD7280A_DEVADDR(dev) << 8 | ch; > + ad7280a_devaddr(dev) << 8 | ch; > st->channels[cnt].scan_index = cnt; > st->channels[cnt].scan_type.sign = 'u'; > st->channels[cnt].scan_type.realbits = 12; > @@ -558,7 +563,7 @@ static int ad7280_attr_init(struct ad7280_state *st) > for (ch = AD7280A_CELL_VOLTAGE_1; ch <= AD7280A_CELL_VOLTAGE_6; > ch++, cnt++) { > st->iio_attr[cnt].address = > - AD7280A_DEVADDR(dev) << 8 | ch; > + ad7280a_devaddr(dev) << 8 | ch; > st->iio_attr[cnt].dev_attr.attr.mode = > 0644; > st->iio_attr[cnt].dev_attr.show = > @@ -574,7 +579,7 @@ static int ad7280_attr_init(struct ad7280_state *st) > &st->iio_attr[cnt].dev_attr.attr; > cnt++; > st->iio_attr[cnt].address = > - AD7280A_DEVADDR(dev) << 8 | > + ad7280a_devaddr(dev) << 8 | > (AD7280A_CB1_TIMER + ch); > st->iio_attr[cnt].dev_attr.attr.mode = > 0644; > @@ -918,7 +923,7 @@ static int ad7280_probe(struct spi_device *spi) > if (ret) > goto error_unregister; > > - ret = ad7280_write(st, AD7280A_DEVADDR(st->slave_num), > + ret = ad7280_write(st, ad7280a_devaddr(st->slave_num), > AD7280A_ALERT, 0, > AD7280A_ALERT_GEN_STATIC_HIGH | > (pdata->chain_last_alert_ignore & 0xF));