On Wed, 2016-08-24 at 17:03 +0200, bigeasy@xxxxxxxxxxxxx wrote: > On 2016-08-24 14:56:12 [+0000], Shevchenko, Andriy wrote: > > Today I discovered that runtime PM rather broken in 8250 _by > > design_. > > It > > definitely requires irq_safe flag to be set in runtime PM. But that > > flag > > demolishes an effort since it keeps parent (and thus all other > > devices > > which parent needs to be on) always on. ^^^^^ (1) > > > > The question is is somebody is going to fix this and when? > > Why is it broken? I meant that without irq_safe flag set there are too many places where "sleep while in atomic" happens. On the other hand the irq_safe flag prevents parent to go to suspend. > As far as I remember the UART (on TI's beagle) can be > shutdown. The result is that the first RX byte will wake up the board > and loosing the first few bytes (depending the baud rate) until the > clock is stable again. This was tested while I moved the PM bits from > omap-serial to the 8250 version. > > Sebastian -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxx> Intel Finland Oy --------------------------------------------------------------------- Intel Finland Oy Registered Address: PL 281, 00181 Helsinki Business Identity Code: 0357606 - 4 Domiciled in Helsinki This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. ��.n��������+%������w��{.n�����{��ǫ����{ay�ʇڙ���f���h������_�(�階�ݢj"��������G����?���&��