Re: [PATCH 1/2] Revert spi/pl022: fix spi-pl022 pm enable at probe

2012-10-17 Thread Mark Brown
On Fri, Oct 05, 2012 at 05:51:11PM +0200, Ulf Hansson wrote: From: Ulf Hansson ulf.hans...@linaro.org This reverts commit 6887237cd7da904184dab2750504040c68f3a080. Applied, thanks. -- Everyone hates slow websites. So

Re: [PATCH 1/2] spi: mxs: Assign message status after transfer finished

2012-10-17 Thread Mark Brown
On Sun, Oct 14, 2012 at 04:32:55AM +0200, Marek Vasut wrote: In the current code implementing the MXS SPI driver, every transferred message had assigned status = 0, which is not correct. Properly assign status returned from the I/O functions. Applied both, thanks.

Re: [PATCH] spi: s3c64xx: use clk_prepare_enable and clk_disable_unprepare

2012-10-17 Thread Mark Brown
On Wed, Oct 03, 2012 at 08:30:12AM +0900, Thomas Abraham wrote: Convert clk_enable/clk_disable to clk_prepare_enable/clk_disable_unprepare calls as required by common clock framework. Applied, thanks. -- Everyone hates

Re: [PATCH 0/3] spi/bcm63xx: misc cleanups and fixes

2012-10-17 Thread Mark Brown
On Wed, Oct 03, 2012 at 11:56:52AM +0200, Florian Fainelli wrote: Hi Mark, Grant, This small series contains cleanups and fixes that had already been previously merged by Grant in his tree, but not pushed upstream. I am resending them rebased on Mark's spi-next branch. Applied all, thanks.

Re: [PATCH v3] SPI: McSPI: allow configuration of pin directions

2012-10-17 Thread Mark Brown
On Sun, Oct 07, 2012 at 06:19:44PM +0200, Daniel Mack wrote: Allow D0 to be an input and D1 to be an output, configurable via platform data and a new DT property. Applied, thanks. -- Everyone hates slow websites. So do

Re: [PATCH 1/1] spi: omap2-mcspi: add option to configure output line for McSPI driver

2012-10-17 Thread Mark Brown
On Mon, Oct 08, 2012 at 04:39:40PM -0600, Stan Hu wrote: McSPI driver previously assumed that D0 was input (MISO) and D1 was output (MOSI). This forces the hardware designer to wire all SPI peripherals in this way when it should be a software configuration option. I applied a similar patch

RE: [PATCH v2 v2 v2 8/8] spi/s3c64xx: use correct dma_transfer_direction type

2012-10-17 Thread Kukjin Kim
Arnd Bergmann wrote: There is a subtle difference between dma_transfer_direction and dma_data_direction: the former is used by the dmaengine framework, while the latter is used by the dma-mapping API. Although the purpose is comparable, the actual values are different and must not be mixed.

Re: [PATCH] spi/pl022: add IDLE state pin management

2012-10-17 Thread Linus Walleij
On Wed, Oct 17, 2012 at 9:24 AM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: On Thu, Oct 11, 2012 at 02:03:51PM +0200, Linus Walleij wrote: From: Patrice Chotard patrice.chot...@stericsson.com This commit allow to put pins in IDLE state in runtime_suspend and in SLEEP state in

[PATCH 0/4] PL022 patch queue

2012-10-17 Thread Linus Walleij
Hi Mark, it turns out the reason that the IDLE state patch did not apply was due to this dependency chain, all tested by me. So I signed them all off and rebased on top of spi-next, hope this works out. Patrice Chotard (1): spi/pl022: add IDLE state pin management Ulf Hansson (3): Revert

[PATCH 1/4] Revert spi/pl022: enable runtime PM

2012-10-17 Thread Linus Walleij
From: Ulf Hansson ulf.hans...@linaro.org This reverts commit 2fb30d1147c599f5657e8c62c862f9a0f58d9d99. This patch is reverted due to wrong runtime PM code. Conflicts: drivers/spi/spi-pl022.c Signed-off-by: Ulf Hansson ulf.hans...@linaro.org Signed-off-by: Linus Walleij

[PATCH 2/4] spi: spi-pl022: Minor simplification for runtime pm

2012-10-17 Thread Linus Walleij
From: Ulf Hansson ulf.hans...@linaro.org In probe pm_runtime_put_autosuspend has the same effect as doing pm_runtime_put. This due to upper layer in driver core is preventing the device from being runtime suspended by a pm_runtime_get*. Signed-off-by: Ulf Hansson ulf.hans...@linaro.org

[PATCH 3/4] spi/pl022: Activate resourses before deactivate them in suspend

2012-10-17 Thread Linus Walleij
From: Ulf Hansson ulf.hans...@linaro.org To be able to deactivate resourses in suspend, the resourses must first be surely active. This is done with a pm_runtime_get_sync. Once the resourses are restored to active state again in resume, the runtime pm usage count can be decreased with a

[PATCH 4/4] spi/pl022: add IDLE state pin management

2012-10-17 Thread Linus Walleij
From: Patrice Chotard patrice.chot...@stericsson.com This commit allow to put pins in IDLE state in runtime_suspend and in SLEEP state in suspend, corresponding to defined semantics in linux/pinctrl/pinctrl-state.h. To do this, just add a boolean parameter runtime to

Re: [PATCH 2/4] spi: spi-pl022: Minor simplification for runtime pm

2012-10-17 Thread Ulf Hansson
Hi, We have discussed this patch previously. I think we shall use it, but we should change the commit msg since it does not reflect the truth. It is no more true that upper layer in driver core is preventing the device from being runtime suspended by a pm_runtime_get*. This was the case earlier.

Re: [PATCH 2/4] spi: spi-pl022: Minor simplification for runtime pm

2012-10-17 Thread Linus Walleij
On Wed, Oct 17, 2012 at 4:39 PM, Ulf Hansson ulf.hans...@linaro.org wrote: We have discussed this patch previously. I think we shall use it, but we should change the commit msg since it does not reflect the truth. It is no more true that upper layer in driver core is preventing the device

[PATCH 2/4 v2] spi: spi-pl022: Minor simplification for runtime pm

2012-10-17 Thread Linus Walleij
From: Ulf Hansson ulf.hans...@linaro.org In probe pm_runtime_put_autosuspend has the same effect as doing pm_runtime_put. Signed-off-by: Ulf Hansson ulf.hans...@linaro.org Signed-off-by: Linus Walleij linus.wall...@linaro.org --- drivers/spi/spi-pl022.c | 5 ++--- 1 file changed, 2