diff options
author | Dan Williams <dcbw@redhat.com> | 2007-08-03 09:40:55 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:50:17 -0400 |
commit | 8362cd413e8116306fafbaf414f0419db0595142 (patch) | |
tree | dad2dc7fe6988d6733eaf2f1f454bf6068133448 /drivers/net/wireless/libertas/wext.c | |
parent | c7fdf26995d5d6ebf1c3314ad001b9a4983c3f04 (diff) |
[PATCH] libertas: fix sparse-reported problems
A few fields being converted to the wrong sized type, and a few missed
endian conversions.
Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/libertas/wext.c')
-rw-r--r-- | drivers/net/wireless/libertas/wext.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/wireless/libertas/wext.c b/drivers/net/wireless/libertas/wext.c index 15395bf0a2ac..3f628223bc6c 100644 --- a/drivers/net/wireless/libertas/wext.c +++ b/drivers/net/wireless/libertas/wext.c | |||
@@ -872,7 +872,7 @@ static struct iw_statistics *wlan_get_wireless_stats(struct net_device *dev) | |||
872 | /* Quality by TX errors */ | 872 | /* Quality by TX errors */ |
873 | priv->wstats.discard.retries = priv->stats.tx_errors; | 873 | priv->wstats.discard.retries = priv->stats.tx_errors; |
874 | 874 | ||
875 | tx_retries = le16_to_cpu(adapter->logmsg.retry); | 875 | tx_retries = le32_to_cpu(adapter->logmsg.retry); |
876 | 876 | ||
877 | if (tx_retries > 75) | 877 | if (tx_retries > 75) |
878 | tx_qual = (90 - tx_retries) * POOR / 15; | 878 | tx_qual = (90 - tx_retries) * POOR / 15; |
@@ -888,10 +888,10 @@ static struct iw_statistics *wlan_get_wireless_stats(struct net_device *dev) | |||
888 | (PERFECT - VERY_GOOD) / 50 + VERY_GOOD; | 888 | (PERFECT - VERY_GOOD) / 50 + VERY_GOOD; |
889 | quality = min(quality, tx_qual); | 889 | quality = min(quality, tx_qual); |
890 | 890 | ||
891 | priv->wstats.discard.code = le16_to_cpu(adapter->logmsg.wepundecryptable); | 891 | priv->wstats.discard.code = le32_to_cpu(adapter->logmsg.wepundecryptable); |
892 | priv->wstats.discard.fragment = le16_to_cpu(adapter->logmsg.rxfrag); | 892 | priv->wstats.discard.fragment = le32_to_cpu(adapter->logmsg.rxfrag); |
893 | priv->wstats.discard.retries = tx_retries; | 893 | priv->wstats.discard.retries = tx_retries; |
894 | priv->wstats.discard.misc = le16_to_cpu(adapter->logmsg.ackfailure); | 894 | priv->wstats.discard.misc = le32_to_cpu(adapter->logmsg.ackfailure); |
895 | 895 | ||
896 | /* Calculate quality */ | 896 | /* Calculate quality */ |
897 | priv->wstats.qual.qual = min_t(u8, quality, 100); | 897 | priv->wstats.qual.qual = min_t(u8, quality, 100); |