Re: [PATCH 00/14] Staging:iio: Cleanup and refactor pre multibuffer.

[Jonathan Cameron <jic23@kernel.org>]

On 04/20/2012 06:00 PM, Lars-Peter Clausen wrote:
> On 04/18/2012 03:13 PM, Jonathan Cameron wrote:
>> Sits on top of the series
>>
>> [PATCH 00/16 V3] IIO: Make the value attribute optional for channels
>>  
>> This series consists of the first chunk of what was
>> [PATCH 1/4] staging:iio: make all buffer access pass through the buffer_list
>> As Lars-Peter rightly pointed out that patch was a whole mess of different
>> elements all bunged in together.
>>
>> I believe this lot are all fairly uncontroversial unless I have trashed
>> a driver by accident. (which is entirely plausible).
> 
> Yep, looks all pretty straight forward. With the compiler warning fixed:
> Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Fixed up and sent on to Greg.
> 
>>
>> There are a few temporary frigs in here to ensure some of the cached
>> values are correct before we put the real change in place.
>> For example the scan_timestamp gets cached both in the core and in the
>> buffers. For this change it would have made sense to drop it from the
>> buffers but then we'd just end up putting it back again a few patches
>> later.
>>
>> Anyhow, please do take a look at what we have here.  Some are
>> trivial, but there are a good few driver reworks that could do
>> with testing (as I'll almost guarantee I've messed something up
>> ripping the original large patch apart!)
>>
>> Thanks,
>>
>> Jonathan Cameron 
>>
>> Jonathan Cameron (14):
>>   staging:iio:buffer trivial use of strtobool to remove dodgy
>>     equivalent.
>>   staging:iio:buffer: pull computation of scan length into a utility
>>     function.
>>   staging:iio: scan_index_timestamp move to iio_dev from buffer
>>   staging:iio: add caching of the number of bytes in a scan.
>>   staging:iio:adc:ad7192 make use of iio_sw_buffer_preenable.
>>   staging:iio:adc:ad7298 use iio_sw_buffer_preenable to avoid code
>>     repitition.
>>   staging:iio:adc:ad7476 use iio_sw_buffer_preenable instead of local
>>     version.
>>   staging:iio:adc:ad7793 use iio_sw_buffer_preenable to avoid
>>     repitition.
>>   staging:iio:adc:ad7887 make use of iio_sw_buffer_preenable.
>>   staging:iio:adc:ad799x use iio_sw_buffer_preenable to avoid code
>>     repitiion
>>   staging:iio:meter:ad7758 use iio_sw_buffer_preenable to avoid code
>>     repition
>>   staging:iio:impedance-analyser make use of iio_sw_buffer_preenable
>>   staging:iio: Add caching of scan_timestamp to the core as well as
>>     buffers.
>>   staging:iio: pull out demux cleanup for a particular buffer.
>>
>>  drivers/staging/iio/accel/adis16201_ring.c      |    5 +-
>>  drivers/staging/iio/accel/adis16203_ring.c      |    5 +-
>>  drivers/staging/iio/accel/adis16204_ring.c      |    5 +-
>>  drivers/staging/iio/accel/adis16209_ring.c      |    6 +-
>>  drivers/staging/iio/accel/adis16240_ring.c      |    5 +-
>>  drivers/staging/iio/accel/lis3l02dq_ring.c      |    6 +-
>>  drivers/staging/iio/adc/ad7192.c                |   24 ++------
>>  drivers/staging/iio/adc/ad7298.h                |    1 -
>>  drivers/staging/iio/adc/ad7298_ring.c           |   26 +++------
>>  drivers/staging/iio/adc/ad7476.h                |    1 -
>>  drivers/staging/iio/adc/ad7476_ring.c           |   38 +-----------
>>  drivers/staging/iio/adc/ad7606_ring.c           |    8 +--
>>  drivers/staging/iio/adc/ad7793.c                |   23 ++------
>>  drivers/staging/iio/adc/ad7887.h                |    1 -
>>  drivers/staging/iio/adc/ad7887_ring.c           |   25 ++------
>>  drivers/staging/iio/adc/ad799x.h                |    1 -
>>  drivers/staging/iio/adc/ad799x_ring.c           |   24 +-------
>>  drivers/staging/iio/adc/max1363_ring.c          |    4 +-
>>  drivers/staging/iio/buffer.h                    |    2 -
>>  drivers/staging/iio/gyro/adis16260_ring.c       |    5 +-
>>  drivers/staging/iio/iio.h                       |    6 ++
>>  drivers/staging/iio/iio_simple_dummy_buffer.c   |   11 +--
>>  drivers/staging/iio/impedance-analyzer/ad5933.c |   11 +---
>>  drivers/staging/iio/imu/adis16400_ring.c        |    4 +-
>>  drivers/staging/iio/industrialio-buffer.c       |   69 +++++++++++++++--------
>>  drivers/staging/iio/meter/ade7758_ring.c        |   22 ++-----
>>  26 files changed, 121 insertions(+), 217 deletions(-)
>>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html