mbox series

[PULL] 2nd set of IIO fixes for the 5.13 cycle.

Message ID 20210521181509.419b9db0@jic23-huawei (mailing list archive)
State Superseded
Headers show
Series [PULL] 2nd set of IIO fixes for the 5.13 cycle. | expand

Pull-request

https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-5.13b

Message

Jonathan Cameron May 21, 2021, 5:15 p.m. UTC
The following changes since commit ba9c25d94dea1a57492a606a1f5dde70d2432583:

  Merge tag 'iio-fixes-5.13a' of https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-linus (2021-05-11 15:17:55 +0200)

are available in the Git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-5.13b

for you to fetch changes up to 923b374318f8aa0b241731e83e7abfb9e5300f29:

  iio: adc: ad7793: Add missing error code in ad7793_setup() (2021-05-16 10:15:47 +0100)

----------------------------------------------------------------
Second set of IIO fixes for the 5.13 cycle

A mixed bag of fixes for various drivers.

adi,ad7124
 - Fix miss balanced regulator enable / disable in error path
 - Fix potential overflow with non sequential channel numbers from dt.
adi,ad7192
 - Avoid disabling clock that was never enabled in error path + remove
 - Avoid nasty corner case if regulator voltage is 0 that would result
   in a good return half way through probe.
adi,ad7746
 - Avoid overwriting num_channels just after setting it correctly.
adi,ad7768
 - Buffer passed to iio_push_to_buffers_with_timestamp() too small and
   not aligned appropriately.
adi,ad7793
 - Missing return code setting in an error path.
adi,ad7923
 - Buffer too small after support for more channels added.
adi,ad5770r
 - Missing fwnode_handle_put in error paths.
fsl,fxa21002c
 - Missing runtime pm put in error path.

----------------------------------------------------------------
Alexandru Ardelean (1):
      iio: adc: ad7192: handle regulator voltage error first

Andy Shevchenko (1):
      iio: dac: ad5770r: Put fwnode in error case during ->probe()

Jonathan Cameron (5):
      iio: adc: ad7124: Fix missbalanced regulator enable / disable on error.
      iio: adc: ad7124: Fix potential overflow due to non sequential channel numbers
      iio: adc: ad7192: Avoid disabling a clock that was never enabled.
      iio: adc: ad7768-1: Fix too small buffer passed to iio_push_to_buffers_with_timestamp()
      iio: adc: ad7923: Fix undersized rx buffer.

Lucas Stankus (1):
      staging: iio: cdc: ad7746: avoid overwrite of num_channels

Rui Miguel Silva (1):
      iio: gyro: fxas21002c: balance runtime power in error path

YueHaibing (1):
      iio: adc: ad7793: Add missing error code in ad7793_setup()

 drivers/iio/adc/ad7124.c           | 36 ++++++++++++++++++++----------------
 drivers/iio/adc/ad7192.c           | 19 ++++++++++---------
 drivers/iio/adc/ad7768-1.c         |  8 ++++++--
 drivers/iio/adc/ad7793.c           |  1 +
 drivers/iio/adc/ad7923.c           |  4 +++-
 drivers/iio/dac/ad5770r.c          | 16 +++++++++++-----
 drivers/iio/gyro/fxas21002c_core.c |  2 ++
 drivers/staging/iio/cdc/ad7746.c   |  1 -
 8 files changed, 53 insertions(+), 34 deletions(-)

Comments

Greg Kroah-Hartman May 21, 2021, 6:12 p.m. UTC | #1
On Fri, May 21, 2021 at 06:15:09PM +0100, Jonathan Cameron wrote:
> The following changes since commit ba9c25d94dea1a57492a606a1f5dde70d2432583:
> 
>   Merge tag 'iio-fixes-5.13a' of https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-linus (2021-05-11 15:17:55 +0200)
> 
> are available in the Git repository at:
> 
>   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-5.13b

Pulled and pushed out, thanks.

greg k-h
Greg Kroah-Hartman May 21, 2021, 6:17 p.m. UTC | #2
On Fri, May 21, 2021 at 08:12:48PM +0200, Greg KH wrote:
> On Fri, May 21, 2021 at 06:15:09PM +0100, Jonathan Cameron wrote:
> > The following changes since commit ba9c25d94dea1a57492a606a1f5dde70d2432583:
> > 
> >   Merge tag 'iio-fixes-5.13a' of https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-linus (2021-05-11 15:17:55 +0200)
> > 
> > are available in the Git repository at:
> > 
> >   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-5.13b
> 
> Pulled and pushed out, thanks.

Oops, nope, didn't work:

 Commit 488abb0f5c83 ("iio: gyro: fxas21002c: balance runtime power in error path")
	committer Signed-off-by missing
	author email:    rui.silva@linaro.org
	committer email: Jonathan.Cameron@huawei.com
	Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>


I've rolled it back in my local tree and never pushed it out to the
public.

Can you fix this up?  linux-next didn't catch this already?

thanks,

greg k-h
Jonathan Cameron May 22, 2021, 7:35 a.m. UTC | #3
On Fri, 21 May 2021 20:17:47 +0200
Greg KH <gregkh@linuxfoundation.org> wrote:

> On Fri, May 21, 2021 at 08:12:48PM +0200, Greg KH wrote:
> > On Fri, May 21, 2021 at 06:15:09PM +0100, Jonathan Cameron wrote:  
> > > The following changes since commit ba9c25d94dea1a57492a606a1f5dde70d2432583:
> > > 
> > >   Merge tag 'iio-fixes-5.13a' of https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-linus (2021-05-11 15:17:55 +0200)
> > > 
> > > are available in the Git repository at:
> > > 
> > >   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-5.13b  
> > 
> > Pulled and pushed out, thanks.  
> 
> Oops, nope, didn't work:
> 
>  Commit 488abb0f5c83 ("iio: gyro: fxas21002c: balance runtime power in error path")
> 	committer Signed-off-by missing
> 	author email:    rui.silva@linaro.org
> 	committer email: Jonathan.Cameron@huawei.com
> 	Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
> 
> 
> I've rolled it back in my local tree and never pushed it out to the
> public.
> 
> Can you fix this up?  linux-next didn't catch this already?
> 

I'll check, but didn't see anything.  Perhaps an email filtering issue my end.

Anyhow, thanks for catching and will send fixed up version in a few mins.

Jonathan

> thanks,
> 
> greg k-h