From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: "Heiko Stübner" <heiko@sntech.de>
Cc: "Stephen Boyd" <sboyd@codeaurora.org>,
"Kevin Hilman" <khilman@kernel.org>,
"Mike Turquette" <mturquette@baylibre.com>,
"Doug Anderson" <dianders@chromium.org>,
linux-clk@vger.kernel.org, lkml <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"Boris Brezillon" <boris.brezillon@free-electrons.com>,
"Alex Elder" <elder@linaro.org>,
"Alexandre Belloni" <alexandre.belloni@free-electrons.com>,
"Stephen Warren" <swarren@wwwdotorg.org>,
"Max Filippov" <jcmvbkbc@gmail.com>,
"Sascha Hauer" <kernel@pengutronix.de>,
"Zhangfei Gao" <zhangfei.gao@linaro.org>,
"Santosh Shilimkar" <ssantosh@kernel.org>,
"Chao Xie" <chao.xie@marvell.com>,
"Jason Cooper" <jason@lakedaemon.net>,
"Stefan Wahren" <stefan.wahren@i2se.com>,
"Andrew Bresticker" <abrestic@chromium.org>,
"Robert Jarzmik" <robert.jarzmik@free.fr>,
"Georgi Djakov" <georgi.djakov@linaro.org>,
"Sylwester Nawrocki" <s.nawrocki@samsung.com>,
"Geert Uytterhoeven" <geert+renesas@glider.be>,
"Barry Song" <baohua@kernel.org>,
"Dinh Nguyen" <dinguyen@opensource.altera.com>,
"Viresh Kumar" <viresh.linux@gmail.com>,
"Gabriel FERNANDEZ" <gabriel.fernandez@st.com>,
"Emilio López" <emilio@elopez.com.ar>,
"Peter De Sc hrijver" <pdeschrijver@nvidia.com>,
"Tero Kristo" <t-kristo@ti.com>,
"Ulf Hansson" <ulf.hansson@linaro.org>,
"Pawel Moll" <pawel.moll@arm.com>,
"Michal Simek" <michal.simek@xilinx.com>,
"Olof Johansson" <olof@lixom.net>,
"Tyler Baker" <tyler.baker@linaro.org>
Subject: Re: [PATCH v3 0/2] clk: improve handling of orphan clocks
Date: Thu, 30 Jul 2015 12:09:08 +0200 [thread overview]
Message-ID: <20150730100908.GI2564@lukather> (raw)
In-Reply-To: <4099394.C7vEjTFs8V@diego>
[-- Attachment #1: Type: text/plain, Size: 1230 bytes --]
Hi Heiko,
On Mon, Jul 27, 2015 at 10:57:33AM +0200, Heiko Stübner wrote:
> > > Also we can try to see if critical clocks aren't being forced on by
> > > applying this patch and looking for clk_get() failures
> >
> > And that shows that the CPU and DDR clocks are not protected, which
> > obviously is pretty mad.
> >
> > I've mass converted all our probing code to use OF_CLK_DECLARE, and
> > make things work again.
> >
> > http://code.bulix.org/5goa5j-88345?raw
> >
> > Is this an acceptable solution?
> >
> > We were already moving to this, I'm not really fond of doing this like
> > that, but I guess this whole debacle makes it necessary.
>
>
> did this lead anywhere meanwhile.
>
> Last I remember the change to orphan handling made sunxi fail, but
> I'm still hoping to get this usable at some point :-)
To be honest, I don't know what the current status is. I haven't get
any news since that mail.
I started to move a significant portion of our clocks to
CLK_OF_DECLARE, but not all of them are (which probably make the
situation worse for the time being).
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2015-07-30 10:09 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-22 20:53 [PATCH v3 0/2] clk: improve handling of orphan clocks Heiko Stuebner
2015-04-22 20:53 ` [PATCH v3 1/2] clk: track the orphan status of clocks and their children Heiko Stuebner
2015-04-30 23:20 ` Stephen Boyd
2015-04-22 20:53 ` [PATCH v3 2/2] clk: prevent orphan clocks from being used Heiko Stuebner
2015-04-30 23:20 ` Stephen Boyd
2015-04-25 12:23 ` [PATCH v3 0/2] clk: improve handling of orphan clocks Stefan Wahren
2015-04-25 13:44 ` Heiko Stübner
2015-04-26 19:58 ` Robert Jarzmik
2015-05-01 0:19 ` Stephen Boyd
2015-05-01 19:59 ` Heiko Stübner
2015-05-01 20:52 ` Stephen Boyd
2015-05-01 22:07 ` Heiko Stübner
2015-05-01 23:40 ` Stephen Boyd
2015-05-07 8:22 ` Tero Kristo
2015-05-07 18:18 ` Stephen Boyd
2015-05-08 11:41 ` Tero Kristo
2015-05-07 15:17 ` Kevin Hilman
2015-05-07 21:03 ` Stephen Boyd
2015-05-08 0:27 ` Kevin Hilman
2015-05-08 6:53 ` Stephen Boyd
2015-05-08 8:13 ` Sascha Hauer
2015-05-08 9:30 ` Heiko Stübner
2015-05-08 9:53 ` Sascha Hauer
2015-05-08 10:02 ` Maxime Ripard
2015-05-12 22:35 ` Stephen Boyd
2015-05-13 13:03 ` Maxime Ripard
2015-05-13 14:33 ` Kevin Hilman
2015-05-13 20:14 ` Maxime Ripard
2015-05-13 20:44 ` Kevin Hilman
2015-05-13 20:51 ` Maxime Ripard
2015-07-27 8:57 ` Heiko Stübner
2015-07-30 10:09 ` Maxime Ripard [this message]
2015-08-11 22:34 ` Stephen Boyd
2015-08-12 8:26 ` Heiko Stübner
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=20150730100908.GI2564@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=abrestic@chromium.org \
--cc=alexandre.belloni@free-electrons.com \
--cc=baohua@kernel.org \
--cc=boris.brezillon@free-electrons.com \
--cc=chao.xie@marvell.com \
--cc=dianders@chromium.org \
--cc=dinguyen@opensource.altera.com \
--cc=elder@linaro.org \
--cc=emilio@elopez.com.ar \
--cc=gabriel.fernandez@st.com \
--cc=geert+renesas@glider.be \
--cc=georgi.djakov@linaro.org \
--cc=heiko@sntech.de \
--cc=jason@lakedaemon.net \
--cc=jcmvbkbc@gmail.com \
--cc=kernel@pengutronix.de \
--cc=khilman@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michal.simek@xilinx.com \
--cc=mturquette@baylibre.com \
--cc=olof@lixom.net \
--cc=pawel.moll@arm.com \
--cc=pdeschrijver@nvidia.com \
--cc=robert.jarzmik@free.fr \
--cc=s.nawrocki@samsung.com \
--cc=sboyd@codeaurora.org \
--cc=ssantosh@kernel.org \
--cc=stefan.wahren@i2se.com \
--cc=swarren@wwwdotorg.org \
--cc=t-kristo@ti.com \
--cc=tyler.baker@linaro.org \
--cc=ulf.hansson@linaro.org \
--cc=viresh.linux@gmail.com \
--cc=zhangfei.gao@linaro.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;
as well as URLs for NNTP newsgroup(s).