On Fri, Dec 20, 2013 at 04:12:55PM +0100, jean-jacques hiblot wrote: > From: jean-jacques hiblot <jjhiblot@xxxxxxxxx> > > Clean-up properly when a transfer fails for whatever reason. > Cancel the transfer when the process is signaled. Please describe what you do a little. I wonder how you can remove so much code while keeping the functionality? > > Signed-off-by: jean-jacques hiblot <jjhiblot@xxxxxxxxxxxxxxx> > - out_8(&iic->cntl, CNTL_HMT); > + DBG(dev, "aborting transfer\n"); > + /* transfer should be aborted within 10ms */ > + end = jiffies + 10; Eeks, msecs_to_jiffies() macro please! And please consider running checkpatch and sparse over your code. Sparse gives, for example: drivers/i2c/busses/i2c-ibm_iic.c:418:24: warning: incorrect type in argument 1 (different address spaces) drivers/i2c/busses/i2c-ibm_iic.c:418:24: expected unsigned char const volatile [noderef] [usertype] <asn:2>*addr drivers/i2c/busses/i2c-ibm_iic.c:418:24: got unsigned char *<noident> (This probably due to patch 1 or 2, I'd guess)
Attachment:
signature.asc
Description: Digital signature
