From: Andrew Lunn <andrew@lunn.ch>
To: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Andrew Lunn <andrew@lunn.ch>,
linux-arm-kernel@lists.infradead.org,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] clk: mvebu: armada 370/XP add clock gating control provider for DT
Date: Sat, 17 Nov 2012 14:54:35 +0100 [thread overview]
Message-ID: <20121117135435.GA13479@lunn.ch> (raw)
In-Reply-To: <50A75BB4.4070003@free-electrons.com>
> > What is the ddr clock for? Does bad things happen if you turn it off?
> > Kirkwood has a similar clock, dunit, which i decided not to export,
> > since when you turn it off, the whole SoC locks up.
>
> Well of course if you code run in DDR then it could be a problem. But
> I think it could be useful to turn it off when going to suspend, it
> the DDR can do self-refresh. In this case it should be possible to run
> the code from SRAM or L2 Cache.
O.K. Just watch out for the lateinit call in the clock framework.
Andrew
WARNING: multiple messages have this Message-ID (diff)
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] clk: mvebu: armada 370/XP add clock gating control provider for DT
Date: Sat, 17 Nov 2012 14:54:35 +0100 [thread overview]
Message-ID: <20121117135435.GA13479@lunn.ch> (raw)
In-Reply-To: <50A75BB4.4070003@free-electrons.com>
> > What is the ddr clock for? Does bad things happen if you turn it off?
> > Kirkwood has a similar clock, dunit, which i decided not to export,
> > since when you turn it off, the whole SoC locks up.
>
> Well of course if you code run in DDR then it could be a problem. But
> I think it could be useful to turn it off when going to suspend, it
> the DDR can do self-refresh. In this case it should be possible to run
> the code from SRAM or L2 Cache.
O.K. Just watch out for the lateinit call in the clock framework.
Andrew
next prev parent reply other threads:[~2012-11-17 13:54 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-15 21:28 [PATCH 0/7] core, cpu and gated clocks for mvebu Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
2012-11-15 21:28 ` [PATCH 1/7] clk: mvebu: add mvebu core clocks Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-2-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-16 12:52 ` Gregory CLEMENT
2012-11-16 12:52 ` Gregory CLEMENT
2012-11-15 21:28 ` [PATCH 2/7] clk: mvebu: add armada-370-xp CPU specific clocks Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-3-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-15 21:44 ` Thomas Petazzoni
2012-11-15 21:44 ` Thomas Petazzoni
2012-11-15 21:28 ` [PATCH 3/7] clk: armada-370-xp: add support for clock framework Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-4-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-15 21:46 ` Thomas Petazzoni
2012-11-15 21:46 ` Thomas Petazzoni
[not found] ` <1353014906-31566-1-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-15 21:28 ` [PATCH 4/7] clocksource: time-armada-370-xp converted to clk framework Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-5-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-15 21:49 ` Thomas Petazzoni
2012-11-15 21:49 ` Thomas Petazzoni
2012-11-15 21:28 ` [PATCH 5/7] clk: mvebu: add clock gating control provider for DT Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
2012-11-16 18:01 ` [PATCH 0/2] Add clock gating support for Armada 370/XP Gregory CLEMENT
2012-11-16 18:01 ` Gregory CLEMENT
2012-11-16 18:01 ` [PATCH 1/2] clk: mvebu: armada 370/XP add clock gating control provider for DT Gregory CLEMENT
2012-11-16 18:01 ` Gregory CLEMENT
2012-11-17 8:26 ` Andrew Lunn
2012-11-17 8:26 ` Andrew Lunn
2012-11-17 9:41 ` Gregory CLEMENT
2012-11-17 9:41 ` Gregory CLEMENT
2012-11-17 13:54 ` Andrew Lunn [this message]
2012-11-17 13:54 ` Andrew Lunn
2012-11-19 4:30 ` Mike Turquette
2012-11-19 4:30 ` Mike Turquette
2012-11-19 15:46 ` Thomas Petazzoni
2012-11-19 15:46 ` Thomas Petazzoni
2012-11-19 15:58 ` Andrew Lunn
2012-11-19 15:58 ` Andrew Lunn
2012-11-19 16:43 ` Thomas Petazzoni
2012-11-19 16:43 ` Thomas Petazzoni
2012-11-19 16:01 ` Sebastian Hesselbarth
2012-11-19 16:01 ` Sebastian Hesselbarth
2012-11-16 18:02 ` [PATCH 2/2] arm: mvebu: armada 370/XP adding clock gating support: dt binding Gregory CLEMENT
2012-11-16 18:02 ` Gregory CLEMENT
2012-11-15 21:28 ` [PATCH 6/7] ARM: dove: switch to DT clock providers Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-7-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-15 21:52 ` Thomas Petazzoni
2012-11-15 21:52 ` Thomas Petazzoni
2012-11-15 21:28 ` [PATCH 7/7] ARM: Kirkwood: " Andrew Lunn
2012-11-15 21:28 ` Andrew Lunn
[not found] ` <1353014906-31566-8-git-send-email-andrew-g2DYL2Zd6BY@public.gmane.org>
2012-11-16 17:33 ` Gregory CLEMENT
2012-11-16 17:33 ` Gregory CLEMENT
2012-11-15 21:54 ` [PATCH 0/7] core, cpu and gated clocks for mvebu Thomas Petazzoni
2012-11-15 21:54 ` Thomas Petazzoni
2012-11-16 7:32 ` Andrew Lunn
2012-11-16 7:32 ` Andrew Lunn
2012-11-16 12:51 ` Gregory CLEMENT
2012-11-16 12:51 ` Gregory CLEMENT
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=20121117135435.GA13479@lunn.ch \
--to=andrew@lunn.ch \
--cc=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sebastian.hesselbarth@gmail.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 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.