nfp: flower: don't try to nack device unregister events
authorJakub Kicinski <jakub.kicinski@netronome.com>
Wed, 7 Nov 2018 01:07:31 +0000 (17:07 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 7 Nov 2018 19:45:22 +0000 (11:45 -0800)
commita558c982a8ab43ec4262e7fd5e8d462e5fdabe45
tree35cec230da591f135cebedd75bf1f75eaa3d7f1b
parente50bfdf74d5f81c81f7a00498aa7fd1cba53b65b
nfp: flower: don't try to nack device unregister events

Returning an error from a notifier means we want to veto the change.
We shouldn't veto NETDEV_UNREGISTER just because we couldn't find
the tracking info for given master.

I can't seem to find a way to trigger this unless we have some
other bug, so it's probably not fix-worthy.

While at it move the checking if the netdev really is of interest
into the handling functions, like we do for other events.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: John Hurley <john.hurley@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/flower/lag_conf.c