Libertas todo: Difference between revisions

From OLPC
Jump to navigation Jump to search
No edit summary
No edit summary
Line 7: Line 7:
* 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)
* collapse command queueing (remove queueing should simplify complexity a lot)
* test individual suspend/resume (CONFIG_USB_SUSPEND)


===David Woodhouse===
===David Woodhouse===

Revision as of 19:15, 10 October 2006

TODO list for Marvell 8388 wireless driver

Marcelo Tosatti

  • ioctl mess
  • 64-bit issues
  • proper locking: add a simple global mutex.
  • 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)

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

Dan Williams

  • confused at IsNetworkCompatible() in wlan_scan.c
  • Break current SIOC*IWENCODE into SIOC*IWENCODE, SIOC*IWENCODEEXT, SIOC*IWAUTH
  • Implement SIOC*IWGENIE
  • Make driver do deferred assocation rather than blocking on assoc in WE handlers like SIOCSIWENCODE