Re: [PATCH V2 0/6] iio: mxs-lradc: Clean up and add datasheet names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 18/07/15 16:38, Marek Vasut wrote:
> On Saturday, July 18, 2015 at 02:30:40 PM, Stefan Wahren wrote:
>> This patch series does some clean up in mxs-lradc and finally add the
>> datasheet names for every usable channel.
>>
>> Changes in V2:
>>   - remove confusing i.MX28 specific comments
>>   - fix some spelling errors
>>   - fix up includes
>>   - separate channel spec for i.MX23 and i.MX28
>>
>> Stefan Wahren (6):
>>   iio: mxs-lradc: clarify supported devices
>>   iio: mxs-lradc: fix some spelling errors
>>   iio: mxs-lradc: add missing include
>>   iio: mxs-lradc: remove unnecessary header includes
>>   iio: mxs-lradc: reorder header includes
>>   iio: mxs-lradc: add datasheet name for every usable channel
>>
>>  .../bindings/staging/iio/adc/mxs-lradc.txt         |    2 +-
>>  drivers/staging/iio/adc/mxs-lradc.c                |  122
>> +++++++++++++------- 2 files changed, 82 insertions(+), 42 deletions(-)
> 
> Entire series
> 
> Reviewed-by: Marek Vasut <marex@xxxxxxx>
Thanks for looking through these. 

Just to let you know, there is currently a rather extended discussion going
on (as part of a suggested topic for the kernel summit) about when reviewed-by
tags are appropriate.

The general feeling is they must in someway indicate some substantial work.  Here
as a result I'd say it was appropriate for patch 6 whereas an Acked-by 
would be more appropriate for the cleanups (where there is nothing 'interesting').
I've left these as reviewed by for this set though!

Note to all reviewers I may also start added reviewed by tags that you haven't sent if
I happen to feel you made a big contribution to review of a patch, but perhaps haven't
revisited later versions.  I'll probably send out an email about this outside
of this thread at some point to get possible wider circulation.

Jonathan
> 
> Best regards,
> Marek Vasut
> 

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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux