Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-22 Thread Lee Jones
On Sun, 23 Jul 2017, Furquan Shaikh wrote: > Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset > state on suspend") changed the behavior on suspend by not putting LPSS > controllers into reset. This was done because S3/S0ix fail if UART > device is put into reset and

Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-22 Thread Lee Jones
On Mon, 21 Aug 2017, Furquan Shaikh wrote: > Hello Lee, > > Gentle ping. Do you see any issues with the following change? Couple of things: 1. Please do not top post. 2. Please do not send contentless pings. If you think a patch had fallen through the cracks (which this one had), please

Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-22 Thread Lee Jones
On Sun, 23 Jul 2017, Furquan Shaikh wrote: > Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset > state on suspend") changed the behavior on suspend by not putting LPSS > controllers into reset. This was done because S3/S0ix fail if UART > device is put into reset and

Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-22 Thread Lee Jones
On Mon, 21 Aug 2017, Furquan Shaikh wrote: > Hello Lee, > > Gentle ping. Do you see any issues with the following change? Couple of things: 1. Please do not top post. 2. Please do not send contentless pings. If you think a patch had fallen through the cracks (which this one had), please

Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-21 Thread Furquan Shaikh
Hello Lee, Gentle ping. Do you see any issues with the following change? Thanks, Furquan On Sun, Jul 23, 2017 at 11:02 PM, Furquan Shaikh wrote: > > Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset > state on suspend") changed the behavior on suspend by

Re: [PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-08-21 Thread Furquan Shaikh
Hello Lee, Gentle ping. Do you see any issues with the following change? Thanks, Furquan On Sun, Jul 23, 2017 at 11:02 PM, Furquan Shaikh wrote: > > Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset > state on suspend") changed the behavior on suspend by not putting LPSS >

[PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-07-24 Thread Furquan Shaikh
Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset state on suspend") changed the behavior on suspend by not putting LPSS controllers into reset. This was done because S3/S0ix fail if UART device is put into reset and no_console_suspend flag is enabled. Because of the above

[PATCH] mfd: intel-lpss: Put I2C and SPI controllers into reset state on suspend

2017-07-24 Thread Furquan Shaikh
Commit 274e43edcda6f ("mfd: intel-lpss: Do not put device in reset state on suspend") changed the behavior on suspend by not putting LPSS controllers into reset. This was done because S3/S0ix fail if UART device is put into reset and no_console_suspend flag is enabled. Because of the above