2021-03-31 18:38:44

by Bhaumik Bhatt

[permalink] [raw]
Subject: [PATCH v6 0/2] Polling for MHI ready

v6:
-Fix return error code for mhi_poll_reg_field()
-Do not recode when returning from mhi_ready_state_transition() if polling fails

v5:
-Use fsleep in place of udelay or usleep_range to accommodate better delay use
-Drop patch for polling during RDDM panic path as new API cannot be used there

v4:
-Added reviewed-by tag
-Return appropriate error code from mhi_poll_reg_field()
-Fixed bug where mhi_poll_reg_field() returns success if polling times out
-Added an interval_us variable in mhi_ready_state_transition()

v3:
-Removed config changes that crept in in the first patch

v2:
-Addressed review comments
-Introduce new patch for to use controller defined read_reg() for polling
-Add usage in RDDM download panic path as well

Bhaumik Bhatt (2):
bus: mhi: core: Introduce internal register poll helper function
bus: mhi: core: Move to polling method to wait for MHI ready

drivers/bus/mhi/core/internal.h | 3 +++
drivers/bus/mhi/core/main.c | 23 +++++++++++++++++++++++
drivers/bus/mhi/core/pm.c | 34 ++++++++++++++++------------------
3 files changed, 42 insertions(+), 18 deletions(-)

--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


2021-03-31 18:43:02

by Manivannan Sadhasivam

[permalink] [raw]
Subject: Re: [PATCH v6 0/2] Polling for MHI ready

On Wed, Mar 31, 2021 at 11:34:23AM -0700, Bhaumik Bhatt wrote:
> v6:
> -Fix return error code for mhi_poll_reg_field()
> -Do not recode when returning from mhi_ready_state_transition() if polling fails
>
> v5:
> -Use fsleep in place of udelay or usleep_range to accommodate better delay use
> -Drop patch for polling during RDDM panic path as new API cannot be used there
>
> v4:
> -Added reviewed-by tag
> -Return appropriate error code from mhi_poll_reg_field()
> -Fixed bug where mhi_poll_reg_field() returns success if polling times out
> -Added an interval_us variable in mhi_ready_state_transition()
>
> v3:
> -Removed config changes that crept in in the first patch
>
> v2:
> -Addressed review comments
> -Introduce new patch for to use controller defined read_reg() for polling
> -Add usage in RDDM download panic path as well
>
> Bhaumik Bhatt (2):
> bus: mhi: core: Introduce internal register poll helper function
> bus: mhi: core: Move to polling method to wait for MHI ready
>

Applied to mhi-next!

Thanks,
Mani

> drivers/bus/mhi/core/internal.h | 3 +++
> drivers/bus/mhi/core/main.c | 23 +++++++++++++++++++++++
> drivers/bus/mhi/core/pm.c | 34 ++++++++++++++++------------------
> 3 files changed, 42 insertions(+), 18 deletions(-)
>
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>