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

Wolfram Sang
2018-04-03T15:26:04Z

[-- Attachment #1: Type: text/plain, Size: 816 bytes --]

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@gmail.com>
> >> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
> >
> > 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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]