linux.git/drivers/iio, branch v6.0-rc2

iio: light: isl29028: Fix the warning in isl29028_remove()

2022-07-19T08:42:11+00:00

The driver use the non-managed form of the register function in
isl29028_remove(). To keep the release order as mirroring the ordering
in probe, the driver should use non-managed form in probe, too.

The following log reveals it:

[   32.374955] isl29028 0-0010: remove
[   32.376861] general protection fault, probably for non-canonical address 0xdffffc0000000006: 0000 [#1] PREEMPT SMP KASAN PTI
[   32.377676] KASAN: null-ptr-deref in range [0x0000000000000030-0x0000000000000037]
[   32.379432] RIP: 0010:kernfs_find_and_get_ns+0x28/0xe0
[   32.385461] Call Trace:
[   32.385807]  sysfs_unmerge_group+0x59/0x110
[   32.386110]  dpm_sysfs_remove+0x58/0xc0
[   32.386391]  device_del+0x296/0xe50
[   32.386959]  cdev_device_del+0x1d/0xd0
[   32.387231]  devm_iio_device_unreg+0x27/0xb0
[   32.387542]  devres_release_group+0x319/0x3d0
[   32.388162]  i2c_device_remove+0x93/0x1f0

Fixes: 2db5054ac28d ("staging: iio: isl29028: add runtime power management support")
Signed-off-by: Zheyu Ma 
Link: https://lore.kernel.org/r/20220717004241.2281028-1-zheyuma97@gmail.com
Cc: 
Signed-off-by: Jonathan Cameron

iio: accel: sca3300: Extend the trigger buffer from 16 to 32 bytes

2022-07-19T08:38:53+00:00

After added inclination angle channels, the trigger buffer size is
insufficient. Extend the buffer size from 16 to 32 bytes, and change
the trigger buffer from the struct to a u8 array to adapt the sensor
with/without inclination angles output.
New trigger buffer data:
  - SCA3300: 3 accel channels, temp, and timestamp.
  - SCL3300: 3 accel channels, temp, 3 incli channels, and timestamp.
Readjustment the scan index to make it consistent with the buffer data.

Signed-off-by: LI Qingwu 
Link: https://lore.kernel.org/r/20220701023030.2527019-2-Qing-wu.Li@leica-geosystems.com.cn
Signed-off-by: Jonathan Cameron

iio: fix iio_format_avail_range() printing for none IIO_VAL_INT

2022-07-19T08:31:27+00:00

iio_format_avail_range() should print range as follow [min, step, max], so
the function was previously calling iio_format_list() with length = 3,
length variable refers to the array size of values not the number of
elements. In case of non IIO_VAL_INT values each element has integer part
and decimal part. With length = 3 this would cause premature end of loop
and result in printing only one element.

Signed-off-by: Fawzi Khaber 
Signed-off-by: Jean-Baptiste Maneyrol 
Fixes: eda20ba1e25e ("iio: core: Consolidate iio_format_avail_{list,range}()")
Link: https://lore.kernel.org/r/20220718130706.32571-1-jmaneyrol@invensense.com
Cc: 
Signed-off-by: Jonathan Cameron

iio: adc: max1027: unlock on error path in max1027_read_single_value()

2022-07-19T08:30:05+00:00

If max1027_wait_eoc() fails then call iio_device_release_direct_mode()
before returning.

Fixes: a0e831653ef9 ("iio: adc: max1027: Introduce an end of conversion helper")
Signed-off-by: Dan Carpenter 
Link: https://lore.kernel.org/r/YsbztVuAXnau2cIZ@kili
Signed-off-by: Jonathan Cameron

iio: magnetometer: hmc5843: Remove duplicate 'the'

2022-07-18T17:53:54+00:00

Fix an obvious typing error, found by spellcheck(1).

Signed-off-by: Jiang Jian 
Link: https://lore.kernel.org/r/20220622035925.5008-1-jiangjian@cdjrlc.com
Signed-off-by: Jonathan Cameron

iio: magn: yas530: Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr() macros

2022-07-18T17:48:19+00:00

Using these new macros allows the compiler to remove the unused dev_pm_ops
structure and related functions if !CONFIG_PM without the need to mark
the functions __maybe_unused.

Signed-off-by: Jonathan Cameron 
Cc: Linus Walleij 
Reviewed-by: Paul Cercueil 
Link: https://lore.kernel.org/r/20220621202719.13644-37-jic23@kernel.org

iio: magnetometer: ak8974: Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr() macros

2022-07-18T17:48:19+00:00

Using these new macros allows the compiler to remove the unused dev_pm_ops
structure and related functions if !CONFIG_PM without the need to mark
the functions __maybe_unused.

Signed-off-by: Jonathan Cameron 
Cc: Linus Walleij 
Reviewed-by: Paul Cercueil 
Link: https://lore.kernel.org/r/20220621202719.13644-36-jic23@kernel.org

iio: light: veml6030: Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr() macros

2022-07-18T17:48:19+00:00

Using these new macros allows the compiler to remove the unused dev_pm_ops
structure and related functions if !CONFIG_PM without the need to mark
the functions __maybe_unused.

Signed-off-by: Jonathan Cameron 
Cc: Rishi Gupta 
Reviewed-by: Rishi Gupta 
Reviewed-by: Paul Cercueil 
Link: https://lore.kernel.org/r/20220621202719.13644-35-jic23@kernel.org

iio: light: vcnl4035: Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr() macros

2022-07-18T17:48:19+00:00

Using these new macros allows the compiler to remove the unused dev_pm_ops
structure and related functions if !CONFIG_PM without the need to mark
the functions __maybe_unused.

Signed-off-by: Jonathan Cameron 
Cc: Parthiban Nallathambi 
Reviewed-by: Paul Cercueil 
Link: https://lore.kernel.org/r/20220621202719.13644-34-jic23@kernel.org

iio: light: vcnl4000: Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr() macros

2022-07-18T17:48:19+00:00

Using these new macros allows the compiler to remove the unused dev_pm_ops
structure and related functions if !CONFIG_PM without the need to mark
the functions __maybe_unused.

Signed-off-by: Jonathan Cameron 
Cc: Mathieu Othacehe 
Reviewed-by: Paul Cercueil 
Link: https://lore.kernel.org/r/20220621202719.13644-33-jic23@kernel.org