Baochen Qiang <quic_bqiang@xxxxxxxxxxx> writes: > On 2/22/2024 2:25 PM, Kalle Valo wrote: >> Baochen Qiang <quic_bqiang@xxxxxxxxxxx> writes: >> >>> On 2/22/2024 1:15 AM, Jeff Johnson wrote: >>>> On 2/20/2024 6:47 PM, Baochen Qiang wrote: >>>>> From: Kalle Valo <quic_kvalo@xxxxxxxxxxx> >>>>> >>>>> Every time the firmware boots we call ath11k_core_qmi_firmware_ready() which >>>>> ends up calling ath11k_thermal_register(). So we try to register thermal >>>>> devices multiple times. And when we power off the firmware during >>>>> suspend/hibernation (implemented in the next patch) we get a warning in resume: >>>>> >>>>> hwmon hwmon4: PM: parent phy0 should not be sleeping >>>>> >>>>> Workaround this similarly like ath11k_mac_register() does by testing >>>>> ATH11K_FLAG_REGISTERED. >>>>> >>>>> Tested-on: WCN6855 hw2.0 PCI >>>>> WLAN.HSP.1.1-03125-QCAHSPSWPL_V1_V2_SILICONZ_LITE-3.6510.30 >>>>> >>>>> Signed-off-by: Kalle Valo <quic_kvalo@xxxxxxxxxxx> >>>> you need to add your own S-O-B >>> Oh, sorry for missing that. >>> >>> Hi Kalle, I see you have put this series in pending branch. I am >>> wondering if I need to send a v2 to add my S-O-B tag, or you have any >>> other plan? >> I can add it in the pending branch if you send it to me as a reply >> to >> this mail. s-o-b is like a signature so I won't add it on my own. > Understood Kalle. So please help add below tag to this patch in > pending branch: > Signed-off-by: Baochen Qiang <quic_bqiang@xxxxxxxxxxx> Thanks, added: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=a88f2007763465f2be307c3a5ed542233bc3c77e -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
