On Fri, 3 Nov 2017 09:33:57 +0100 SF Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> wrote: > From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > Date: Fri, 3 Nov 2017 09:26:28 +0100 > > * Add a jump target so that a call of the function "mutex_unlock" is stored > only twice in this function implementation. > > * Replace two calls by goto statements. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Again, a reasonable little tidy up. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to check we haven't missed anything. Thanks, Jonathan > --- > drivers/staging/iio/cdc/ad7746.c | 17 +++++++++-------- > 1 file changed, 9 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c > index a124853a05f0..c4a864725376 100644 > --- a/drivers/staging/iio/cdc/ad7746.c > +++ b/drivers/staging/iio/cdc/ad7746.c > @@ -302,23 +302,24 @@ static inline ssize_t ad7746_start_calib(struct device *dev, > mutex_lock(&chip->lock); > regval |= chip->config; > ret = i2c_smbus_write_byte_data(chip->client, AD7746_REG_CFG, regval); > - if (ret < 0) { > - mutex_unlock(&chip->lock); > - return ret; > - } > + if (ret < 0) > + goto unlock; > > do { > msleep(20); > ret = i2c_smbus_read_byte_data(chip->client, AD7746_REG_CFG); > - if (ret < 0) { > - mutex_unlock(&chip->lock); > - return ret; > - } > + if (ret < 0) > + goto unlock; > + > } while ((ret == regval) && timeout--); > > mutex_unlock(&chip->lock); > > return len; > + > +unlock: > + mutex_unlock(&chip->lock); > + return ret; > } > > static ssize_t ad7746_start_offset_calib(struct device *dev, _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel