Hi John, On Wed, Apr 22, 2020 at 10:46 PM John Stultz <john.stultz@xxxxxxxxxx> wrote: > 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. IOMMU support is optional. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds