Messages in this thread | | | Date | Fri, 9 Sep 2011 22:22:07 -0700 | From | Dan Carpenter <> | Subject | Re: [PATCH 1/2] staging: et131x: Fix checkpatch whitespace warnings |
| |
On Sat, Sep 03, 2011 at 12:49:06PM +0100, Mark Einon wrote: > --- a/drivers/staging/et131x/et131x_netdev.c > +++ b/drivers/staging/et131x/et131x_netdev.c > @@ -408,7 +408,7 @@ void et131x_tx_timeout(struct net_device *netdev) > unsigned long flags; > > /* If the device is closed, ignore the timeout */ > - if (~(adapter->flags & fMP_ADAPTER_INTERRUPT_IN_USE)); > + if (~(adapter->flags & fMP_ADAPTER_INTERRUPT_IN_USE)) > return; >
You fixed a bug here. That's not mentioned in the commit message. Why would you want to keep that a secret? Also it should go in a separate commit.
But otherwise this patch looks good.
regards, dan carpenter
| |