Re: [PATCH] i2c: imx: add slave support. v2 status

2016-12-01 Thread Frkuska, Joshua
Hi Maxim, Dmitriy, Wolfram, If there is no immediate plan for a third release of the below patch set, would it be possible to continue with merging v2 after addressing the remaining concerns? Thank you and regards, Joshua Hi Maxim, On 2016-03-04 11:06:10 in the thread "

Re: [PATCH] i2c: imx: add slave support. v2 status

2016-10-31 Thread Maxim Syrchin
Thanks, Maxim Syrchin 27.10.2016 10:31, Frkuska, Joshua пишет: Hi Maxim, Dmitriy, Wolfram, If there is no immediate plan for a third release of the below patch set, would it be possible to continue with merging v2 after addressing the remaining concerns? Thank you and regards, Joshua Hi M

Re: [PATCH] i2c: imx: add slave support. v2 status

2016-10-30 Thread Frkuska, Joshua
Hi Maxim, Dmitriy, Wolfram, If there is no immediate plan for a third release of the below patch set, would it be possible to continue with merging v2 after addressing the remaining concerns? Thank you and regards, Joshua Hi Maxim, On 2016-03-04 11:06:10 in the thread "Re: [PATCH]

Re: [PATCH] i2c: imx: add slave support. v2 status

2016-10-27 Thread Maxim Syrchin
xim, On 2016-03-04 11:06:10 in the thread "Re: [PATCH] i2c: imx: add slave support. v2" referenced here: https://patchwork.ozlabs.org/patch/573353/ you said: Hi Wolfram, I'm now working on creating new driver version. I think I'll be able to sent it soon. Do you still plan

Re: Re: [PATCH] i2c: imx: add slave support. v2 status

2016-10-27 Thread Frkuska, Joshua
] i2c: imx: add slave support. v2" referenced here: https://patchwork.ozlabs.org/patch/573353/ you said: Hi Wolfram, I'm now working on creating new driver version. I think I'll be able to sent it soon. Do you still plan to release a driver update for an i2c imx driver slave su

Re: [PATCH] i2c: imx: add slave support. v2

2016-03-04 Thread Maxim Syrchin
Hi Wolfram, I'm now working on creating new driver version. I think I'll be able to sent it soon. 04.03.2016 0:35, Wolfram Sang пишет: There are might be race conditions. Can you name them Most of races are fixed already. There were some issues with interrupt latencies - sometimes slave int

Re: [PATCH] i2c: imx: add slave support. v2

2016-03-03 Thread Wolfram Sang
On Tue, Jan 26, 2016 at 07:14:40PM +0300, Dmitriy Baranov wrote: > Add I2C slave provider using the generic slave interface. > It also supports master transactions when the slave in the idle mode. > > Issues: > Changes work only in PIO mode (when driver doesn`t use DMA) This is fine with me. We d

[PATCH] i2c: imx: add slave support. v2

2016-01-26 Thread Dmitriy Baranov
Add I2C slave provider using the generic slave interface. It also supports master transactions when the slave in the idle mode. Issues: Changes work only in PIO mode (when driver doesn`t use DMA) It weren`t tested with DMA is enabled (in PIO mode it works well) There are might be race conditions.

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Wolfram Sang
> Thus, if you don`t mind, we wont send fixed version for now. Perfect! signature.asc Description: Digital signature

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Dmitriy Baranov
Vladimir, Thanks for the comment. We are both the authors of this patch. Could you please squash the changes (care about indentation in Kconfig also) and resend it as v2 (probably you may want to wait for review comments some more time)? I suppose that after review, there will be many other thi

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Vladimir Zapolskiy
On 26.01.2016 11:54, Dmitriy Baranov wrote: > Sorry, we should have added selecting this in our patch. > The following fixes it: > > Subject: [PATCH] Select I2C_SLAVE for i2c-imx driver because it uses the > generic slave interface. > > Signed-off-by: Dmitriy Baranov > Signed-off-by: Maxim Syrc

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Dmitriy Baranov
Sorry, we should have added selecting this in our patch. The following fixes it: Subject: [PATCH] Select I2C_SLAVE for i2c-imx driver because it uses the generic slave interface. Signed-off-by: Dmitriy Baranov Signed-off-by: Maxim Syrchin --- drivers/i2c/busses/Kconfig | 1 + 1 file changed

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Wolfram Sang
On Tue, Jan 26, 2016 at 11:22:24AM +0300, Dmitriy Baranov wrote: > Thank you for testing our patch. > > Due to using the generic slave interface, It should be enabled in the config > file. > Please add the following in the config file: > CONFIG_I2C_SLAVE=y For now, you need to select it to make r

Re: [PATCH] i2c: imx: add slave support

2016-01-26 Thread Dmitriy Baranov
Thank you for testing our patch. Due to using the generic slave interface, It should be enabled in the config file. Please add the following in the config file: CONFIG_I2C_SLAVE=y On 25.01.2016 21:09, kbuild test robot wrote: Hi Dmitriy, [auto build test ERROR on wsa/i2c/for-next] [also bu

Re: [PATCH] i2c: imx: add slave support

2016-01-25 Thread kbuild test robot
Hi Dmitriy, [auto build test ERROR on wsa/i2c/for-next] [also build test ERROR on v4.5-rc1 next-20160125] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Dmitriy-Baranov/i2c-imx-add-slave-suppo

[PATCH] i2c: imx: add slave support

2016-01-25 Thread Dmitriy Baranov
Add I2C slave provider using the generic slave interface. It also supports master transactions when the slave in the idle mode. Changes work only in PIO mode (when driver doesn`t use DMA) These changes weren`t tested with DMA is enabled. Signed-off-by: Dmitriy Baranov Signed-off-by: Maxim Syrchin