[PATCH 2/7 v3] i2c-eg20t: Modify returned value s32 to long

2011-10-05 Thread Tomoya MORINAGA
Type of wait_event_timeout is long not s32. This patch replaces s32 with long. Additionally, delete negative processing(ret < 0). Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/drivers/i2c/busses/i2c

[PATCH 3/7 v1] i2c-eg20t: Fix 10bit access issue

2011-10-05 Thread Tomoya MORINAGA
Reported-by: Jeffrey (Sheng-Hui) Chu Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/i2c/busses/i2c-eg20t.c b/drivers/i2c/busses/i2c-eg20t.c index c5b9924..aa1705e 100644 --- a/drivers/i2c/busses

[PATCH 4/7 v3] i2c-eg20t: Separate error processing

2011-10-05 Thread Tomoya MORINAGA
From: Tomoya MORINAGA Error processing for NACK or wait-event must be precessed separately. So divide wait-event error processing into NACK-receiving and timeout. Add arbitration lost processing. Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c | 171 +

[PATCH 6/7 v3] i2c-eg20t: Fix flag setting issue

2011-10-05 Thread Tomoya MORINAGA
Currently, in case occurring abnormal event, internal flag variable(=pch_event_flag) is not reset. This patch fixes the issue. Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/i2c/busses/i2c-eg20t.

[PATCH 5/7 v3] i2c-eg20t: add stop sequence in case wait-event timeout occurs

2011-10-05 Thread Tomoya MORINAGA
add stop sequence in case wait-event timeout in write processing. (read processing already had it) Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/i2c/busses/i2c-eg20t.c b/drivers/i2c/busses/i2c-

[PATCH 3/7 v3] i2c-eg20t: delete 10bit access processing

2011-10-05 Thread Tomoya MORINAGA
From: Tomoya MORINAGA Linux I2C core doesn't support 10bit access formally. Additionally, we can't test with 10bit mode. This patch deletes the 10bit access processing. Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-

[PATCH 7/7 v3] i2c-eg20t: Add initialize processing in case i2c-error occurs

2011-10-05 Thread Tomoya MORINAGA
In case disconnecting physical connection, need to initialize i2c device for retry access. This patch adds initialize process in case bus-idle fails and Lost arbitration. Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-

[PATCH 1/7 v3] i2c-eg20t: Fix bus-idle waiting issue

2011-10-05 Thread Tomoya MORINAGA
Currently, when checking whether bus is idle or not, if timeout occurs, this function always returns success(zero). This patch fixes the issue. Signed-off-by: Tomoya MORINAGA --- drivers/i2c/busses/i2c-eg20t.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --gi

Re: [PATCH 2/7 v2] i2c-eg20t: delete 10bit access processing

2011-10-05 Thread Tomoya MORINAGA
(2011/10/06 2:29), Jeffrey (Sheng-Hui) Chu wrote: Is there any way I can help getting the code fixed instead of removed? OK. I will add a 10bit access code again. I will post v3 patch series soon. Please check the code. -- tomoya ROHM Co., Ltd. -- To unsubscribe from this list: send the line

RE: [PATCH 2/7 v2] i2c-eg20t: delete 10bit access processing

2011-10-05 Thread Jeffrey (Sheng-Hui) Chu
Is there any way I can help getting the code fixed instead of removed? Regards, -Jeffrey -Original Message- From: linux-i2c-ow...@vger.kernel.org [mailto:linux-i2c-ow...@vger.kernel.org] On Behalf Of Tomoya MORINAGA Sent: Wednesday, October 05, 2011 12:18 AM To: Jean Delvare; Ben Dooks;

Re: [GIT PULL] I2C: OMAP: misc. cleanup for v3.2

2011-10-05 Thread Kevin Hilman
Ben Dooks writes: > On Mon, Sep 26, 2011 at 03:30:50PM -0700, Kevin Hilman wrote: >> ping >> >> On 09/06/2011 03:31 PM, Kevin Hilman wrote: >> >Hi Ben, >> > >> >On 08/23/2011 05:10 PM, Kevin Hilman wrote: >> >>Ben, >> >> >> >>Here's one more I2C cleanup series for v3.2. >> >> >> >>It applies on