Re: [PATCH 2/3] mac80211: IEEE802.11e/WMM TS management and DLS support

[Zhu Yi <yi.zhu@intel.com>]

On Wed, 2007-05-23 at 23:42 -0700, Michael Wu wrote:
> The dls_info code looks like it was copied from the sta_info code and modified 
> to store what you need. No good.
> 
> Instead, you should modify struct sta_info to hold the information necessary. 

OK.

> > + if (dls_link_status(&sdata->u.sta, hdr.addr1) == DLS_STATUS_OK){
> How is this line suppose to work? I don't see hdr.addr1 initialized at any 
> point before this.

DLS must be setup before it can be used. The addr1 is inserted into the
dls hash table when the QSTA received a successful DLS response or
accepted a DLS request.

Thanks,
-yi