From: lorenzo.pieralisi@arm.com (Lorenzo Pieralisi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH RFC v2 2/2] Documentation: arm: define DT C-states bindings
Date: Mon, 27 Jan 2014 11:41:26 +0000 [thread overview]
Message-ID: <20140127114125.GD16639@e102568-lin.cambridge.arm.com> (raw)
In-Reply-To: <20140125.101546.1134862463481697494.apm@brigitte.kvy.fi>
On Sat, Jan 25, 2014 at 08:15:46AM +0000, Antti P Miettinen wrote:
> From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> Subject: [PATCH RFC v2 2/2] Documentation: arm: define DT C-states bindings
> Date: Mon, 20 Jan 2014 17:47:59 +0000
> > + - latency
> > + Usage: Required
> > + Value type: <prop-encoded-array>
> > + Definition: List of u32 values representing worst case latency
> > + in microseconds required to enter and exit the
> > + C-state, one value per OPP [2]. The list should
> > + be specified in the same order as the operating
> > + points property list of the cpu this state is
> > + valid on.
> > + If no OPP bindings are present, the latency value
> > + is associated with the current OPP of CPUs in the
> > + system.
>
> I'm afraid the CPU OPP is not enough to capture the variance in
> latencies. Especially memory frequency affects some of the latencies
> very stronly.
That's why I defined the worst case. How did you implemented it in your
idle drivers ? That would help generalize it, after all these bindings
are there to simplify drivers upstreaming, feedback welcome.
Thanks,
Lorenzo
next prev parent reply other threads:[~2014-01-27 11:41 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-20 17:47 [PATCH RFC v2 0/2] ARM: defining power states DT bindings Lorenzo Pieralisi
2014-01-20 17:47 ` [PATCH RFC v2 1/2] Documentation: arm: add cache " Lorenzo Pieralisi
2014-01-21 11:49 ` Dave Martin
2014-01-21 14:47 ` Lorenzo Pieralisi
2014-01-27 12:58 ` Russell King - ARM Linux
2014-01-27 18:10 ` Lorenzo Pieralisi
2014-01-20 17:47 ` [PATCH RFC v2 2/2] Documentation: arm: define DT C-states bindings Lorenzo Pieralisi
2014-01-21 11:16 ` Vincent Guittot
2014-01-21 13:31 ` Lorenzo Pieralisi
2014-01-21 14:35 ` Amit Kucheria
2014-01-21 15:23 ` Lorenzo Pieralisi
2014-01-22 11:52 ` Mark Brown
2014-01-22 16:23 ` Lorenzo Pieralisi
2014-01-22 18:17 ` Mark Brown
2014-01-22 11:42 ` Mark Brown
2014-01-22 16:33 ` Lorenzo Pieralisi
2014-01-22 18:11 ` Mark Brown
2014-01-22 19:20 ` Lorenzo Pieralisi
2014-01-24 8:40 ` Vincent Guittot
2014-01-24 17:58 ` Lorenzo Pieralisi
2014-01-28 8:24 ` Vincent Guittot
2014-01-29 12:42 ` Lorenzo Pieralisi
2014-01-25 8:15 ` Antti P Miettinen
2014-01-27 11:41 ` Lorenzo Pieralisi [this message]
2014-01-27 12:48 ` Antti P Miettinen
2014-01-27 18:22 ` Lorenzo Pieralisi
[not found] <20140127155924.GA2178@e103592.cambridge.arm.com>
2014-01-29 12:33 ` Lorenzo Pieralisi
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=20140127114125.GD16639@e102568-lin.cambridge.arm.com \
--to=lorenzo.pieralisi@arm.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 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).