[PATCH 2/2] clk: versal: Cleanup driver

Michal Simek michal.simek at amd.com
Mon Nov 10 13:33:33 CET 2025


Remove unneeded debug messages, parenthesis and fix error message.

Signed-off-by: Michal Simek <michal.simek at amd.com>
---

 drivers/clk/clk_versal.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/drivers/clk/clk_versal.c b/drivers/clk/clk_versal.c
index 139f429d609b..78a2410ca21c 100644
--- a/drivers/clk/clk_versal.c
+++ b/drivers/clk/clk_versal.c
@@ -738,8 +738,6 @@ static ulong versal_clk_get_rate(struct clk *clk)
 	if (id >= clock_max_idx)
 		return -ENODEV;
 
-	debug("%s\n", __func__);
-
 	clk_id = priv->clk[id].clk_id;
 
 	versal_clock_get_rate(clk_id, &clk_rate);
@@ -756,8 +754,6 @@ static ulong versal_clk_set_rate(struct clk *clk, ulong rate)
 	u32 div;
 	int ret;
 
-	debug("%s\n", __func__);
-
 	if (id >= clock_max_idx)
 		return -ENODEV;
 
@@ -783,7 +779,7 @@ static ulong versal_clk_set_rate(struct clk *clk, ulong rate)
 	} while (((clk_id >> NODE_SUBCLASS_SHIFT) &
 		 NODE_CLASS_MASK) != NODE_SUBCLASS_CLOCK_REF);
 
-	printf("Clock didn't has Divisors:0x%x\n", priv->clk[id].clk_id);
+	printf("Clock has no divider: 0x%x\n", clk_id);
 
 	return clk_rate;
 }
@@ -799,10 +795,9 @@ static int versal_clk_enable(struct clk *clk)
 
 	clk_id = priv->clk[id].clk_id;
 
-	if (versal_clock_gate(clk_id)) {
+	if (versal_clock_gate(clk_id))
 		return xilinx_pm_request(PM_CLOCK_ENABLE, clk_id, 0, 0, 0,
 					 0, 0, NULL);
-	}
 
 	return 0;
 }
-- 
2.43.0

base-commit: 04d178e16e6cf0209efa1b424a86c72067dea255


More information about the U-Boot mailing list