[v2] Expose GT CNTFRQ as a CPU property to support AST2600

[PATCH v2 0/4] Expose GT CNTFRQ as a CPU property to support AST2600

Andrew Jeffery posted 4 patches 4 years, 4 months ago

Diff against v1 v3
Download series mbox

Test asan passed

Test checkpatch passed

Test FreeBSD passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test docker-quick@centos7 passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191203041440.6275-1-andrew@aj.id.au

Maintainers: Peter Maydell <peter.maydell@linaro.org>

There is a newer version of this series

hw/arm/aspeed_ast2600.c |  3 +++
target/arm/cpu.c        | 41 +++++++++++++++++++++++++++++++++--------
target/arm/cpu.h        | 28 ++++++++++++++++++++++++++++
target/arm/helper.c     | 24 ++++++++++++++++++------
4 files changed, 82 insertions(+), 14 deletions(-)

Expand all Fold all

[PATCH v2 0/4] Expose GT CNTFRQ as a CPU property to support AST2600

Posted by Andrew Jeffery 4 years, 4 months ago

Hello,

This is a v2 of the belated follow-up from a few of my earlier attempts to fix
up the ARM generic timer for correct behaviour on the ASPEED AST2600 SoC. The
AST2600 clocks the generic timer at the rate of HPLL, which is configured to
1125MHz.  This is significantly quicker than the currently hard-coded generic
timer rate of 62.5MHz and so we see "sticky" behaviour in the guest.

v1 can be found here:

https://patchwork.ozlabs.org/cover/1201887/

Changes since v1:

* Fix a user mode build failure from partial renaming of gt_cntfrq_period_ns()
* Add tags from Cedric and Richard

Please review.

Andrew

Andrew Jeffery (4):
  target/arm: Remove redundant scaling of nexttick
  target/arm: Abstract the generic timer frequency
  target/arm: Prepare generic timer for per-platform CNTFRQ
  ast2600: Configure CNTFRQ at 1125MHz

 hw/arm/aspeed_ast2600.c |  3 +++
 target/arm/cpu.c        | 41 +++++++++++++++++++++++++++++++++--------
 target/arm/cpu.h        | 28 ++++++++++++++++++++++++++++
 target/arm/helper.c     | 24 ++++++++++++++++++------
 4 files changed, 82 insertions(+), 14 deletions(-)

-- 
2.20.1

Re: [PATCH v2 0/4] Expose GT CNTFRQ as a CPU property to support AST2600

Posted by Philippe Mathieu-Daudé 4 years, 4 months ago

On 12/3/19 5:14 AM, Andrew Jeffery wrote:
> Hello,
> 
> This is a v2 of the belated follow-up from a few of my earlier attempts to fix
> up the ARM generic timer for correct behaviour on the ASPEED AST2600 SoC. The
> AST2600 clocks the generic timer at the rate of HPLL, which is configured to
> 1125MHz.  This is significantly quicker than the currently hard-coded generic
> timer rate of 62.5MHz and so we see "sticky" behaviour in the guest.

Glad you fixed this! I hit the same problem with the Raspi4.