From: Patrice Chotard <patrice.chotard@st.com>
Date: Tue, 17 Oct 2017 09:21:33 +0000 (+0200)
Subject: i2c: stm32f7_i2c: fix usage of useless local variable
X-Git-Tag: v2017.11-rc3~45^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d10bd6cfd85c2a9f27c950e22e83d27ff978e7e8;p=platform%2Fkernel%2Fu-boot.git

i2c: stm32f7_i2c: fix usage of useless local variable

Remove useless local variable "s" and use directly
function's parameter "output"

Signed-off-by: Patrice Chotard <patrice.chotard@st.com>
---

diff --git a/drivers/i2c/stm32f7_i2c.c b/drivers/i2c/stm32f7_i2c.c
index 346748d..8662487 100644
--- a/drivers/i2c/stm32f7_i2c.c
+++ b/drivers/i2c/stm32f7_i2c.c
@@ -640,7 +640,7 @@ static int stm32_i2c_compute_timing(struct stm32_i2c_priv *i2c_priv,
 				      struct stm32_i2c_setup *setup,
 				      struct stm32_i2c_timings *output)
 {
-	struct stm32_i2c_timings *v, *_v, s;
+	struct stm32_i2c_timings *v, *_v;
 	struct list_head solutions;
 	int ret;
 
@@ -676,16 +676,10 @@ static int stm32_i2c_compute_timing(struct stm32_i2c_priv *i2c_priv,
 	if (ret)
 		goto exit;
 
-	ret = stm32_i2c_choose_solution(setup, &solutions, &s);
+	ret = stm32_i2c_choose_solution(setup, &solutions, output);
 	if (ret)
 		goto exit;
 
-	output->presc = s.presc;
-	output->scldel = s.scldel;
-	output->sdadel = s.sdadel;
-	output->scll = s.scll;
-	output->sclh = s.sclh;
-
 	debug("%s: Presc: %i, scldel: %i, sdadel: %i, scll: %i, sclh: %i\n",
 	      __func__, output->presc,
 	      output->scldel, output->sdadel,