Libertas todo: Difference between revisions
Jump to navigation
Jump to search
No edit summary |
|||
(5 intermediate revisions by 4 users not shown) | |||
Line 2: | Line 2: | ||
===Marcelo Tosatti=== |
===Marcelo Tosatti=== |
||
* ioctl mess |
|||
* 64-bit issues |
* 64-bit issues |
||
* proper locking: add a simple global mutex. |
* proper locking: add a simple global mutex. |
||
* kernel style documentation (Documentation/kernel-doc-nano-HOWTO.txt) |
* kernel style documentation (Documentation/kernel-doc-nano-HOWTO.txt) |
||
* test individual suspend/resume (CONFIG_USB_SUSPEND) |
* test individual suspend/resume (CONFIG_USB_SUSPEND) |
||
* firmware must report correct association error codes, must send that |
|||
* multiple rdmac commands from ioctl (wlanconfig) causes driver to detect device as "removed" |
|||
down to userspace |
|||
* reloading the firmware can crash the machine after power up (reloading code) |
|||
* wlan_do_getcis_ioctl copies Adapter->CisInfoBuf to userspace, which is never initialized. |
|||
===David Woodhouse=== |
===David Woodhouse=== |
||
* #ifdef __KERNEL__ in wlan_scan.h? !! |
|||
* lots of typedef struct { ... } foo |
|||
** most of them removed, a few remaining (marcelo) |
|||
* __attribute__ ((packed)) wlan_ioctl_user_scan_cfg why packet? |
* __attribute__ ((packed)) wlan_ioctl_user_scan_cfg why packet? |
||
** some packed attributes are useless, some are not. We removed some already, need to finish (marcelo) |
|||
* and if it's for compatibility with hardware, the u64 should be either __le64 or __be64 |
|||
===Dan Williams=== |
===Dan Williams=== |
||
Line 24: | Line 18: | ||
* Implement SIOC*IWGENIE |
* Implement SIOC*IWGENIE |
||
* Allocate the WPA IE on the fly rather than using a constant sized buffer |
* Allocate the WPA IE on the fly rather than using a constant sized buffer |
||
[[Category:Network]] |
|||
[[Category:Developers]] |
Latest revision as of 23:59, 1 February 2007
TODO list for Marvell 8388 wireless driver
Marcelo Tosatti
- 64-bit issues
- proper locking: add a simple global mutex.
- kernel style documentation (Documentation/kernel-doc-nano-HOWTO.txt)
- test individual suspend/resume (CONFIG_USB_SUSPEND)
- firmware must report correct association error codes, must send that
down to userspace
David Woodhouse
- __attribute__ ((packed)) wlan_ioctl_user_scan_cfg why packet?
Dan Williams
- Clean up IsNetworkCompatible() in wlan_scan.c
- Break current SIOC*IWENCODE into SIOC*IWENCODE, SIOC*IWENCODEEXT, SIOC*IWAUTH
- Implement SIOC*IWGENIE
- Allocate the WPA IE on the fly rather than using a constant sized buffer