Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/8/22 16:34, Michal Simek wrote: On 3/8/22 16:32, Marek Vasut wrote: On 3/8/22 14:46, Michal Simek wrote: On 3/8/22 14:39, Marek Vasut wrote: On 3/8/22 14:21, Michal Simek wrote: On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com I'm confused, is this a patch I should try to apply somehow ? It should be applied before that dwc3 patch. If you don't want to apply it via your usb tree I can take it via my tree or ask Tom to take it. Please just collect the patches in the right order and resubmit them. They were sent in the right order. 1/2 header fix, 2/2 dwc3 fix. Both of them are in patchwork. I received random lore patch link here, another half of series in another thread and now I should look up yet more patches in patchwork. I am really confused, I do not know what you expect me to do about this, randomly try to chase down patches in multiple patch trackers, try to apply them somehow, and likely get it all wrong? Please, collect and resubmit the patches which you want applied, that way we can be certain I apply the right patches in the right order.
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/8/22 16:32, Marek Vasut wrote: On 3/8/22 14:46, Michal Simek wrote: On 3/8/22 14:39, Marek Vasut wrote: On 3/8/22 14:21, Michal Simek wrote: On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com I'm confused, is this a patch I should try to apply somehow ? It should be applied before that dwc3 patch. If you don't want to apply it via your usb tree I can take it via my tree or ask Tom to take it. Please just collect the patches in the right order and resubmit them. They were sent in the right order. 1/2 header fix, 2/2 dwc3 fix. Both of them are in patchwork. M
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/8/22 14:46, Michal Simek wrote: On 3/8/22 14:39, Marek Vasut wrote: On 3/8/22 14:21, Michal Simek wrote: On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com I'm confused, is this a patch I should try to apply somehow ? It should be applied before that dwc3 patch. If you don't want to apply it via your usb tree I can take it via my tree or ask Tom to take it. Please just collect the patches in the right order and resubmit them.
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/8/22 14:39, Marek Vasut wrote: On 3/8/22 14:21, Michal Simek wrote: On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com I'm confused, is this a patch I should try to apply somehow ? It should be applied before that dwc3 patch. If you don't want to apply it via your usb tree I can take it via my tree or ask Tom to take it. Thanks, Michal
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/8/22 14:21, Michal Simek wrote: On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com I'm confused, is this a patch I should try to apply somehow ?
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/4/22 18:33, Marek Vasut wrote: On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461 This patch is fixing the problem https://lore.kernel.org/r/eabd31079152659556432fed561cd3a98300c8f5.1646745547.git.michal.si...@xilinx.com Thanks, Michal
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/4/22 05:42, Marek Vasut wrote: On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks. Fails to build, please fix. Patch dropped: https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/399461
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 3/3/22 09:40, Bin Meng wrote: On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: When usb3-phy label is found, PHY driver is called and serdes line is initialized. This is preparation for serdes/psgtr driver to configure GT lines based on description in DT. Signed-off-by: Michal Simek --- drivers/usb/dwc3/dwc3-generic.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Bin Meng Applied, thanks.
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On Wed, Feb 23, 2022 at 10:56 PM Michal Simek wrote: > > When usb3-phy label is found, PHY driver is called and serdes line is > initialized. This is preparation for serdes/psgtr driver to configure GT > lines based on description in DT. > > Signed-off-by: Michal Simek > --- > > drivers/usb/dwc3/dwc3-generic.c | 17 + > 1 file changed, 17 insertions(+) > Reviewed-by: Bin Meng
Re: [PATCH] usb: dwc3: Add support for usb3-phy PHY configuration
On 2/23/22 15:56, Michal Simek wrote:
When usb3-phy label is found, PHY driver is called and serdes line is
initialized. This is preparation for serdes/psgtr driver to configure GT
lines based on description in DT.
Signed-off-by: Michal Simek
---
drivers/usb/dwc3/dwc3-generic.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/usb/dwc3/dwc3-generic.c b/drivers/usb/dwc3/dwc3-generic.c
index 01bd0ca190e3..b1e019b5b91a 100644
--- a/drivers/usb/dwc3/dwc3-generic.c
+++ b/drivers/usb/dwc3/dwc3-generic.c
@@ -409,6 +409,17 @@ static int dwc3_glue_probe(struct udevice *dev)
struct udevice *child = NULL;
int index = 0;
int ret;
+ struct phy phy;
+
+ ret = generic_phy_get_by_name(dev, "usb3-phy", &phy);
+ if (!ret) {
+ ret = generic_phy_init(&phy);
+ if (ret)
+ return ret;
+ } else if (ret != -ENOENT) {
+ debug("could not get phy (err %d)\n", ret);
+ return ret;
+ }
glue->regs = dev_read_addr(dev);
@@ -420,6 +431,12 @@ static int dwc3_glue_probe(struct udevice *dev)
if (ret)
return ret;
+ if (phy.dev) {
+ ret = generic_phy_power_on(&phy);
+ if (ret)
+ return ret;
+ }
+
ret = device_find_first_child(dev, &child);
if (ret)
return ret;
CC Bin please.
