Messages in this thread | | | Date | Mon, 18 Jan 2021 16:20:14 +0000 | From | Mark Brown <> | Subject | Re: [PATCH v2 11/13] regulator: hi6421v600-regulator: move it from staging |
| |
On Mon, Jan 18, 2021 at 05:02:45PM +0100, Mauro Carvalho Chehab wrote: > Mark Brown <broonie@kernel.org> escreveu:
> > If for some reason the PMIC is sufficiently fragile to need a delay > > between enables it's not clear why the driver is doing it before > > enabling rather than after, presumably there's issues with the regulator > > ramping up and stabalising its output
> I don't have any datasheets or documentation from this device, except for > the Linaro's official driver for Kernel 4.9 (from where this driver > was originally ported), and a high-level documentation for this > hardware at 96boards site, which doesn't cover any details.
This misses the point. To repeat, if the driver is hard coding a delay why is the driver doing this *before* rather than *after* enabling? [unhandled content-type:application/pgp-signature] | |