Wen Gong <quic_wgong@xxxxxxxxxxx> writes: > On 11/24/2021 3:39 PM, Kalle Valo wrote: >> Wen Gong <quic_wgong@xxxxxxxxxxx> writes: >> >>> On 10/22/2021 8:37 PM, P Praneesh wrote: >>> >>> ... >>> >>>> --- >>>> ... >>>> - ch->maxpower = channel->max_power * 2; >>>> - ch->maxregpower = channel->max_reg_power * 2; >>>> - ch->antennamax = channel->max_antenna_gain * 2; >>>> ... >>>> +int ath11k_wmi_update_scan_chan_list(struct ath11k *ar, >>>> + struct scan_req_params *arg) >>>> ... >>>> + ch->maxpower = channel->max_power; >>>> + ch->maxregpower = channel->max_reg_power; >>>> + ch->antennamax = channel->max_antenna_gain; >>>> + >>>> ... >>> I see it also have another patch to remove the "*2" for power. >>> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?id=9212c1b9e80a869e732769a4fe7f82d392b219be >>> >>> ath11k: send proper txpower and maxregpower values to firmware. >>> >>> so are you the same reason to remove "*2" ? >> I'm not quite getting what you are saying, can you elaborate more? Did >> this patch break something? > > Oh. Not see break anything. > > I just want to know why remove the "*2". Did you see any issue if NOT > remove "*2"? All I know is that commit 9212c1b9e80a ("ath11k: send proper txpower and maxregpower values to firmware") mentions about cca getting stuck: ath11k: send proper txpower and maxregpower values to firmware Set proper values for max_regpower, max_power, max_antenna_gain as it is because firmware will convert power values to 0.5dbm steps by multiplying it with 2. If txpower is not set, it will lead to cca stuck resulting in latency issues for QCN9074. Tested-on: QCN9074 hw1.0 PCI WLAN.HK.2.4.0.1-01386-QCAHKSWPL_SILICONZ-1 -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
