mt76: mt76x02: fix coverage_class type
authorLorenzo Bianconi <lorenzo@kernel.org>
Sun, 8 Dec 2019 07:25:38 +0000 (08:25 +0100)
committerFelix Fietkau <nbd@nbd.name>
Fri, 14 Feb 2020 09:06:00 +0000 (10:06 +0100)
Fix coverage_class type in mt76x02_dev data structure since
coverage_class can be negative to enable dynack (just supported by
ath9k). Set 0 as minimum value for coverage_class

Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt76x02.h
drivers/net/wireless/mediatek/mt76/mt76x02_util.c

index ba4c14eea22d99c1d178d114eb639d41ce77303f..57665516d00e2719c2c7aaa2f6213fe3a6cddfff 100644 (file)
@@ -116,7 +116,7 @@ struct mt76x02_dev {
 
        bool no_2ghz;
 
-       u8 coverage_class;
+       s16 coverage_class;
        u8 slottime;
 
        struct mt76x02_dfs_pattern_detector dfs_pd;
index e5685d9534e43ecec8ce3cfb84295e05fd6ea10f..ab717d82d771bbf2242ed2590efab9a930553d51 100644 (file)
@@ -545,7 +545,7 @@ void mt76x02_set_coverage_class(struct ieee80211_hw *hw,
        struct mt76x02_dev *dev = hw->priv;
 
        mutex_lock(&dev->mt76.mutex);
-       dev->coverage_class = coverage_class;
+       dev->coverage_class = max_t(s16, coverage_class, 0);
        mt76x02_set_tx_ackto(dev);
        mutex_unlock(&dev->mt76.mutex);
 }