* [PATCH 0/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
@ 2023-03-05 21:02 Bastian Germann
2023-03-06 6:19 ` Kalle Valo
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
0 siblings, 2 replies; 9+ messages in thread
From: Bastian Germann @ 2023-03-05 21:02 UTC (permalink / raw)
To: toke, Kalle Valo; +Cc: Bastian Germann, linux-wireless, netdev, linux-kernel
Drop a wrongly claimed USB ID.
Bastian Germann (1):
wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
drivers/net/wireless/ath/ath9k/hif_usb.c | 2 --
1 file changed, 2 deletions(-)
--
2.39.2
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-05 21:02 [PATCH 0/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID Bastian Germann
@ 2023-03-06 6:19 ` Kalle Valo
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
1 sibling, 0 replies; 9+ messages in thread
From: Kalle Valo @ 2023-03-06 6:19 UTC (permalink / raw)
To: Bastian Germann; +Cc: toke, linux-wireless, netdev, linux-kernel
Bastian Germann <bage@debian.org> writes:
> Drop a wrongly claimed USB ID.
>
> Bastian Germann (1):
> wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
>
> drivers/net/wireless/ath/ath9k/hif_usb.c | 2 --
> 1 file changed, 2 deletions(-)
I can't find the actual patch anywhere, I only see the cover letter:
https://lore.kernel.org/all/20230305210245.9831-1-bage@debian.org/
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-05 21:02 [PATCH 0/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID Bastian Germann
2023-03-06 6:19 ` Kalle Valo
@ 2023-03-06 12:50 ` Bastian Germann
2023-03-06 13:07 ` Kalle Valo
` (2 more replies)
1 sibling, 3 replies; 9+ messages in thread
From: Bastian Germann @ 2023-03-06 12:50 UTC (permalink / raw)
To: toke, Kalle Valo; +Cc: Bastian Germann, linux-wireless, netdev, linux-kernel
The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
correctly claimed to be supported by carl9170.
Supposedly, the successor 802AIN2 which has an ath9k compatible chip
whose USB ID (unknown) could be inserted instead.
Drop the ID from the wrong driver.
Signed-off-by: Bastian Germann <bage@debian.org>
---
drivers/net/wireless/ath/ath9k/hif_usb.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
index f521dfa2f194..1395536720b0 100644
--- a/drivers/net/wireless/ath/ath9k/hif_usb.c
+++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
@@ -42,8 +42,6 @@ static const struct usb_device_id ath9k_hif_usb_ids[] = {
{ USB_DEVICE(0x0cf3, 0x7015),
.driver_info = AR9287_USB }, /* Atheros */
- { USB_DEVICE(0x1668, 0x1200),
- .driver_info = AR9287_USB }, /* Verizon */
{ USB_DEVICE(0x0cf3, 0x7010),
.driver_info = AR9280_USB }, /* Atheros */
--
2.39.2
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
@ 2023-03-06 13:07 ` Kalle Valo
2023-03-06 13:36 ` Bastian Germann
2023-03-06 13:59 ` Toke Høiland-Jørgensen
2023-03-15 10:14 ` Kalle Valo
2 siblings, 1 reply; 9+ messages in thread
From: Kalle Valo @ 2023-03-06 13:07 UTC (permalink / raw)
To: Bastian Germann; +Cc: toke, linux-wireless, netdev, linux-kernel
Bastian Germann <bage@debian.org> writes:
> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
> correctly claimed to be supported by carl9170.
>
> Supposedly, the successor 802AIN2 which has an ath9k compatible chip
> whose USB ID (unknown) could be inserted instead.
>
> Drop the ID from the wrong driver.
>
> Signed-off-by: Bastian Germann <bage@debian.org>
Thanks, I see this patch now.
I guess there's a bug report somewhere, do you have a link?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 13:07 ` Kalle Valo
@ 2023-03-06 13:36 ` Bastian Germann
2023-03-06 13:49 ` Kalle Valo
0 siblings, 1 reply; 9+ messages in thread
From: Bastian Germann @ 2023-03-06 13:36 UTC (permalink / raw)
To: Kalle Valo; +Cc: toke, linux-wireless, netdev, linux-kernel
Am 06.03.23 um 14:07 schrieb Kalle Valo:
> Bastian Germann <bage@debian.org> writes:
>
>> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
>> correctly claimed to be supported by carl9170.
>>
>> Supposedly, the successor 802AIN2 which has an ath9k compatible chip
>> whose USB ID (unknown) could be inserted instead.
>>
>> Drop the ID from the wrong driver.
>>
>> Signed-off-by: Bastian Germann <bage@debian.org>
>
> Thanks, I see this patch now.
>
> I guess there's a bug report somewhere, do you have a link?
No, I happened to find this by chance while packaging the ath9k and carl9170 firmware for Debian,
which have the ID represented in an XML format:
https://salsa.debian.org/debian/open-ath9k-htc-firmware/-/blob/master/debian/firmware-ath9k-htc.metainfo.xml
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 13:36 ` Bastian Germann
@ 2023-03-06 13:49 ` Kalle Valo
2023-03-06 13:51 ` Bastian Germann
0 siblings, 1 reply; 9+ messages in thread
From: Kalle Valo @ 2023-03-06 13:49 UTC (permalink / raw)
To: Bastian Germann; +Cc: toke, linux-wireless, netdev, linux-kernel
Bastian Germann <bage@debian.org> writes:
> Am 06.03.23 um 14:07 schrieb Kalle Valo:
>> Bastian Germann <bage@debian.org> writes:
>>
>>> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
>>> correctly claimed to be supported by carl9170.
>>>
>>> Supposedly, the successor 802AIN2 which has an ath9k compatible chip
>>> whose USB ID (unknown) could be inserted instead.
>>>
>>> Drop the ID from the wrong driver.
>>>
>>> Signed-off-by: Bastian Germann <bage@debian.org>
>>
>> Thanks, I see this patch now.
>>
>> I guess there's a bug report somewhere, do you have a link?
>
> No, I happened to find this by chance while packaging the ath9k and
> carl9170 firmware for Debian,
> which have the ID represented in an XML format:
> https://salsa.debian.org/debian/open-ath9k-htc-firmware/-/blob/master/debian/firmware-ath9k-htc.metainfo.xml
Do you mind if we add this (without the link) to the commit log? It's
good to always document the background of the patch.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 13:49 ` Kalle Valo
@ 2023-03-06 13:51 ` Bastian Germann
0 siblings, 0 replies; 9+ messages in thread
From: Bastian Germann @ 2023-03-06 13:51 UTC (permalink / raw)
To: Kalle Valo; +Cc: toke, linux-wireless, netdev, linux-kernel
Am 06.03.23 um 14:49 schrieb Kalle Valo:
> Bastian Germann <bage@debian.org> writes:
>
>> Am 06.03.23 um 14:07 schrieb Kalle Valo:
>>> Bastian Germann <bage@debian.org> writes:
>>>
>>>> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
>>>> correctly claimed to be supported by carl9170.
>>>>
>>>> Supposedly, the successor 802AIN2 which has an ath9k compatible chip
The "which" should be removed as well.
>>>> whose USB ID (unknown) could be inserted instead.
>>>>
>>>> Drop the ID from the wrong driver.
>>>>
>>>> Signed-off-by: Bastian Germann <bage@debian.org>
>>>
>>> Thanks, I see this patch now.
>>>
>>> I guess there's a bug report somewhere, do you have a link?
>>
>> No, I happened to find this by chance while packaging the ath9k and
>> carl9170 firmware for Debian,
>> which have the ID represented in an XML format:
>> https://salsa.debian.org/debian/open-ath9k-htc-firmware/-/blob/master/debian/firmware-ath9k-htc.metainfo.xml
>
> Do you mind if we add this (without the link) to the commit log? It's
> good to always document the background of the patch.
Please go ahead.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
2023-03-06 13:07 ` Kalle Valo
@ 2023-03-06 13:59 ` Toke Høiland-Jørgensen
2023-03-15 10:14 ` Kalle Valo
2 siblings, 0 replies; 9+ messages in thread
From: Toke Høiland-Jørgensen @ 2023-03-06 13:59 UTC (permalink / raw)
To: Bastian Germann, Kalle Valo
Cc: Bastian Germann, linux-wireless, netdev, linux-kernel
Bastian Germann <bage@debian.org> writes:
> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
> correctly claimed to be supported by carl9170.
>
> Supposedly, the successor 802AIN2 which has an ath9k compatible chip
> whose USB ID (unknown) could be inserted instead.
>
> Drop the ID from the wrong driver.
>
> Signed-off-by: Bastian Germann <bage@debian.org>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
2023-03-06 13:07 ` Kalle Valo
2023-03-06 13:59 ` Toke Høiland-Jørgensen
@ 2023-03-15 10:14 ` Kalle Valo
2 siblings, 0 replies; 9+ messages in thread
From: Kalle Valo @ 2023-03-15 10:14 UTC (permalink / raw)
To: Bastian Germann
Cc: toke, Bastian Germann, linux-wireless, netdev, linux-kernel
Bastian Germann <bage@debian.org> wrote:
> The USB device 1668:1200 is Qwest/Actiontec 802AIN which is also
> correctly claimed to be supported by carl9170.
>
> Supposedly, the successor 802AIN2 has an ath9k compatible chip
> whose USB ID (unknown) could be inserted instead.
>
> Drop the ID from the wrong driver. I happened to find this by chance while
> packaging the ath9k and carl9170 firmware for Debian.
>
> Signed-off-by: Bastian Germann <bage@debian.org>
> Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
> Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Patch applied to ath-next branch of ath.git, thanks.
f94d7a3a5107 wifi: ath9k: Remove Qwest/Actiontec 802AIN ID
--
https://patchwork.kernel.org/project/linux-wireless/patch/20230306125041.2221-1-bage@debian.org/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2023-03-15 10:16 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-05 21:02 [PATCH 0/1] wifi: ath9k: Remove Qwest/Actiontec 802AIN ID Bastian Germann
2023-03-06 6:19 ` Kalle Valo
2023-03-06 12:50 ` [PATCH 1/1] " Bastian Germann
2023-03-06 13:07 ` Kalle Valo
2023-03-06 13:36 ` Bastian Germann
2023-03-06 13:49 ` Kalle Valo
2023-03-06 13:51 ` Bastian Germann
2023-03-06 13:59 ` Toke Høiland-Jørgensen
2023-03-15 10:14 ` Kalle Valo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.