mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-07-21 10:24:33 +00:00
hw/clock: Let clock_set_mul_div() return a boolean value
Let clock_set_mul_div() return a boolean value whether the clock has been updated or not, similarly to clock_set(). Return early when clock_set_mul_div() is called with same mul/div values the clock has. Acked-by: Luc Michel <luc@lmichel.fr> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Message-Id: <20240325152827.73817-2-philmd@linaro.org>
This commit is contained in:
parent
e66d741467
commit
52405b7f69
|
@ -279,6 +279,10 @@ You can change the multiplier and divider of a clock at runtime,
|
||||||
so you can use this to model clock controller devices which
|
so you can use this to model clock controller devices which
|
||||||
have guest-programmable frequency multipliers or dividers.
|
have guest-programmable frequency multipliers or dividers.
|
||||||
|
|
||||||
|
Similary to ``clock_set()``, ``clock_set_mul_div()`` returns ``true`` if
|
||||||
|
the clock state was modified; that is, if the multiplier or the diviser
|
||||||
|
or both were changed by the call.
|
||||||
|
|
||||||
Note that ``clock_set_mul_div()`` does not automatically call
|
Note that ``clock_set_mul_div()`` does not automatically call
|
||||||
``clock_propagate()``. If you make a runtime change to the
|
``clock_propagate()``. If you make a runtime change to the
|
||||||
multiplier or divider you must call clock_propagate() yourself.
|
multiplier or divider you must call clock_propagate() yourself.
|
||||||
|
|
|
@ -143,14 +143,20 @@ char *clock_display_freq(Clock *clk)
|
||||||
return freq_to_str(clock_get_hz(clk));
|
return freq_to_str(clock_get_hz(clk));
|
||||||
}
|
}
|
||||||
|
|
||||||
void clock_set_mul_div(Clock *clk, uint32_t multiplier, uint32_t divider)
|
bool clock_set_mul_div(Clock *clk, uint32_t multiplier, uint32_t divider)
|
||||||
{
|
{
|
||||||
assert(divider != 0);
|
assert(divider != 0);
|
||||||
|
|
||||||
|
if (clk->multiplier == multiplier && clk->divider == divider) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
trace_clock_set_mul_div(CLOCK_PATH(clk), clk->multiplier, multiplier,
|
trace_clock_set_mul_div(CLOCK_PATH(clk), clk->multiplier, multiplier,
|
||||||
clk->divider, divider);
|
clk->divider, divider);
|
||||||
clk->multiplier = multiplier;
|
clk->multiplier = multiplier;
|
||||||
clk->divider = divider;
|
clk->divider = divider;
|
||||||
|
|
||||||
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void clock_initfn(Object *obj)
|
static void clock_initfn(Object *obj)
|
||||||
|
|
|
@ -357,6 +357,8 @@ char *clock_display_freq(Clock *clk);
|
||||||
* @multiplier: multiplier value
|
* @multiplier: multiplier value
|
||||||
* @divider: divider value
|
* @divider: divider value
|
||||||
*
|
*
|
||||||
|
* @return: true if the clock is changed.
|
||||||
|
*
|
||||||
* By default, a Clock's children will all run with the same period
|
* By default, a Clock's children will all run with the same period
|
||||||
* as their parent. This function allows you to adjust the multiplier
|
* as their parent. This function allows you to adjust the multiplier
|
||||||
* and divider used to derive the child clock frequency.
|
* and divider used to derive the child clock frequency.
|
||||||
|
@ -374,6 +376,6 @@ char *clock_display_freq(Clock *clk);
|
||||||
* Note that this function does not call clock_propagate(); the
|
* Note that this function does not call clock_propagate(); the
|
||||||
* caller should do that if necessary.
|
* caller should do that if necessary.
|
||||||
*/
|
*/
|
||||||
void clock_set_mul_div(Clock *clk, uint32_t multiplier, uint32_t divider);
|
bool clock_set_mul_div(Clock *clk, uint32_t multiplier, uint32_t divider);
|
||||||
|
|
||||||
#endif /* QEMU_HW_CLOCK_H */
|
#endif /* QEMU_HW_CLOCK_H */
|
||||||
|
|
Loading…
Reference in a new issue