aspeed/timer: use the APB frequency from the SCU
The timer controller can be driven by either an external 1MHz clock or by the APB clock. Today, the model makes the assumption that the APB frequency is always set to 24MHz but this is incorrect. The AST2400 SoC on the palmetto machines uses a 48MHz input clock source and the APB can be set to 48MHz. The consequence is a general system slowdown. The QEMU machines using the AST2500 SoC do not seem impacted today because the APB frequency is still set to 24MHz. We fix the timer frequency for all SoCs by linking the Timer model to the SCU model. The APB frequency driving the timers is now the one configured for the SoC. Signed-off-by: Cédric Le Goater <clg@kaod.org> Reviewed-by: Joel Stanley <joel@jms.id.au> Reviewed-by: Andrew Jeffery <andrew@aj.id.au> Message-id: 20180622075700.5923-4-clg@kaod.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
e2a11ca859
commit
9b945a9ee3
|
@ -127,6 +127,8 @@ static void aspeed_soc_init(Object *obj)
|
||||||
|
|
||||||
object_initialize(&s->timerctrl, sizeof(s->timerctrl), TYPE_ASPEED_TIMER);
|
object_initialize(&s->timerctrl, sizeof(s->timerctrl), TYPE_ASPEED_TIMER);
|
||||||
object_property_add_child(obj, "timerctrl", OBJECT(&s->timerctrl), NULL);
|
object_property_add_child(obj, "timerctrl", OBJECT(&s->timerctrl), NULL);
|
||||||
|
object_property_add_const_link(OBJECT(&s->timerctrl), "scu",
|
||||||
|
OBJECT(&s->scu), &error_abort);
|
||||||
qdev_set_parent_bus(DEVICE(&s->timerctrl), sysbus_get_default());
|
qdev_set_parent_bus(DEVICE(&s->timerctrl), sysbus_get_default());
|
||||||
|
|
||||||
object_initialize(&s->i2c, sizeof(s->i2c), TYPE_ASPEED_I2C);
|
object_initialize(&s->i2c, sizeof(s->i2c), TYPE_ASPEED_I2C);
|
||||||
|
|
|
@ -10,8 +10,10 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#include "qemu/osdep.h"
|
#include "qemu/osdep.h"
|
||||||
|
#include "qapi/error.h"
|
||||||
#include "hw/sysbus.h"
|
#include "hw/sysbus.h"
|
||||||
#include "hw/timer/aspeed_timer.h"
|
#include "hw/timer/aspeed_timer.h"
|
||||||
|
#include "hw/misc/aspeed_scu.h"
|
||||||
#include "qemu-common.h"
|
#include "qemu-common.h"
|
||||||
#include "qemu/bitops.h"
|
#include "qemu/bitops.h"
|
||||||
#include "qemu/timer.h"
|
#include "qemu/timer.h"
|
||||||
|
@ -26,7 +28,6 @@
|
||||||
#define TIMER_CLOCK_USE_EXT true
|
#define TIMER_CLOCK_USE_EXT true
|
||||||
#define TIMER_CLOCK_EXT_HZ 1000000
|
#define TIMER_CLOCK_EXT_HZ 1000000
|
||||||
#define TIMER_CLOCK_USE_APB false
|
#define TIMER_CLOCK_USE_APB false
|
||||||
#define TIMER_CLOCK_APB_HZ 24000000
|
|
||||||
|
|
||||||
#define TIMER_REG_STATUS 0
|
#define TIMER_REG_STATUS 0
|
||||||
#define TIMER_REG_RELOAD 1
|
#define TIMER_REG_RELOAD 1
|
||||||
|
@ -80,11 +81,11 @@ static inline bool timer_external_clock(AspeedTimer *t)
|
||||||
return timer_ctrl_status(t, op_external_clock);
|
return timer_ctrl_status(t, op_external_clock);
|
||||||
}
|
}
|
||||||
|
|
||||||
static uint32_t clock_rates[] = { TIMER_CLOCK_APB_HZ, TIMER_CLOCK_EXT_HZ };
|
|
||||||
|
|
||||||
static inline uint32_t calculate_rate(struct AspeedTimer *t)
|
static inline uint32_t calculate_rate(struct AspeedTimer *t)
|
||||||
{
|
{
|
||||||
return clock_rates[timer_external_clock(t)];
|
AspeedTimerCtrlState *s = timer_to_ctrl(t);
|
||||||
|
|
||||||
|
return timer_external_clock(t) ? TIMER_CLOCK_EXT_HZ : s->scu->apb_freq;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline uint32_t calculate_ticks(struct AspeedTimer *t, uint64_t now_ns)
|
static inline uint32_t calculate_ticks(struct AspeedTimer *t, uint64_t now_ns)
|
||||||
|
@ -449,6 +450,16 @@ static void aspeed_timer_realize(DeviceState *dev, Error **errp)
|
||||||
int i;
|
int i;
|
||||||
SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
|
SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
|
||||||
AspeedTimerCtrlState *s = ASPEED_TIMER(dev);
|
AspeedTimerCtrlState *s = ASPEED_TIMER(dev);
|
||||||
|
Object *obj;
|
||||||
|
Error *err = NULL;
|
||||||
|
|
||||||
|
obj = object_property_get_link(OBJECT(dev), "scu", &err);
|
||||||
|
if (!obj) {
|
||||||
|
error_propagate(errp, err);
|
||||||
|
error_prepend(errp, "required link 'scu' not found: ");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
s->scu = ASPEED_SCU(obj);
|
||||||
|
|
||||||
for (i = 0; i < ASPEED_TIMER_NR_TIMERS; i++) {
|
for (i = 0; i < ASPEED_TIMER_NR_TIMERS; i++) {
|
||||||
aspeed_init_one_timer(s, i);
|
aspeed_init_one_timer(s, i);
|
||||||
|
|
|
@ -24,6 +24,8 @@
|
||||||
|
|
||||||
#include "qemu/timer.h"
|
#include "qemu/timer.h"
|
||||||
|
|
||||||
|
typedef struct AspeedSCUState AspeedSCUState;
|
||||||
|
|
||||||
#define ASPEED_TIMER(obj) \
|
#define ASPEED_TIMER(obj) \
|
||||||
OBJECT_CHECK(AspeedTimerCtrlState, (obj), TYPE_ASPEED_TIMER);
|
OBJECT_CHECK(AspeedTimerCtrlState, (obj), TYPE_ASPEED_TIMER);
|
||||||
#define TYPE_ASPEED_TIMER "aspeed.timer"
|
#define TYPE_ASPEED_TIMER "aspeed.timer"
|
||||||
|
@ -55,6 +57,8 @@ typedef struct AspeedTimerCtrlState {
|
||||||
uint32_t ctrl;
|
uint32_t ctrl;
|
||||||
uint32_t ctrl2;
|
uint32_t ctrl2;
|
||||||
AspeedTimer timers[ASPEED_TIMER_NR_TIMERS];
|
AspeedTimer timers[ASPEED_TIMER_NR_TIMERS];
|
||||||
|
|
||||||
|
AspeedSCUState *scu;
|
||||||
} AspeedTimerCtrlState;
|
} AspeedTimerCtrlState;
|
||||||
|
|
||||||
#endif /* ASPEED_TIMER_H */
|
#endif /* ASPEED_TIMER_H */
|
||||||
|
|
Loading…
Reference in a new issue