Re: [PATCH] ath11k: pci: avoid unsafe register access during shutdown

Balsam Chihi <[email protected]>
Newsgroups org.infradead.lists.ath11k,org.kernel.vger.linux-wireless
Message-ID <CAONkfw452N+cOrLo4p7KXOE_W6p=CFHanBReDdnenXrgEY6Cww@mail.gmail.com>
On Tue, Aug 19, 2025 at 7:36 PM Ameer Antar <[email protected]> wrote:
>
> Balsam, do you have a patch for this? I also have cards that require different
> bin files for each band. I haven't been able to figure out how to do this via
> configuration rather than rebuilding drivers.
>

Hello Ameer,

Yes, indeed we have to rebuild ath11k but once with the patched
core.c, not each time we add a card.
Then you can add text files named with the pci device address and the
path to your board-2.bin inside,
for each instance of the driver.
That way we avoid building the driver or the kernel each time we add a
wifi card.
I'm cleaning up the patch and I will send it to the mailing list if
you are interested.

Best regards,
Balsam

> Thanks.
> -Ameer
>
> ----- Original Message -----
> From: Balsam Chihi <[email protected]>
> To: Baochen Qiang <[email protected]>
> Cc: [email protected], [email protected],
> [email protected]
> Subject: Re: [PATCH] ath11k: pci: avoid unsafe register access during shutdown
> Date: 8/19/25 4:50:47 AM
>
> >Hello,
> >I Have a question on the same subject but not related to the bug.
> >I have many identical PCIe WiFi cards on the system and I need to load
> >a different board-2.bin on each one (1 adapter per frequency band).
> >Is there any kernel built-in solution for that?
> >I could not find any, so I had to create a patch that loads a
> >different board-2.bin for each card based on the pci device address.
> >I patched ath11k's core.c file so that it reads a string from a text
> >file in the rootfs (board-2.bin path per wifi card).
> >If the text file is not found, the driver loads the default file
> >provided by linux-firmware for all cards, the original behaviour.
> >Please let me know if there is a better solution for that.
> >Best regards.
> >Balsam
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.