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

Chen Ni posted 1 patch 1 year, 7 months ago
drivers/clk/qcom/kpss-xcc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
[PATCH] clk: qcom: kpss-xcc: Return of_clk_add_hw_provider to transfer the error
Posted by Chen Ni 1 year, 7 months ago
Return of_clk_add_hw_provider() in order to transfer the error if it
fails.

Fixes: 09be1a39e685 ("clk: qcom: kpss-xcc: register it as clk provider")
Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
---
 drivers/clk/qcom/kpss-xcc.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/qcom/kpss-xcc.c b/drivers/clk/qcom/kpss-xcc.c
index 23b0b11f0007..e7cfa8d22044 100644
--- 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 = {
-- 
2.25.1
Re: [PATCH] clk: qcom: kpss-xcc: Return of_clk_add_hw_provider to transfer the error
Posted by Bjorn Andersson 1 year, 7 months ago
On Thu, 04 Jul 2024 15:36:06 +0800, Chen Ni wrote:
> Return of_clk_add_hw_provider() in order to transfer the error if it
> fails.
> 
> 

Applied, thanks!

[1/1] clk: qcom: kpss-xcc: Return of_clk_add_hw_provider to transfer the error
      commit: 9db4585eca22fcd0422a94ac792f87dcbf74b643

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>
Re: [PATCH] clk: qcom: kpss-xcc: Return of_clk_add_hw_provider to transfer the error
Posted by Dmitry Baryshkov 1 year, 7 months ago
On Thu, Jul 04, 2024 at 03:36:06PM GMT, Chen Ni wrote:
> Return of_clk_add_hw_provider() in order to transfer the error if it
> fails.
> 
> Fixes: 09be1a39e685 ("clk: qcom: kpss-xcc: register it as clk provider")
> Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
> ---
>  drivers/clk/qcom/kpss-xcc.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> 

-- 
With best wishes
Dmitry