On Thursday 24 December 2020 13:15:21 Peter Chen wrote:
>
> > > > + /* Old bindings miss the PHY handle */
> > > > + phy = of_phy_get(dev->of_node, "usb3-phy");
> > > > + if (IS_ERR(phy) && PTR_ERR(phy) == -EPROBE_DEFER)
> > > > + return -EPROBE_DEFER;
> > >
> > > Doesn't need to judge IS_ERR(phy).
> >
> > Ok, I can remove it. I used same condition which is already in SATA and PCIe
> > phy code.
> >
> > > > + else if (IS_ERR(phy))
> > > > + goto phy_out;
> > > > +
> > > > + ret = phy_init(phy);
> > > > + if (ret)
> > > > + goto phy_put;
> > > > +
> > > > + ret = phy_set_mode(phy, PHY_MODE_USB_HOST_SS);
> > > > + if (ret)
> > > > + goto phy_exit;
> > > > +
> > > > + ret = phy_power_on(phy);
> > > > + if (ret == -EOPNOTSUPP) {
> > > > + /* Skip initializatin of XHCI PHY when it is unsupported by
> > firmware */
> > > > + dev_warn(dev, "PHY unsupported by firmware\n");
> > > > + xhci->quirks |= XHCI_SKIP_PHY_INIT;
> > > > + }
> > > > + if (ret)
> > > > + goto phy_exit;
> > > > +
> > > > + phy_power_off(phy);
> > > > +phy_exit:
> > > > + phy_exit(phy);
> > > > +phy_put:
> > > > + of_phy_put(phy);
> > > > +phy_out:
> > > > +
> > >
> > > You do power on and off again only want to know if PHY has already
> > > powered at ATF, right?
> >
> > I need to know if power on/off procedure is supported by ATF. And if not
> > (indicated by -EOPNOTSUPP) then I need to ensure that usb hdc code would not
> > try to call phy_power_on() as it would cause failure as described in the commit
> > message. You can look at those other two commits for PCIe and SATA. Same
> > thing is needed for USB.
>
> If not supported by ATF, then where to power on and off PHY since no other place calls PHY APIs? Is it always on?
Yes, in this case (when -EOPNOTSUPP is returned) SMC API is not
supported by ATF, and PHY is always on.
