Re: [PATCH v2 2/6] i2c: i2c-stm32f7: Add slave support

From: Wolfram Sang
Date: Tue Apr 03 2018 - 11:26:11 EST


On Mon, Mar 26, 2018 at 10:41:51AM +0200, Pierre Yves MORDRET wrote:
>
>
> On 03/25/2018 08:16 PM, Wolfram Sang wrote:
> > On Wed, Mar 21, 2018 at 05:48:56PM +0100, Pierre-Yves MORDRET wrote:
> >> This patch adds slave support for I2C controller embedded in STM32F7 SoC
> >>
> >> Signed-off-by: M'boumba Cedric Madianga <cedric.madianga@xxxxxxxxx>
> >> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@xxxxxx>
> >
> > Looks OK from a first look. What kind of tests did you do?
>
> As mentioned for 10-bit, I'm using 2 I2C instances from the SoC.
> Here are the tests I dit:
> - Master/slave send in 7 and 10 bits
> - master/slave recv in 7 and 10 bits
> - E2PROM Read/Write

As far as I understand the code now, both instances can be master /
slave simultanously on the same bus?

Attachment: signature.asc
Description: PGP signature