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

From: Pierre Yves MORDRET
Date: Wed Apr 04 2018 - 04:13:49 EST




On 04/03/2018 05:26 PM, Wolfram Sang wrote:
> 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?
>

Correct.