On 9/17/19 12:14 PM, Peter Maydell wrote: >> +static Property arm_cpu_gt_cntfrq_property = >> + DEFINE_PROP_UINT64("cntfrq", ARMCPU, gt_cntfrq, >> + (1000 * 1000 * 1000) / GTIMER_SCALE); > I think it would be helpful to have a comment saynig what units > this property is in. >
Should this be NANOSECONDS_PER_SECOND? It's certainly a suspicious use of 1e9 otherwise. r~