From ec956adf378a4b3bf133a2c1b6614c4891743843 Mon Sep 17 00:00:00 2001 From: David Lechner Date: Thu, 8 Jan 2026 12:05:54 -0600 Subject: [PATCH 1/2] clk: clk-uclass: fix format specifier for ofnode name Change the format specifier from %p to %s when printing the ofnode name so that the actual name is printed instead of the pointer address. Signed-off-by: David Lechner --- drivers/clk/clk-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c index 0584429bed6..0e56d207b0e 100644 --- a/drivers/clk/clk-uclass.c +++ b/drivers/clk/clk-uclass.c @@ -410,7 +410,7 @@ int clk_get_by_name_nodev(ofnode node, const char *name, struct clk *clk) { int index = 0; - debug("%s(node=%p, name=%s, clk=%p)\n", __func__, + debug("%s(node=%s, name=%s, clk=%p)\n", __func__, ofnode_get_name(node), name, clk); clk->dev = NULL; From 2540dd7be91608af2df7138e9fd60aa9778b7290 Mon Sep 17 00:00:00 2001 From: David Lechner Date: Thu, 8 Jan 2026 12:05:55 -0600 Subject: [PATCH 2/2] clk: clk-uclass: used dev name in debug message Consistently use the device name in debug messages. The clk-uclass file previously had a mix of printing the dev pointer and the device name. Changing all to use the device name makes the debug messages more useful. Signed-off-by: David Lechner --- drivers/clk/clk-uclass.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c index 0e56d207b0e..c33f415917e 100644 --- a/drivers/clk/clk-uclass.c +++ b/drivers/clk/clk-uclass.c @@ -122,7 +122,8 @@ static int clk_get_by_indexed_prop(struct udevice *dev, const char *prop_name, int ret; struct ofnode_phandle_args args; - debug("%s(dev=%p, index=%d, clk=%p)\n", __func__, dev, index, clk); + debug("%s(dev=%s, index=%d, clk=%p)\n", __func__, dev_read_name(dev), + index, clk); assert(clk); clk->dev = NULL; @@ -183,8 +184,8 @@ int clk_get_bulk(struct udevice *dev, struct clk_bulk *bulk) bulk_get_err: err = clk_release_all(bulk->clks, bulk->count); if (err) - debug("%s: could not release all clocks for %p\n", - __func__, dev); + debug("%s: could not release all clocks for %s\n", + __func__, dev_read_name(dev)); return ret; } @@ -217,8 +218,8 @@ static int clk_set_default_parents(struct udevice *dev, num_parents = dev_count_phandle_with_args(dev, "assigned-clock-parents", "#clock-cells", 0); if (num_parents < 0) { - debug("%s: could not read assigned-clock-parents for %p\n", - __func__, dev); + debug("%s: could not read assigned-clock-parents for %s\n", + __func__, dev_read_name(dev)); return 0; } @@ -467,7 +468,7 @@ int clk_request(struct udevice *dev, struct clk *clk) { const struct clk_ops *ops; - debug("%s(dev=%p, clk=%p)\n", __func__, dev, clk); + debug("%s(dev=%s, clk=%p)\n", __func__, dev_read_name(dev), clk); if (!clk) return 0; ops = clk_dev_ops(dev);