All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	Mike Turquette <mturquette-l0cyMroinI0@public.gmane.org>
Subject: Re: [RFC v2 4/9] of: add clock providers
Date: Fri, 13 Jan 2012 08:05:21 -0600	[thread overview]
Message-ID: <4F103A21.1050403@gmail.com> (raw)
In-Reply-To: <20120113135036.GC17029-rvtDTF3kK1ictlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>

On 01/13/2012 07:50 AM, Shawn Guo wrote:
> On Mon, Dec 12, 2011 at 03:02:04PM -0700, Grant Likely wrote:
>> +==Clock providers==
>> +
>> +Required properties:
>> +#clock-cells:	   Number of cells in a clock specifier; typically will be
>> +		   set to 1
> 
> Shouldn't it be 0 typically, which means it represents only one clock
> in the node?
> 

We have no idea what will be typical. I would say: Typically 0 for nodes
with a single clock output and 1 for nodes with multiple clock outputs.

Rob

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robherring2@gmail.com>
To: Shawn Guo <shawn.guo@linaro.org>
Cc: Grant Likely <grant.likely@secretlab.ca>,
	Sascha Hauer <kernel@pengutronix.de>,
	devicetree-discuss@lists.ozlabs.org,
	linux-kernel@vger.kernel.org,
	Rob Herring <rob.herring@calxeda.com>,
	Mike Turquette <mturquette@ti.com>
Subject: Re: [RFC v2 4/9] of: add clock providers
Date: Fri, 13 Jan 2012 08:05:21 -0600	[thread overview]
Message-ID: <4F103A21.1050403@gmail.com> (raw)
In-Reply-To: <20120113135036.GC17029@S2101-09.ap.freescale.net>

On 01/13/2012 07:50 AM, Shawn Guo wrote:
> On Mon, Dec 12, 2011 at 03:02:04PM -0700, Grant Likely wrote:
>> +==Clock providers==
>> +
>> +Required properties:
>> +#clock-cells:	   Number of cells in a clock specifier; typically will be
>> +		   set to 1
> 
> Shouldn't it be 0 typically, which means it represents only one clock
> in the node?
> 

We have no idea what will be typical. I would say: Typically 0 for nodes
with a single clock output and 1 for nodes with multiple clock outputs.

Rob

  parent reply	other threads:[~2012-01-13 14:05 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-12 22:02 [RFC v2 1/9] arm/versatile*: merge all versatile struct clk definitions Grant Likely
2011-12-12 22:02 ` Grant Likely
     [not found] ` <1323727329-4989-1-git-send-email-grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
2011-12-12 22:02   ` [RFC v2 2/9] arm/versatile*: Consolidate clk_ops and setvco implementations Grant Likely
2011-12-12 22:02     ` Grant Likely
2011-12-12 22:02   ` [RFC v2 3/9] of: Add of_property_match_string() to find index into a string list Grant Likely
2011-12-12 22:02     ` Grant Likely
2011-12-12 22:02   ` [RFC v2 4/9] of: add clock providers Grant Likely
2011-12-12 22:02     ` Grant Likely
     [not found]     ` <1323727329-4989-4-git-send-email-grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
2011-12-12 23:29       ` Jamie Iles
2011-12-12 23:29         ` Jamie Iles
2011-12-13 17:54         ` Grant Likely
2011-12-13 17:54           ` Grant Likely
2011-12-13 18:01           ` Rob Herring
2011-12-13 18:03             ` Grant Likely
2011-12-15 13:51           ` Shawn Guo
2011-12-15 13:51             ` Shawn Guo
     [not found]             ` <20111215135129.GA2831-+NayF8gZjK2ctlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2011-12-15 14:23               ` Rob Herring
2011-12-15 14:23                 ` Rob Herring
2011-12-15 15:13                 ` Shawn Guo
2011-12-15 15:13                   ` Shawn Guo
     [not found]                   ` <20111215151335.GB2831-+NayF8gZjK2ctlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2011-12-15 17:37                     ` Grant Likely
2011-12-15 17:37                       ` Grant Likely
2012-01-10 21:33       ` Jamie Iles
2012-01-10 21:33         ` Jamie Iles
2012-01-12  4:46         ` Grant Likely
2012-01-12  4:46           ` Grant Likely
2012-01-12 10:07           ` Jamie Iles
2012-01-12 18:44             ` Turquette, Mike
2012-01-12 19:16               ` Grant Likely
     [not found]           ` <CACxGe6vqb8AZcZb5WMvfmLsUZH7=SEtBrJCSfsFmDYpKX42MzA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-01-13 12:47             ` Shawn Guo
2012-01-13 12:47               ` Shawn Guo
2012-01-14  4:30               ` Turquette, Mike
2012-01-14  5:40                 ` Shawn Guo
2012-01-13 13:50       ` Shawn Guo
2012-01-13 13:50         ` Shawn Guo
     [not found]         ` <20120113135036.GC17029-rvtDTF3kK1ictlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
2012-01-13 14:05           ` Rob Herring [this message]
2012-01-13 14:05             ` Rob Herring
     [not found]             ` <4F103A21.1050403-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-01-13 14:38               ` Shawn Guo
2012-01-13 14:38                 ` Shawn Guo
2012-01-17 20:44     ` Stephen Warren
2012-01-17 22:47       ` Grant Likely
2012-01-17 23:37         ` Turquette, Mike
     [not found]           ` <CAJOA=zMgGsZKGQ-EfSnXDY_WoFrzG6XPZtuxANFKqdL8CAaYrw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-01-17 23:49             ` Grant Likely
2012-01-17 23:49               ` Grant Likely
2012-01-18  0:05             ` Stephen Warren
2012-01-18  0:05               ` Stephen Warren
2011-12-12 22:02   ` [RFC v2 6/9] arm/dt: add devicetree support to sp804 timer support Grant Likely
2011-12-12 22:02     ` Grant Likely
2011-12-12 23:54     ` Rob Herring
     [not found]       ` <4EE69446.2060009-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-12-13  0:29         ` Grant Likely
2011-12-12 22:02   ` [RFC v2 9/9] arm/highbank: Use clock binding common support code Grant Likely
2011-12-12 22:02     ` Grant Likely
2011-12-12 22:02 ` [RFC v2 5/9] dt/clock: Add handling for fixed clocks and a clock node setup iterator Grant Likely
     [not found]   ` <1323727329-4989-5-git-send-email-grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
2011-12-15 15:19     ` Shawn Guo
2011-12-15 15:19       ` Shawn Guo
2011-12-12 22:02 ` [RFC v2 7/9] arm/dt: Common plat-versatile support for icst and sp804 based system clocks Grant Likely
     [not found]   ` <1323727329-4989-7-git-send-email-grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
2012-01-17 21:05     ` Stephen Warren
2012-01-17 21:05       ` Stephen Warren
2012-01-17 22:02       ` Rob Herring
2012-01-17 22:59       ` Grant Likely
2011-12-12 22:02 ` [RFC v2 8/9] dt/arm: versatile add clock parsing 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=4F103A21.1050403@gmail.com \
    --to=robherring2-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mturquette-l0cyMroinI0@public.gmane.org \
    --cc=rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org \
    --cc=shawn.guo-QSEj5FYQhm4dnm+yROfE0A@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.