diff options
author | Philippe Mathieu-Daudé <f4bug@amsat.org> | 2020-08-28 10:02:46 +0100 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2020-08-28 10:02:46 +0100 |
commit | 739fa3255492f1c7541db1c7a9795fcf4b472c91 (patch) | |
tree | 66fd7f9d85ea237f11df037bd0b0344365c63ce2 /include | |
parent | f129360ca15723e43a8c44d1a2f025c0df1270cc (diff) | |
download | qemu-739fa3255492f1c7541db1c7a9795fcf4b472c91.zip qemu-739fa3255492f1c7541db1c7a9795fcf4b472c91.tar.gz qemu-739fa3255492f1c7541db1c7a9795fcf4b472c91.tar.bz2 |
hw/qdev-clock: Avoid calling qdev_connect_clock_in after DeviceRealize
Clock canonical name is set in device_set_realized (see the block
added to hw/core/qdev.c in commit 0e6934f264).
If we connect a clock after the device is realized, this code is
not executed. This is currently not a problem as this name is only
used for trace events, however this disrupt tracing.
Add a comment to document qdev_connect_clock_in() must be called
before the device is realized, and assert this condition.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-id: 20200803105647.22223-5-f4bug@amsat.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/hw/qdev-clock.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/hw/qdev-clock.h b/include/hw/qdev-clock.h index a897f7c..64ca4d2 100644 --- a/include/hw/qdev-clock.h +++ b/include/hw/qdev-clock.h @@ -70,6 +70,8 @@ Clock *qdev_get_clock_out(DeviceState *dev, const char *name); * * Set the source clock of input clock @name of device @dev to @source. * @source period update will be propagated to @name clock. + * + * Must be called before @dev is realized. */ void qdev_connect_clock_in(DeviceState *dev, const char *name, Clock *source); |