clk: qcom: kpss-xcc: Return of_clk_add_hw_provider to transfer the error

Chen Ni · andersson · commit 9db4585eca22 · 2024-07-06T12:58:02.000-05:00
Return of_clk_add_hw_provider() in order to transfer the error if it fails. Fixes: 09be1a3 ("clk: qcom: kpss-xcc: register it as clk provider") Signed-off-by: Chen Ni <nichen@iscas.ac.cn> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Link: https://lore.kernel.org/r/20240704073606.1976936-1-nichen@iscas.ac.cn Signed-off-by: Bjorn Andersson <andersson@kernel.org>
diff --git a/drivers/clk/qcom/kpss-xcc.c b/drivers/clk/qcom/kpss-xcc.c
@@ -58,9 +58,7 @@ static int kpss_xcc_driver_probe(struct platform_device *pdev)
 	if (IS_ERR(hw))
 		return PTR_ERR(hw);
 
-	of_clk_add_hw_provider(dev->of_node, of_clk_hw_simple_get, hw);
-
-	return 0;
+	return of_clk_add_hw_provider(dev->of_node, of_clk_hw_simple_get, hw);
 }
 
 static struct platform_driver kpss_xcc_driver = {

Original file line number	Diff line number	Diff line change
`@@ -58,9 +58,7 @@ static int kpss_xcc_driver_probe(struct platform_device *pdev)`
`58`	`58`	`if (IS_ERR(hw))`
`59`	`59`	`return PTR_ERR(hw);`
`60`	`60`
`61`		`- of_clk_add_hw_provider(dev->of_node, of_clk_hw_simple_get, hw);`
`62`		`-`
`63`		`- return 0;`
	`61`	`+ return of_clk_add_hw_provider(dev->of_node, of_clk_hw_simple_get, hw);`
`64`	`62`	`}`
`65`	`63`
`66`	`64`	`static struct platform_driver kpss_xcc_driver = {`