Re: [PATCH 0/4] i2c: sh_mobile: fix STOP condition generation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 17, 2013 at 10:45:53AM +0100, Guennadi Liakhovetski wrote:
> Currently the i2c-sh_mobile.c driver generates STOP conditions after each 
> message. This makes any multi-message transfers, e.g. reading, impossible 
> with the AS3711 PMIC on kzm9g. This patch series fixes this by only 
> generating the STOP condition after the last message or if the I2C_M_STOP 
> flag is set.
> 
> Guennadi Liakhovetski (4):
>   i2c: i2c-sh_mobile.c: cosmetic: trivially simplify 2 functions
>   i2c: sh_mobile: fix timeout error handling
>   i2c: sh_mobile: eliminate an open-coded "goto" loop
>   i2c: sh_mobile: don't send a stop condition by default inside
>     transfers

Looks good to me. I have to mention usleep_range, though. I leave it to
you if you want to fix it or not. Other than that, I would apply the
series.
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux