* Romain Naour <romain.naour@xxxxxxxx> [220217 09:09]: > On u-boot devicetree the uart4 node is missing, but we don't plan to use the gps > from uboot :) > Should I add the uart4 node? There should be no need for that, kernel should be able to initialize it properly no matter what the state is from the bootloader. > Since removing the uart quirk had some other side effect, I did a shameless hack > in 8250_omap.c to disable autosuspend. > > - pm_runtime_put_autosuspend(port->dev); > + pm_runtime_dont_use_autosuspend(port->dev); > > With that the uart is always up and running. Yes but note that 8250_omap autosuspend does not do anything unless the timeouts are manually enabled by the userspace. They are initialized to -1 during probe. > > The test script I posted does that for all the uarts, probably best not > > to do that until the other issues are sorted out :) If so, maybe the issue > > on close is that the uart has already autoidled. > > Indeed. But I'm not sure why the autosuspend is enabled by default? See above, it's always been initialized to -1 by default so it won't do anything. But if you ran the test script I posted, autosuspend timeout got enabled for all the uarts. But maybe the issue you posted is yet another issue that I don't quite understand yet. To me it seems we should always have runtime PM functions enable the serial port to usable state and get rid of the conditional enable for probe and dma. Regards, Tony
