From: David Wu Date: Sat, 22 Oct 2016 08:43:42 +0000 (+0800) Subject: i2c: rk3x: Give the tuning value 0 during rk3x_i2c_v0_calc_timings X-Git-Tag: v4.8.7~138 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9c619c2a8d4d14c7e9fe741239b7b42f610f09d9;p=thirdparty%2Fkernel%2Fstable.git i2c: rk3x: Give the tuning value 0 during rk3x_i2c_v0_calc_timings commit 399c168ab5ab5e12ed55b6c91d61c24eb84c9164 upstream. We found a bug that i2c transfer sometimes failed on 3066a board with stabel-4.8, the con register would be updated by uninitialized tuning value, it made the i2c transfer failed. So give the tuning value to be zero during rk3x_i2c_v0_calc_timings. Signed-off-by: David Wu Tested-by: Andy Yan Reviewed-by: Douglas Anderson Signed-off-by: Wolfram Sang Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c index 5c5b7cada8bee..dfae43523d345 100644 --- a/drivers/i2c/busses/i2c-rk3x.c +++ b/drivers/i2c/busses/i2c-rk3x.c @@ -694,6 +694,8 @@ static int rk3x_i2c_v0_calc_timings(unsigned long clk_rate, t_calc->div_low--; t_calc->div_high--; + /* Give the tuning value 0, that would not update con register */ + t_calc->tuning = 0; /* Maximum divider supported by hw is 0xffff */ if (t_calc->div_low > 0xffff) { t_calc->div_low = 0xffff;