Search Linux Wireless

re: rndis_wlan: split getting current channel to separate function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Jussi Kivilinna <jussi.kivilinna@xxxxxxxx>:

Quoting Dan Carpenter <dan.carpenter@xxxxxxxxxx>:

Hello Jussi,

The patch b5257c952: "rndis_wlan: workaround device not returning bss
for currently connected AP" from Nov 9, 2011, leads to the following
warning:
drivers/net/wireless/rndis_wlan.c:1374:36: warning: cast to restricted __le16 drivers/net/wireless/rndis_wlan.c:1374:36: warning: cast from restricted __le32

 1372
 1373          if (beacon_interval)
 1374                  *beacon_interval = le16_to_cpu(config.beacon_period);
 1375          return channel;

The problem is that config.beacon_period is declared as le32 but we're
using it as le16.

Ah yes, I'll post patch soon. Thanks.

Well.. I ran into interesting problem where driver stops working if I correct le16_to_cpu to le32_to_cpu. But if I instead pass beacon_interval=0 to cfg80211, driver start working again. So this is bug fixing another problem, I'll check it more throughtly when I have more time.

-Jussi


-Jussi


regards,
dan carpenter






--
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





--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux