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

2016-12-01 Thread Frkuska, Joshua
Hello Maxim, I have a few questions for you. Please see my comments inline below. In addition, I have modified your patch-set slightly and I would like to progress it to merger if you do not have any issues with this. I would like to sync with you here before moving forward and submitting a

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

2016-12-01 Thread Frkuska, Joshua
Hello Maxim, I have a few questions for you. Please see my comments inline below. In addition, I have modified your patch-set slightly and I would like to progress it to merger if you do not have any issues with this. I would like to sync with you here before moving forward and submitting a

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

2016-10-31 Thread Maxim Syrchin
Hello, Please find some comments below. 31.10.2016 5:14, Frkuska, Joshua пишет: Hello Maxim, Thank you very much for the intermediate patch. I am in the process of reviewing it. Please let me clarify a few questions I have. 1. What alternative to "bus busy/bus free/IBB" polling do you

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

2016-10-31 Thread Maxim Syrchin
Hello, Please find some comments below. 31.10.2016 5:14, Frkuska, Joshua пишет: Hello Maxim, Thank you very much for the intermediate patch. I am in the process of reviewing it. Please let me clarify a few questions I have. 1. What alternative to "bus busy/bus free/IBB" polling do you

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

2016-10-30 Thread Frkuska, Joshua
Hello Maxim, Thank you very much for the intermediate patch. I am in the process of reviewing it. Please let me clarify a few questions I have. 1. What alternative to "bus busy/bus free/IBB" polling do you have in mind? This seems like a reasonable approach to me. 2. What are the major

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

2016-10-30 Thread Frkuska, Joshua
Hello Maxim, Thank you very much for the intermediate patch. I am in the process of reviewing it. Please let me clarify a few questions I have. 1. What alternative to "bus busy/bus free/IBB" polling do you have in mind? This seems like a reasonable approach to me. 2. What are the major

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

2016-10-27 Thread Maxim Syrchin
Hi, Sorry for huge delay in answering. Unfortunately we don't have enough resources now to prepare clean enough patch to be accepted by community. Please find the latest version attached. Driver has passed stress tests, but looks like it need seriuos refactoring (it is unnecessarily

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

2016-10-27 Thread Maxim Syrchin
Hi, Sorry for huge delay in answering. Unfortunately we don't have enough resources now to prepare clean enough patch to be accepted by community. Please find the latest version attached. Driver has passed stress tests, but looks like it need seriuos refactoring (it is unnecessarily

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

2016-10-27 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:

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

2016-10-27 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: