diff mbox series

wifi: mt76: support ww power config in dts node

Message ID 70b8acb74fef12b10a74501cccd20e0c12f3f134.1675348583.git.deren.wu@mediatek.com (mailing list archive)
State Accepted
Delegated to: Felix Fietkau
Headers show
Series wifi: mt76: support ww power config in dts node | expand

Commit Message

Deren Wu Feb. 2, 2023, 2:47 p.m. UTC 
support new node for WW regulatory domain

Tested-by: Michael Lo <michael.lo@mediatek.com>
Signed-off-by: Deren Wu <deren.wu@mediatek.com>
---
 drivers/net/wireless/mediatek/mt76/eeprom.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Kalle Valo Feb. 8, 2023, 9:35 a.m. UTC | #1 
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?
Deren Wu Feb. 8, 2023, 10:04 a.m. UTC | #2 
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
Kalle Valo Feb. 13, 2023, 1:08 p.m. UTC | #3 
"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 mbox series

Patch

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",