Hello, On Thu, Jun 27, 2019 at 03:15:10AM +0000, Peter Chen wrote: > > > On 19-06-26 02:40, Peter Chen wrote: > > > > > > > Subject: [PATCH] ARM: imx25: provide a fixed regulator for usb phys > > > > > > > > The usb phys are internal to the SoC and so it their 5V supply. With > > > > this regulator added explicitly the following (harmless) boot messages > > > > go away: > > > > > > > > usb_phy_generic usbphy:usb-phy@0: usbphy:usb-phy@0 supply vcc > > > > not found, using dummy regulator > > > > usb_phy_generic usbphy:usb-phy@1: usbphy:usb-phy@1 supply vcc > > > > not found, using dummy regulator > > > > > > > > > > To eliminate the warning message, I suggest doing below changes, as > > > vcc supply is not mandatory. > > > > > > diff --git a/drivers/usb/phy/phy-generic.c > > > b/drivers/usb/phy/phy-generic.c index a53b89be5324..01a5ff1a0515 > > > 100644 > > > --- a/drivers/usb/phy/phy-generic.c > > > +++ b/drivers/usb/phy/phy-generic.c > > > @@ -275,7 +275,7 @@ int usb_phy_gen_create_phy(struct device *dev, struct > > > usb_phy_generic *nop, > > > } > > > } > > > > > > - nop->vcc = devm_regulator_get(dev, "vcc"); > > > + nop->vcc = devm_regulator_get_optional(dev, "vcc"); > > > > Is the regulator optional? IMHO this shouldn't be the fix. I think the > > right fix is Uwe's > > approach. > > > > Add Felipe. > > Some USB PHY's power are from the core system's power (eg, DDR), and some are > fixed at the board and no switch for it. So, it is transparent for software > at some cases.
It's not clear to me how to proceed. There are two opposing opinions and I don't know enough about USB on mx25 to judge myself. Felipe? Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |
