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

[Lars-Peter Clausen <lars@xxxxxxxxxx>]

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

> 
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html