2010-07-10 20:55:41

by Denis Kirjanov

[permalink] [raw]
Subject: [PATCH -staging] slicoss: README update

Update README to remove completed tasks

Signed-off-by: Denis Kirjanov <[email protected]>
---
drivers/staging/slicoss/README | 7 -------
1 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/slicoss/README b/drivers/staging/slicoss/README
index 70f4909..862a87d 100644
--- a/drivers/staging/slicoss/README
+++ b/drivers/staging/slicoss/README
@@ -7,14 +7,9 @@ This driver is supposed to support:
The driver was actually tested on Oasis and Kalahari cards.

TODO:
- - move firmware loading to request_firmware()
- remove direct memory access of structures
- any remaining sparse and checkpatch.pl warnings

- - use net_device_ops
- - use dev->stats rather than adapter->stats
- - don't cast netdev_priv it is already void
- - use compare_ether_addr
- GET RID OF MACROS
- work on all architectures
- without CONFIG_X86_64 confusion
@@ -25,10 +20,8 @@ TODO:
- no new SIOCDEVPRIVATE ioctl allowed
- don't use module_param for configuring interrupt mitigation
use ethtool instead
- - reorder code to elminate use of forward declarations
- don't keep private linked list of drivers.
- remove all the gratiutous debug infrastructure
- - use PCI_DEVICE()
- do ethtool correctly using ethtool_ops
- NAPI?
- wasted overhead of extra stats


2010-07-22 18:13:49

by Greg KH

[permalink] [raw]
Subject: Re: Staging: slicoss: README update

On Sat, Jul 10, 2010 at 08:55:28PM +0000, Denis Kirjanov wrote:
> Update README to remove completed tasks

I didn't apply this as I couldn't take the last one.

Oh, also, please change your "From:" to not have "hera" in the address,
I doubt that is correct...

thanks,

greg k-h