Hi Wolfram, On Fri, Jun 23, 2023 at 12:10:07PM +0200, Wolfram Sang wrote: > On Fri, Jun 09, 2023 at 12:55:13AM +0200, Andi Shyti wrote: > > From: Alexey Khoroshilov <khoroshilov@xxxxxxxxx> > > > > pm_runtime_set_suspended() does not lead to call of suspend callback, > > so clk may be left undisabled in hix5hd2_i2c_remove(). > > > > By the way, the patch adds error handling for clk_prepare_enable(). > > > > Found by Linux Driver Verification project (linuxtesting.org). > > > > Signed-off-by: Alexey Khoroshilov <khoroshilov@xxxxxxxxx> > > Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxx> > > I have now applied "[PATCH 05/15] i2c: busses: hix5hd2: Use > devm_clk_get_enabled()". Can this patch then be dropped entirely? yes, this patch can be dropped. When I resent this I didn't plan that refactoring. Thanks, Andi
