netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: Mitch Bradley <wmb@firmworks.com>,
	patches@linaro.org, netdev@vger.kernel.org,
	devicetree-discuss@lists.ozlabs.org,
	Jason Liu <jason.hui@linaro.org>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Jeremy Kerr <jeremy.kerr@canonical.com>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Shawn Guo <shawn.guo@freescale.com>
Subject: Re: [PATCH 1/3] serial/imx: add device tree support
Date: Tue, 21 Jun 2011 19:53:56 +0100	[thread overview]
Message-ID: <20110621185356.GJ23234@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <BANLkTiny9xDwiUww9Ys+jQZ0SHdQATABUA@mail.gmail.com>

On Tue, Jun 21, 2011 at 12:42:14PM -0600, Grant Likely wrote:
> On Tue, Jun 21, 2011 at 12:32 PM, Mitch Bradley <wmb@firmworks.com> wrote:
> > I wonder if it makes sense to create a new device node "/linux-devices" to express a desired mapping from device nodes to /dev entries?  The properties could be the names of device special files and the values the corresponding node phandles.
> 
> I've been trying /really/ hard to avoid doing something like that
> because a lot of the time the desired Linux dev name is a
> implementation detail, and a potentially unstable one at that.  If
> Linux requires certain devices to have certain names because that is
> how it hooks up clocks

As I keep on saying, we don't _have_ to have to match on device name.
If DT can come up with a better way to bind a clock to a particular
device/connection name then DT can provide its own clk_get()
implementation which does that.

clk_get() has the struct device, and therefore can get at the DT
information itself to read out whatever properties are required.  But,
we must not get away from the fact that clk_get()'s second argument
should _never_ be used as a unique clock name itself.

At the moment, until we do have some kind of DT based clk_get(), the
easiest way to move clk-API using drivers over is to use the device
name in the static clk_lookup tables.

It's all an implementation detail, one which is (thankfully) hidden
behind a proper API which will be _completely_ transparent to drivers
using the clk API in the _proper_ way.

  reply	other threads:[~2011-06-21 18:54 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-18 15:19 [PATCH 0/3] Add basic device support for imx51 babbage Shawn Guo
     [not found] ` <1308410354-21387-1-git-send-email-shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-06-18 15:19   ` [PATCH 1/3] serial/imx: add device tree support Shawn Guo
     [not found]     ` <1308410354-21387-2-git-send-email-shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-06-18 16:19       ` Grant Likely
2011-06-19  7:02         ` Wolfram Sang
2011-06-19  7:30         ` Shawn Guo
2011-06-19 15:05           ` Grant Likely
2011-06-19 15:15             ` Rob Herring
2011-06-19 18:44               ` Grant Likely
2011-06-21 13:55           ` Shawn Guo
2011-06-21 18:32             ` Mitch Bradley
2011-06-21 18:42               ` Grant Likely
2011-06-21 18:53                 ` Russell King - ARM Linux [this message]
2011-06-21 19:02                   ` Grant Likely
     [not found]                 ` <BANLkTiny9xDwiUww9Ys+jQZ0SHdQATABUA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-06-21 19:04                   ` Mitch Bradley
2011-06-21 19:13             ` Grant Likely
     [not found]               ` <BANLkTim4vunTm1176MRw3pqRZnunu4=9jg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-06-21 19:35                 ` Mitch Bradley
2011-06-21 19:38                   ` Grant Likely
2011-06-21 22:08                     ` Mitch Bradley
2011-06-21 22:33                       ` Grant Likely
     [not found]                         ` <BANLkTikBUo3jO=R0amkdxU1V3QMJoOUkEw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-06-21 22:52                           ` Segher Boessenkool
     [not found]                             ` <ba379b3b42f1659f4f6649c78b8bb1b2-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2011-06-21 22:58                               ` Grant Likely
2011-06-22 15:33                 ` Shawn Guo
2011-06-22 15:52                   ` Grant Likely
     [not found]                     ` <BANLkTimsk2eWKPi2hnHbaAn-vSMKmdYsGQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-06-23  0:12                       ` Shawn Guo
     [not found]                         ` <20110623001234.GB25799-+NayF8gZjK2ctlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2011-06-23  3:35                           ` Grant Likely
2011-06-23 18:38               ` Shawn Guo
2011-06-23 23:11                 ` Grant Likely
     [not found]                   ` <BANLkTi=8Mks6T85WnEyihmB21U1j7reHrQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-06-24  3:49                     ` Shawn Guo
     [not found]                       ` <20110624034859.GB19188-+NayF8gZjK2ctlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2011-06-24  4:05                         ` Grant Likely
2011-06-18 16:21     ` Arnd Bergmann
     [not found]       ` <201106181821.46574.arnd-r2nGTMty4D4@public.gmane.org>
2011-06-18 16:26         ` Grant Likely
     [not found]           ` <20110618162655.GK8195-e0URQFbLeQY2iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2011-06-18 18:11             ` Arnd Bergmann
2011-06-19  7:41           ` Shawn Guo
2011-06-19  7:40         ` Shawn Guo
2011-06-18 15:19   ` [PATCH 2/3] net/fec: " Shawn Guo
2011-06-18 16:22     ` Grant Likely
     [not found]       ` <20110618162220.GI8195-e0URQFbLeQY2iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2011-06-19  7:46         ` Shawn Guo
     [not found]     ` <1308410354-21387-3-git-send-email-shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-06-18 18:27       ` Arnd Bergmann
     [not found]         ` <201106182027.22974.arnd-r2nGTMty4D4@public.gmane.org>
2011-06-19  0:24           ` Grant Likely
2011-06-19  7:55         ` Shawn Guo
2011-06-19 11:39     ` Greg Ungerer
2011-06-19 12:11       ` Arnd Bergmann
2011-06-19 15:09         ` Rob Herring
2011-06-19 18:43           ` Grant Likely
2011-06-20  0:05         ` Greg Ungerer
2011-06-18 15:19   ` [PATCH 3/3] ARM: mx5: add basic device tree support for imx51 babbage Shawn Guo
     [not found]     ` <1308410354-21387-4-git-send-email-shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-06-18 16:24       ` Grant Likely
2011-06-18 16:29   ` [PATCH 0/3] Add basic device " 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=20110621185356.GJ23234@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=jason.hui@linaro.org \
    --cc=jeremy.kerr@canonical.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=patches@linaro.org \
    --cc=s.hauer@pengutronix.de \
    --cc=shawn.guo@freescale.com \
    --cc=wmb@firmworks.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).