On Wed, Apr 22, 2020 at 12:54 AM Mark Brown <broonie@xxxxxxxxxx> wrote: > On Tue, Apr 21, 2020 at 06:16:31PM -0700, John Stultz wrote: > > > The second reverts the default timeout back to 0: > > https://lore.kernel.org/lkml/20200413204253.84991-1-john.stultz@xxxxxxxxxx/ > > If you're reverting the timeout we should revert the regulator change > too I think. Maybe? The main issue for me was my change was clearly breaking users with dts with missing dependencies where their setup was working before. I sort of feel like having a dtb with missing dependencies is less valid than wanting to load module dependencies from userland, but they were working first, so we have to keep them happy :) And at least now the latter can add the timeout boot argument to make it work. For your case, I'm not sure if the timeout would run afoul on the nfs root mounting case this one tripped over. thanks -john