It was superfluous and confusing. It's better to
define the structure explicitly.

Signed-off-by: Michal Kazior <[email protected]>
---
 drivers/net/wireless/ath/ath10k/mac.c | 31 +++++++++++++------------------
 1 file changed, 13 insertions(+), 18 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
b/drivers/net/wireless/ath/ath10k/mac.c
index 5721196..f4e76e7 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -34,27 +34,22 @@
 /* Rates */
 /*********/
 
-#define RATETAB_ENT(_rate, _rateid, _flags) { \
-       .bitrate                = (_rate), \
-       .flags                  = (_flags), \
-       .hw_value               = (_rateid), \
-}
-
 static struct ieee80211_rate ath10k_rates[] = {
        /* CCK */
-       RATETAB_ENT(10,  0, 0),
-       RATETAB_ENT(20,  0, 0),
-       RATETAB_ENT(55,  0, 0),
-       RATETAB_ENT(110, 0, 0),
+       { .bitrate = 10 },
+       { .bitrate = 20 },
+       { .bitrate = 55 },
+       { .bitrate = 110 },
+
        /* OFDM */
-       RATETAB_ENT(60,  0, 0),
-       RATETAB_ENT(90,  0, 0),
-       RATETAB_ENT(120, 0, 0),
-       RATETAB_ENT(180, 0, 0),
-       RATETAB_ENT(240, 0, 0),
-       RATETAB_ENT(360, 0, 0),
-       RATETAB_ENT(480, 0, 0),
-       RATETAB_ENT(540, 0, 0),
+       { .bitrate = 60 },
+       { .bitrate = 90 },
+       { .bitrate = 120 },
+       { .bitrate = 180 },
+       { .bitrate = 240 },
+       { .bitrate = 360 },
+       { .bitrate = 480 },
+       { .bitrate = 540 },
 };
 
 #define ath10k_a_rates (ath10k_rates + 4)
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to