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: firefly_0821_release~1362 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=3614d69125af1d0d1ebd29a755b7888a950a6367;p=firefly-linux-kernel-4.4.55.git i2c: rk3x: Give the tuning value 0 during rk3x_i2c_v0_calc_timings 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. Change-Id: I8686b8525e7fc8adc896f60dec4ae74d6c2a173c Signed-off-by: David Wu Tested-by: Andy Yan Reviewed-by: Douglas Anderson Signed-off-by: Wolfram Sang Cc: stable@kernel.org --- diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c index 7b65a2218685..df8a4b20d566 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;