devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mike Turquette <mturquette@linaro.org>
To: Peter De Schrijver <pdeschrijver@nvidia.com>
Cc: Tero Kristo <t-kristo@ti.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 0/4] clk: dt: add support for default rate/parent
Date: Mon, 24 Mar 2014 17:38:47 -0700	[thread overview]
Message-ID: <20140325003847.32624.58084@quantum> (raw)
In-Reply-To: <20140321081217.GD26674@tbergstrom-lnx.Nvidia.com>

Quoting Peter De Schrijver (2014-03-21 01:12:17)
> On Thu, Mar 20, 2014 at 10:23:08PM +0100, Mike Turquette wrote:
> > Quoting Tero Kristo (2014-03-05 05:10:17)
> > > Ping.
> > > 
> > > Mike, any feedback on this?
> > 
> > Hi Tero,
> > 
> > Have you seen Sylwester's approach[1]? I prefer it since it is more
> > device-oriented and less "centralized". The clock consumer enumerates
> > the default parent or rate of a consumed clock. This can be made to work
> 
> That assumes driver writers are aware of the clock tree topology. IME that's
> seldomly the case.

It assumes no such thing.

The point of Sylwester's patch is that if a driver consumes a clock and
needs to do the very typical setup regarding that clock's rate or
parent, then we now have a sensible way to express that in DT.

One of the strengths of DT is that the C code does not have to know all
of the details about topology or how things are hooked up. We can hide
some of those cute embedded nonsense hacks in DTS and the device
integrator can manage it there.

Regards,
Mike

> 
> Cheers,
> 
> Peter.

  reply	other threads:[~2014-03-25  0:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-13  9:00 [PATCH 0/4] clk: dt: add support for default rate/parent Tero Kristo
2014-02-13  9:00 ` [PATCH 1/4] clk: ti: mux: add support for default-parenting Tero Kristo
2014-02-13  9:00 ` [PATCH 2/4] clk: add support for default-rate Tero Kristo
2014-02-13  9:00 ` [PATCH 3/4] clk: ti: add support for default-rate property from DT Tero Kristo
2014-02-28 22:51   ` Tony Lindgren
2014-02-13  9:00 ` [PATCH 4/4] clk: ti: omap4: set default-parents and default-rates using DT Tero Kristo
2014-03-05 13:10 ` [PATCH 0/4] clk: dt: add support for default rate/parent Tero Kristo
2014-03-20 21:23   ` Mike Turquette
2014-03-21  8:02     ` Tero Kristo
2014-03-21  8:12     ` Peter De Schrijver
2014-03-25  0:38       ` Mike Turquette [this message]
2014-03-25  7:28         ` Peter De Schrijver

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=20140325003847.32624.58084@quantum \
    --to=mturquette@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=pdeschrijver@nvidia.com \
    --cc=t-kristo@ti.com \
    /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;
as well as URLs for NNTP newsgroup(s).