On 13/06/16 11:13, Jun Li wrote: > Hi > >> diff --git a/drivers/usb/common/common.c b/drivers/usb/common/common.c >> index e3d0161..d7ec471 100644 >> --- a/drivers/usb/common/common.c >> +++ b/drivers/usb/common/common.c >> @@ -238,6 +238,33 @@ int of_usb_update_otg_caps(struct device_node *np, } >> EXPORT_SYMBOL_GPL(of_usb_update_otg_caps); >> >> +#ifdef CONFIG_USB_OTG >> +/** >> + * of_usb_get_otg - get the OTG controller linked to the USB controller >> + * @np: Pointer to the device_node of the USB controller >> + * @otg_caps: Pointer to the target usb_otg_caps to be set > > Remove otg_caps. Sure. > >> + * >> + * Returns the OTG controller device or NULL on error. >> + */ >> +struct device *of_usb_get_otg(struct device_node *np) { >> + struct device_node *otg_np; >> + struct platform_device *pdev; >> + >> + otg_np = of_parse_phandle(np, "otg-controller", 0); >> + if (!otg_np) >> + return NULL; >> + >> + pdev = of_find_device_by_node(otg_np); >> + of_node_put(otg_np); >> + if (!pdev) >> + return NULL; >> + >> + return &pdev->dev; >> +} >> +EXPORT_SYMBOL_GPL(of_usb_get_otg); >> +#endif >> + >> #endif >> >> MODULE_LICENSE("GPL"); >> diff --git a/include/linux/usb/of.h b/include/linux/usb/of.h index >> de3237f..499a4e8 100644 >> --- a/include/linux/usb/of.h >> +++ b/include/linux/usb/of.h >> @@ -40,6 +40,15 @@ static inline struct device_node >> *usb_of_get_child_node } #endif >> >> +#if IS_ENABLED(CONFIG_OF) && IS_ENABLED(CONFIG_USB_OTG) struct device >> +*of_usb_get_otg(struct device_node *np); #else static inline struct >> +device *of_usb_get_otg(struct device_node *np) { >> + return NULL; >> +} >> +#endif >> + >> #if IS_ENABLED(CONFIG_OF) && IS_ENABLED(CONFIG_USB_SUPPORT) enum >> usb_phy_interface of_usb_get_phy_mode(struct device_node *np); #else >> -- >> 2.7.4 > -- cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html