]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
net: phy: icplus: implement .did_interrupt for IP101A/G
authorMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Sun, 18 Nov 2018 21:23:58 +0000 (22:23 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 19 Nov 2018 00:16:20 +0000 (16:16 -0800)
commitf7e290fbeb336421ba6237548b693c9afb4d75fa
tree18ad7ffc5439aaaa0cc9e92f8ec633c3716ca81d
parenta872c388f9d9e515d9c4102110d0bf0fd7a9778a
net: phy: icplus: implement .did_interrupt for IP101A/G

The IP101A_G_IRQ_CONF_STATUS register has bits to detect which
interrupts have fired. Implement the .did_interrupt callback to let the
PHY core know whether the interrupt was for this specific PHY.

This is useful for debugging interrupt problems with 32-pin IP101GR PHYs
where the interrupt line is shared with the RX_ERR (receive error
status) signal. The default values are:
- RX_ERR is enabled by default (LOW means that there is no receive
  error)
- the PHY's interrupt line is configured "active low" by default

Without any additional changes there is a flood of interrupts if the
RX_ERR/INTR32 signal is configured in RX_ERR mode (which is the
default). Having a did_interrupt ensures that the PHY core returns
IRQ_NONE instead of endlessly triggering the PHY state machine.
Additionally the kernel will report this after a while:
  irq 28: nobody cared (try booting with the "irqpoll" option)

Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/icplus.c