Re: [PATCH V3] mmc: Set suspend/resume bus operations if CONFIG_PM_RUNTIME is used

2012-01-10 Thread Ulf Hansson
Datta, Shubhrajyoti wrote: Hi Dmitry, On Mon, Jan 9, 2012 at 11:34 PM, Dmitry Shmidt dimitr...@google.com wrote: Signed-off-by: Dmitry Shmidt dimitr...@google.com --- drivers/mmc/card/block.c |4 ++-- drivers/mmc/core/bus.c | 27 +-- include/linux/mmc/card.h |

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Adrian Hunter
On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when inserting/removing a card since the slot is probably not glitch free, or that

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Adrian Hunter
On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when inserting/removing a card since the slot is probably not glitch free, or that

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Ulf Hansson
Adrian Hunter wrote: On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when inserting/removing a card since the slot is probably not

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Ulf Hansson
Adrian Hunter wrote: On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when inserting/removing a card since the slot is probably not

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Adrian Hunter
On 10/01/12 12:59, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when

Re: [PATCH] mmc: core: Force a detect to handle non-properly removed cards

2012-01-10 Thread Adrian Hunter
On 10/01/12 13:03, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 16:27, Ulf Hansson wrote: Adrian Hunter wrote: On 09/01/12 15:14, Ulf Hansson wrote: My concern is more about what we actually can trust; either the GPIO irq which likely is giving more than one irq when

[PATCH] mmc: tmio_mmc, sdhi: update input clock frequency after resume

2012-01-10 Thread Guennadi Liakhovetski
After a runtime or system-wide suspend the clock frequency can change, therefore it must be re-read. Signed-off-by: Guennadi Liakhovetski g.liakhovet...@gmx.de --- drivers/mmc/host/sh_mobile_sdhi.c | 10 ++ drivers/mmc/host/tmio_mmc_pio.c | 31 +--

Re: [PATCH 09/11] mmc: sh_mobile_sdhi: do not manage PM clocks manually on ARM

2012-01-10 Thread Guennadi Liakhovetski
Hi Magnus On Fri, 6 Jan 2012, Magnus Damm wrote: Hi Guennadi, On Wed, Jan 4, 2012 at 11:17 PM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On ARM the same clock is used by the PM subsystem and by the driver directly. This leads to the clock staying permanently on, independent of

[PATCH 09/11 v2] mmc: sh_mobile_sdhi: do not manage PM clocks manually on ARM

2012-01-10 Thread Guennadi Liakhovetski
On sh-mobile platforms the MMC clock frequency for the TMIO MMC unit is obtained from the same clock, as the one, that runtime power-manages the controller. The SDHI glue code has to access that clock directly, bypassing the runtime PM framework, to get its frequency, but it shouldn't enable or

Re: [PATCH V3] mmc: Set suspend/resume bus operations if CONFIG_PM_RUNTIME is used

2012-01-10 Thread Dmitry Shmidt
On Mon, Jan 9, 2012 at 11:12 PM, Datta, Shubhrajyoti shubhrajy...@ti.com wrote: Hi Dmitry, On Mon, Jan 9, 2012 at 11:34 PM, Dmitry Shmidt dimitr...@google.com wrote: Signed-off-by: Dmitry Shmidt dimitr...@google.com ---  drivers/mmc/card/block.c |    4 ++--  drivers/mmc/core/bus.c   |   27

[PATCH V4] mmc: Set suspend/resume bus operations if CONFIG_PM_RUNTIME is used

2012-01-10 Thread Dmitry Shmidt
Signed-off-by: Dmitry Shmidt dimitr...@google.com --- drivers/mmc/card/block.c |4 ++-- drivers/mmc/core/bus.c | 25 - include/linux/mmc/card.h |2 +- 3 files changed, 11 insertions(+), 20 deletions(-) diff --git a/drivers/mmc/card/block.c

Re: [PATCH V4] mmc: Set suspend/resume bus operations if CONFIG_PM_RUNTIME is used

2012-01-10 Thread Barry Song
2012/1/10 Dmitry Shmidt dimitr...@google.com: Signed-off-by: Dmitry Shmidt dimitr...@google.com Hi Dmitry, after you move mmc_bus_suspend, mmc_bus_resume to dev_pm_ops, while doing hiberantion by swsusp, suspend/resume will not be called by the hibernation flow since it will move to call

Re: [PATCH] mmc: core: move suspend/resume to dev_pm_ops and add hibernation support

2012-01-10 Thread Barry Song
2012/1/9 Shubhrajyoti Datta omaplinuxker...@gmail.com: Hi Bin, On Mon, Jan 9, 2012 at 11:52 AM, Barry Song barry.s...@csr.com wrote: From: Bin Shi bin@csr.com This patch moves suspend/resume to dev_pm_ops and add hibernation support. It was tested on CSR SiRFprimaII cortex-a9 platform.

[PATCH] mmc: core: UHS sdio card that fails should not exceed 50MHz

2012-01-10 Thread Philip Rakity
A UHS sdio card that fails initialization at 1.8v signaling is not in UHS mode. We cannot use the speed in the the cis to reflect the bus speed as this is the maxiumum value and will not reflect the fact that the host is operating at a lower (non uhs) bus speed. Signed-off-by: Philip Rakity

Re: [PATCH] mmc: core: move suspend/resume to dev_pm_ops and add hibernation support

2012-01-10 Thread Shubhrajyoti Datta
Hi Barry, On Wed, Jan 11, 2012 at 8:24 AM, Barry Song 21cn...@gmail.com wrote: 2012/1/9 Shubhrajyoti Datta omaplinuxker...@gmail.com: Hi Bin, On Mon, Jan 9, 2012 at 11:52 AM, Barry Song barry.s...@csr.com wrote: From: Bin Shi bin@csr.com This patch moves suspend/resume to dev_pm_ops