Sender: kvalo=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.4.1 smtp.codeaurora.org 14883C43164
From: Kalle Valo <kvalo@codeaurora.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Loic Poulain <loic.poulain@linaro.org>,  ath11k@lists.infradead.org,  linux-arm-msm <linux-arm-msm@vger.kernel.org>,  linux-wireless@vger.kernel.org,  regressions@lists.linux.dev
Subject: Re: [regression] mhi: ath11k resume fails on some devices
References: <871r5p0x2u.fsf@codeaurora.org>
	<CAMZdPi8UJLvBFQd8-nf-iHAQh8cEuihq97PUFfZ7Q=rxRQoPsg@mail.gmail.com>
	<20210916111218.GA12918@thinkpad>
	<CAMZdPi94607mZorp+Zmkw3seWXak6p9Jr05CQ5hhfgKQoG8n7Q@mail.gmail.com>
	<20210916163529.GA9027@thinkpad>
Date: Thu, 16 Sep 2021 19:42:02 +0300
In-Reply-To: <20210916163529.GA9027@thinkpad> (Manivannan Sadhasivam's message
	of "Thu, 16 Sep 2021 22:05:29 +0530")
Message-ID: <87k0jgxyjp.fsf@codeaurora.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

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

> On Thu, Sep 16, 2021 at 01:18:22PM +0200, Loic Poulain wrote:
>> Le jeu. 16 sept. 2021 =C3=A0 13:12, Manivannan Sadhasivam <
>> manivannan.sadhasivam@linaro.org> a =C3=A9crit :
>>=20
>
> [...]
>
>> > If things seems to work fine without that patch, then it implies that
>> > setting M0
>> > state works during resume. I think we should just revert that patch.
>> >
>> > Loic, did that patch fix any issue for you or it was a cosmetic fix on=
ly?
>>=20
>>=20
>> It fixes sdx modem resuming issue, without that we don=E2=80=99t know mo=
dem needs
>> to be reinitialized.
>>=20
>
> Okay. Then in that case, the recovery mechanism has to be added to the at=
h11k
> MHI controller.

What does that mean in practise, do you have any pointers or examples? I
have no clue what you are proposing :)

> If that's too much of work for Kalle, then I'll look into it. But I might=
 get
> time only after Plumbers.

I'm busy, as always, so not sure when I'm able to do it either. I think
we should seriously consider reverting 020d3b26c07a and adding it back
after ath11k is able to handle this new situation.

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

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatc=
hes