* [PATCH v2 0/2] Fix incorrect USB3 phy parsing in tegra-xudc @ 2024-03-07 3:03 Wayne Chang 2024-03-07 3:03 ` [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy Wayne Chang 2024-03-07 3:03 ` [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic Wayne Chang 0 siblings, 2 replies; 6+ messages in thread From: Wayne Chang @ 2024-03-07 3:03 UTC (permalink / raw) To: waynec, jonathanh, thierry.reding, jckuo, vkoul, kishon, gregkh Cc: linux-phy, linux-tegra, linux-kernel, linux-usb The patch series introduces a new API for retrieving the port number of the PHY. And then utilized this API to address an issue in the USB3 PHY retrieval logic. Wayne Chang (2): phy: tegra: xusb: Add API to retrieve the port number of phy usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic drivers/phy/tegra/xusb.c | 13 ++++++++++ drivers/usb/gadget/udc/tegra-xudc.c | 39 ++++++++++++++++++----------- include/linux/phy/tegra/xusb.h | 1 + 3 files changed, 39 insertions(+), 14 deletions(-) base-commit: 90d35da658da8cff0d4ecbb5113f5fac9d00eb72 -- 2.25.1 -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy 2024-03-07 3:03 [PATCH v2 0/2] Fix incorrect USB3 phy parsing in tegra-xudc Wayne Chang @ 2024-03-07 3:03 ` Wayne Chang 2024-03-07 13:43 ` Jon Hunter 2024-04-05 16:59 ` Vinod Koul 2024-03-07 3:03 ` [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic Wayne Chang 1 sibling, 2 replies; 6+ messages in thread From: Wayne Chang @ 2024-03-07 3:03 UTC (permalink / raw) To: waynec, jonathanh, thierry.reding, jckuo, vkoul, kishon, gregkh Cc: linux-phy, linux-tegra, linux-kernel, linux-usb, stable This patch introduces a new API, tegra_xusb_padctl_get_port_number, to the Tegra XUSB Pad Controller driver. This API is used to identify the USB port that is associated with a given PHY. The function takes a PHY pointer for either a USB2 PHY or USB3 PHY as input and returns the corresponding port number. If the PHY pointer is invalid, it returns -ENODEV. Cc: stable@vger.kernel.org Signed-off-by: Wayne Chang <waynec@nvidia.com> --- V1 -> V2:cc stable drivers/phy/tegra/xusb.c | 13 +++++++++++++ include/linux/phy/tegra/xusb.h | 1 + 2 files changed, 14 insertions(+) diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c index 142ebe0247cc..983a6e6173bd 100644 --- a/drivers/phy/tegra/xusb.c +++ b/drivers/phy/tegra/xusb.c @@ -1531,6 +1531,19 @@ int tegra_xusb_padctl_get_usb3_companion(struct tegra_xusb_padctl *padctl, } EXPORT_SYMBOL_GPL(tegra_xusb_padctl_get_usb3_companion); +int tegra_xusb_padctl_get_port_number(struct phy *phy) +{ + struct tegra_xusb_lane *lane; + + if (!phy) + return -ENODEV; + + lane = phy_get_drvdata(phy); + + return lane->index; +} +EXPORT_SYMBOL_GPL(tegra_xusb_padctl_get_port_number); + MODULE_AUTHOR("Thierry Reding <treding@nvidia.com>"); MODULE_DESCRIPTION("Tegra XUSB Pad Controller driver"); MODULE_LICENSE("GPL v2"); diff --git a/include/linux/phy/tegra/xusb.h b/include/linux/phy/tegra/xusb.h index 70998e6dd6fd..6ca51e0080ec 100644 --- a/include/linux/phy/tegra/xusb.h +++ b/include/linux/phy/tegra/xusb.h @@ -26,6 +26,7 @@ void tegra_phy_xusb_utmi_pad_power_down(struct phy *phy); int tegra_phy_xusb_utmi_port_reset(struct phy *phy); int tegra_xusb_padctl_get_usb3_companion(struct tegra_xusb_padctl *padctl, unsigned int port); +int tegra_xusb_padctl_get_port_number(struct phy *phy); int tegra_xusb_padctl_enable_phy_sleepwalk(struct tegra_xusb_padctl *padctl, struct phy *phy, enum usb_device_speed speed); int tegra_xusb_padctl_disable_phy_sleepwalk(struct tegra_xusb_padctl *padctl, struct phy *phy); -- 2.25.1 -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy 2024-03-07 3:03 ` [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy Wayne Chang @ 2024-03-07 13:43 ` Jon Hunter 2024-04-05 16:59 ` Vinod Koul 1 sibling, 0 replies; 6+ messages in thread From: Jon Hunter @ 2024-03-07 13:43 UTC (permalink / raw) To: Wayne Chang, thierry.reding, jckuo, vkoul, kishon, gregkh Cc: linux-phy, linux-tegra, linux-kernel, linux-usb, stable On 07/03/2024 03:03, Wayne Chang wrote: > This patch introduces a new API, tegra_xusb_padctl_get_port_number, > to the Tegra XUSB Pad Controller driver. This API is used to identify > the USB port that is associated with a given PHY. > > The function takes a PHY pointer for either a USB2 PHY or USB3 PHY as input > and returns the corresponding port number. If the PHY pointer is invalid, > it returns -ENODEV. > > Cc: stable@vger.kernel.org > Signed-off-by: Wayne Chang <waynec@nvidia.com> > --- > V1 -> V2:cc stable > drivers/phy/tegra/xusb.c | 13 +++++++++++++ > include/linux/phy/tegra/xusb.h | 1 + > 2 files changed, 14 insertions(+) > > diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c > index 142ebe0247cc..983a6e6173bd 100644 > --- a/drivers/phy/tegra/xusb.c > +++ b/drivers/phy/tegra/xusb.c > @@ -1531,6 +1531,19 @@ int tegra_xusb_padctl_get_usb3_companion(struct tegra_xusb_padctl *padctl, > } > EXPORT_SYMBOL_GPL(tegra_xusb_padctl_get_usb3_companion); > > +int tegra_xusb_padctl_get_port_number(struct phy *phy) > +{ > + struct tegra_xusb_lane *lane; > + > + if (!phy) > + return -ENODEV; > + > + lane = phy_get_drvdata(phy); > + > + return lane->index; > +} > +EXPORT_SYMBOL_GPL(tegra_xusb_padctl_get_port_number); > + > MODULE_AUTHOR("Thierry Reding <treding@nvidia.com>"); > MODULE_DESCRIPTION("Tegra XUSB Pad Controller driver"); > MODULE_LICENSE("GPL v2"); > diff --git a/include/linux/phy/tegra/xusb.h b/include/linux/phy/tegra/xusb.h > index 70998e6dd6fd..6ca51e0080ec 100644 > --- a/include/linux/phy/tegra/xusb.h > +++ b/include/linux/phy/tegra/xusb.h > @@ -26,6 +26,7 @@ void tegra_phy_xusb_utmi_pad_power_down(struct phy *phy); > int tegra_phy_xusb_utmi_port_reset(struct phy *phy); > int tegra_xusb_padctl_get_usb3_companion(struct tegra_xusb_padctl *padctl, > unsigned int port); > +int tegra_xusb_padctl_get_port_number(struct phy *phy); > int tegra_xusb_padctl_enable_phy_sleepwalk(struct tegra_xusb_padctl *padctl, struct phy *phy, > enum usb_device_speed speed); > int tegra_xusb_padctl_disable_phy_sleepwalk(struct tegra_xusb_padctl *padctl, struct phy *phy); Reviewed-by: Jon Hunter <jonathanh@nvidia.com> Tested-by: Jon Hunter <jonathanh@nvidia.com> Thanks! Jon -- nvpublic -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy 2024-03-07 3:03 ` [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy Wayne Chang 2024-03-07 13:43 ` Jon Hunter @ 2024-04-05 16:59 ` Vinod Koul 1 sibling, 0 replies; 6+ messages in thread From: Vinod Koul @ 2024-04-05 16:59 UTC (permalink / raw) To: Wayne Chang Cc: jonathanh, thierry.reding, jckuo, kishon, gregkh, linux-phy, linux-tegra, linux-kernel, linux-usb, stable On 07-03-24, 11:03, Wayne Chang wrote: > This patch introduces a new API, tegra_xusb_padctl_get_port_number, > to the Tegra XUSB Pad Controller driver. This API is used to identify > the USB port that is associated with a given PHY. > > The function takes a PHY pointer for either a USB2 PHY or USB3 PHY as input > and returns the corresponding port number. If the PHY pointer is invalid, > it returns -ENODEV. Acked-by: Vinod Koul <vkoul@kernel.org> -- ~Vinod -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic 2024-03-07 3:03 [PATCH v2 0/2] Fix incorrect USB3 phy parsing in tegra-xudc Wayne Chang 2024-03-07 3:03 ` [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy Wayne Chang @ 2024-03-07 3:03 ` Wayne Chang 2024-03-07 13:43 ` Jon Hunter 1 sibling, 1 reply; 6+ messages in thread From: Wayne Chang @ 2024-03-07 3:03 UTC (permalink / raw) To: waynec, jonathanh, thierry.reding, jckuo, vkoul, kishon, gregkh Cc: linux-phy, linux-tegra, linux-kernel, linux-usb, stable This commit resolves an issue in the tegra-xudc USB gadget driver that incorrectly fetched USB3 PHY instances. The problem stemmed from the assumption of a one-to-one correspondence between USB2 and USB3 PHY names and their association with physical USB ports in the device tree. Previously, the driver associated USB3 PHY names directly with the USB3 instance number, leading to mismatches when mapping the physical USB ports. For instance, if using USB3-1 PHY, the driver expect the corresponding PHY name as 'usb3-1'. However, the physical USB ports in the device tree were designated as USB2-0 and USB3-0 as we only have one device controller, causing a misalignment. This commit rectifies the issue by adjusting the PHY naming logic. Now, the driver correctly correlates the USB2 and USB3 PHY instances, allowing the USB2-0 and USB3-1 PHYs to form a physical USB port pair while accurately reflecting their configuration in the device tree by naming them USB2-0 and USB3-0, respectively. The change ensures that the PHY and PHY names align appropriately, resolving the mismatch between physical USB ports and their associated names in the device tree. Fixes: b4e19931c98a ("usb: gadget: tegra-xudc: Support multiple device modes") Cc: stable@vger.kernel.org Signed-off-by: Wayne Chang <waynec@nvidia.com> --- V1 -> V2:no change drivers/usb/gadget/udc/tegra-xudc.c | 39 ++++++++++++++++++----------- 1 file changed, 25 insertions(+), 14 deletions(-) diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c index cb85168fd00c..7aa46d426f31 100644 --- a/drivers/usb/gadget/udc/tegra-xudc.c +++ b/drivers/usb/gadget/udc/tegra-xudc.c @@ -3491,8 +3491,8 @@ static void tegra_xudc_device_params_init(struct tegra_xudc *xudc) static int tegra_xudc_phy_get(struct tegra_xudc *xudc) { - int err = 0, usb3; - unsigned int i; + int err = 0, usb3_companion_port; + unsigned int i, j; xudc->utmi_phy = devm_kcalloc(xudc->dev, xudc->soc->num_phys, sizeof(*xudc->utmi_phy), GFP_KERNEL); @@ -3520,7 +3520,7 @@ static int tegra_xudc_phy_get(struct tegra_xudc *xudc) if (IS_ERR(xudc->utmi_phy[i])) { err = PTR_ERR(xudc->utmi_phy[i]); dev_err_probe(xudc->dev, err, - "failed to get usb2-%d PHY\n", i); + "failed to get PHY for phy-name usb2-%d\n", i); goto clean_up; } else if (xudc->utmi_phy[i]) { /* Get usb-phy, if utmi phy is available */ @@ -3539,19 +3539,30 @@ static int tegra_xudc_phy_get(struct tegra_xudc *xudc) } /* Get USB3 phy */ - usb3 = tegra_xusb_padctl_get_usb3_companion(xudc->padctl, i); - if (usb3 < 0) + usb3_companion_port = tegra_xusb_padctl_get_usb3_companion(xudc->padctl, i); + if (usb3_companion_port < 0) continue; - snprintf(phy_name, sizeof(phy_name), "usb3-%d", usb3); - xudc->usb3_phy[i] = devm_phy_optional_get(xudc->dev, phy_name); - if (IS_ERR(xudc->usb3_phy[i])) { - err = PTR_ERR(xudc->usb3_phy[i]); - dev_err_probe(xudc->dev, err, - "failed to get usb3-%d PHY\n", usb3); - goto clean_up; - } else if (xudc->usb3_phy[i]) - dev_dbg(xudc->dev, "usb3-%d PHY registered", usb3); + for (j = 0; j < xudc->soc->num_phys; j++) { + snprintf(phy_name, sizeof(phy_name), "usb3-%d", j); + xudc->usb3_phy[i] = devm_phy_optional_get(xudc->dev, phy_name); + if (IS_ERR(xudc->usb3_phy[i])) { + err = PTR_ERR(xudc->usb3_phy[i]); + dev_err_probe(xudc->dev, err, + "failed to get PHY for phy-name usb3-%d\n", j); + goto clean_up; + } else if (xudc->usb3_phy[i]) { + int usb2_port = + tegra_xusb_padctl_get_port_number(xudc->utmi_phy[i]); + int usb3_port = + tegra_xusb_padctl_get_port_number(xudc->usb3_phy[i]); + if (usb3_port == usb3_companion_port) { + dev_dbg(xudc->dev, "USB2 port %d is paired with USB3 port %d for device mode port %d\n", + usb2_port, usb3_port, i); + break; + } + } + } } return err; -- 2.25.1 -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic 2024-03-07 3:03 ` [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic Wayne Chang @ 2024-03-07 13:43 ` Jon Hunter 0 siblings, 0 replies; 6+ messages in thread From: Jon Hunter @ 2024-03-07 13:43 UTC (permalink / raw) To: Wayne Chang, thierry.reding, jckuo, vkoul, kishon, gregkh Cc: linux-phy, linux-tegra, linux-kernel, linux-usb, stable On 07/03/2024 03:03, Wayne Chang wrote: > This commit resolves an issue in the tegra-xudc USB gadget driver that > incorrectly fetched USB3 PHY instances. The problem stemmed from the > assumption of a one-to-one correspondence between USB2 and USB3 PHY > names and their association with physical USB ports in the device tree. > > Previously, the driver associated USB3 PHY names directly with the USB3 > instance number, leading to mismatches when mapping the physical USB > ports. For instance, if using USB3-1 PHY, the driver expect the > corresponding PHY name as 'usb3-1'. However, the physical USB ports in > the device tree were designated as USB2-0 and USB3-0 as we only have > one device controller, causing a misalignment. > > This commit rectifies the issue by adjusting the PHY naming logic. > Now, the driver correctly correlates the USB2 and USB3 PHY instances, > allowing the USB2-0 and USB3-1 PHYs to form a physical USB port pair > while accurately reflecting their configuration in the device tree by > naming them USB2-0 and USB3-0, respectively. > > The change ensures that the PHY and PHY names align appropriately, > resolving the mismatch between physical USB ports and their associated > names in the device tree. > > Fixes: b4e19931c98a ("usb: gadget: tegra-xudc: Support multiple device modes") > Cc: stable@vger.kernel.org > Signed-off-by: Wayne Chang <waynec@nvidia.com> > --- > V1 -> V2:no change > drivers/usb/gadget/udc/tegra-xudc.c | 39 ++++++++++++++++++----------- > 1 file changed, 25 insertions(+), 14 deletions(-) > > diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c > index cb85168fd00c..7aa46d426f31 100644 > --- a/drivers/usb/gadget/udc/tegra-xudc.c > +++ b/drivers/usb/gadget/udc/tegra-xudc.c > @@ -3491,8 +3491,8 @@ static void tegra_xudc_device_params_init(struct tegra_xudc *xudc) > > static int tegra_xudc_phy_get(struct tegra_xudc *xudc) > { > - int err = 0, usb3; > - unsigned int i; > + int err = 0, usb3_companion_port; > + unsigned int i, j; > > xudc->utmi_phy = devm_kcalloc(xudc->dev, xudc->soc->num_phys, > sizeof(*xudc->utmi_phy), GFP_KERNEL); > @@ -3520,7 +3520,7 @@ static int tegra_xudc_phy_get(struct tegra_xudc *xudc) > if (IS_ERR(xudc->utmi_phy[i])) { > err = PTR_ERR(xudc->utmi_phy[i]); > dev_err_probe(xudc->dev, err, > - "failed to get usb2-%d PHY\n", i); > + "failed to get PHY for phy-name usb2-%d\n", i); > goto clean_up; > } else if (xudc->utmi_phy[i]) { > /* Get usb-phy, if utmi phy is available */ > @@ -3539,19 +3539,30 @@ static int tegra_xudc_phy_get(struct tegra_xudc *xudc) > } > > /* Get USB3 phy */ > - usb3 = tegra_xusb_padctl_get_usb3_companion(xudc->padctl, i); > - if (usb3 < 0) > + usb3_companion_port = tegra_xusb_padctl_get_usb3_companion(xudc->padctl, i); > + if (usb3_companion_port < 0) > continue; > > - snprintf(phy_name, sizeof(phy_name), "usb3-%d", usb3); > - xudc->usb3_phy[i] = devm_phy_optional_get(xudc->dev, phy_name); > - if (IS_ERR(xudc->usb3_phy[i])) { > - err = PTR_ERR(xudc->usb3_phy[i]); > - dev_err_probe(xudc->dev, err, > - "failed to get usb3-%d PHY\n", usb3); > - goto clean_up; > - } else if (xudc->usb3_phy[i]) > - dev_dbg(xudc->dev, "usb3-%d PHY registered", usb3); > + for (j = 0; j < xudc->soc->num_phys; j++) { > + snprintf(phy_name, sizeof(phy_name), "usb3-%d", j); > + xudc->usb3_phy[i] = devm_phy_optional_get(xudc->dev, phy_name); > + if (IS_ERR(xudc->usb3_phy[i])) { > + err = PTR_ERR(xudc->usb3_phy[i]); > + dev_err_probe(xudc->dev, err, > + "failed to get PHY for phy-name usb3-%d\n", j); > + goto clean_up; > + } else if (xudc->usb3_phy[i]) { > + int usb2_port = > + tegra_xusb_padctl_get_port_number(xudc->utmi_phy[i]); > + int usb3_port = > + tegra_xusb_padctl_get_port_number(xudc->usb3_phy[i]); > + if (usb3_port == usb3_companion_port) { > + dev_dbg(xudc->dev, "USB2 port %d is paired with USB3 port %d for device mode port %d\n", > + usb2_port, usb3_port, i); > + break; > + } > + } > + } > } > > return err; Reviewed-by: Jon Hunter <jonathanh@nvidia.com> Tested-by: Jon Hunter <jonathanh@nvidia.com> Thanks! Jon -- nvpublic -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-04-05 16:59 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-03-07 3:03 [PATCH v2 0/2] Fix incorrect USB3 phy parsing in tegra-xudc Wayne Chang 2024-03-07 3:03 ` [PATCH v2 1/2] phy: tegra: xusb: Add API to retrieve the port number of phy Wayne Chang 2024-03-07 13:43 ` Jon Hunter 2024-04-05 16:59 ` Vinod Koul 2024-03-07 3:03 ` [PATCH v2 2/2] usb: gadget: tegra-xudc: Fix USB3 PHY retrieval logic Wayne Chang 2024-03-07 13:43 ` Jon Hunter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).