All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kalle Valo <kvalo@codeaurora.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Carl Huang <cjhuang@codeaurora.org>,
	 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
Date: Fri, 24 Sep 2021 12:07:41 +0300	[thread overview]
Message-ID: <8735putk82.fsf@codeaurora.org> (raw)
In-Reply-To: <20210923085926.GD6083@thinkpad> (Manivannan Sadhasivam's message of "Thu, 23 Sep 2021 14:29:26 +0530")

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

> For aid debugging, please see the state the device is in during mhi_pm_resume().
> You can use below diff:
>
> diff --git a/drivers/bus/mhi/core/pm.c b/drivers/bus/mhi/core/pm.c
> index fb99e3727155..482d55dd209e 100644
> --- a/drivers/bus/mhi/core/pm.c
> +++ b/drivers/bus/mhi/core/pm.c
> @@ -898,6 +898,9 @@ int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
>         if (MHI_PM_IN_ERROR_STATE(mhi_cntrl->pm_state))
>                 return -EIO;
>  
> +       dev_info(dev, "Device state: %s\n",
> +                TO_MHI_STATE_STR(mhi_get_mhi_state(mhi_cntrl)));
> +
>         if (mhi_get_mhi_state(mhi_cntrl) != MHI_STATE_M3)
>                 return -EINVAL;

This is what I get with my NUC testbox:

[  970.488202] ACPI: EC: event unblocked
[  970.492484] hpet: Lost 1587 RTC interrupts
[  970.492749] mhi mhi0: Device state: RESET
[  970.492805] ath11k_pci 0000:06:00.0: failed to set mhi state: RESUME(6)

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

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

WARNING: multiple messages have this Message-ID (diff)
From: Kalle Valo <kvalo@codeaurora.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Carl Huang <cjhuang@codeaurora.org>,
	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
Date: Fri, 24 Sep 2021 12:07:41 +0300	[thread overview]
Message-ID: <8735putk82.fsf@codeaurora.org> (raw)
In-Reply-To: <20210923085926.GD6083@thinkpad> (Manivannan Sadhasivam's message of "Thu, 23 Sep 2021 14:29:26 +0530")

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

> For aid debugging, please see the state the device is in during mhi_pm_resume().
> You can use below diff:
>
> diff --git a/drivers/bus/mhi/core/pm.c b/drivers/bus/mhi/core/pm.c
> index fb99e3727155..482d55dd209e 100644
> --- a/drivers/bus/mhi/core/pm.c
> +++ b/drivers/bus/mhi/core/pm.c
> @@ -898,6 +898,9 @@ int mhi_pm_resume(struct mhi_controller *mhi_cntrl)
>         if (MHI_PM_IN_ERROR_STATE(mhi_cntrl->pm_state))
>                 return -EIO;
>  
> +       dev_info(dev, "Device state: %s\n",
> +                TO_MHI_STATE_STR(mhi_get_mhi_state(mhi_cntrl)));
> +
>         if (mhi_get_mhi_state(mhi_cntrl) != MHI_STATE_M3)
>                 return -EINVAL;

This is what I get with my NUC testbox:

[  970.488202] ACPI: EC: event unblocked
[  970.492484] hpet: Lost 1587 RTC interrupts
[  970.492749] mhi mhi0: Device state: RESET
[  970.492805] ath11k_pci 0000:06:00.0: failed to set mhi state: RESUME(6)

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

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

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

  parent reply	other threads:[~2021-09-24  9:07 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-16  8:00 [regression] mhi: ath11k resume fails on some devices Kalle Valo
2021-09-16  8:00 ` Kalle Valo
2021-09-16 10:18 ` Loic Poulain
2021-09-16 10:18   ` Loic Poulain
2021-09-16 10:18   ` Loic Poulain
2021-09-16 11:12   ` Manivannan Sadhasivam
2021-09-16 11:12     ` Manivannan Sadhasivam
     [not found]     ` <CAMZdPi94607mZorp+Zmkw3seWXak6p9Jr05CQ5hhfgKQoG8n7Q@mail.gmail.com>
2021-09-16 16:35       ` Manivannan Sadhasivam
2021-09-16 16:35         ` Manivannan Sadhasivam
2021-09-16 16:42         ` Kalle Valo
2021-09-16 16:42           ` Kalle Valo
2021-09-16 16:42           ` Kalle Valo
2021-09-16 17:19           ` Manivannan Sadhasivam
2021-09-16 17:19             ` Manivannan Sadhasivam
2021-09-23  8:34             ` Carl Huang
2021-09-23  8:34               ` Carl Huang
2021-09-23  8:59               ` Manivannan Sadhasivam
2021-09-23  8:59                 ` Manivannan Sadhasivam
2021-09-23  9:26                 ` Carl Huang
2021-09-23  9:26                   ` Carl Huang
2021-09-23 10:50                   ` Loic Poulain
2021-09-23 10:50                     ` Loic Poulain
2021-09-23 10:50                     ` Loic Poulain
2021-09-24  9:07                 ` Kalle Valo [this message]
2021-09-24  9:07                   ` Kalle Valo
2021-09-24  9:07                   ` Kalle Valo
2021-09-24  9:57                   ` Manivannan Sadhasivam
2021-09-24  9:57                     ` Manivannan Sadhasivam
2021-10-07  9:55                     ` Kalle Valo
2021-10-07  9:55                       ` Kalle Valo
2021-10-21 10:01                       ` Manivannan Sadhasivam
2021-10-21 10:01                         ` Manivannan Sadhasivam
2021-09-24  8:36   ` Kalle Valo
2021-09-24  8:36     ` Kalle Valo
2021-09-24  8:36     ` Kalle Valo
2021-09-24  9:43     ` Loic Poulain
2021-09-24  9:43       ` Loic Poulain
2021-09-24  9:43       ` Loic Poulain
2021-09-24 10:00       ` Manivannan Sadhasivam
2021-09-24 10:00         ` Manivannan Sadhasivam
2021-10-07  9:48       ` Kalle Valo
2021-10-07  9:48         ` Kalle Valo
2021-10-19 12:12         ` Kalle Valo
2021-10-19 12:12           ` Kalle Valo
2021-10-21 10:03           ` Manivannan Sadhasivam
2021-10-21 10:03             ` Manivannan Sadhasivam
2021-11-12 11:36             ` Thorsten Leemhuis
2021-11-12 11:36               ` Thorsten Leemhuis
2021-11-18 17:41             ` Manivannan Sadhasivam
2021-11-18 17:41               ` Manivannan Sadhasivam
2021-12-01  7:34               ` Thorsten Leemhuis
2021-12-01  7:34                 ` Thorsten Leemhuis
2021-09-25 17:40 ` Thorsten Leemhuis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8735putk82.fsf@codeaurora.org \
    --to=kvalo@codeaurora.org \
    --cc=ath11k@lists.infradead.org \
    --cc=cjhuang@codeaurora.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=loic.poulain@linaro.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=regressions@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.