net: sunhme: Unify IRQ requesting
authorSean Anderson <seanga2@gmail.com>
Fri, 24 Mar 2023 17:51:30 +0000 (13:51 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 27 Mar 2023 07:41:56 +0000 (08:41 +0100)
commit27b9ea8f37a6f5d6bd4c6867fd1910a978371572
treee6473723bffe655118c903ebb34f4c73984485b3
parent3427372d0bd8c75036768d017074c29c99ce6342
net: sunhme: Unify IRQ requesting

Instead of registering one interrupt handler for all four SBUS Quattro
HMEs, let each HME register its own handler. To make this work, we don't
handle the IRQ if none of the status bits are set. This reduces the
complexity of the driver, and makes it easier to ensure things happen
before/after enabling IRQs.

I'm not really sure why we request IRQs in two different places (and leave
them running after removing the driver!). A lot of things in this driver
seem to just be crusty, and not necessarily intentional. I'm assuming
that's the case here as well.

This really needs to be tested by someone with an SBUS Quattro card.

Signed-off-by: Sean Anderson <seanga2@gmail.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sun/sunhme.c