[PATCH v2] iio: adc: meson_saradc: Better handle BL30 not releaseing the hardware

Jonathan Cameron jic23 at kernel.org
Sat Feb 25 08:36:32 PST 2023


On Sun, 19 Feb 2023 22:05:16 +0100
Martin Blumenstingl <martin.blumenstingl at googlemail.com> wrote:

> Hello Uwe,
> 
> I think there's a typo in the subject line:
> s/releaseing/releasing/
> 
> On Sun, Feb 19, 2023 at 9:44 PM Uwe Kleine-König
> <u.kleine-koenig at pengutronix.de> wrote:
> >
> > meson_sar_adc_lock() might return an error if BL30 doesn't release its
> > lock on the hardware. Just returning early from .remove() is wrong
> > however as this keeps the clocks and regulators on which is never
> > cleaned up later.
> >
> > Given the BL30 not giving up its lock is a strong hint for broken
> > behaviour, and there is nothing we can do about that: Just clean up
> > ignoring the fact that we're not holding the lock.
> >
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>  
> Other than the typo (which maybe Jonathan can fix up while applying):
> Reviewed-by: Martin Blumenstingl <martin.blumenstingl at googlemail.com>
Fixed up typo and applied to the togreg branch of iio.git.

Note I'll not push that out other than as testing until I can rebase on rc1.

Thanks,

Jonathan

> 
> 
> Thank you!
> Martin




More information about the linux-amlogic mailing list