On Thu, Apr 14, 2016 at 01:35:14PM -0500, David Lechner wrote:
The initial use for this is for PHYs that have a mode related to USB OTG.
There are several SoCs (e.g. TI OMAP and DA8xx) that have a mode setting
in the USB PHY to override OTG VBUS and ID signals.
Of course, the enum can be expaned in the future to include modes for
other types of PHYs as well.
Suggested-by: Kishon Vijay Abraham I <kishon@xxxxxx>
Signed-off-by: David Lechner <david@xxxxxxxxxxxxxx>
* This is a new patch to avoid exporting a symbol from the phy-da8xx-usb driver.
drivers/phy/phy-core.c | 15 +++++++++++++++
include/linux/phy/phy.h | 15 +++++++++++++++
2 files changed, 30 insertions(+)
diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index e7e574d..fe0344c 100644
@@ -342,6 +342,21 @@ int phy_power_off(struct phy *phy)
+int phy_set_mode(struct phy *phy, enum phy_mode mode)
+ int ret;
+ if (!phy || !phy->ops->set_mode)
+ return 0;
+ ret = phy->ops->set_mode(phy, mode);
+ return ret;
Sorry for my late comments, have been busy on other things.
As I commented in v2, isn't it a better idea to not adding this callback
and let the da8xx phy driver set the mode register in _probe() based on
musb core only calls the *optional* _set_mode() during init, so I don't
see any problem if the phy set the mode in its probe.