]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
lan78xx: Avoid spurious kevent 4 "error"
authorPhil Elwell <phil@raspberrypi.org>
Wed, 11 Apr 2018 11:02:47 +0000 (12:02 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Apr 2018 14:32:43 +0000 (10:32 -0400)
lan78xx_defer_event generates an error message whenever the work item
is already scheduled. lan78xx_open defers three events -
EVENT_STAT_UPDATE, EVENT_DEV_OPEN and EVENT_LINK_RESET. Being aware
of the likelihood (or certainty) of an error message, the DEV_OPEN
event is added to the set of pending events directly, relying on
the subsequent deferral of the EVENT_LINK_RESET call to schedule the
work.  Take the same precaution with EVENT_STAT_UPDATE to avoid a
totally unnecessary error message.

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/lan78xx.c

index 145bb7cbf5b2bec5a6d5e0a81ce8079a81cbb23a..bdb696612e11f066d37b506bbc164e155d80d469 100644 (file)
@@ -2503,7 +2503,7 @@ static void lan78xx_init_stats(struct lan78xx_net *dev)
        dev->stats.rollover_max.eee_tx_lpi_transitions = 0xFFFFFFFF;
        dev->stats.rollover_max.eee_tx_lpi_time = 0xFFFFFFFF;
 
-       lan78xx_defer_kevent(dev, EVENT_STAT_UPDATE);
+       set_bit(EVENT_STAT_UPDATE, &dev->flags);
 }
 
 static int lan78xx_open(struct net_device *net)