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

From: Greg KH
Date: Sat Jun 28 2014 - 12:03:34 EST


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>
--
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/