staging: unisys: visornic - correctly reset flag prior to send_enbdis()
authorTim Sell <Timothy.Sell@unisys.com>
Tue, 28 Jul 2015 16:29:08 +0000 (12:29 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 31 Jul 2015 23:10:35 +0000 (16:10 -0700)
Because devdata->enab_dis_acked is the flag used to determine whether an
enable/disable operation to the IO partition has completed, it should
always be cleared prior to initiating the operation.  The call added to
visornic_enable_with_timeout() added in this patch makes the usage there
consistent with visornic_disable_with_timeout().

Signed-off-by: Tim Sell <Timothy.Sell@unisys.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visornic/visornic_main.c

index 801e66abf58e10829881620b840a70c273dc1855..02906ef6c4c68a5fe4fa85aef599d4a222b16ace 100644 (file)
@@ -680,6 +680,7 @@ visornic_enable_with_timeout(struct net_device *netdev, const int timeout)
 
        spin_lock_irqsave(&devdata->priv_lock, flags);
        devdata->enabled = 1;
+       devdata->enab_dis_acked = 0;
 
        /* now we're ready, let's send an ENB to uisnic but until we get
         * an ACK back from uisnic, we'll drop the packets