Libertas todo: Difference between revisions

From OLPC
Jump to navigation Jump to search
No edit summary
 
 
(14 intermediate revisions by 10 users not shown)
Line 1: Line 1:
==TODO list for Marvell 8388 wireless driver==
==TODO list for Marvell 8388 wireless driver==


* Marcelo Tosatti
===Marcelo Tosatti===


* 64-bit issues
- ioctl mess
* 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===
- 64-bit issues
* __attribute__ ((packed)) wlan_ioctl_user_scan_cfg why packet?


===Dan Williams===
- proper locking: add a simple global mutex.
* 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


[[Category:Network]]
- kernel style documentation (Documentation/kernel-doc-nano-HOWTO.txt)
[[Category:Developers]]

- collapse command queueing (remove queueing should simplify complexity a lot)

* 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?

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

- confused at IsNetworkCompatible() in wlan_scan.c

- isn't that something that the higher 802.11 layers should do?

* Dan Williams

- foobar

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