From 29701960b3ba792c8897cce23605ee79ec91c89e Mon Sep 17 00:00:00 2001 From: Mitchell Horne Date: Fri, 8 Mar 2024 10:08:22 -0400 Subject: [PATCH] clk_fixed: call clkdom_dump() for verbose boot This is standard practice for clock drivers that register clocks dynamically. Nothing else uses the CLK_DEBUG macro. The result is that the name and frequency of the fixed clock is printed for a verbose boot, which may aid in debugging. Reviewed by: manu MFC after: 1 week Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D44265 (cherry picked from commit 6e66bb9fc3d026765113fa1258cd12292da9309a) --- sys/dev/extres/clk/clk_fixed.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/sys/dev/extres/clk/clk_fixed.c b/sys/dev/extres/clk/clk_fixed.c index 1a61f0013bb..62c094a65da 100644 --- a/sys/dev/extres/clk/clk_fixed.c +++ b/sys/dev/extres/clk/clk_fixed.c @@ -263,9 +263,10 @@ clk_fixed_attach(device_t dev) rv = ENXIO; goto fail; } -#ifdef CLK_DEBUG - clkdom_dump(sc->clkdom); -#endif + + if (bootverbose) + clkdom_dump(sc->clkdom); + OF_prop_free(__DECONST(char *, def.clkdef.name)); OF_prop_free(def.clkdef.parent_names); return (bus_generic_attach(dev));