Messages in this thread | | | Date | Thu, 23 Apr 2020 15:35:14 +0100 | From | Mark Brown <> | Subject | Re: [PATCH v5 0/6] driver core: Improve and cleanup driver_deferred_probe_check_state() |
| |
On Wed, Apr 22, 2020 at 01:45:49PM -0700, John Stultz wrote: > On Wed, Apr 22, 2020 at 12:54 AM Mark Brown <broonie@kernel.org> wrote:
> > 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.
Given that it's basically entirely about glitching displays rather than an unrecoverable break I suspect that anyone using NFS root is some combination of unaffected or doesn't care if they see the timeout kick in. [unhandled content-type:application/pgp-signature] | |