* [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
[not found] <cover.1513248071.git.lorenzo.bianconi@redhat.com>
@ 2017-12-14 10:46 ` Lorenzo Bianconi
2017-12-14 15:29 ` Felix Fietkau
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Lorenzo Bianconi @ 2017-12-14 10:46 UTC (permalink / raw)
To: nbd; +Cc: colin.king, linux-wireless
Fix typo in 5GHz power vs channel eeprom parsing
Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
---
drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
index 440b7e7d73a9..29dc52ef629d 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
+++ b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
@@ -524,7 +524,7 @@ mt76x2_get_power_info_5g(struct mt76x2_dev *dev, struct mt76x2_tx_power_info *t,
if (channel >= 192)
delta_idx = 4;
- else if (channel >= 484)
+ else if (channel >= 184)
delta_idx = 3;
else if (channel < 44)
delta_idx = 3;
--
2.13.6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
2017-12-14 10:46 ` [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g() Lorenzo Bianconi
@ 2017-12-14 15:29 ` Felix Fietkau
2017-12-14 15:42 ` Colin Ian King
2018-01-08 17:30 ` Kalle Valo
2 siblings, 0 replies; 6+ messages in thread
From: Felix Fietkau @ 2017-12-14 15:29 UTC (permalink / raw)
To: Lorenzo Bianconi; +Cc: colin.king, linux-wireless
On 2017-12-14 11:46, Lorenzo Bianconi wrote:
> Fix typo in 5GHz power vs channel eeprom parsing
>
> Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Acked-by: Felix Fietkau <nbd@nbd.name>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
2017-12-14 10:46 ` [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g() Lorenzo Bianconi
2017-12-14 15:29 ` Felix Fietkau
@ 2017-12-14 15:42 ` Colin Ian King
2017-12-14 16:20 ` Kalle Valo
2018-01-08 17:30 ` Kalle Valo
2 siblings, 1 reply; 6+ messages in thread
From: Colin Ian King @ 2017-12-14 15:42 UTC (permalink / raw)
To: Lorenzo Bianconi, nbd; +Cc: linux-wireless
On 14/12/17 10:46, Lorenzo Bianconi wrote:
> Fix typo in 5GHz power vs channel eeprom parsing
>
> Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
> ---
> drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
> index 440b7e7d73a9..29dc52ef629d 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
> @@ -524,7 +524,7 @@ mt76x2_get_power_info_5g(struct mt76x2_dev *dev, struct mt76x2_tx_power_info *t,
>
> if (channel >= 192)
> delta_idx = 4;
> - else if (channel >= 484)
> + else if (channel >= 184)
> delta_idx = 3;
> else if (channel < 44)
> delta_idx = 3;
>
any possibility of adding:
Reported-by: Colin Ian King <colin.king@canonical.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
2017-12-14 15:42 ` Colin Ian King
@ 2017-12-14 16:20 ` Kalle Valo
2018-01-08 17:32 ` Kalle Valo
0 siblings, 1 reply; 6+ messages in thread
From: Kalle Valo @ 2017-12-14 16:20 UTC (permalink / raw)
To: Colin Ian King; +Cc: Lorenzo Bianconi, nbd, linux-wireless
Colin Ian King <colin.king@canonical.com> writes:
> On 14/12/17 10:46, Lorenzo Bianconi wrote:
>> Fix typo in 5GHz power vs channel eeprom parsing
>>
>> Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
>> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
>> ---
>> drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
>> index 440b7e7d73a9..29dc52ef629d 100644
>> --- a/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
>> +++ b/drivers/net/wireless/mediatek/mt76/mt76x2_eeprom.c
>> @@ -524,7 +524,7 @@ mt76x2_get_power_info_5g(struct mt76x2_dev *dev, struct mt76x2_tx_power_info *t,
>>
>> if (channel >= 192)
>> delta_idx = 4;
>> - else if (channel >= 484)
>> + else if (channel >= 184)
>> delta_idx = 3;
>> else if (channel < 44)
>> delta_idx = 3;
>>
> any possibility of adding:
>
> Reported-by: Colin Ian King <colin.king@canonical.com>
Sure, I'll add that during commit.
--
Kalle Valo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
2017-12-14 10:46 ` [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g() Lorenzo Bianconi
2017-12-14 15:29 ` Felix Fietkau
2017-12-14 15:42 ` Colin Ian King
@ 2018-01-08 17:30 ` Kalle Valo
2 siblings, 0 replies; 6+ messages in thread
From: Kalle Valo @ 2018-01-08 17:30 UTC (permalink / raw)
To: Lorenzo Bianconi; +Cc: nbd, colin.king, linux-wireless
Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:
> Fix typo in 5GHz power vs channel eeprom parsing
>
> Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
> Acked-by: Felix Fietkau <nbd@nbd.name>
> Reported-by: Colin Ian King <colin.king@canonical.com>
Patch applied to wireless-drivers-next.git, thanks.
c3929a980b16 mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
--
https://patchwork.kernel.org/patch/10111813/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g()
2017-12-14 16:20 ` Kalle Valo
@ 2018-01-08 17:32 ` Kalle Valo
0 siblings, 0 replies; 6+ messages in thread
From: Kalle Valo @ 2018-01-08 17:32 UTC (permalink / raw)
To: Colin Ian King; +Cc: Lorenzo Bianconi, nbd, linux-wireless
Kalle Valo <kvalo@codeaurora.org> writes:
> Colin Ian King <colin.king@canonical.com> writes:
>
>> On 14/12/17 10:46, Lorenzo Bianconi wrote:
>>> Fix typo in 5GHz power vs channel eeprom parsing
>>>
>>> Fixes: 7bc04215a66b ("mt76: add driver code for MT76x2e")
>>> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
[...]
>> any possibility of adding:
>>
>> Reported-by: Colin Ian King <colin.king@canonical.com>
>
> Sure, I'll add that during commit.
Actually patchwork added that automatically based on your email, just as
it does with Acked-by. Very nice!
--
Kalle Valo
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-01-08 17:32 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1513248071.git.lorenzo.bianconi@redhat.com>
2017-12-14 10:46 ` [PATCH] mt76x2: eeprom: fix typo in mt76x2_get_power_info_5g() Lorenzo Bianconi
2017-12-14 15:29 ` Felix Fietkau
2017-12-14 15:42 ` Colin Ian King
2017-12-14 16:20 ` Kalle Valo
2018-01-08 17:32 ` Kalle Valo
2018-01-08 17:30 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).