Re: [PATCH 0/8 V2] staging:iio:mixed bag of fixes and cleanups.

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

 



Gah, sorry about lack of title on the cover letter.

Doh.

There you go Jon, I also mess up patch sets in the silliest ways.
> Hi Greg,
> 
> A poorly ordered (sorry about that) set of fixes combined
> with cleanups suggested by Arnd.  This is a precursor
> to the more intersting (and intrusive) changes to follow.
> 
> As per earlier email this is based on, staging-next
> with:
> 
> [PATCH V4]TAOS 258x: Device Driver
> [PATCH 1/1] IIO: DAC: AD5446 add support for AD5541A devices
> [PATCH 1/1] IIO: DAC: New driver for AD5791/AD5781 High Resolution Voltage Output DACs
> [PATCH 1/3] IIO: GYRO: Add MACRO for gyro quadrature correction
> (and rest of series)
> 
> on top.  You might not have the AD5791 one yet, but Michael
> should send it to you shortly.
> 
> Original message was:
> 
> I should probably have broken these up into more coherent subsets
> and made at least the allocate priv_data one an RFC.
> May do so before sending on to Greg.
> 
> Patches 3, 4 and 6 are direct results of suggestions from Arnd Bergmann.
> Basically it's the easy bits of the discussion before we get onto
> the more 'interesting' ones.
> 
> 1, 2 are things I spotted whilst working on the irq_chip stuff
> Thomas Gleixner suggested.  They are just cleanups of some bizare
> corners. That rewrite of triggers currently requires some
> exports to be added to allow a modular build and I need to chase
> down whether we are simply the first people to have a reasonable
> case for it, or whether there is something more fundamental
> going on.
> 
> The others are bugs / cleanups I came across whilst hammering
> max1363 with the new example code. Some of the timestamp bit
> was pointed out by Michael Hennerich a while back, but turns
> out the driver was broken as well as not obeying the ABI.
> 
> Thanks,
> 
> 
> Jonathan Cameron (8):
>   staging:iio:trigger remove export of iio_trigger_find_by_name, use
>     sysfs_streq for matching
>   staging:iio: iio_trigger_unregister - remove rather strange search
>     for what we already have.
>   staging:iio: replace rip_lots naming with read_first_n
>   staging:iio: Add ability to allocate private data space to
>     iio_allocate_device
>   staging:iio:adc:max1363 fix timestamp handling.
>   staging:iio:adc: max1363 dev_info -> indio_dev in naming for
>     consistency
>   staging:iio:max1363 take advantage of new iio_device_allocate private
>     data.
>   staging:iio:documentation make read_size signed to allow for errors.
> 
>  drivers/staging/iio/Documentation/device.txt       |    2 +-
>  drivers/staging/iio/Documentation/generic_buffer.c |    2 +-
>  drivers/staging/iio/accel/adis16201_core.c         |    2 +-
>  drivers/staging/iio/accel/adis16203_core.c         |    2 +-
>  drivers/staging/iio/accel/adis16204_core.c         |    2 +-
>  drivers/staging/iio/accel/adis16209_core.c         |    2 +-
>  drivers/staging/iio/accel/adis16220_core.c         |    2 +-
>  drivers/staging/iio/accel/adis16240_core.c         |    2 +-
>  drivers/staging/iio/accel/kxsd9.c                  |    2 +-
>  drivers/staging/iio/accel/lis3l02dq_core.c         |    2 +-
>  drivers/staging/iio/accel/sca3000_core.c           |    2 +-
>  drivers/staging/iio/accel/sca3000_ring.c           |    8 +-
>  drivers/staging/iio/adc/ad7150.c                   |    2 +-
>  drivers/staging/iio/adc/ad7152.c                   |    2 +-
>  drivers/staging/iio/adc/ad7291.c                   |    2 +-
>  drivers/staging/iio/adc/ad7298_core.c              |    2 +-
>  drivers/staging/iio/adc/ad7314.c                   |    2 +-
>  drivers/staging/iio/adc/ad7476_core.c              |    2 +-
>  drivers/staging/iio/adc/ad7606_core.c              |    2 +-
>  drivers/staging/iio/adc/ad7745.c                   |    2 +-
>  drivers/staging/iio/adc/ad7780.c                   |    2 +-
>  drivers/staging/iio/adc/ad7816.c                   |    2 +-
>  drivers/staging/iio/adc/ad7887_core.c              |    2 +-
>  drivers/staging/iio/adc/ad799x_core.c              |    2 +-
>  drivers/staging/iio/adc/adt7310.c                  |    2 +-
>  drivers/staging/iio/adc/adt7410.c                  |    2 +-
>  drivers/staging/iio/adc/adt75.c                    |    2 +-
>  drivers/staging/iio/adc/max1363.h                  |    2 -
>  drivers/staging/iio/adc/max1363_core.c             |  189 ++++++++++----------
>  drivers/staging/iio/adc/max1363_ring.c             |   21 ++-
>  drivers/staging/iio/addac/adt7316.c                |    2 +-
>  drivers/staging/iio/dac/ad5446.c                   |    2 +-
>  drivers/staging/iio/dac/ad5504.c                   |    2 +-
>  drivers/staging/iio/dac/ad5624r_spi.c              |    2 +-
>  drivers/staging/iio/dac/ad5791.c                   |    2 +-
>  drivers/staging/iio/dac/max517.c                   |    2 +-
>  drivers/staging/iio/dds/ad5930.c                   |    2 +-
>  drivers/staging/iio/dds/ad9832.c                   |    2 +-
>  drivers/staging/iio/dds/ad9834.c                   |    2 +-
>  drivers/staging/iio/dds/ad9850.c                   |    2 +-
>  drivers/staging/iio/dds/ad9852.c                   |    2 +-
>  drivers/staging/iio/dds/ad9910.c                   |    2 +-
>  drivers/staging/iio/dds/ad9951.c                   |    2 +-
>  drivers/staging/iio/gyro/adis16060_core.c          |    2 +-
>  drivers/staging/iio/gyro/adis16080_core.c          |    2 +-
>  drivers/staging/iio/gyro/adis16130_core.c          |    2 +-
>  drivers/staging/iio/gyro/adis16260_core.c          |    2 +-
>  drivers/staging/iio/gyro/adxrs450_core.c           |    2 +-
>  drivers/staging/iio/iio.h                          |   17 ++-
>  drivers/staging/iio/imu/adis16300_core.c           |    2 +-
>  drivers/staging/iio/imu/adis16350_core.c           |    2 +-
>  drivers/staging/iio/imu/adis16400_core.c           |    2 +-
>  drivers/staging/iio/industrialio-core.c            |   15 ++-
>  drivers/staging/iio/industrialio-ring.c            |   12 +-
>  drivers/staging/iio/industrialio-trigger.c         |   27 +--
>  drivers/staging/iio/kfifo_buf.c                    |    8 +-
>  drivers/staging/iio/kfifo_buf.h                    |   10 +-
>  drivers/staging/iio/light/isl29018.c               |    2 +-
>  drivers/staging/iio/light/tsl2563.c                |    2 +-
>  drivers/staging/iio/light/tsl2583.c                |    2 +-
>  drivers/staging/iio/magnetometer/ak8975.c          |    2 +-
>  drivers/staging/iio/magnetometer/hmc5843.c         |    2 +-
>  drivers/staging/iio/meter/ade7753.c                |    2 +-
>  drivers/staging/iio/meter/ade7754.c                |    2 +-
>  drivers/staging/iio/meter/ade7758_core.c           |    2 +-
>  drivers/staging/iio/meter/ade7759.c                |    2 +-
>  drivers/staging/iio/meter/ade7854.c                |    2 +-
>  drivers/staging/iio/resolver/ad2s120x.c            |    2 +-
>  drivers/staging/iio/resolver/ad2s1210.c            |    2 +-
>  drivers/staging/iio/resolver/ad2s90.c              |    2 +-
>  drivers/staging/iio/ring_generic.h                 |   10 +-
>  drivers/staging/iio/ring_sw.c                      |   13 +-
>  drivers/staging/iio/ring_sw.h                      |   14 +-
>  drivers/staging/iio/trigger.h                      |    7 -
>  74 files changed, 242 insertions(+), 231 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


[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