From: Julien Grall <julien.grall@linaro.org>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Suriyan Ramasami <suriyan.r@gmail.com>, xen-devel@lists.xen.org
Subject: Re: ARM: EXYNOS 5410 - DOM0 not being scheduled
Date: Tue, 18 Mar 2014 12:04:12 +0000 [thread overview]
Message-ID: <5328363C.6030407@linaro.org> (raw)
In-Reply-To: <1395136344.12847.13.camel@kazak.uk.xensource.com>
Hi Ian,
On 03/18/2014 09:52 AM, Ian Campbell wrote:
> On Mon, 2014-03-17 at 22:15 +0000, Julien Grall wrote:
>>> 2. arch_timer_rate coming up as 0 in dom0 inspite of the dtb defining
>>> clock-frequency.
>>
>> The clock-frequency properties is not given to dom0. It should be able
>> to retrieve the frequency from the timer (CNTFRQ). That would mean that
>> u-boot is not correctly set up the frequency (it's the only one able to
>> do it as it need to be done in secure world).
>
> If the clock-frequency property is present in the host DT then that
> would imply that secure world is not setting CNTFRQ or is setting it
> wrong (since the property is effectively a workaround for that exact
> bug). Perhaps we should propagate it if it is present?
It's easy to do it for dom0, what about guests? I think it would be
better to let the bootloader to set up the frequency. I'm a bit
surprised that it's not already the case for this board.
For instance, on the Arndale the property "clock-frequency" exists but
CNTFRQ is also correctly set.
> While grepping about this I noticed that make_timer_node doesn't use
> DT_MATCH_TIMER, which I guess is just an accidental oversight, perhaps
> if someone is fixing the clock-frequency issue they could throw in a
> second patch for this too?
I think it's an error during the cleanup.
Regards,
--
Julien Grall
next prev parent reply other threads:[~2014-03-18 12:04 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-15 14:44 ARM: EXYNOS 5410 - DOM0 not being scheduled Suriyan Ramasami
2014-03-16 20:31 ` Julien Grall
2014-03-17 13:17 ` Suriyan Ramasami
2014-03-17 13:21 ` Julien Grall
2014-03-17 13:32 ` Suriyan Ramasami
2014-03-17 13:52 ` Julien Grall
2014-03-17 17:39 ` Suriyan Ramasami
2014-03-17 18:10 ` Suriyan Ramasami
2014-03-17 18:22 ` Suriyan Ramasami
2014-03-17 22:15 ` Julien Grall
2014-03-18 9:52 ` Ian Campbell
2014-03-18 12:04 ` Julien Grall [this message]
2014-03-18 13:37 ` Ian Campbell
2014-03-18 15:41 ` Suriyan Ramasami
2014-03-18 16:09 ` Julien Grall
2014-03-17 22:03 ` Julien Grall
2014-03-17 10:09 ` Ian Campbell
2014-03-17 13:24 ` Suriyan Ramasami
2014-03-17 13:36 ` Ian Campbell
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=5328363C.6030407@linaro.org \
--to=julien.grall@linaro.org \
--cc=Ian.Campbell@citrix.com \
--cc=suriyan.r@gmail.com \
--cc=xen-devel@lists.xen.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.