Re: [PATCH 2/8] mac80211: A-MPDU Rx add MLME structures

Johannes Berg
2007-12-19T15:45:51Z

[-- Attachment #1: Type: text/plain, Size: 727 bytes --]

On Tue, 2007-12-18 at 15:41 +0200, Ron Rindjunsky wrote:
> >
> > > + struct sta_ampdu_mlme ampdu_mlme;
> > > + int timer_to_tid[STA_TID_NUM]; /* convert timer id to tid */
> >
> > Since you only use that as an identity mapping it can well be a u8
> > array. Not that I like the trick used, but I don't see any good way to
> > avoid it either.
> >
>
> yes... that was quite an irritating issue, had to use this kind of
> mechanism here.
> i will check changing type to u8 (need to exmine if it will cause any
> addresses problems)

I just checked, it's an unsigned long and not an address so you can
really pass anything to it. Therefore, I think using a u8 array would be
appropriate.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]