Message ID | cover.ec85a2db47c3e10de70fba144e1bc75caf2f0a91.1576215453.git-series.andrew@aj.id.au |
---|---|
Headers | show |
Series | Expose GT CNTFRQ as a CPU property to support AST2600 | expand |
On Fri, 13 Dec 2019 at 05:48, Andrew Jeffery <andrew@aj.id.au> wrote: > > Hello, > > This is a v3 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. > > v2 can be found here: > > https://patchwork.ozlabs.org/cover/1203474/ > > Changes since v2: > > * Address some minor review comments from Philippe and add tags > > 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 Applied to target-arm.next, thanks. -- PMM
On Wed, 18 Dec 2019, at 01:55, Peter Maydell wrote: > On Fri, 13 Dec 2019 at 05:48, Andrew Jeffery <andrew@aj.id.au> wrote: > > > > Hello, > > > > This is a v3 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. > > > > v2 can be found here: > > > > https://patchwork.ozlabs.org/cover/1203474/ > > > > Changes since v2: > > > > * Address some minor review comments from Philippe and add tags > > > > 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 > > > > > Applied to target-arm.next, thanks. Thanks for your feedback throughout. Andrew