clk: rp1: Fix rp1_pll_divider_determine_rate

pelwell · pelwell · commit 6d7283b90220 · 2025-12-18T18:37:55.000Z
The determine_rate member of clk_ops returns the rate to the caller by
modifying the pass-by-reference req structure. Its actual return value
is a status code.

Signed-off-by: Phil Elwell &lt;phil@raspberrypi.com&gt;
diff --git a/drivers/clk/clk-rp1.c b/drivers/clk/clk-rp1.c
@@ -771,9 +771,7 @@ static unsigned long rp1_pll_divider_recalc_rate(struct clk_hw *hw,
 static int rp1_pll_divider_determine_rate(struct clk_hw *hw,
 					  struct clk_rate_request *req)
 {
-	req->rate = clk_divider_ops.determine_rate(hw, req);
-
-	return 0;
+	return clk_divider_ops.determine_rate(hw, req);
 }
 
 static int rp1_clock_is_on(struct clk_hw *hw)

Original file line number	Diff line number	Diff line change
`@@ -771,9 +771,7 @@ static unsigned long rp1_pll_divider_recalc_rate(struct clk_hw *hw,`
`771`	`771`	`static int rp1_pll_divider_determine_rate(struct clk_hw *hw,`
`772`	`772`	`struct clk_rate_request *req)`
`773`	`773`	`{`
`774`		`- req->rate = clk_divider_ops.determine_rate(hw, req);`
`775`		`-`
`776`		`- return 0;`
	`774`	`+ return clk_divider_ops.determine_rate(hw, req);`
`777`	`775`	`}`
`778`	`776`
`779`	`777`	`static int rp1_clock_is_on(struct clk_hw *hw)`