1 From f606aab3f1a49d723d66e14e545f6ca45005bda6 Mon Sep 17 00:00:00 2001
2 From: Kewei Xu <kewei.xu@mediatek.com>
3 Date: Thu, 17 Feb 2022 20:22:43 +0800
4 Subject: [PATCH 04/16] i2c: mediatek: modify bus speed calculation formula
6 When clock-div is 0 or greater than 1, the bus speed
7 calculated by the old speed calculation formula will be
8 larger than the target speed. So we update the formula.
10 Signed-off-by: Kewei Xu <kewei.xu@mediatek.com>
11 Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
12 Reviewed-by: Qii Wang <qii.wang@mediatek.com>
13 Signed-off-by: Wolfram Sang <wsa@kernel.org>
15 drivers/i2c/busses/i2c-mt65xx.c | 51 ++++++++++++++++++++++++++-------
16 1 file changed, 41 insertions(+), 10 deletions(-)
18 --- a/drivers/i2c/busses/i2c-mt65xx.c
19 +++ b/drivers/i2c/busses/i2c-mt65xx.c
22 #define MAX_SAMPLE_CNT_DIV 8
23 #define MAX_STEP_CNT_DIV 64
24 -#define MAX_CLOCK_DIV 256
25 +#define MAX_CLOCK_DIV_8BITS 256
26 +#define MAX_CLOCK_DIV_5BITS 32
27 #define MAX_HS_STEP_CNT_DIV 8
28 -#define I2C_STANDARD_MODE_BUFFER (1000 / 2)
29 -#define I2C_FAST_MODE_BUFFER (300 / 2)
30 -#define I2C_FAST_MODE_PLUS_BUFFER (20 / 2)
31 +#define I2C_STANDARD_MODE_BUFFER (1000 / 3)
32 +#define I2C_FAST_MODE_BUFFER (300 / 3)
33 +#define I2C_FAST_MODE_PLUS_BUFFER (20 / 3)
35 #define I2C_CONTROL_RS (0x1 << 1)
36 #define I2C_CONTROL_DMA_EN (0x1 << 2)
37 @@ -604,6 +605,31 @@ static int mtk_i2c_max_step_cnt(unsigned
38 return MAX_STEP_CNT_DIV;
41 +static int mtk_i2c_get_clk_div_restri(struct mtk_i2c *i2c,
42 + unsigned int sample_cnt)
44 + int clk_div_restri = 0;
46 + if (i2c->dev_comp->ltiming_adjust == 0)
49 + if (sample_cnt == 1) {
50 + if (i2c->ac_timing.inter_clk_div == 0)
55 + if (i2c->ac_timing.inter_clk_div == 0)
56 + clk_div_restri = -1;
57 + else if (i2c->ac_timing.inter_clk_div == 1)
63 + return clk_div_restri;
67 * Check and Calculate i2c ac-timing
69 @@ -732,6 +758,7 @@ static int mtk_i2c_calculate_speed(struc
70 unsigned int best_mul;
73 + int clk_div_restri = 0;
75 if (target_speed > I2C_MAX_HIGH_SPEED_MODE_FREQ)
76 target_speed = I2C_MAX_HIGH_SPEED_MODE_FREQ;
77 @@ -749,7 +776,8 @@ static int mtk_i2c_calculate_speed(struc
78 * optimizing for sample_cnt * step_cnt being minimal
80 for (sample_cnt = 1; sample_cnt <= MAX_SAMPLE_CNT_DIV; sample_cnt++) {
81 - step_cnt = DIV_ROUND_UP(opt_div, sample_cnt);
82 + clk_div_restri = mtk_i2c_get_clk_div_restri(i2c, sample_cnt);
83 + step_cnt = DIV_ROUND_UP(opt_div + clk_div_restri, sample_cnt);
84 cnt_mul = step_cnt * sample_cnt;
85 if (step_cnt > max_step_cnt)
87 @@ -763,7 +791,7 @@ static int mtk_i2c_calculate_speed(struc
89 base_sample_cnt = sample_cnt;
90 base_step_cnt = step_cnt;
91 - if (best_mul == opt_div)
92 + if (best_mul == (opt_div + clk_div_restri))
96 @@ -774,7 +802,8 @@ static int mtk_i2c_calculate_speed(struc
97 sample_cnt = base_sample_cnt;
98 step_cnt = base_step_cnt;
100 - if ((clk_src / (2 * sample_cnt * step_cnt)) > target_speed) {
101 + if ((clk_src / (2 * (sample_cnt * step_cnt - clk_div_restri))) >
103 /* In this case, hardware can't support such
106 @@ -803,13 +832,16 @@ static int mtk_i2c_set_speed(struct mtk_
107 target_speed = i2c->speed_hz;
108 parent_clk /= i2c->clk_src_div;
110 - if (i2c->dev_comp->timing_adjust)
111 - max_clk_div = MAX_CLOCK_DIV;
112 + if (i2c->dev_comp->timing_adjust && i2c->dev_comp->ltiming_adjust)
113 + max_clk_div = MAX_CLOCK_DIV_5BITS;
114 + else if (i2c->dev_comp->timing_adjust)
115 + max_clk_div = MAX_CLOCK_DIV_8BITS;
119 for (clk_div = 1; clk_div <= max_clk_div; clk_div++) {
120 clk_src = parent_clk / clk_div;
121 + i2c->ac_timing.inter_clk_div = clk_div - 1;
123 if (target_speed > I2C_MAX_FAST_MODE_PLUS_FREQ) {
124 /* Set master code speed register */
125 @@ -856,7 +888,6 @@ static int mtk_i2c_set_speed(struct mtk_
129 - i2c->ac_timing.inter_clk_div = clk_div - 1;