[PATCH] clk: clocking-wizard: Fix output clock register offset for Versal platforms

Shubhrajyoti Datta posted 1 patch 6 months, 4 weeks ago
drivers/clk/xilinx/clk-xlnx-clock-wizard.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] clk: clocking-wizard: Fix output clock register offset for Versal platforms
Posted by Shubhrajyoti Datta 6 months, 4 weeks ago
The output clock register offset used in clk_wzrd_register_output_clocks
was incorrectly referencing 0x3C instead of 0x38, which caused
misconfiguration of output dividers on Versal platforms.

Correcting the off-by-one error ensures proper configuration of output
clocks.

Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
---

 drivers/clk/xilinx/clk-xlnx-clock-wizard.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/xilinx/clk-xlnx-clock-wizard.c b/drivers/clk/xilinx/clk-xlnx-clock-wizard.c
index 8abf12f88eb2..4f8ed6d1e5fd 100644
--- a/drivers/clk/xilinx/clk-xlnx-clock-wizard.c
+++ b/drivers/clk/xilinx/clk-xlnx-clock-wizard.c
@@ -1138,7 +1138,7 @@ static int clk_wzrd_register_output_clocks(struct device *dev, int nr_outputs)
 						(dev,
 						 clkout_name, clk_name, 0,
 						 clk_wzrd->base,
-						 (WZRD_CLK_CFG_REG(is_versal, 3) + i * 8),
+						 (WZRD_CLK_CFG_REG(is_versal, 2) + i * 8),
 						 WZRD_CLKOUT_DIVIDE_SHIFT,
 						 WZRD_CLKOUT_DIVIDE_WIDTH,
 						 CLK_DIVIDER_ONE_BASED |
-- 
2.17.1
Re: [PATCH] clk: clocking-wizard: Fix output clock register offset for Versal platforms
Posted by Stephen Boyd 6 months, 1 week ago
Quoting Shubhrajyoti Datta (2025-09-05 02:10:02)
> The output clock register offset used in clk_wzrd_register_output_clocks
> was incorrectly referencing 0x3C instead of 0x38, which caused
> misconfiguration of output dividers on Versal platforms.
> 
> Correcting the off-by-one error ensures proper configuration of output
> clocks.
> 
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
> ---

Applied to clk-next