On Thu, Feb 21, 2008 at 10:29:17AM +0100, Johannes Berg wrote:
>
> > > --- everything.orig/drivers/net/wireless/rndis_wlan.c 2008-02-20 11:43:22.000000000 +0100
> > > +++ everything/drivers/net/wireless/rndis_wlan.c 2008-02-20 11:45:35.000000000 +0100
> > > @@ -228,9 +228,9 @@ struct NDIS_WLAN_BSSID_EX {
> > > struct NDIS_802_11_SSID Ssid;
> > > __le32 Privacy;
> > > __le32 Rssi;
> > > - enum NDIS_802_11_NETWORK_TYPE NetworkTypeInUse;
> > > + __le32 NetworkTypeInUse;
> > > struct NDIS_802_11_CONFIGURATION Configuration;
> > > - enum NDIS_802_11_NETWORK_INFRASTRUCTURE InfrastructureMode;
> > > + __le32 InfrastructureMode;
> > > u8 SupportedRates[NDIS_802_11_LENGTH_RATES_EX];
> > > __le32 IELength;
> > > u8 IEs[0];
> >
> > This suggests that the values in the enum need to be explicitly
> > defined, does it not?
>
> Yeah, well, it is probably understood that those values are from
> hardware. We don't always explicitly define them even when they are
> defined elsewhere since afaik C guarantees starting at 0 and increasing
> order.
Ah, nevermind...I see that the enum values are being converted to/from
cpu-endian before use.
Thanks,
John
--
John W. Linville
linville@xxxxxxxxxxxxx
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
