Re: [PATCH RFC 3/6] net: phy: Threaded interrupts allow some simplification

On 09/28/2016 03:28 PM, Andrew Lunn wrote:

quoted

quoted

quoted

The PHY interrupts are now handled in a threaded interrupt handler,
which can sleep. The work queue is no longer needed, phy_change() can
be called directly. Additionally, none of the callers of
phy_mac_interrupt() did so in interrupt context, so fully remove the

  I did intend to call it from interrupt context (from the ravb driver).

quoted

work queue, and document that phy_mac_interrupt() should not be called
in interrupt context.

  It was intentionally made callable from the interrupt context, I'd prefer
if you wouldn't change that.

   OTOH, it's still not very handy to call because of the 'new_link'
parameter which I'm not sure I can provide...

Hi Sergei

If there is a need for it, i will leave the work queue and keep this
code unchanged.

   Let's hear what Florian says...

   Andrew

MBR, Sergei