[v3] clk: s3c2410: removed unneeded variable in s3c24xx_clkout_set_parent

Commit Message

Mutharaju, Prasanna (P.) Nov. 30, 2015, 10:08 a.m. UTC

From: Prasanna Karthik <[e-mail]>

Remove unneeded variable used to store return value.

Signed-off-by: Prasanna Karthik <[e-mail]>

---
v3: Modified subject and changelog comments
---
---
 drivers/clk/samsung/clk-s3c2410-dclk.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Krzysztof Kozlowski Nov. 30, 2015, 10:18 a.m. UTC | #1

2015-11-30 19:08 GMT+09:00 Mutharaju, Prasanna (P.) <[e-mail]>:
> From: Prasanna Karthik <[e-mail]>
>
> Remove unneeded variable used to store return value.
>
> Signed-off-by: Prasanna Karthik <[e-mail]>

Thanks,

Reviewed-by: Krzysztof Kozlowski <[e-mail]>

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to [e-mail]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Sylwester Nawrocki/Kernel \(PLT\) /SRPOL/Staff Engineer/Samsung Electronics Dec. 16, 2015, 6:03 p.m. UTC | #2

On 30/11/15 11:08, Mutharaju, Prasanna (P.) wrote:
> From: Prasanna Karthik <[e-mail]>
> 
> Remove unneeded variable used to store return value.
> 
> Signed-off-by: Prasanna Karthik <[e-mail]>

Patch applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to [e-mail]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/clk/samsung/clk-s3c2410-dclk.c b/drivers/clk/samsung/clk-s3c2410-dclk.c
index e9eb935..ec6fb14 100644
--- a/drivers/clk/samsung/clk-s3c2410-dclk.c
+++ b/drivers/clk/samsung/clk-s3c2410-dclk.c
@@ -77,12 +77,11 @@  static u8 s3c24xx_clkout_get_parent(struct clk_hw *hw)
 static int s3c24xx_clkout_set_parent(struct clk_hw *hw, u8 index)
 {
 	struct s3c24xx_clkout *clkout = to_s3c24xx_clkout(hw);
-	int ret = 0;
 
 	s3c2410_modify_misccr((clkout->mask << clkout->shift),
 			      (index << clkout->shift));
 
-	return ret;
+	return 0;
 }
 
 static const struct clk_ops s3c24xx_clkout_ops = {