clk: zynqmp: Check the return type

Shubhrajyoti Datta · bebarino · commit 47d0fbd1cd42 · 2021-08-28T21:26:04.000-07:00
Currently the return value of of_clk_add_hw_provider is ignored. lets check and return value. Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com> Link: https://lore.kernel.org/r/20210818065929.12835-2-shubhrajyoti.datta@xilinx.com Signed-off-by: Stephen Boyd <sboyd@kernel.org>
diff --git a/drivers/clk/zynqmp/clkc.c b/drivers/clk/zynqmp/clkc.c
@@ -762,9 +762,7 @@ static int zynqmp_clk_setup(struct device_node *np)
 	zynqmp_register_clocks(np);
 
 	zynqmp_data->num = clock_max_idx;
-	of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data);
-
-	return 0;
+	return of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data);
 }
 
 static int zynqmp_clock_probe(struct platform_device *pdev)

Original file line number	Diff line number	Diff line change
`@@ -762,9 +762,7 @@ static int zynqmp_clk_setup(struct device_node *np)`
`762`	`762`	`zynqmp_register_clocks(np);`
`763`	`763`
`764`	`764`	`zynqmp_data->num = clock_max_idx;`
`765`		`- of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data);`
`766`		`-`
`767`		`- return 0;`
	`765`	`+ return of_clk_add_hw_provider(np, of_clk_hw_onecell_get, zynqmp_data);`
`768`	`766`	`}`
`769`	`767`
`770`	`768`	`static int zynqmp_clock_probe(struct platform_device *pdev)`