diff options
| author | Niklas Cassel | 2018-02-09 17:22:47 +0100 |
|---|---|---|
| committer | David S. Miller | 2018-02-09 20:23:04 +0100 |
| commit | 1029117127540fef4edcf4f0887dc3e1f7d5adb2 (patch) | |
| tree | ebec54574b541a88c0465c9a2b59ce45ace5faa1 /drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c | |
| parent | net: stmmac: rename GMAC_INT_DEFAULT_MASK for dwmac4 (diff) | |
| download | kernel-qcow2-linux-1029117127540fef4edcf4f0887dc3e1f7d5adb2.tar.gz kernel-qcow2-linux-1029117127540fef4edcf4f0887dc3e1f7d5adb2.tar.xz kernel-qcow2-linux-1029117127540fef4edcf4f0887dc3e1f7d5adb2.zip | |
net: stmmac: remove redundant enable of PMT irq
For dwmac4, GMAC_INT_DEFAULT_ENABLE already includes
GMAC_INT_PMT_EN, so it is redundant to check if hw->pmt
is set, and if so, setting the bit again.
For dwmac1000, GMAC_INT_DEFAULT_MASK does not include
GMAC_INT_DISABLE_PMT, so it is redundant to check if
hw->pmt is set, and if so, clearing an already cleared bit.
Improve code readability by removing this redundant code.
Signed-off-by: Niklas Cassel <niklas.cassel@axis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c')
| -rw-r--r-- | drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c b/drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c index 6badc63d8e6d..63795ecafc8d 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac4_core.c @@ -63,8 +63,7 @@ static void dwmac4_core_init(struct mac_device_info *hw, /* Enable GMAC interrupts */ value = GMAC_INT_DEFAULT_ENABLE; - if (hw->pmt) - value |= GMAC_INT_PMT_EN; + if (hw->pcs) value |= GMAC_PCS_IRQ_DEFAULT; |