From: Kalle Valo <kvalo@kernel.org>
To: Jeff Johnson <quic_jjohnson@quicinc.com>
Cc: Miaoqing Pan <quic_miaoqing@quicinc.com>,
<ath11k@lists.infradead.org>, <linux-wireless@vger.kernel.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] wifi: ath11k: add support for QCA6698AQ
Date: Thu, 21 Nov 2024 13:05:22 +0200 [thread overview]
Message-ID: <8734jkyhhp.fsf@kernel.org> (raw)
In-Reply-To: <87o72k4hxk.fsf@kernel.org> (Kalle Valo's message of "Tue, 12 Nov 2024 16:59:03 +0200")
Kalle Valo <kvalo@kernel.org> writes:
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>
>> On 10/30/2024 5:05 PM, Miaoqing Pan wrote:
>>> QCA6698AQ IP core is the same as WCN6855 hw2.1, they share the same
>>> PCI device ID, the same major and minor version numbers, the same
>>> register address, and same HAL descriptors, etc. The most significant
>>> difference is that QCA6698AQ has different RF, IPA, thermal, etc.
>>>
>>> Follow the approach done in commit 5dc9d1a55e95 ("wifi: ath11k: add
>>> support for QCA2066"), enumerate the subversion number to identify the
>>> specific card.
>>>
>>> Tested-on: QCA6698AQ hw2.1 PCI WLAN.HSP.1.1-04479-QCAHSPSWPL_V1_V2_SILICONZ_IOE-1
>>>
>>> Signed-off-by: Miaoqing Pan <quic_miaoqing@quicinc.com>
>>
>> Am I correct that this satisfies at least a subset of the devices you were
>> trying to support with the following?
>> https://lore.kernel.org/all/20241024002514.92290-1-quic_miaoqing@quicinc.com/
>>
>> If so, can you reply to that thread, pointing to this new patch, and elaborate
>> on what is now supported and what still requires ath11k to support
>> board-specific firmware overrides?
>
> That question is still open but otherwise this patch looks good to me.
> But I'll give an Acked-by only after I understand what's that "NFA765
> m.2 card for IoT platforms" mentioned in the other thread is.
Miaoqing provided the info offlist so:
Acked-by: Kalle Valo <kvalo@kernel.org>
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
next prev parent reply other threads:[~2024-11-21 11:05 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-31 0:05 [PATCH] wifi: ath11k: add support for QCA6698AQ Miaoqing Pan
2024-10-31 14:58 ` Jeff Johnson
2024-11-12 14:59 ` Kalle Valo
2024-11-21 11:05 ` Kalle Valo [this message]
2024-10-31 15:31 ` Jeff Johnson
2024-11-21 16:12 ` Jeff Johnson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8734jkyhhp.fsf@kernel.org \
--to=kvalo@kernel.org \
--cc=ath11k@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=quic_jjohnson@quicinc.com \
--cc=quic_miaoqing@quicinc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.