diff mbox series

usb: dwc3: qcom: skip interconnect init for ACPI probe

Message ID 20210311060318.25418-1-shawn.guo@linaro.org
State Accepted
Commit 5e4010e36a58978e42b2ee13739ff9b50209c830
Headers show
Series usb: dwc3: qcom: skip interconnect init for ACPI probe | expand

Commit Message

Shawn Guo March 11, 2021, 6:03 a.m. UTC
The ACPI probe starts failing since commit bea46b981515 ("usb: dwc3:
qcom: Add interconnect support in dwc3 driver"), because there is no
interconnect support for ACPI, and of_icc_get() call in
dwc3_qcom_interconnect_init() will just return -EINVAL.

Fix the problem by skipping interconnect init for ACPI probe, and then
the NULL icc_path_ddr will simply just scheild all ICC calls.

Fixes: bea46b981515 ("usb: dwc3: qcom: Add interconnect support in dwc3 driver")
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
 drivers/usb/dwc3/dwc3-qcom.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Sergei Shtylyov March 11, 2021, 8:32 a.m. UTC | #1
Hello!

On 11.03.2021 9:03, Shawn Guo wrote:

> The ACPI probe starts failing since commit bea46b981515 ("usb: dwc3:
> qcom: Add interconnect support in dwc3 driver"), because there is no
> interconnect support for ACPI, and of_icc_get() call in
> dwc3_qcom_interconnect_init() will just return -EINVAL.
> 
> Fix the problem by skipping interconnect init for ACPI probe, and then
> the NULL icc_path_ddr will simply just scheild all ICC calls.

    Scheild?

> Fixes: bea46b981515 ("usb: dwc3: qcom: Add interconnect support in dwc3 driver")
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
[...]

MBR, Sergei
patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #2
Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

On Thu, 11 Mar 2021 14:03:18 +0800 you wrote:
> The ACPI probe starts failing since commit bea46b981515 ("usb: dwc3:

> qcom: Add interconnect support in dwc3 driver"), because there is no

> interconnect support for ACPI, and of_icc_get() call in

> dwc3_qcom_interconnect_init() will just return -EINVAL.

> 

> Fix the problem by skipping interconnect init for ACPI probe, and then

> the NULL icc_path_ddr will simply just scheild all ICC calls.

> 

> [...]


Here is the summary with links:
  - usb: dwc3: qcom: skip interconnect init for ACPI probe
    https://git.kernel.org/qcom/c/5e4010e36a58

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/drivers/usb/dwc3/dwc3-qcom.c b/drivers/usb/dwc3/dwc3-qcom.c
index 175a609cd0af..7dc74885a635 100644
--- a/drivers/usb/dwc3/dwc3-qcom.c
+++ b/drivers/usb/dwc3/dwc3-qcom.c
@@ -244,6 +244,9 @@  static int dwc3_qcom_interconnect_init(struct dwc3_qcom *qcom)
 	struct device *dev = qcom->dev;
 	int ret;
 
+	if (has_acpi_companion(dev))
+		return 0;
+
 	qcom->icc_path_ddr = of_icc_get(dev, "usb-ddr");
 	if (IS_ERR(qcom->icc_path_ddr)) {
 		dev_err(dev, "failed to get usb-ddr path: %ld\n",