On Wed, Jan 16, 2013 at 09:48:57AM +0800, Shawn Guo wrote: > On Wed, Jan 16, 2013 at 09:18:46AM +0800, Peter Chen wrote: > > On Tue, Jan 15, 2013 at 07:33:16PM +0800, Shawn Guo wrote: > > > On Tue, Jan 15, 2013 at 02:08:34PM +0800, Peter Chen wrote: > > > > For mxs-phy user i.mx6q, the PHY's clock is controlled by > > > > hardware automatically, the software only needs to enable it > > > > at probe, disable it at remove. During the runtime, > > > > we don't need to control it. So for the usbphy clk policy: > > > > > > > > - Keep refcount for usbphy as clk framework needs to know if > > > > it is off or on. > > > > I thought it might be okay to access anatop for enabling the clock in > usb driver. But I found it's just too ugly to do that when I look at > the code a little bit closer. I prefer to keep clock driver unchanged > and just call clk_prepare_enable() at platform level to enable the > clock. I really dislike the code accessing anatop for enabling the > clock. That's the part different from you v1 patch. It is a way, but how to consolidate other mxx-phy users, like mx28. It needs to call clk_prepare_enable runtime. > > > I send the v2 patch due to below reasons, > > There are other PLL3 users, it is better keep refcount > > to all its children, or the user will be confused when the > > PLL3 goes to disable, but the usb otg part is still used > > > This is the part I'm not sure how it works. You said, the PLL will be > managed by hardware during USB suspend/resume. I assume that it means > the PLL will be powered off when USB suspends and be powered on when > USB resumes. If that's the case, how other users of the PLL work when > PLL is powered off by USB hardware. It is a little complicated: You can understand both PLL7 and PLL3 like below: PLL_POWER_BIT && usb_is_not_suspend For PLL7: USB host 1 is the only user, so we can keep PLL7_POWER_BIT is always on and let the usb hardware controller it automatically. Other way is let the clock framework and usb hardware controller it together. For PLL3, as there are other users, so software must control PLL3_POWER_BIT, or the PLL3 will be off if the usb enters suspend. This PLL_POWER_BIT is not related to usbphy clock gate, this power bit can be usbphy's parent. > > Shawn > > > For PLL7, we can do it, and let the PLL7 be out of clock framework > > management. > > > > But I still think keep usbphy as PLL child is better solution, > > in that case, the clock maintainer can know the real clock usage. > > -- Best Regards, Peter Chen -- 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
