From: Peng Fan <van.freenix@gmail.com>
To: Ian Campbell <ian.campbell@citrix.com>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
George Dunlap <george.dunlap@citrix.com>,
Julien Grall <julien.grall@citrix.com>,
David Vrabel <david.vrabel@citrix.com>,
xen-devel@lists.xenproject.org
Subject: Re: [RFC V2] xen: interface: introduce pvclk interface
Date: Wed, 20 Jan 2016 21:52:34 +0800 [thread overview]
Message-ID: <20160120135232.GA10911@linux-7smt.suse> (raw)
In-Reply-To: <1453292827.26343.98.camel@citrix.com>
Hi Ian, Stefano
On Wed, Jan 20, 2016 at 12:27:07PM +0000, Ian Campbell wrote:
>On Wed, 2016-01-20 at 12:06 +0000, Stefano Stabellini wrote:
>> On Wed, 20 Jan 2016, Peng Fan wrote:
>> > To my use case, Dom0 and DomU both use device tree, I need to build
>> > the mapping table between id and name, since I use name to lookup
>> > the clk in backend, like this:
>> > "clk = __clk_loopkup(clkname); clk_prepare_enable(clk)". Maybe ACPI
>> > is another different case.
>>
>> Theoretically on systems using ACPI there is no need to fiddle with
>> clocks,
>
>I mentioned ACPI in my replies to v1 more as a placeholder for "other
>firmware descriptions than DT", in order that the pv protocol we end up
>with does not end up being DT specific, which would be a mistake
>irrespective of what may or may not be required for non-DT firmware
>descriptions.
Thanks for clarifying. Beside this, are you ok with the xenstore node description in this file?
Thanks,
Peng.
>
>Ian.
>
>> see
>>
>> Documentation/arm64/arm-acpi.txt
>>
>>
>> "In DT, clocks need to be specified
>> and the drivers need to take them into account. In ACPI, the assumption
>> is that UEFI will leave the device in a reasonable default state,
>> including
>> any clock settings. If for some reason the driver needs to change a
>> clock
>> value, this can be done in an ACPI method; all the driver needs to do is
>> invoke the method and not concern itself with what the method needs to do
>> to change the clock. Changing the hardware can then take place over time
>> by changing what the ACPI method does, and not the driver.
>>
>> In DT, the parameters needed by the driver to set up clocks as in the
>> example
>> above are known as "bindings"; in ACPI, these are known as "Device
>> Properties"
>> and provided to a driver via the _DSD object."
>>
>>
>> However currently we don't have the ability to run ACPI in DomU guests
>> on ARM. Even if we had, there is no way to call native ACPI methods from
>> any guests other than Dom0, even on x86. We just have to hope that
>> clocks don't need to be reset on ACPI systems.
prev parent reply other threads:[~2016-01-20 13:53 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-20 8:31 [RFC V2] xen: interface: introduce pvclk interface Peng Fan
2016-01-20 9:05 ` Juergen Gross
2016-01-20 9:25 ` Peng Fan
2016-01-20 10:16 ` Jan Beulich
2016-01-20 10:40 ` Juergen Gross
2016-01-20 11:48 ` Peng Fan
2016-01-20 12:11 ` Juergen Gross
2016-01-20 14:13 ` Peng Fan
2016-01-20 10:14 ` Jan Beulich
2016-01-20 11:40 ` Peng Fan
2016-01-20 12:01 ` Jan Beulich
2016-01-20 14:05 ` Peng Fan
2016-01-20 14:16 ` Jan Beulich
2016-01-20 14:37 ` Peng Fan
2016-01-20 14:49 ` Ian Campbell
2016-01-20 14:52 ` Jan Beulich
2016-01-21 1:29 ` Peng Fan
2016-01-21 7:53 ` Jan Beulich
2016-01-21 8:59 ` Peng Fan
2016-01-21 10:19 ` Ian Campbell
2016-01-21 11:55 ` Peng Fan
2016-01-21 12:26 ` Ian Campbell
2016-01-21 12:35 ` Peng Fan
2016-01-21 12:49 ` Ian Campbell
2016-01-22 2:19 ` Peng Fan
2016-01-23 15:26 ` Peng Fan
2016-01-21 12:55 ` Stefano Stabellini
2016-01-21 13:11 ` Ian Campbell
2016-01-21 16:11 ` Stefano Stabellini
2016-01-22 2:51 ` Peng Fan
2016-01-21 10:21 ` Jan Beulich
2016-01-21 12:06 ` Peng Fan
2016-01-21 12:52 ` Jan Beulich
2016-01-22 1:56 ` Peng Fan
2016-01-22 7:36 ` Jan Beulich
2016-01-22 9:27 ` Peng Fan
2016-01-22 10:25 ` Jan Beulich
2016-01-22 12:12 ` Peng Fan
2016-01-22 12:33 ` Jan Beulich
2016-01-22 13:55 ` Stefano Stabellini
2016-01-20 12:06 ` Stefano Stabellini
2016-01-20 12:27 ` Ian Campbell
2016-01-20 13:52 ` Peng Fan [this message]
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=20160120135232.GA10911@linux-7smt.suse \
--to=van.freenix@gmail.com \
--cc=david.vrabel@citrix.com \
--cc=george.dunlap@citrix.com \
--cc=ian.campbell@citrix.com \
--cc=julien.grall@citrix.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xenproject.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).