* [PULL] First set of IIO new device support etc for the 5.5 cycle.
@ 2019-10-12 9:24 Jonathan Cameron
2019-10-12 10:51 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2019-10-12 9:24 UTC (permalink / raw)
To: gregkh, linux-iio
The following changes since commit b73b93a2af3392b9b7b8ba7e818ee767499f9655:
iio: adc: ad7192: Add sysfs ABI documentation (2019-09-08 10:34:49 +0100)
are available in the Git repository at:
https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-for-5.5a
for you to fetch changes up to 181bea8c04479f8132010146af525957648c70a1:
iio: pressure: bmp280: remove stray newline (2019-10-07 22:13:51 +0100)
----------------------------------------------------------------
First set of IIO new device support, cleanups and features for the 5.5 cycle
The usual mixed backs of new device support being added to drivers,
long term reworks continuing and little per driver cleanups and
features.
Also a few trivial counter subsystem tidy ups on behalf of William.
Core new feature
* Device label support. A long requested feature no one got around to
implementing before. Allows DT based provision of a 'label' that
identifies a device uniquely within a system. This differs from existing
'name' which is meant to be the part number.
New device support
* ingenic-adc
- Support for the JZ4770 SoC ADC including bindings.
* inv_mpu6050
- Add support for magnetometer in MPU925x parts.
Fiddly to do as this is actually a separate device sitting inside the
package, but with the master device being able to schedule reads etc.
Will only run if the auxiliary bus is not in use for any other devices.
Features
* ad7192
- Userspace calibration controls to do zero and full scale.
* st_lsm6dsx
- Enable latched interrupts by default for sensors events with related clear.
- Motion events and related wakeup source. This needed quite a bit of
refactoring as well.
Cleanups and minor features
* ad7192
- sysfs ABI docs
* ad7949
- Remove code to readback configuration word as driver never actually enabled
it.
- Fix incorrect xfer length. Not actually known to cause problems other
than wasted bus usage.
* adis library and drivers
- Locking rework to simplify locking in general and avoid using the
core mlock except for it's intending use to protect IIO state changes.
* adis16080
- Replace core mlock usage with local lock with more appropriate scope.
* adis16130
- Remove pointless mlock usage.
* adis16240
- Remove include of gpio.h as no gpio usage.
* atlas-ph-sensor
- Improve logical ordering of buffer predisable / postenable functions.
This is part of a longer term rework Alexandru is driving towards.
* bh1750
- Fix up a static compiler warning and make the code more readable.
- yaml conversion of binding + MAINTAINERS entry.
* bmp280
- Drop a stray newline.
* cm36651
- Drop a redundant assignment
* itg3200
- Alignment cleanup.
* max31856
- Add missing of_node and parent references, useful to identify the device.
* sc27xx_adc
- Use devm_hwspin_lock_request_specific rather than local rolled version.
* stm32-lptimer counter
- kernel-doc warning.
* stm32-timer counter
- kernel-doc warning.
- Alignment cleanup.
* sx9500
- Improve logical ordering of buffer predisable / postenable functions.
This is part of a longer term rework Alexandru is driving towards.
* tcs3414
- Improve logical ordering of buffer predisable / postenable functions.
This is part of a longer term rework Alexandru is driving towards.
----------------------------------------------------------------
Alexandru Ardelean (15):
iio: tcs3414: fix iio_triggered_buffer_{pre,post}enable positions
iio: gyro: adis16130: remove mlock usage
iio: gyro: adis16080: replace mlock with own lock
iio: proximity: sx9500: fix iio_triggered_buffer_{predisable,postenable} positions
iio: imu: adis: rename txrx_lock -> state_lock
iio: imu: adis: add unlocked read/write function versions
iio: imu: adis[16480]: group RW into a single lock in adis_enable_irq()
iio: imu: adis: create an unlocked version of adis_check_status()
iio: imu: adis: create an unlocked version of adis_reset()
iio: imu: adis: protect initial startup routine with state lock
iio: imu: adis: group single conversion under a single state lock
iio: imu: adis16400: rework locks using ADIS library's state lock
iio: gyro: adis16136: rework locks using ADIS library's state lock
iio: imu: adis16480: use state lock for filter freq set
iio: chemical: atlas-ph-sensor: fix iio_triggered_buffer_predisable() position
Andrea Merello (3):
iio: ad7949: kill pointless "readback"-handling code
iio: max31856: add missing of_node and parent references to iio_dev
iio: ad7949: fix incorrect SPI xfer len
Artur Rojek (3):
dt-bindings: iio/adc: Add a compatible string for JZ4770 SoC ADC
dt-bindings: iio/adc: Add AUX2 channel idx for JZ4770 SoC ADC
IIO: Ingenic JZ47xx: Add support for JZ4770 SoC ADC.
Baolin Wang (1):
iio: adc: sc27xx: Use devm_hwspin_lock_request_specific() to simplify code
Bartosz Golaszewski (1):
iio: pressure: bmp280: remove stray newline
Colin Ian King (3):
iio: light: cm36651: redundant assignment to variable ret
counter: stm32: clean up indentation issue
iio: gyro: clean up indentation issue
Fabrice Gasnier (2):
counter: stm32-timer-cnt: fix a kernel-doc warning
counter: stm32-lptimer-cnt: fix a kernel-doc warning
Jean-Baptiste Maneyrol (7):
iio: imu: inv_mpu6050: disable i2c mux for MPU925x
iio: imu: inv_mpu6050: add header include protection macro
iio: imu: inv_mpu6050: add defines for supporting 9-axis chips
iio: imu: inv_mpu6050: fix objects syntax in Makefile
iio: imu: inv_mpu6050: helpers for using i2c master on auxiliary bus
iio: imu: inv_mpu6050: add MPU925x magnetometer support
iio: imu: inv_mpu6050: add fifo support for magnetometer data
Krzysztof Wilczynski (1):
iio: light: bh1750: Resolve compiler warning and make code more readable
Lorenzo Bianconi (2):
iio: imu: st_lsm6dsx: enable LIR for sensor events
iio: imu: st_lsm6dsx: enable clear on read for latched interrupts
Mircea Caprioru (2):
iio: adc: ad_sigma_delta: Export ad_sd_calibrate
staging: iio: adc: ad7192: Add system calibration support
Phil Reid (2):
dt-binding: iio: Add optional label property
iio: core: Add optional symbolic label to device attributes
Rohit Sarkar (1):
staging: iio: ADIS16240: Remove unused include
Sean Nyekjaer (5):
iio: imu: st_lsm6dsx: move interrupt thread to core
iio: imu: st_lsm6dsx: add motion events
iio: imu: st_lsm6dsx: add wakeup-source option
iio: imu: st_lsm6dsx: always enter interrupt thread
iio: imu: st_lsm6dsx: add motion report function and call from interrupt
Tomasz Duszynski (2):
dt-bindings: iio: light: bh1750: convert bindings to yaml
MAINTAINERS: add entry for ROHM BH1750 driver
Documentation/ABI/testing/sysfs-bus-iio-adc-ad7192 | 24 ++
.../devicetree/bindings/iio/adc/ingenic,adc.txt | 1 +
.../devicetree/bindings/iio/iio-bindings.txt | 5 +
.../devicetree/bindings/iio/light/bh1750.txt | 18 -
.../devicetree/bindings/iio/light/bh1750.yaml | 43 ++
MAINTAINERS | 6 +
drivers/counter/stm32-lptimer-cnt.c | 2 +-
drivers/counter/stm32-timer-cnt.c | 6 +-
drivers/iio/adc/ad7949.c | 33 +-
drivers/iio/adc/ad_sigma_delta.c | 3 +-
drivers/iio/adc/ingenic-adc.c | 149 +++++--
drivers/iio/adc/sc27xx_adc.c | 16 +-
drivers/iio/chemical/atlas-ph-sensor.c | 8 +-
drivers/iio/gyro/adis16080.c | 8 +-
drivers/iio/gyro/adis16130.c | 2 -
drivers/iio/gyro/adis16136.c | 31 +-
drivers/iio/gyro/itg3200_core.c | 2 +-
drivers/iio/imu/adis.c | 95 ++---
drivers/iio/imu/adis16400.c | 51 ++-
drivers/iio/imu/adis16480.c | 17 +-
drivers/iio/imu/adis_buffer.c | 4 +-
drivers/iio/imu/inv_mpu6050/Makefile | 7 +-
drivers/iio/imu/inv_mpu6050/inv_mpu_aux.c | 204 ++++++++++
drivers/iio/imu/inv_mpu6050/inv_mpu_aux.h | 19 +
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 152 +++++++-
drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 60 ++-
drivers/iio/imu/inv_mpu6050/inv_mpu_iio.h | 70 +++-
drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c | 356 +++++++++++++++++
drivers/iio/imu/inv_mpu6050/inv_mpu_magn.h | 36 ++
drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 11 +-
drivers/iio/imu/inv_mpu6050/inv_mpu_trigger.c | 86 +++-
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h | 50 +++
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 78 +---
drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 434 ++++++++++++++++++++-
drivers/iio/industrialio-core.c | 17 +
drivers/iio/light/bh1750.c | 4 +-
drivers/iio/light/cm36651.c | 2 +-
drivers/iio/light/tcs3414.c | 30 +-
drivers/iio/pressure/bmp280-core.c | 1 -
drivers/iio/proximity/sx9500.c | 16 +-
drivers/iio/temperature/max31856.c | 2 +
drivers/staging/iio/accel/adis16240.c | 1 -
drivers/staging/iio/adc/ad7192.c | 79 +++-
include/dt-bindings/iio/adc/ingenic,adc.h | 1 +
include/linux/iio/adc/ad_sigma_delta.h | 2 +
include/linux/iio/iio.h | 2 +
include/linux/iio/imu/adis.h | 154 +++++++-
47 files changed, 2077 insertions(+), 321 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/iio/light/bh1750.txt
create mode 100644 Documentation/devicetree/bindings/iio/light/bh1750.yaml
create mode 100644 drivers/iio/imu/inv_mpu6050/inv_mpu_aux.c
create mode 100644 drivers/iio/imu/inv_mpu6050/inv_mpu_aux.h
create mode 100644 drivers/iio/imu/inv_mpu6050/inv_mpu_magn.c
create mode 100644 drivers/iio/imu/inv_mpu6050/inv_mpu_magn.h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PULL] First set of IIO new device support etc for the 5.5 cycle.
2019-10-12 9:24 [PULL] First set of IIO new device support etc for the 5.5 cycle Jonathan Cameron
@ 2019-10-12 10:51 ` Greg KH
2019-10-12 11:16 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2019-10-12 10:51 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: linux-iio
On Sat, Oct 12, 2019 at 10:24:36AM +0100, Jonathan Cameron wrote:
> The following changes since commit b73b93a2af3392b9b7b8ba7e818ee767499f9655:
>
> iio: adc: ad7192: Add sysfs ABI documentation (2019-09-08 10:34:49 +0100)
>
> are available in the Git repository at:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-for-5.5a
I'm getting build warnings in drivers/iio/imu/adis16400.c, is that to be
expected?
drivers/iio/imu/adis16400.c: In function ‘adis16334_get_freq’:
drivers/iio/imu/adis16400.c:334:4: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
334 | t >>= ADIS16334_RATE_DIV_SHIFT;
| ^
drivers/iio/imu/adis16400.c: In function ‘adis16400_get_freq’:
drivers/iio/imu/adis16400.c:360:11: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
360 | uint16_t t;
| ^
CC [M] drivers/iio/light/bh1750.o
drivers/iio/imu/adis16400.c: In function ‘adis16400_read_raw’:
drivers/iio/imu/adis16400.c:557:10: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
557 | int16_t val16;
| ^~~~~
drivers/iio/imu/adis16400.c: In function ‘adis16400_write_raw’:
drivers/iio/imu/adis16400.c:424:14: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
424 | (val16 & ~0x07) | i);
| ~~~~~~~^~~~~~~~
drivers/iio/imu/adis16400.c:411:11: note: ‘val16’ was declared here
411 | uint16_t val16;
| ^~~~~
Can you fix that up first?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PULL] First set of IIO new device support etc for the 5.5 cycle.
2019-10-12 10:51 ` Greg KH
@ 2019-10-12 11:16 ` Jonathan Cameron
2019-10-12 15:24 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2019-10-12 11:16 UTC (permalink / raw)
To: Greg KH; +Cc: linux-iio
On Sat, 12 Oct 2019 12:51:19 +0200
Greg KH <gregkh@linuxfoundation.org> wrote:
> On Sat, Oct 12, 2019 at 10:24:36AM +0100, Jonathan Cameron wrote:
> > The following changes since commit b73b93a2af3392b9b7b8ba7e818ee767499f9655:
> >
> > iio: adc: ad7192: Add sysfs ABI documentation (2019-09-08 10:34:49 +0100)
> >
> > are available in the Git repository at:
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-for-5.5a
>
> I'm getting build warnings in drivers/iio/imu/adis16400.c, is that to be
> expected?
>
> drivers/iio/imu/adis16400.c: In function ‘adis16334_get_freq’:
> drivers/iio/imu/adis16400.c:334:4: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> 334 | t >>= ADIS16334_RATE_DIV_SHIFT;
> | ^
> drivers/iio/imu/adis16400.c: In function ‘adis16400_get_freq’:
> drivers/iio/imu/adis16400.c:360:11: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> 360 | uint16_t t;
> | ^
> CC [M] drivers/iio/light/bh1750.o
> drivers/iio/imu/adis16400.c: In function ‘adis16400_read_raw’:
> drivers/iio/imu/adis16400.c:557:10: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> 557 | int16_t val16;
> | ^~~~~
> drivers/iio/imu/adis16400.c: In function ‘adis16400_write_raw’:
> drivers/iio/imu/adis16400.c:424:14: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> 424 | (val16 & ~0x07) | i);
> | ~~~~~~~^~~~~~~~
> drivers/iio/imu/adis16400.c:411:11: note: ‘val16’ was declared here
> 411 | uint16_t val16;
> | ^~~~~
>
>
> Can you fix that up first?
>
> thanks,
>
> greg k-h
Hi Greg,
I got these as well, but as they weren't the result of anything done
in this series (seems to be GCC 8 getting fussier) I pointed them out
to Alex and he sent a fix. Was about to queue it up for next series.
I'll tack it on here and send you a new pull in a few mins.
Thanks,
Jonathan
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PULL] First set of IIO new device support etc for the 5.5 cycle.
2019-10-12 11:16 ` Jonathan Cameron
@ 2019-10-12 15:24 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2019-10-12 15:24 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: linux-iio
On Sat, Oct 12, 2019 at 12:16:19PM +0100, Jonathan Cameron wrote:
> On Sat, 12 Oct 2019 12:51:19 +0200
> Greg KH <gregkh@linuxfoundation.org> wrote:
>
> > On Sat, Oct 12, 2019 at 10:24:36AM +0100, Jonathan Cameron wrote:
> > > The following changes since commit b73b93a2af3392b9b7b8ba7e818ee767499f9655:
> > >
> > > iio: adc: ad7192: Add sysfs ABI documentation (2019-09-08 10:34:49 +0100)
> > >
> > > are available in the Git repository at:
> > >
> > > https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-for-5.5a
> >
> > I'm getting build warnings in drivers/iio/imu/adis16400.c, is that to be
> > expected?
> >
> > drivers/iio/imu/adis16400.c: In function ‘adis16334_get_freq’:
> > drivers/iio/imu/adis16400.c:334:4: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> > 334 | t >>= ADIS16334_RATE_DIV_SHIFT;
> > | ^
> > drivers/iio/imu/adis16400.c: In function ‘adis16400_get_freq’:
> > drivers/iio/imu/adis16400.c:360:11: warning: ‘t’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> > 360 | uint16_t t;
> > | ^
> > CC [M] drivers/iio/light/bh1750.o
> > drivers/iio/imu/adis16400.c: In function ‘adis16400_read_raw’:
> > drivers/iio/imu/adis16400.c:557:10: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> > 557 | int16_t val16;
> > | ^~~~~
> > drivers/iio/imu/adis16400.c: In function ‘adis16400_write_raw’:
> > drivers/iio/imu/adis16400.c:424:14: warning: ‘val16’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> > 424 | (val16 & ~0x07) | i);
> > | ~~~~~~~^~~~~~~~
> > drivers/iio/imu/adis16400.c:411:11: note: ‘val16’ was declared here
> > 411 | uint16_t val16;
> > | ^~~~~
> >
> >
> > Can you fix that up first?
> >
> > thanks,
> >
> > greg k-h
> Hi Greg,
>
> I got these as well, but as they weren't the result of anything done
> in this series (seems to be GCC 8 getting fussier) I pointed them out
> to Alex and he sent a fix. Was about to queue it up for next series.
> I'll tack it on here and send you a new pull in a few mins.
The warning goes away when I drop this pull request, so something in
here triggered it :(
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-12 15:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-12 9:24 [PULL] First set of IIO new device support etc for the 5.5 cycle Jonathan Cameron
2019-10-12 10:51 ` Greg KH
2019-10-12 11:16 ` Jonathan Cameron
2019-10-12 15:24 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).