Re: [PATCH 4.19 09/78] iio: adc: mcp3422: fix locking on error path

[Sasha Levin]

On Wed, Sep 16, 2020 at 09:31:00AM +0200, Pavel Machek wrote:
> Hi!
>
> > [ Upstream commit a139ffa40f0c24b753838b8ef3dcf6ad10eb7854 ]
> >
> > Reading from the chip should be unlocked on error path else the lock
> > could never being released.
> >
> > Fixes: 07914c84ba30 ("iio: adc: Add driver for Microchip MCP3422/3/4 high resolution ADC")
> > Fixes: 3f1093d83d71 ("iio: adc: mcp3422: fix locking scope")
>
> Well, 3f1093d83d71 is only applied later in the stable series, so this
> introduces spurious unlock.
>
> Ideally this should go just after 3f1093d83d71 ("iio: adc: mcp3422:
> fix locking scope").

Patches rearranged, thanks!

--
Thanks,
Sasha