From: sylvain.rochet@finsecur.com (Sylvain Rochet)
To: linux-arm-kernel@lists.infradead.org
Subject: AT91 slow clock mode regression/fixes, improvement proposal
Date: Thu, 18 Dec 2014 22:23:30 +0100 [thread overview]
Message-ID: <20141218212330.GA19367@gradator.net> (raw)
In-Reply-To: <20141218203904.GA25215@gradator.net>
Hello,
On Thu, Dec 18, 2014 at 09:39:04PM +0100, Sylvain Rochet wrote:
>
> About e0c8ba9b0ec3154e87da747098ee56e96ca3cee6:
>
> I have mixed feeling about moving the PLL enabling from slow clock mode
> to master clock mode. Starting PLL is almost all about waiting, waiting,
> and waiting until they are stable enough to be used, the few CPU
> instructions required to switch ON the PLL is nothing compared to the
> wait time.
>
> To be sure I benchmarked the required time to set up the UTMI PLL (using
> a GPIO + scope) on my AT91SAM9G35-CM module in both slow clock and
> master clock mode, I found out the required time to start up the PLL is,
> as expected, -exactly- the same.
>
> But, previously, we were waiting with the CPU in slow clock mode, when
> the CPU power consumption is very very low, now we are waiting when the
> CPU is in full speed, which is worse.
>
> Or, I am missing something ?
Responding myself on this point, just did further mesurements, and...
well, this can be discarded. I didn't notice at first that PLL startup
time are only about to take ~500 us, which is a negligible amount of
time and is actually a bit faster (just a bit) on master clock. I
really should have checked the timing as well with PLL start up
disabled, sorry :)
Sylvain
next prev parent reply other threads:[~2014-12-18 21:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-18 20:39 AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2014-12-18 21:23 ` Sylvain Rochet [this message]
2014-12-19 2:50 ` Yang, Wenyou
2014-12-19 15:04 ` [PATCH] pm: at91: pm_slowclock: improve reliability of suspend/resume Sylvain Rochet
2014-12-19 15:26 ` AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2014-12-22 8:32 ` Yang, Wenyou
2014-12-22 10:03 ` Sylvain Rochet
2015-01-05 3:32 ` Yang, Wenyou
2015-01-06 14:15 ` [PATCH] pm: at91: pm_slowclock: improve reliability of suspend/resume Sylvain Rochet
2015-01-06 14:16 ` Sylvain Rochet
2015-01-06 14:25 ` AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2015-01-07 1:17 ` Yang, Wenyou
2015-01-07 9:14 ` Alexandre Belloni
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=20141218212330.GA19367@gradator.net \
--to=sylvain.rochet@finsecur.com \
--cc=linux-arm-kernel@lists.infradead.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.