Re: Runtime PM support in serial core?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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����?���&��




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux