From: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
To: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: devicetree-discuss
<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>
Subject: Re: clock bindings
Date: Thu, 31 Mar 2011 16:45:47 -0600 [thread overview]
Message-ID: <20110331224547.GH437@ponder.secretlab.ca> (raw)
In-Reply-To: <4D498AAC.3090009-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
On Wed, Feb 02, 2011 at 10:47:40AM -0600, Rob Herring wrote:
> I've started looking at the DT clock bindings in more depth. To what
> level should the clock tree be defined in the DT? Should it be a
> one-to-one correlation of current struct clk nodes to node in DT
> where each node is a single input and output? Or only define a
> single (or few) node with the inputs (oscillators) and many outputs
> for SOC's clock controller (i.e. MX51 CCM).
>
> If the DT itself does not have a hierarchical construction of nodes
> that matches the clock tree hierarchy, then creating the hierarchy
> at run-time will be a challenge and not be very efficient. It would
> require 1 pass for each clock node type to create all the nodes, and
> then another pass to setup the hierarchy.
Wow, it takes me a long time to reply sometimes.
I'm not sure at the moment. A month ago I would have said
"everything", but I've thought about it a lot more and I think it is
more important that dt and non-dt users share the same code base for
setting up internal-to-the-soc clocks. Anything available externally
will need a node describing it in the dt, which means the support code
shifts from registering all the clocks to matching up dt nodes to
existing clocks where possible.
For the time being, I've pushed back on registering all devices from
the device tree for the same reason which has given some breathing
room on the topic. It will be a topic for discussion at UDS.
g.
next prev parent reply other threads:[~2011-03-31 22:45 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-02 16:47 clock bindings Rob Herring
[not found] ` <4D498AAC.3090009-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-03-31 22:45 ` Grant Likely [this message]
[not found] ` <20110331224547.GH437-e0URQFbLeQY2iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2011-04-01 6:43 ` Shawn Guo
[not found] ` <20110401064303.GH25866-+NayF8gZjK2ctlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2011-04-05 4:50 ` Grant Likely
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110331224547.GH437@ponder.secretlab.ca \
--to=grant.likely-s3s/wqlpoipyb63q8fvjnq@public.gmane.org \
--cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox