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

On 9/28/2016 2:46 PM, Sergei Shtylyov wrote:

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...

quoted

Signed-off-by: Andrew Lunn <andrew@lunn.ch>

[...]

MBR, Sergei