On 08/24/2016 04:25 PM, gwenhael wrote:
> Hi,
>
> On 24/08/2016 16:09, Lars-Peter Clausen wrote:
>> Hi,
>>
>> Thanks for the patch, very good work. Looks mostly good, a few comments
>> inline.
>>
>> Also please run the patch through scripts/checkpatch.pl there are a couple
>> of whitespace issues here and there (extra space at the end of the line,
>> etc.).
> Thanks for your review. It's strange, on my computer no warnings about
> whitespace
> are displayed by checkpatch.pl. I need to check by hand.
Strange, I get the following:
ERROR: trailing whitespace
#94: FILE: drivers/iio/dac/ad8801.c:39:
+^I $
ERROR: trailing whitespace
#131: FILE: drivers/iio/dac/ad8801.c:76:
+^I$
ERROR: trailing whitespace
#155: FILE: drivers/iio/dac/ad8801.c:100:
+^I^I$
ERROR: trailing whitespace
#167: FILE: drivers/iio/dac/ad8801.c:112:
+const struct iio_chan_spec ad8801_channels[] = { $
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
