On Tue, Sep 15, 2015 at 11:24 PM, Peter Chen <peter.chen@xxxxxxxxxxxxx> wrote: > but drivers/usb/gadget/udc/fsl_mxc_udc.c has three clocks Ok, looking at the clock driver I see that they used to associate three clocks with mxc-ehci: clk_register_clkdev(clk[usb_div], "per", "mxc-ehci.0"); clk_register_clkdev(clk[ipg], "ipg", "mxc-ehci.0"); clk_register_clkdev(clk[usbotg_gate], "ahb", "mxc-ehci.0"); So your approach looks good. I only think that mx25 and mx35 dtsi should be also updated to have the 'three-clock' property. Thanks -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html