Message ID | 1551885142-22365-1-git-send-email-bich.hemon@st.com |
---|---|
State | Accepted |
Headers | show |
Series | i2c: i2c-stm32f7: improve loopback in timing algorithm | expand |
Hi Reviewed-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com> Thanks On 3/6/19 4:12 PM, Bich HEMON wrote: > From: Nicolas Le Bayon <nicolas.le.bayon@st.com> > > This avoids useless loops inside the I2C timing algorithm. > Actually, we support only one possible solution per prescaler value. > So after finding a solution with a prescaler, the algorithm can > switch directly to the next prescaler value. > > Signed-off-by: Nicolas Le Bayon <nicolas.le.bayon@st.com> > Signed-off-by: Bich Hemon <bich.hemon@st.com> > --- > drivers/i2c/busses/i2c-stm32f7.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-stm32f7.c b/drivers/i2c/busses/i2c-stm32f7.c > index 4284fc9..48337be 100644 > --- a/drivers/i2c/busses/i2c-stm32f7.c > +++ b/drivers/i2c/busses/i2c-stm32f7.c > @@ -476,8 +476,12 @@ static int stm32f7_i2c_compute_timing(struct stm32f7_i2c_dev *i2c_dev, > > list_add_tail(&v->node, > &solutions); > + break; > } > } > + > + if (p_prev == p) > + break; > } > } > >
On Wed, Mar 06, 2019 at 03:12:32PM +0000, Bich HEMON wrote: > From: Nicolas Le Bayon <nicolas.le.bayon@st.com> > > This avoids useless loops inside the I2C timing algorithm. > Actually, we support only one possible solution per prescaler value. > So after finding a solution with a prescaler, the algorithm can > switch directly to the next prescaler value. > > Signed-off-by: Nicolas Le Bayon <nicolas.le.bayon@st.com> > Signed-off-by: Bich Hemon <bich.hemon@st.com> Applied to for-next, thanks! Pierre-Yves, there are two binding patches for stm32 still open, if you have some time for it...
diff --git a/drivers/i2c/busses/i2c-stm32f7.c b/drivers/i2c/busses/i2c-stm32f7.c index 4284fc9..48337be 100644 --- a/drivers/i2c/busses/i2c-stm32f7.c +++ b/drivers/i2c/busses/i2c-stm32f7.c @@ -476,8 +476,12 @@ static int stm32f7_i2c_compute_timing(struct stm32f7_i2c_dev *i2c_dev, list_add_tail(&v->node, &solutions); + break; } } + + if (p_prev == p) + break; } }