Re: [PULL] 1st set of IIO fixes for the 5.16 cycle (suggest queuing for merge window as nothing urgent)

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

On Fri, 22 Oct 2021 16:23:45 +0100
Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote:

> The following changes since commit 37f12202c5d28291ba5f83ce229771447ce9148f:
> 
>   staging: r8188eu: prevent array underflow in rtw_hal_update_ra_mask() (2021-10-05 12:35:30 +0200)
> 
> are available in the Git repository at:
> 
>   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-for-5.16a
> 
> for you to fetch changes up to 486a25084155bf633768c26f022201c051d6fd95:
> 
>   iio: buffer: Fix memory leak in iio_buffers_alloc_sysfs_and_mask() (2021-10-21 10:26:38 +0100)
> 
> ----------------------------------------------------------------

Hi Greg,

Doesn't make much sense to rush this set in, so I'd suggest just merging
it alongside the other pull request to char-misc-next.

I thought about just merging this into the iio-togreg branch and
sending you one pull request, but decided this was simpler.  Let me know
if you'd prefer it done a different way.

*fingers crossed* I haven't messed these up this time!

The last few patches hit linux-next today but most in both these pull
requests have been soaking for a while.

Thanks,

Jonathan

> First set of IIO fixes for the 5.16 cycle
> 
> As these are very late in the 5.15 cycle and non are particularly urgent,
> they can wait for the merge window.
> 
> Key element in this set is Yang Yingliang has identified a number of
> issues in error paths introduced recently when we added multiple
> buffer support.
> 
> Other fixes:
> * adi,ad5662
>   - Fix handling of i2c_master_send() return value.
> * adi,ad5766
>   - Fix a wrong dt-property name that indicated wrong units and
>     did not mach the bindings.
>   - Associated 'fix' of the bindings example to have a possible scale.
> * st,pressure-spi
>   - Add some missing entries to the spi_device_id table to ensure
>     auto-loading works.
> * ti,tsc2046
>   - Fix a backwards comparison leading to a false dev_warn
> 
> ----------------------------------------------------------------
> Mark Brown (1):
>       iio: st_pressure_spi: Add missing entries SPI to device ID table
> 
> Mihail Chindris (2):
>       drivers: iio: dac: ad5766: Fix dt property name
>       Documentation:devicetree:bindings:iio:dac: Fix val
> 
> Oleksij Rempel (1):
>       iio: adc: tsc2046: fix scan interval warning
> 
> Pekka Korpinen (1):
>       iio: dac: ad5446: Fix ad5622_write() return value
> 
> Yang Yingliang (7):
>       iio: buffer: check return value of kstrdup_const()
>       iio: buffer: Fix memory leak in __iio_buffer_alloc_sysfs_and_mask()
>       iio: buffer: Fix double-free in iio_buffers_alloc_sysfs_and_mask()
>       iio: buffer: Fix memory leak in iio_buffer_register_legacy_sysfs_groups()
>       iio: core: check return value when calling dev_set_name()
>       iio: core: fix double free in iio_device_unregister_sysfs()
>       iio: buffer: Fix memory leak in iio_buffers_alloc_sysfs_and_mask()
> 
>  .../devicetree/bindings/iio/dac/adi,ad5766.yaml    |  2 +-
>  drivers/iio/adc/ti-tsc2046.c                       |  2 +-
>  drivers/iio/dac/ad5446.c                           |  9 ++++++-
>  drivers/iio/dac/ad5766.c                           |  6 ++---
>  drivers/iio/industrialio-buffer.c                  | 28 +++++++++++++---------
>  drivers/iio/industrialio-core.c                    |  9 ++++++-
>  drivers/iio/pressure/st_pressure_spi.c             |  4 ++++
>  7 files changed, 42 insertions(+), 18 deletions(-)