Message ID | 70b8acb74fef12b10a74501cccd20e0c12f3f134.1675348583.git.deren.wu@mediatek.com |
---|---|
State | New |
Headers | show |
Series | wifi: mt76: support ww power config in dts node | expand |
Deren Wu <deren.wu@mediatek.com> writes:
> support new node for WW regulatory domain
The commit log doesn't really explain much. What does this patch do
exactly?
Hi Kalle, On Wed, 2023-02-08 at 11:35 +0200, Kalle Valo wrote: > Deren Wu <deren.wu@mediatek.com> writes: > > > support new node for WW regulatory domain > > The commit log doesn't really explain much. What does this patch do > exactly? > In STA mode, if the host stays in ww regulatory doamin ("00"), NL80211_DFS_UNSET would be set to driver. In such case, mt76 cannot find out the proper dts node to limit power settings. We apply this patch to have a proper power mapping for ww domain. Regards, Deren
"Deren Wu (武德仁)" <Deren.Wu@mediatek.com> writes: > Hi Kalle, > > On Wed, 2023-02-08 at 11:35 +0200, Kalle Valo wrote: >> Deren Wu <deren.wu@mediatek.com> writes: >> >> > support new node for WW regulatory domain >> >> The commit log doesn't really explain much. What does this patch do >> exactly? >> > In STA mode, if the host stays in ww regulatory doamin ("00"), > NL80211_DFS_UNSET would be set to driver. In such case, mt76 cannot > find out the proper dts node to limit power settings. We apply this > patch to have a proper power mapping for ww domain. Ok. In the future, please always this kind of information to the commit log. One vague phrase is not a proper commit log, more info about that in the wiki below.
diff --git a/drivers/net/wireless/mediatek/mt76/eeprom.c b/drivers/net/wireless/mediatek/mt76/eeprom.c index 9bc8758573fc..dce851d42e08 100644 --- a/drivers/net/wireless/mediatek/mt76/eeprom.c +++ b/drivers/net/wireless/mediatek/mt76/eeprom.c @@ -138,6 +138,7 @@ mt76_find_power_limits_node(struct mt76_dev *dev) { struct device_node *np = dev->dev->of_node; const char *const region_names[] = { + [NL80211_DFS_UNSET] = "ww", [NL80211_DFS_ETSI] = "etsi", [NL80211_DFS_FCC] = "fcc", [NL80211_DFS_JP] = "jp",