From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Peter De Schrijver
<pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: Mike Turquette
<mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Prashant Gaikwad
<pgaikwad-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
Thierry Reding
<thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>,
Joseph Lo <josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
Paul Walmsley <pwalmsley-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 00/12] Introduce common infra for tegra clocks
Date: Tue, 01 Oct 2013 11:39:09 -0600 [thread overview]
Message-ID: <524B08BD.2020902@wwwdotorg.org> (raw)
In-Reply-To: <1379946381-20125-1-git-send-email-pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
On 09/23/2013 08:24 AM, Peter De Schrijver wrote:
> This patchset introduces common infrastructure for clocks which exist in
> several Tegra SoCs. We also move Tegra114 to this new infrastructure.
> Tegra20 and Tegra30 will be moved later in separate patchsets.
>
> Changes since v1:
> + move common clks allocation to patch 2
> + adapt also Tegra20 and Tegra30 to the changes in patch 2
>
> Boottested on dalmore, beaverboard and seaboard.
What commit is this series based on? I applied the two pre-requisites:
ARM: tegra114: add missing clocks to binding.eml
clk: tegra: replace enum tegra114_clk by binding header.eml
... but still got a bunch of conflicts when applying this. I guess at
least commit 819c1de "clk: add CLK_SET_RATE_NO_REPARENT flag" isn't in
the history of this series. Consequently, I gave up and didn't test this.
Also, I had given a lot of feedback on v1. Admittedly I should have
reviewed v2 since it appeared just before I reviewed v1, but I expect
most of the feedback still applies given the limited list of changes
listed above. Certainly, this series still removes the "dsib" clock yet
I expect it shouldn't (patch 1, in the change to
tegra114_periph_clk_init()), and that's the very first piece of feedback
I gave on v1, and also causes the first "git am" conflict with
CLK_SET_RATE_NO_REPARENT.
next prev parent reply other threads:[~2013-10-01 17:39 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-23 14:24 [PATCH v2 00/12] Introduce common infra for tegra clocks Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 01/12] clk: tegra: simplify periph clock data Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 02/12] clk: tegra: common periph_clk_enb_refcnt and clks Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 03/12] clk: tegra: Add TEGRA_PERIPH_NO_DIV flag Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 05/12] clk: tegra: add header for common tegra clock IDs Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 06/12] clk: tegra: add common infra for DT clocks Peter De Schrijver
2013-09-23 14:24 ` [PATCH v2 07/12] clk: tegra: move audio clk to common file Peter De Schrijver
2013-09-23 14:25 ` [PATCH v2 08/12] clk: tegra: move periph clocks " Peter De Schrijver
[not found] ` <1379946381-20125-1-git-send-email-pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-09-23 14:24 ` [PATCH v2 04/12] clk: tegra: move some PLLC and PLLXC init to clk-pll.c Peter De Schrijver
2013-09-23 14:25 ` [PATCH v2 09/12] clk: tegra: move PMC clocks to common file Peter De Schrijver
2013-09-23 14:25 ` [PATCH v2 10/12] clk: tegra: move fixed " Peter De Schrijver
2013-09-23 14:25 ` [PATCH v2 11/12] clk: tegra: introduce common tegra_osc_clk_init Peter De Schrijver
2013-09-23 14:25 ` [PATCH v2 12/12] clk: tegra: introduce common gen4 super clock Peter De Schrijver
2013-10-01 17:39 ` Stephen Warren [this message]
[not found] ` <524B08BD.2020902-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-03 9:16 ` [PATCH v2 00/12] Introduce common infra for tegra clocks 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=524B08BD.2020902@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=pgaikwad-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=pwalmsley-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@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