Re: [PATCH] i2c: mediatek: Add to support continuous mode

2020-06-30 Thread Yingjoe Chen
Hi Qiangming, When you send new version, you should - Add version number in subject, so we know this is a new one. - List what's changed in this patch after ---, so reviewer knows where we should check. On Tue, 2020-06-23 at 15:42 +0800, Qiangming Xia wrote: > From: "qiangming.xia" > >

Re: [PATCH] i2c: mediatek: Add to support continuous mode

2020-06-28 Thread Qii Wang
Hi Qiangming: Do you have the specific timing cost data about the "continuous mode"? Is it better than the default multi-write mode(one message by one message) ?I need to know if this patch is very necessary. On Fri, 2020-06-19 at 16:06 +0800, Qiangming Xia wrote: > From: "qiangming.xia"

Re: [PATCH] i2c: mediatek: Add to support continuous mode

2020-06-20 Thread Yingjoe Chen
On Fri, 2020-06-19 at 16:06 +0800, Qiangming Xia wrote: > From: "qiangming.xia" Please make 'From:' the same to Signed-off-by. > Mediatek i2c controller support for continuous mode, > it allow to transfer once multiple writing messages of equal length. So the limitations are writing to

Re: [PATCH] i2c: mediatek: Add to support continuous mode

2020-06-19 Thread Qii Wang
On Fri, 2020-06-19 at 16:06 +0800, Qiangming Xia wrote: > From: "qiangming.xia" > > Mediatek i2c controller support for continuous mode, > it allow to transfer once multiple writing messages of equal length. > For example, a slave need write a serial of non-continuous > offset range in