From: Sriram R <srirrama@codeaurora.org>
To: Sven Eckelmann <sven@narfation.org>
Cc: ath11k@lists.infradead.org, sw@simonwunderlich.de
Subject: Re: ath11k: Country code setting ignored on bootup
Date: Thu, 05 Nov 2020 17:55:49 +0530 [thread overview]
Message-ID: <2fb984817cc1255b9686a9aa8f16c75b@codeaurora.org> (raw)
In-Reply-To: <1657154.X513TT2pbd@ripper>
On 2020-11-04 22:00, Sven Eckelmann wrote:
> On Wednesday, 4 November 2020 17:25:53 CET Sriram R wrote:
>> btw, i dont see any possible issues in current usage of
>> ATH11k_FLAG_REGISTERED in
>> ath11k_debug_pdev_create, ath11k_mac_register and
>> ath11k_mac_allocate
>> since they sequenced and fall within the same context. Do you face any
>> issues?
>
> A firmware crash before ATH11K_QMI_EVENT_FW_READY was
> processed/received. So
> we might want to recreate the mac/debugfs/... even when it was already
> created
> - at least this is the stuff mentioned in the commit(s) which introduce
> the
> ATH11k_FLAG_REGISTERED in these positions.
Yeah right, i think it was added to avoid multiple driver registration.
The ATH11K_FLAG_REGISTERED currently implies core registration complete
where
mac registration is one part of it. Agree that checking this flag in reg
path
is not the right approach to decide if mac is registered and default
rules are
applied for it.
I can also prepare a patch to avoid the issue you had mentioned and a
similar race as below (which i noted with this patch in my setup)and
see if it works fine for all use cases.
Please note below on the 4th line, the Default regd for pdev 0 gets
overwritten from US to IN(the country set before ath11k bind), which is
not desirable.
[ 949.056244] ath11k c000000.wifi1: Default REGD update pdev 0 US
[ 949.056493] ath11k c000000.wifi1: Default REGD update pdev 1 US
[ 949.056595] ath11k c000000.wifi1: Default REGD update pdev 2 US
[ 949.061231] ath11k c000000.wifi1: Default REGD update pdev 0 IN
[ 949.063915] ath11k c000000.wifi1: User REGD update pdev 1 IN
[ 949.066156] ath11k c000000.wifi1: User REGD update pdev 2 IN
Thanks and Regards,
Sriram.R
>
> Kind regards,
> Sven
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
prev parent reply other threads:[~2020-11-05 12:25 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-30 12:34 Country code setting ignored on bootup Sven Eckelmann
2020-10-30 12:34 ` ath11k: " Sven Eckelmann
[not found] ` <BYAPR02MB41208890EFBE3E186114FBC1F7EF0@BYAPR02MB4120.namprd02.prod.outlook.com>
2020-11-04 16:25 ` Sriram R
2020-11-04 16:30 ` Sven Eckelmann
2020-11-05 12:25 ` Sriram R [this message]
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=2fb984817cc1255b9686a9aa8f16c75b@codeaurora.org \
--to=srirrama@codeaurora.org \
--cc=ath11k@lists.infradead.org \
--cc=sven@narfation.org \
--cc=sw@simonwunderlich.de \
/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.