Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

staging: iio: ad9850.c: code cleanup

checkpath.pl was complaining about value_mask:
ERROR: Macros with complex values should be enclosed in parenthesis

I fixed this by simply removing it since it's not used (as well as another
macro). Got rid of the un-necessary error_ret label as well.

Signed-off-by: Guillaume Morin <guillaume@morinfr.org>
Reported-by: Michael Welling <mwelling@ieee.org>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

authored by

Guillaume Morin and committed by
Jonathan Cameron
87a46909 fc167f62

-6
-6
drivers/staging/iio/frequency/ad9850.c
··· 21 21 22 22 #define DRV_NAME "ad9850" 23 23 24 - #define value_mask (u16)0xf000 25 - #define addr_shift 12 26 - 27 24 /* Register format: 4 bits addr + 12 bits value */ 28 25 struct ad9850_config { 29 26 u8 control[5]; ··· 47 50 mutex_lock(&st->lock); 48 51 49 52 ret = spi_sync_transfer(st->sdev, &xfer, 1); 50 - if (ret) 51 - goto error_ret; 52 - error_ret: 53 53 mutex_unlock(&st->lock); 54 54 55 55 return ret ? ret : len;