clk: socfpga: remove redundant assignment on division

ColinIanKing · bebarino · commit 08d92c7a4737 · 2022-01-05T16:32:43.000-08:00
The variable parent_rate is being divided by div and the result is re-assigned to parent_rate before being returned. The assignment is redundant, replace /= operator with just / operator. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Link: https://lore.kernel.org/r/20211221003750.212780-1-colin.i.king@gmail.com Acked-by: Dinh Nguyen <dinguyen@kernel.org> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
diff --git a/drivers/clk/socfpga/clk-pll-s10.c b/drivers/clk/socfpga/clk-pll-s10.c
@@ -113,7 +113,7 @@ static unsigned long clk_boot_clk_recalc_rate(struct clk_hw *hwclk,
 		SWCTRLBTCLKSEL_MASK) >>
 		SWCTRLBTCLKSEL_SHIFT);
 	div += 1;
-	return parent_rate /= div;
+	return parent_rate / div;
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ static unsigned long clk_boot_clk_recalc_rate(struct clk_hw *hwclk,`
`113`	`113`	`SWCTRLBTCLKSEL_MASK) >>`
`114`	`114`	`SWCTRLBTCLKSEL_SHIFT);`
`115`	`115`	`div += 1;`
`116`		`- return parent_rate /= div;`
	`116`	`+ return parent_rate / div;`
`117`	`117`	`}`
`118`	`118`
`119`	`119`