Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

[Jeff Johnson]

On 3/18/2024 5:15 AM, Vladimir Mesh. wrote:
> Dear Sirs, 
> 
> I have a problem using ath12k driver with wcn7850 adapters.
> ath12k driver does not allow the system to switch to suspend/sleep mode.
> Please fix it! 

This is a known issue both for ath11k and ath12k
https://bugzilla.kernel.org/show_bug.cgi?id=214649

There is already a proposed fix for ath11k that requires changes to other
components (MHI & QRTR) that is pending approval from those maintainers:
https://lore.kernel.org/ath11k/20240305021320.3367-1-quic_bqiang@quicinc.com/

Once that is in place, ath12k can also be addressed.

/jeff

Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

[Kalle Valo]

Jeff Johnson <quic_jjohnson@quicinc.com> writes:

> On 3/18/2024 5:15 AM, Vladimir Mesh. wrote:
>> Dear Sirs, 
>> 
>> I have a problem using ath12k driver with wcn7850 adapters.
>> ath12k driver does not allow the system to switch to suspend/sleep mode.
>> Please fix it! 
>
> This is a known issue both for ath11k and ath12k
> https://bugzilla.kernel.org/show_bug.cgi?id=214649
>
> There is already a proposed fix for ath11k that requires changes to other
> components (MHI & QRTR) that is pending approval from those maintainers:
> https://lore.kernel.org/ath11k/20240305021320.3367-1-quic_bqiang@quicinc.com/
>
> Once that is in place, ath12k can also be addressed.

BTW it would be good to submit corresponding ath12k patches as well.
That way people could test them and provide feedback.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

[Owen Mesh.]

Thanks for the info!
I'll wait to hear about the solution.
In the meantime,  will use a proven method - run the "sudo modprobe -r
ath12k" command before leaving home.


пн, 18 мар. 2024 г. в 18:00, Kalle Valo <kvalo@kernel.org>:
>
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>
> > On 3/18/2024 5:15 AM, Vladimir Mesh. wrote:
> >> Dear Sirs,
> >>
> >> I have a problem using ath12k driver with wcn7850 adapters.
> >> ath12k driver does not allow the system to switch to suspend/sleep mode.
> >> Please fix it!
> >
> > This is a known issue both for ath11k and ath12k
> > https://bugzilla.kernel.org/show_bug.cgi?id=214649
> >
> > There is already a proposed fix for ath11k that requires changes to other
> > components (MHI & QRTR) that is pending approval from those maintainers:
> > https://lore.kernel.org/ath11k/20240305021320.3367-1-quic_bqiang@quicinc.com/
> >
> > Once that is in place, ath12k can also be addressed.
>
> BTW it would be good to submit corresponding ath12k patches as well.
> That way people could test them and provide feedback.
>
> --
> https://patchwork.kernel.org/project/linux-wireless/list/
>
> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

[Manivannan Sadhasivam]

On Mon, Mar 18, 2024 at 08:51:04AM -0700, Jeff Johnson wrote:
> On 3/18/2024 5:15 AM, Vladimir Mesh. wrote:
> > Dear Sirs, 
> > 
> > I have a problem using ath12k driver with wcn7850 adapters.
> > ath12k driver does not allow the system to switch to suspend/sleep mode.
> > Please fix it! 
> 
> This is a known issue both for ath11k and ath12k
> https://bugzilla.kernel.org/show_bug.cgi?id=214649
> 
> There is already a proposed fix for ath11k that requires changes to other
> components (MHI & QRTR) that is pending approval from those maintainers:
> https://lore.kernel.org/ath11k/20240305021320.3367-1-quic_bqiang@quicinc.com/
> 

Just to clarify. There is no 'pending approval' for MHI/QRTR. Both patches are
reviewed and pending to get merged.

I will merge the MHI patch once merge window closes. If Kalle wants, we can do
an immutable branch between mhi and ath11k trees. Otherwise, ath11k patches can
get merged for 6.11.

QRTR patch should go through netdev.

- Mani

> Once that is in place, ath12k can also be addressed.
> 
> /jeff
> 

-- 
மணிவண்ணன் சதாசிவம்

Re: ath12k: driver does not allow the system to switch to suspend/sleep mode

[Kalle Valo]

Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> writes:

> On Mon, Mar 18, 2024 at 08:51:04AM -0700, Jeff Johnson wrote:
>> On 3/18/2024 5:15 AM, Vladimir Mesh. wrote:
>> > Dear Sirs, 
>> > 
>> > I have a problem using ath12k driver with wcn7850 adapters.
>> > ath12k driver does not allow the system to switch to suspend/sleep mode.
>> > Please fix it! 
>> 
>> This is a known issue both for ath11k and ath12k
>> https://bugzilla.kernel.org/show_bug.cgi?id=214649
>> 
>> There is already a proposed fix for ath11k that requires changes to other
>> components (MHI & QRTR) that is pending approval from those maintainers:
>> https://lore.kernel.org/ath11k/20240305021320.3367-1-quic_bqiang@quicinc.com/
>> 
>
> Just to clarify. There is no 'pending approval' for MHI/QRTR. Both patches are
> reviewed and pending to get merged.
>
> I will merge the MHI patch once merge window closes. If Kalle wants, we can do
> an immutable branch between mhi and ath11k trees. Otherwise, ath11k patches can
> get merged for 6.11.

A lot of people are waiting for hibernation feature so I really would
prefer to have an immutable branch which could be merged to our ath.git
tree.

> QRTR patch should go through netdev.

I would be faster to apply that patch to ath.git, but I'll discuss with
Jakub about that.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches