Libertas todo: Difference between revisions

From OLPC
Jump to navigation Jump to search
 
(8 intermediate revisions by 7 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)
* collapse command queueing (remove queueing should simplify complexity a lot)
* 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===
* confused at IsNetworkCompatible() in wlan_scan.c
* Clean up IsNetworkCompatible() in wlan_scan.c
* Break current SIOC*IWENCODE into SIOC*IWENCODE, SIOC*IWENCODEEXT, SIOC*IWAUTH
* Break current SIOC*IWENCODE into SIOC*IWENCODE, SIOC*IWENCODEEXT, SIOC*IWAUTH
* Implement SIOC*IWGENIE
* Implement SIOC*IWGENIE
* Make driver do deferred assocation rather than blocking on assoc in WE handlers like SIOCSIWENCODE
* 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