From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Fri, 22 Apr 2011 00:09:18 -0700 Subject: [PATCH RFC] clk: add support for automatic parent handling In-Reply-To: References: <1303308457-7501-1-git-send-email-u.kleine-koenig@pengutronix.de> <20110420185922.GD31131@pengutronix.de> <20110421072259.GG31131@pengutronix.de> <20110421103117.GB16776@sirena.org.uk> <20110421114220.GJ13688@atomide.com> Message-ID: <20110422070918.GB841@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Thomas Gleixner [110421 07:49]: > On Thu, 21 Apr 2011, Tony Lindgren wrote: > > > * Mark Brown [110421 03:29]: > > > On Thu, Apr 21, 2011 at 11:12:36AM +0200, Thomas Gleixner wrote: > > > > On Thu, 21 Apr 2011, Uwe Kleine-K?nig wrote: > > > > > > > > If there were a pointer tracking the parent you still needed to program > > > > > out the get_parent function anyhow to set the pointer initially. But I > > > > > agree that in other situations having a pointer is more comfortable and > > > > > saves ugly error handling e.g. in set_parent. > > > > > > > That's nonsense. Why do you want a get_parent() function at all? > > > > parent is set up when the clock is established in the clock tree. And > > > > that's not done by some random driver. That's a property of the clock. > > > > > > That's not universal - right now the drivers can and do know about this > > > on some platforms. In some cases there will be requirements beyond the > > > clock rate which mean that the driver will need to be able to tell the > > > framework that, for example, it needs one clock to be in the same clock > > > domain as another. > > > > Also there can be multiple parent clocks. An example are the timers on > > omaps where the parent clock can be either the 32KiHz clock or external > > source clock. ?his may need to be reprogrammed dynamically in some cases > > for advanced idle modes as one of the parent clocks may be shut down. > > If you need to propagate from bottom up, then you need a list of > childs in struct clk. Right ? Yes we need to be able to reprogram all children if the parent clock needs to be reprogrammed for cpufreq/DVFS. Also a child may need to change the parent clock like I wrote above. However, it might be possible to deal with this by registering the same child for both parents and handle that directly in the driver for the special cases if that simplifies things. In some clk_set_rate cases for a child the parent clock may need to be reprogrammed to provide the desired rate. I don't know how common this is, and this may be doable by requesting both parent and child in the driver if that simplifies things. Would be good to hear Paul's comments on this, as he probably knows best what's really needed so I've added him to Cc. Regards, Tony