diff options
author | Mark Einon <mark.einon@gmail.com> | 2013-09-11 14:14:46 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-09-17 07:47:43 -0700 |
commit | 966630bc79fd16df43e4dc172771371f59d94849 (patch) | |
tree | 8773128543a9df7583ddeae2e17ba7ec95c9e1db /drivers/staging/et131x | |
parent | 0cdc6ee80d4f0a56fffd259d69966c3b76a09c7e (diff) | |
download | blackbird-op-linux-966630bc79fd16df43e4dc172771371f59d94849.tar.gz blackbird-op-linux-966630bc79fd16df43e4dc172771371f59d94849.zip |
staging: et131x: Add some items to the TODO list
Adding some trivial formatting suggestions made by Dan Carpenter to
the TODO.
Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/et131x')
-rw-r--r-- | drivers/staging/et131x/README | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/staging/et131x/README b/drivers/staging/et131x/README index 9272a24ae61c..8da96a6d2c92 100644 --- a/drivers/staging/et131x/README +++ b/drivers/staging/et131x/README @@ -11,7 +11,12 @@ TODO: - Look at reducing the number of spinlocks - Simplify code in nic_rx_pkts(), when determining multicast_pkts_rcvd - Implement NAPI support - - in et131x_tx(), don't return NETDEV_TX_BUSY, just drop the packet with kfree_skb(). + - In et131x_tx(), don't return NETDEV_TX_BUSY, just drop the packet with kfree_skb(). + - Reduce the number of split lines by careful consideration of variable names etc. + - Do this in et131x.c: + struct fbr_lookup *fbr; + fbr = rx_local->fbr[id]; + Then replace all the instances of "rx_local->fbr[id]" with fbr. Please send patches to: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |