Re: [PATCH v4 1/1] staging: iio: ad9850.c: code cleanup

From: Jonathan Cameron
Date: Sun Jun 29 2014 - 06:51:21 EST


On 28/06/14 17:03, Greg KH wrote:
On Sat, Jun 28, 2014 at 05:54:39PM +0200, Guillaume Morin wrote:
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@xxxxxxxxxxx>
Reported-by: Michael Welling <mwelling@xxxxxxxx>

---
Changes since v3:
- Added Reported-by: Michael Welling <mwelling@xxxxxxxx>

drivers/staging/iio/frequency/ad9850.c | 6 ------
1 file changed, 6 deletions(-)

Very good, thanks for pushing through and doing this.

Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

Applied to the togreg branch of iio.git

Thanks,
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/