This is a note to let you know that I've just added the patch titled Revert "usb: dwc3: disable USB core PHY management" to the 6.0-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: revert-usb-dwc3-disable-usb-core-phy-management.patch and it can be found in the queue-6.0 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From 5c294de36e7fb3e0cba0c4e1ef9a5f57bc080d0f Mon Sep 17 00:00:00 2001 From: Johan Hovold <johan+linaro@xxxxxxxxxx> Date: Thu, 3 Nov 2022 15:46:48 +0100 Subject: Revert "usb: dwc3: disable USB core PHY management" From: Johan Hovold <johan+linaro@xxxxxxxxxx> commit 5c294de36e7fb3e0cba0c4e1ef9a5f57bc080d0f upstream. This reverts commit 6000b8d900cd5f52fbcd0776d0cc396e88c8c2ea. The offending commit disabled the USB core PHY management as the dwc3 already manages the PHYs in question. Unfortunately some platforms have started relying on having USB core also controlling the PHY and this is specifically currently needed on some Exynos platforms for PHY calibration or connected device may fail to enumerate. The PHY calibration was previously handled in the dwc3 driver, but to work around some issues related to how the dwc3 driver interacts with xhci (e.g. using multiple drivers) this was moved to USB core by commits 34c7ed72f4f0 ("usb: core: phy: add support for PHY calibration") and a0a465569b45 ("usb: dwc3: remove generic PHY calibrate() calls"). The same PHY obviously should not be controlled from two different places, which for example do no agree on the PHY mode or power state during suspend, but as the offending patch was backported to stable, let's revert it for now. Reported-by: Stefan Agner <stefan@xxxxxxxx> Link: https://lore.kernel.org/lkml/808bdba846bb60456adf10a3016911ee@xxxxxxxx/ Fixes: 6000b8d900cd ("usb: dwc3: disable USB core PHY management") Cc: stable@xxxxxxxxxxxxxxx Signed-off-by: Johan Hovold <johan+linaro@xxxxxxxxxx> Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> Acked-by: Thinh Nguyen <Thinh.Nguyen@xxxxxxxxxxxx> Link: https://lore.kernel.org/r/20221103144648.14197-1-johan+linaro@xxxxxxxxxx Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/usb/dwc3/host.c | 10 ---------- 1 file changed, 10 deletions(-) --- a/drivers/usb/dwc3/host.c +++ b/drivers/usb/dwc3/host.c @@ -11,13 +11,8 @@ #include <linux/of.h> #include <linux/platform_device.h> -#include "../host/xhci-plat.h" #include "core.h" -static const struct xhci_plat_priv dwc3_xhci_plat_priv = { - .quirks = XHCI_SKIP_PHY_INIT, -}; - static void dwc3_host_fill_xhci_irq_res(struct dwc3 *dwc, int irq, char *name) { @@ -97,11 +92,6 @@ int dwc3_host_init(struct dwc3 *dwc) goto err; } - ret = platform_device_add_data(xhci, &dwc3_xhci_plat_priv, - sizeof(dwc3_xhci_plat_priv)); - if (ret) - goto err; - memset(props, 0, sizeof(struct property_entry) * ARRAY_SIZE(props)); if (dwc->usb3_lpm_capable) Patches currently in stable-queue which might be from johan+linaro@xxxxxxxxxx are queue-6.0/arm64-dts-qcom-sc8280xp-fix-ufs_card_phy-ref-clock.patch queue-6.0/arm64-dts-qcom-sc8280xp-crd-specify-which-ldo-modes-.patch queue-6.0/arm64-dts-qcom-sc8280xp-fix-usb-phy-pcs-registers.patch queue-6.0/arm64-dts-qcom-sc8280xp-fix-ufs-phy-serdes-size.patch queue-6.0/arm64-dts-qcom-sc8280xp-correct-ref-clock-for-ufs_me.patch queue-6.0/arm64-dts-qcom-sa8295p-adp-specify-which-ldo-modes-a.patch queue-6.0/arm64-dts-qcom-sc8280xp-drop-broken-dp-phy-nodes.patch queue-6.0/arm64-dts-qcom-sc8280xp-fix-usb0-phy-pcs_misc-regist.patch queue-6.0/arm64-dts-qcom-sc8280xp-fix-usb1-phy-rx1-registers.patch queue-6.0/revert-usb-dwc3-disable-usb-core-phy-management.patch