Hi Bernhard, On 28/10/23 14:24, Bernhard Beschow wrote:
Signed-off-by: Bernhard Beschow <shen...@gmail.com> --- hw/misc/imx6_ccm.c | 41 ++++++++++++++--------------------------- hw/misc/trace-events | 15 +++++++++++++++ 2 files changed, 29 insertions(+), 27 deletions(-)
+# imx6_ccm.c +imx6_analog_get_periph_clk(uint32_t freq) "freq = %u"
Preferably explicit the unit, as "freq = %u Hz".
+imx6_analog_get_pll2_clk(uint32_t freq) "freq = %u" +imx6_analog_get_pll2_pfd0_clk(uint32_t freq) "freq = %u" +imx6_analog_get_pll2_pfd2_clk(uint32_t freq) "freq = %u"
+imx6_ccm_get_ahb_clk(uint32_t freq) "freq = %u" +imx6_ccm_get_ipg_clk(uint32_t freq) "freq = %u" +imx6_ccm_get_per_clk(uint32_t freq) "freq = %u" +imx6_ccm_get_clock_frequency(unsigned clock, uint32_t freq) "(Clock = %d) = %u"
'freq' is uint64_t, but I suppose 32-bit is enough, so I'm not against the implicit cast. Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>