RE: [PATCH] i2c: designware: fix slave omitted IC_INTR_STOP_DET

[Michael.Wu]

On 10/14/20 6:53 PM, jarkko.nikula@xxxxxxxxxxxxxxx wrote:
> Thanks for the patch. I was thinking this too after your report but
> haven't found actually time to look at implementing it.
> 
> But what I was thinking it is probably good to have two patches. First
> patch that changes only i2c_dw_read_clear_intrbits_slave() semantics so
> that it's called only once like here and second patch that does other
> logic changes. Makes easier to catch possible regressions I think.
> 
> Jarkko

I can't agree with you more.
I'll separate them into two patches in next version.

By the way, I found a way to compile my patch by myself via
	"make multi_v7_defconfig"
and
	"make drivers/i2c/busses/i2c-designware-slave.o".

I've done and there were no warnings about
drivers/i2c/busses/i2c-designware-slave.c:13.
--
Michael Wu