[lkml]   [2017]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [RFC PATCH v2 1/3] PCI: rockchip: Add support for pcie wake irq
Hi Tony,

On 08/23/2017 01:26 AM, Tony Lindgren wrote:
> OK, let's fix any wakeriq ordering issues to make it more
> usable. Sounds like in your case the wakeirq needs to be enabled
> late and disabled early, while in my test cases I can keep it
> enabled basically any time.

yes, in my case it's a level triggered irq, which needs to be disabled
when receive it(by irq_pm_check_wakeup(my hack) or inside of the custom
irq handler)

and for eage irq, maybe we should enable it right after(or before) the
driver activate wake function(for example activate WOWLAN or WOLAN),
otherwise would it be possible to miss some irqs(triggered before we
actually enable the wake irq)?

> If this is for suspend/resume, You could just register the
> wakeirq on suspend and then remove it on resume. We do have at
> least network drivers doing device_init_wakeup(dev, true) and
> device_init_wakeup(dev, false) as needed for WOL, see for example
> bfin_mac_ethtool_setwol().
> Regards,
> Tony

 \ /
  Last update: 2017-08-23 03:34    [W:0.157 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site