Libertas todo

From OLPC
Revision as of 15:57, 6 October 2006 by DanWilliams (talk | contribs) (WE stuff)
Jump to: navigation, search

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)

  • 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

- 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