Message ID | 20171228024714.GA3110@MacBook-Air (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, 27 Dec 2017 18:47:18 -0800 Ji-Hun Kim <jihuun.k@gmail.com> wrote: > Clean up checkpatch warning: > CHECK: Unnecessary parentheses around 'st->devid != ID_AD7195' > > Signed-off-by: Ji-Hun Kim <jihuun.k@gmail.com> I've personally never really cared about this particular one as removing the brackets doesn't make the code easier to read. However, it is worthwhile to suppress checkpatch warnings so we can see the ones that matter. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/staging/iio/adc/ad7192.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/iio/adc/ad7192.c b/drivers/staging/iio/adc/ad7192.c > index cadfb96..f015955 100644 > --- a/drivers/staging/iio/adc/ad7192.c > +++ b/drivers/staging/iio/adc/ad7192.c > @@ -271,7 +271,7 @@ static int ad7192_setup(struct ad7192_state *st, > if (pdata->sinc3_en) > st->mode |= AD7192_MODE_SINC3; > > - if (pdata->refin2_en && (st->devid != ID_AD7195)) > + if (pdata->refin2_en && st->devid != ID_AD7195) > st->conf |= AD7192_CONF_REFSEL; > > if (pdata->chop_en) { -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/staging/iio/adc/ad7192.c b/drivers/staging/iio/adc/ad7192.c index cadfb96..f015955 100644 --- a/drivers/staging/iio/adc/ad7192.c +++ b/drivers/staging/iio/adc/ad7192.c @@ -271,7 +271,7 @@ static int ad7192_setup(struct ad7192_state *st, if (pdata->sinc3_en) st->mode |= AD7192_MODE_SINC3; - if (pdata->refin2_en && (st->devid != ID_AD7195)) + if (pdata->refin2_en && st->devid != ID_AD7195) st->conf |= AD7192_CONF_REFSEL; if (pdata->chop_en) {
Clean up checkpatch warning: CHECK: Unnecessary parentheses around 'st->devid != ID_AD7195' Signed-off-by: Ji-Hun Kim <jihuun.k@gmail.com> --- drivers/staging/iio/adc/ad7192.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)