Libertas todo

From OLPC
Revision as of 19:54, 6 October 2006 by MarceloTosatti (talk | contribs)
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigation Jump to search
The printable version is no longer supported and may have rendering errors. Please update your browser bookmarks and please use the default browser print function instead.

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

- foobar