Hi Joe On Tue, Dec 07, 2021 at 06:47:45AM -0800, Joe Perches wrote: > On Tue, 2021-12-07 at 14:24 +0200, Sakari Ailus wrote: > > Hi Joe (and Paul), > > > I guess this patch is fine as such but there seems to be a problem in error > > handling here: all regulators are disabled if there's a problem enabling > > one of them. > > > > Would it be possible to fix this as well? > > I've no hardware to test, so I've no idea if that's the right thing to do. I don't have the hardware either. But I can tell that you shouldn't disable a regulator you haven't enabled to begin with. Bugs (fixes of which probably should go to stable trees) need to be fixed before reworking the code. -- Regards, Sakari Ailus