On Fri, 2020-12-18 at 19:38 +0100, Guido Günther wrote: > Use the typical startup times from the data sheet so boards get a > reasonable default. Not setting any enable time can lead to board > hangs > when e.g. clocks are enabled too soon afterwards. > > This fixes gpu power domain resume on the Librem 5. > > Signed-off-by: Guido Günther <[email protected]> > > --- > v2 > - As per review comment by Matti Vaittinen > > https://lore.kernel.org/lkml/[email protected]/ > Use defines instead of plain values > - As per review comment by Mark Brown > https://lore.kernel.org/lkml/[email protected]/ > Drop cover letter > --- > drivers/regulator/bd718x7-regulator.c | 27 ++++++++++++++++++++++++ > include/linux/mfd/rohm-bd718x7.h | 30
Thanks again Guido. I might have preferred having the defines in bd718x7-regulator.c as they are not likely to be used outside this file. That would have strictly limited the change to regulator subsystem. Having them in the header is still fine with me if it works for Mark & others. (I don't think these defines need much of changes in the future). Reviewed-by: Matti Vaittinen <[email protected]> Best Regards Matti Vaittinen
