diff options
| author | Heiner Kallweit <hkallweit1@gmail.com> | 2017-12-01 01:55:15 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-12-04 18:23:41 +0300 |
| commit | a6d1642dab0c49829cda61508cbdc97172815ff7 (patch) | |
| tree | 842ff147579f10acc49df85afa5a40da3c52084f /include/linux/phy.h | |
| parent | c6d3c96f8d52b8f16de73e778717641e659c4ccc (diff) | |
| download | linux-a6d1642dab0c49829cda61508cbdc97172815ff7.tar.xz | |
net: phy: core: remove now uneeded disabling of interrupts
After commits c974bdbc3e "net: phy: Use threaded IRQ, to allow IRQ from
sleeping devices" and 664fcf123a30 "net: phy: Threaded interrupts allow
some simplification" all relevant code pieces run in process context
anyway and I don't think we need the disabling of interrupts any longer.
Interestingly enough, latter commit already removed the comment
explaining why interrupts need to be temporarily disabled.
On my system phy interrupt mode works fine with this patch.
However I may miss something, especially in the context of shared phy
interrupts, therefore I'd appreciate if more people could test this.
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/phy.h')
| -rw-r--r-- | include/linux/phy.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/phy.h b/include/linux/phy.h index 4962af37722a..50030da01664 100644 --- a/include/linux/phy.h +++ b/include/linux/phy.h @@ -468,7 +468,6 @@ struct phy_device { /* Interrupt and Polling infrastructure */ struct work_struct phy_queue; struct delayed_work state_queue; - atomic_t irq_disable; struct mutex lock; |