[PATCH-V2 03/12] i2c: pxa: Return I2C_RETRY when timeout in pio mode
Vaibhav Hiremath
vaibhav.hiremath at linaro.org
Tue Jun 16 00:41:36 PDT 2015
On Tuesday 16 June 2015 11:01 AM, Shubhrajyoti Datta wrote:
> On Mon, Jun 15, 2015 at 9:19 PM, Vaibhav Hiremath
> <vaibhav.hiremath at linaro.org> wrote:
>> From: Shouming Wang <wangshm at marvell.com>
>>
>> In case of timeout in pio mode of operation return I2C_RETRY.
>> This behavior will be same as interrupt mode of operation.
>>
>> Signed-off-by: Shouming Wang <wangshm at marvell.com>
>> [vaibhav.hiremath at linaro.org: Updated changelog]
>> Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath at linaro.org>
>> ---
>> drivers/i2c/busses/i2c-pxa.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
>> index 023e59f..632008f 100644
>> --- a/drivers/i2c/busses/i2c-pxa.c
>> +++ b/drivers/i2c/busses/i2c-pxa.c
>> @@ -745,8 +745,10 @@ static int i2c_pxa_do_pio_xfer(struct pxa_i2c *i2c,
>> ret = i2c->msg_idx;
>>
>> out:
>> - if (timeout == 0)
>> + if (timeout == 0) {
>> i2c_pxa_scream_blue_murder(i2c, "timeout");
>> + ret = I2C_RETRY;
>
> Can we use standard return types.
>
Unfortunately NO.
I2C_RETRY is used in many places, internal to the i2c-pcxa driver.
This return value is internal only and used internally to the driver.
Thanks,
Vaibhav
More information about the linux-arm-kernel
mailing list