Re: [PATCH 1/1] New driver: rtl8xxxu (mac80211)

[Jes Sorensen <Jes.Sorensen@redhat.com>]

Johannes Berg <johannes@sipsolutions.net> writes:
> On Sat, 2015-09-05 at 09:32 +0530, Sujith Manoharan wrote:
>> Jes Sorensen wrote:
>> > > Ah, this makes sense. There's an rc_update call or something like 
>> > > that,
>> > > that might have the requisite information. Not really sure off 
>> > > the top
>> > > of my head.
>> 
>> ieee80211_sta doesn't contain the preamble information, though ?
>> 
>> The bss information holds the short preamble information, so that can
>> probably be used - using BSS_CHANGED_ERP_PREAMBLE.
>
> Yes, this is a good point, and the preamble can't really be different
> for different stations since all the stations in the BSS (and even
> outside it on the same channel) must be able to decode it.
>
> So yeah - Jes, you can remove that and use the
> BSS_CHANGED_ERP_PREAMBLE.

Finally went back to look at this - I believe this is exactly what I am
doing in the driver right now. Pulling the short_preamble info from
BSS_CHANGED_ERP_PREAMBLE and storing it sta->drv_priv for use in the
tx function.

Cheers,
Jes