Re: [PATCH 1/9] I2C: mv64xxx: work around signals causing I2C transactions to be aborted

2013-05-17 Thread Jean-Francois Moine
On Thu, 16 May 2013 21:30:59 +0100 Russell King rmk+ker...@arm.linux.org.uk wrote: Do not use interruptible waits in an I2C driver; if a process uses signals (eg, Xorg uses SIGALRM and SIGPIPE) then these signals can cause the I2C driver to abort a transaction in progress by another driver,

Re: [PATCH 1/9] I2C: mv64xxx: work around signals causing I2C transactions to be aborted

2013-05-17 Thread Russell King - ARM Linux
On Fri, May 17, 2013 at 08:37:43AM +0200, Jean-Francois Moine wrote: On Thu, 16 May 2013 21:30:59 +0100 Russell King rmk+ker...@arm.linux.org.uk wrote: Do not use interruptible waits in an I2C driver; if a process uses signals (eg, Xorg uses SIGALRM and SIGPIPE) then these signals can

Re: [PATCH 1/9] I2C: mv64xxx: work around signals causing I2C transactions to be aborted

2013-05-17 Thread Wolfram Sang
On Thu, May 16, 2013 at 09:30:59PM +0100, Russell King wrote: Do not use interruptible waits in an I2C driver; if a process uses signals (eg, Xorg uses SIGALRM and SIGPIPE) then these signals can cause the I2C driver to abort a transaction in progress by another driver, which can cause that

Re: [PATCH 1/9] I2C: mv64xxx: work around signals causing I2C transactions to be aborted

2013-05-17 Thread Mark A. Greer
On Thu, May 16, 2013 at 09:30:59PM +0100, Russell King wrote: Do not use interruptible waits in an I2C driver; if a process uses signals (eg, Xorg uses SIGALRM and SIGPIPE) then these signals can cause the I2C driver to abort a transaction in progress by another driver, which can cause that

[PATCH 1/9] I2C: mv64xxx: work around signals causing I2C transactions to be aborted

2013-05-16 Thread Russell King
Do not use interruptible waits in an I2C driver; if a process uses signals (eg, Xorg uses SIGALRM and SIGPIPE) then these signals can cause the I2C driver to abort a transaction in progress by another driver, which can cause that driver to fail. I2C drivers are not expected to abort transactions