Search Linux Wireless

Re: [PATCH 1/3] ath11k: Change number of TCL rings to one for QCA6390

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2021-09-28 23:12, Kalle Valo wrote:
Jouni Malinen <jouni@xxxxxxxxxxxxxx> writes:

From: Baochen Qiang <bqiang@xxxxxxxxxxxxxx>

Some targets, QCA6390 for example, use only one TCL ring,
it is better to initialize only one ring and leave others
untouched for such targets.

Tested-on: QCA6390 hw2.0 PCI WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1

Signed-off-by: Baochen Qiang <bqiang@xxxxxxxxxxxxxx>
Signed-off-by: Jouni Malinen <jouni@xxxxxxxxxxxxxx>

It's better? Please be more specific. Does this fix a bug or is this
just a theoretical fix you found during code review?

Yes, this is just a theoretical fix. By "better" I mean there is no need to initialize
the other two TCL rings for QCA6390 since they are not used.



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux