From: Prashant Gaikwad <pgaikwad@nvidia.com>
To: Peter De Schrijver <pdeschrijver@nvidia.com>
Cc: "linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Grant Likely <grant.likely@secretlab.ca>,
Rob Herring <rob.herring@calxeda.com>,
Rob Landley <rob@landley.net>,
Stephen Warren <swarren@wwwdotorg.org>,
Russell King <linux@arm.linux.org.uk>,
Simon Glass <sjg@chromium.org>, Rhyland Klein <rklein@nvidia.com>,
Pritesh Raithatha <praithatha@nvidia.com>,
Linus Walleij <linus.walleij@linaro.org>,
Hiroshi Doyu <hdoyu@nvidia.com>,
Laxman Dewangan <ldewangan@nvidia.com>,
Andrew Chew <AChew@nvidia.com>,
Mike Turquette <mturquette@linaro.org>,
Thierry Reding <thierry.reding@avionic-design.de>,
"devicetree-discuss@lists.ozlabs.org"
<devicetree-discuss@lists.ozlabs.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v8 10/14] ARM: tegra: Define Tegra114 CAR binding
Date: Thu, 4 Apr 2013 12:44:06 +0530 [thread overview]
Message-ID: <515D283E.5060209@nvidia.com> (raw)
In-Reply-To: <20130403141916.GS18519@tbergstrom-lnx.Nvidia.com>
On Wednesday 03 April 2013 07:49 PM, Peter De Schrijver wrote:
> On Mon, Mar 25, 2013 at 12:15:47PM +0100, Prashant Gaikwad wrote:
>> On Friday 22 March 2013 06:09 PM, Peter De Schrijver wrote:
>>> The device tree binding models Tegra114 CAR (Clock And Reset) as a single
>>> monolithic clock provider.
>>>
>>> Signed-off-by: Peter De Schrijver <pdeschrijver@nvidia.com>
>>> ---
>> <snip>
>>
>>> + 263 cclk_lp
>>> + 264 dfll_ref
>>> + 265 dfll_soc
>>> +
>> Peter,
>>
>> Please remove following clock IDs from binding.
>>
>>> +Mux clocks
>>> +
> ...
>>> + 306 clk_out_1_mux
> Is at least used by the init list. So we can't remove that?
Init list is internal to driver and does not use device tree, so we can
keep that ID in enum but remove from DT binding so that we don't have to
change the binding later.
>
> Cheers,
>
> Peter.
next prev parent reply other threads:[~2013-04-04 7:14 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-22 12:39 [PATCH v8 00/14] Tegra114 clockframework Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 02/14] clk: tegra: Refactor PLL programming code Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 03/14] clk: tegra: Add TEGRA_PLL_BYPASS flag Peter De Schrijver
[not found] ` <1363956029-31873-1-git-send-email-pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-22 12:39 ` [PATCH v8 01/14] clk: tegra: provide dummy cpu car ops Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 04/14] clk: tegra: introduce TEGRA_PLL_HAS_LOCK_ENABLE Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 07/14] clk: tegra: Add new fields and PLL types for Tegra114 Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 08/14] clk: tegra: Add flags to tegra_clk_periph() Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 09/14] clk: tegra: Workaround for Tegra114 MSENC problem Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 12/14] clk: tegra: devicetree match for nvidia,tegra114-car Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 13/14] ARM: dt: Add references to tegra_car clocks Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 14/14] clk: tegra: Remove forced clk_enable of uartd Peter De Schrijver
2013-03-22 19:05 ` [PATCH v8 00/14] Tegra114 clockframework Stephen Warren
2013-03-25 22:44 ` Stephen Warren
2013-03-22 12:39 ` [PATCH v8 05/14] clk: tegra: Add PLL post divider table Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 06/14] clk: tegra: move from a lock bit idx to a lock mask Peter De Schrijver
2013-03-22 12:39 ` [PATCH v8 10/14] ARM: tegra: Define Tegra114 CAR binding Peter De Schrijver
2013-03-25 11:15 ` Prashant Gaikwad
[not found] ` <515031E3.40007-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-04-03 14:19 ` Peter De Schrijver
2013-04-04 7:14 ` Prashant Gaikwad [this message]
2013-03-22 12:39 ` [PATCH v8 11/14] clk: tegra: Implement clocks for Tegra114 Peter De Schrijver
2013-03-22 12:55 ` [PATCH v8 00/14] Tegra114 clockframework 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=515D283E.5060209@nvidia.com \
--to=pgaikwad@nvidia.com \
--cc=AChew@nvidia.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=hdoyu@nvidia.com \
--cc=ldewangan@nvidia.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mturquette@linaro.org \
--cc=pdeschrijver@nvidia.com \
--cc=praithatha@nvidia.com \
--cc=rklein@nvidia.com \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
--cc=sjg@chromium.org \
--cc=swarren@wwwdotorg.org \
--cc=thierry.reding@avionic-design.de \
/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).