Re: [PATCH v2] regulators: bd718x7: Add enable times

[Vaittinen, Matti]

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 <agx@xxxxxxxxxxx>
> 
> ---
> v2
> - As per review comment by Matti Vaittinen
>   
> https://lore.kernel.org/lkml/beba25e85db20649aa040fc0ae549895c9265f6f.camel@xxxxxxxxxxxxxxxxx/
>   Use defines instead of plain values
> - As per review comment by Mark Brown
>   https://lore.kernel.org/lkml/20201216130637.GC4861@xxxxxxxxxxxxx/
>   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 <matti.vaittinen@xxxxxxxxxxxxxxxxx>


Best Regards
 Matti Vaittinen