staging: slicoss: Add MAINTAINERS entry, break README into TODO & README

Adding a MAINTAINERS entry with content from the README.
Move the TODO items from the README to a separate TODO file.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/MAINTAINERS b/MAINTAINERS
index b2cf5cfb4d29de3dc27e351ba090134bfd624de6..2af6445fd9e7031003f85019cfe28b646b2e85ee 100644 (file)
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8311,6 +8311,12 @@ M:       Teddy Wang <teddy.wang@siliconmotion.com.cn>
 S:     Odd Fixes
 F:     drivers/staging/sm7xxfb/
 
+STAGING - SLICOSS
+M:     Lior Dotan <liodot@gmail.com>
+M:     Christopher Harrer <charrer@alacritech.com>
+S:     Odd Fixes
+F:     drivers/staging/slicoss/
+
 STAGING - SOFTLOGIC 6x10 MPEG CODEC
 M:     Ismael Luceno <ismael.luceno@corp.bluecherry.net>
 S:     Supported

diff --git a/drivers/staging/slicoss/README b/drivers/staging/slicoss/README
index 53052c4e78ae71ffe4f6fa37c4b73ecda91e30b2..4fa50e73ce86a2a42884ca95d9105ff4ffed694d 100644 (file)
--- a/drivers/staging/slicoss/README
+++ b/drivers/staging/slicoss/README
@@ -5,43 +5,3 @@ This driver is supposed to support:
        Kalahari cards (dual and quad port PCI-e Gigabit) copper and fiber
 
 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
-       - GET RID OF MACROS
-       - work on all architectures
-          - without CONFIG_X86_64 confusion
-          - do 64 bit correctly
-          - don't depend on order of union
-       - get rid of ASSERT(), use BUG() instead but only where necessary
-         looks like most aren't really useful
-       - 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
-       - state variables for things that are
-         easily available and shouldn't be kept in card structure, cardnum, ...
-         slotnumber, events, ...
-       - get rid of slic_spinlock wrapper
-       - volatile == bad design => bad code
-       - locking too fine grained, not designed just throw more locks
-         at problem
-
-
-Please send patches to:
-        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-and Cc: Lior Dotan <liodot@gmail.com> and Christopher Harrer
-<charrer@alacritech.com> as well as they are also able to test out any
-changes.

diff --git a/drivers/staging/slicoss/TODO b/drivers/staging/slicoss/TODO
new file mode 100644 (file)
index 0000000..62ff100
--- /dev/null
+++ b/drivers/staging/slicoss/TODO
@@ -0,0 +1,38 @@
+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
+       - GET RID OF MACROS
+       - work on all architectures
+          - without CONFIG_X86_64 confusion
+          - do 64 bit correctly
+          - don't depend on order of union
+       - get rid of ASSERT(), use BUG() instead but only where necessary
+         looks like most aren't really useful
+       - 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
+       - state variables for things that are
+         easily available and shouldn't be kept in card structure, cardnum, ...
+         slotnumber, events, ...
+       - get rid of slic_spinlock wrapper
+       - volatile == bad design => bad code
+       - locking too fine grained, not designed just throw more locks
+         at problem
+
+Please send patches to:
+        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+and Cc: Lior Dotan <liodot@gmail.com> and Christopher Harrer
+<charrer@alacritech.com> as well as they are also able to test out any
+changes.