diff options
author | Larry Finger <Larry.Finger@lwfinger.net> | 2008-10-09 20:38:52 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-10-31 19:00:11 -0400 |
commit | 2e20cc3986cbee410fbe8e3e116bdcb12d70fcce (patch) | |
tree | f72330c504b68dae64ea3d62d2f6a124d9ec2fe3 | |
parent | 78236571a538860dc2f0842ff6c7789522eb1e63 (diff) |
p54: Move rx_mtu to struct bootrec_desc
The patch entitled "[PATCH] p54: Fix sparse warnings" added the __le16
variable rx_mtu to struct bootrec, but it could equally well be placed
in the struct bootrec_desc, which overlays the 'data' section of bootrec.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/p54/p54common.c | 2 | ||||
-rw-r--r-- | drivers/net/wireless/p54/p54common.h | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/wireless/p54/p54common.c b/drivers/net/wireless/p54/p54common.c index b3c297ed00cd..f8cdd88e010e 100644 --- a/drivers/net/wireless/p54/p54common.c +++ b/drivers/net/wireless/p54/p54common.c | |||
@@ -183,7 +183,7 @@ int p54_parse_firmware(struct ieee80211_hw *dev, const struct firmware *fw) | |||
183 | priv->headroom = desc->headroom; | 183 | priv->headroom = desc->headroom; |
184 | priv->tailroom = desc->tailroom; | 184 | priv->tailroom = desc->tailroom; |
185 | if (le32_to_cpu(bootrec->len) == 11) | 185 | if (le32_to_cpu(bootrec->len) == 11) |
186 | priv->rx_mtu = le16_to_cpu(bootrec->rx_mtu); | 186 | priv->rx_mtu = le16_to_cpu(desc->rx_mtu); |
187 | else | 187 | else |
188 | priv->rx_mtu = (size_t) | 188 | priv->rx_mtu = (size_t) |
189 | 0x620 - priv->tx_hdr_len; | 189 | 0x620 - priv->tx_hdr_len; |
diff --git a/drivers/net/wireless/p54/p54common.h b/drivers/net/wireless/p54/p54common.h index 2fa994cfcfed..22af41c7dab6 100644 --- a/drivers/net/wireless/p54/p54common.h +++ b/drivers/net/wireless/p54/p54common.h | |||
@@ -19,7 +19,6 @@ struct bootrec { | |||
19 | __le32 code; | 19 | __le32 code; |
20 | __le32 len; | 20 | __le32 len; |
21 | u32 data[10]; | 21 | u32 data[10]; |
22 | __le16 rx_mtu; | ||
23 | } __attribute__((packed)); | 22 | } __attribute__((packed)); |
24 | 23 | ||
25 | struct bootrec_exp_if { | 24 | struct bootrec_exp_if { |
@@ -39,6 +38,8 @@ struct bootrec_desc { | |||
39 | u8 tailroom; | 38 | u8 tailroom; |
40 | u8 unimportant[6]; | 39 | u8 unimportant[6]; |
41 | u8 rates[16]; | 40 | u8 rates[16]; |
41 | u8 padding2[4]; | ||
42 | __le16 rx_mtu; | ||
42 | } __attribute__((packed)); | 43 | } __attribute__((packed)); |
43 | 44 | ||
44 | #define BR_CODE_MIN 0x80000000 | 45 | #define BR_CODE_MIN 0x80000000 |