Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> writes: > On Thu, Sep 16, 2021 at 01:18:22PM +0200, Loic Poulain wrote: >> Le jeu. 16 sept. 2021 à 13:12, Manivannan Sadhasivam < >> manivannan.sadhasivam@xxxxxxxxxx> a écrit : >> > > [...] > >> > 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 only? >> >> >> It fixes sdx modem resuming issue, without that we don’t know modem needs >> to be reinitialized. >> > > Okay. Then in that case, the recovery mechanism has to be added to the ath11k > 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. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches