On Wed, Sep 28, 2016 at 03:13:46PM +0300, Sergei Shtylyov wrote: > On 9/28/2016 2:46 PM, Sergei Shtylyov wrote: > > >>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). > > > >>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. Andrew
