Re: [PATCH RESEND] mmc: core: Improve support for deferred regulators

2014-04-25 Thread Andrew Bresticker
Hi Tim, > It is important that callers of mmc_regulator_get_supply know if either > vmmc or vqmmc are present but not yet available. To support this need, > modify this function to return zero except when either of the regulator > get operations fail with -EPROBE_DEFER. Current callers don't che

[PATCH RESEND] mmc: core: Improve support for deferred regulators

2014-04-25 Thread Tim Kryger
It is important that callers of mmc_regulator_get_supply know if either vmmc or vqmmc are present but not yet available. To support this need, modify this function to return zero except when either of the regulator get operations fail with -EPROBE_DEFER. Current callers don't check the return val

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Russell King - ARM Linux
On Fri, Apr 25, 2014 at 02:15:30PM +0100, Russell King - ARM Linux wrote: > On Fri, Apr 25, 2014 at 03:08:49PM +0200, Markus Pargmann wrote: > > I just tried the different parts of the patch. Without the following > > change, emmc works: > > > > --- a/drivers/mmc/host/sdhci.c > > +++ b/drivers/mmc

[PATCH 25/32] mmc: sdhci-esdhc-imx: remove emulation of uhs_mode

2014-04-25 Thread Russell King
We no longer need to emulate the uhs_mode field of the host control2 register - the main sdhci driver never reads this back to evaluate the current mode as it caches the current mode instead. Signed-off-by: Russell King --- drivers/mmc/host/sdhci-esdhc-imx.c | 17 +++-- 1 file change

[PATCH 26/32] mmc: sdhci-of-esdhc: remove platform_suspend/platform_resume callbacks

2014-04-25 Thread Russell King
We don't need these hooks in order to insert code in these paths, we can just provide our own handlers and call the main sdhci handlers as appropriate. Signed-off-by: Russell King --- drivers/mmc/host/sdhci-of-esdhc.c | 55 +-- 1 file changed, 36 insertions(+)

[PATCH 29/32] mmc: sdhci: move regulator handling into sdhci_set_power()

2014-04-25 Thread Russell King
Move the regulator handling into sdhci_set_power() rather than being in sdhci_do_set_ios(). This wraps all power control up into this function. Signed-off-by: Russell King --- drivers/mmc/host/sdhci.c | 72 +++- 1 file changed, 35 insertions(+), 37 de

[PATCH 31/32] mmc: sdhci: track whether preset mode is currently enabled in hardware

2014-04-25 Thread Russell King
Track whether preset mode is currently enabled in hardware, and use that when making decisions elsewhere in the code rather than reading the register and checking the bit. Signed-off-by: Russell King --- drivers/mmc/host/sdhci.c | 44 ++-- include/linux/m

[PATCH 30/32] mmc: sdhci: move remaining power handling into sdhci_set_power()

2014-04-25 Thread Russell King
Move the remaining parts of the power handling in sdhci_do_set_ios() into sdhci_set_power(). Signed-off-by: Russell King --- drivers/mmc/host/sdhci.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 956

[PATCH 21/32] mmc: sdhci: set_uhs_signaling() need not return a value

2014-04-25 Thread Russell King
The set_uhs_signaling() method gives the impression that it can fail, but anything returned from the method is entirely ignored by the sdhci driver. So returning failure has no effect. So, kill the idea that it's possible for this to return an error by removing the returned value. Signed-off-by:

[PATCH 24/32] mmc: sdhci: clean up sdhci_execute_tuning() decision

2014-04-25 Thread Russell King
Clean up the code in sdhci_execute_tuning() so the decision whether to execute tuning is clearer - and despite this reflecting what the original code was doing, it shows that it may not be what the author actually intended. Signed-off-by: Russell King --- drivers/mmc/host/sdhci.c | 31 ++

[PATCH 27/32] mmc: sdhci: remove platform_suspend/platform_resume callbacks

2014-04-25 Thread Russell King
The only user (sdhci-of-esdhc) no longer uses these callbacks, so lets remove them to discourage any further use. Acked-by: Ulf Hansson Signed-off-by: Russell King --- drivers/mmc/host/sdhci.c | 6 -- drivers/mmc/host/sdhci.h | 2 -- 2 files changed, 8 deletions(-) diff --git a/drivers/mmc

[PATCH 28/32] mmc: sdhci-tegra: get rid of special PRESENT_STATE register handling

2014-04-25 Thread Russell King
sdhci-tegra provides a get_ro method, which overrides the checking of the write protect bit in the PRESENT_STATE register in sdhci.c: if (host->flags & SDHCI_DEVICE_DEAD) is_readonly = 0; else if (host->ops->get_ro) is_readonly = host->ops->get_ro(ho

[PATCH 32/32] mmc: sdhci: fix SDHCI dependencies

2014-04-25 Thread Russell King
Signed-off-by: Russell King --- drivers/mmc/host/Kconfig | 63 1 file changed, 32 insertions(+), 31 deletions(-) diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig index 8aaf8c1f3f63..d8c1f31ab37f 100644 --- a/drivers/mmc/host/Kconfi

[PATCH 22/32] mmc: sdhci: convert sdhci_set_uhs_signaling() into a library function

2014-04-25 Thread Russell King
Add sdhci_set_uhs_signaling() and always call the set_uhs_signaling method. This avoids quirks being added into sdhci_set_uhs_signaling(). Signed-off-by: Russell King --- drivers/mmc/host/sdhci-acpi.c | 2 ++ drivers/mmc/host/sdhci-bcm-kona.c | 1 + drivers/mmc/host/sdhci-bcm2835.c |

[PATCH 23/32] mmc: sdhci: cache timing information locally

2014-04-25 Thread Russell King
Rather than reading back the timing information from the registers, cache it locally. This allows implementations to translate the UHS timing by overriding the set_uhs_signaling() method as required without also having to emulate the SDHCI_HOST_CONTROL2 register. Signed-off-by: Russell King ---

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Russell King - ARM Linux
On Fri, Apr 25, 2014 at 02:15:30PM +0100, Russell King - ARM Linux wrote: > On Fri, Apr 25, 2014 at 03:08:49PM +0200, Markus Pargmann wrote: > > I just tried the different parts of the patch. Without the following > > change, emmc works: > > > > --- a/drivers/mmc/host/sdhci.c > > +++ b/drivers/mmc

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Russell King - ARM Linux
On Fri, Apr 25, 2014 at 03:08:49PM +0200, Markus Pargmann wrote: > I just tried the different parts of the patch. Without the following > change, emmc works: > > --- a/drivers/mmc/host/sdhci.c > +++ b/drivers/mmc/host/sdhci.c > @@ -1507,12 +1507,6 @@ static void sdhci_do_set_ios(struct sdhci_host

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Markus Pargmann
On Fri, Apr 25, 2014 at 01:49:45PM +0100, Russell King - ARM Linux wrote: > On Fri, Apr 25, 2014 at 02:38:20PM +0200, Markus Pargmann wrote: > > Hi, > > > > On Wed, Apr 23, 2014 at 08:07:57PM +0100, Russell King wrote: > > > Patch suggested by Dong Aisheng , this avoids > > > additional clock star

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Russell King - ARM Linux
On Fri, Apr 25, 2014 at 02:38:20PM +0200, Markus Pargmann wrote: > Hi, > > On Wed, Apr 23, 2014 at 08:07:57PM +0100, Russell King wrote: > > Patch suggested by Dong Aisheng , this avoids > > additional clock start/stop cycles during the transition to 1.8V > > signalling mode. > > > > Signed-off-b

Re: [PATCH 21/38] mmc: sdhci: hack up driver to make it more compliant with UHS-1

2014-04-25 Thread Markus Pargmann
Hi, On Wed, Apr 23, 2014 at 08:07:57PM +0100, Russell King wrote: > Patch suggested by Dong Aisheng , this avoids > additional clock start/stop cycles during the transition to 1.8V > signalling mode. > > Signed-off-by: Russell King I tested the series on imx6s with a RIoT board. With this patch

Re: [PATCH 00/38] MMC updates, plus CuBox-i WiFi support

2014-04-25 Thread Ulf Hansson
On 25 April 2014 13:20, Russell King - ARM Linux wrote: > On Fri, Apr 25, 2014 at 01:18:28PM +0200, Ulf Hansson wrote: >> On 25 April 2014 11:03, Russell King - ARM Linux >> wrote: >> > On Thu, Apr 24, 2014 at 01:13:05PM +0200, Ulf Hansson wrote: >> >> On 24 April 2014 12:57, Russell King - ARM

Re: [PATCH 00/38] MMC updates, plus CuBox-i WiFi support

2014-04-25 Thread Russell King - ARM Linux
On Fri, Apr 25, 2014 at 01:18:28PM +0200, Ulf Hansson wrote: > On 25 April 2014 11:03, Russell King - ARM Linux > wrote: > > On Thu, Apr 24, 2014 at 01:13:05PM +0200, Ulf Hansson wrote: > >> On 24 April 2014 12:57, Russell King - ARM Linux > >> wrote: > >> > This is nothing new or unexpected -

Re: [PATCH 00/38] MMC updates, plus CuBox-i WiFi support

2014-04-25 Thread Ulf Hansson
On 25 April 2014 11:03, Russell King - ARM Linux wrote: > On Thu, Apr 24, 2014 at 01:13:05PM +0200, Ulf Hansson wrote: >> On 24 April 2014 12:57, Russell King - ARM Linux >> wrote: >> > This is nothing new or unexpected - it was last posted back in February, >> > and I elected that it should be

Re: [PATCH v7 4/4] memstick: Add realtek USB memstick host driver

2014-04-25 Thread Dan Carpenter
On Wed, Apr 16, 2014 at 09:13:52AM +0100, Lee Jones wrote: > It seems Max and Alex are less than interested in this patch, so I'm > going to apply it anyway. In your last review you mentioned that it > looked good to your inexpert eye. Can I apply your Ack to that effect? > Just apply it... He s

Re: [PATCH 00/38] MMC updates, plus CuBox-i WiFi support

2014-04-25 Thread Russell King - ARM Linux
On Thu, Apr 24, 2014 at 01:13:05PM +0200, Ulf Hansson wrote: > On 24 April 2014 12:57, Russell King - ARM Linux > wrote: > > This is nothing new or unexpected - it was last posted back in February, > > and I elected that it should be held off until after the last merge > > window. > > > > Unfortu

Re: [PATCH v2] mmc: core: Try other signal levels during power up

2014-04-25 Thread Ulf Hansson
On 24 April 2014 23:44, Tim Kryger wrote: > The eMMC signalling voltage is determined by VCCQ which is provided to > the card by the host. Signalling is not required to begin at 3.3v and, > if the host and card both support a particular VCC/VCCQ combination, it > can be used immediately. > > In c

Re: [PATCH] mmc: sdhci: Use mmc core regulator infrastucture

2014-04-25 Thread Ulf Hansson
On 25 April 2014 00:36, Tim Kryger wrote: > Switch the common SDHCI code over to use mmc_host's regulator pointers > and remove the ones in the sdhci_host structure. Additionally, use the > common mmc_regulator_get_supply function to get the regulators and set > the ocr_avail mask. > > This chang

Re: [PATCH v7 4/4] memstick: Add realtek USB memstick host driver

2014-04-25 Thread Roger
On 04/16/2014 04:13 PM, Lee Jones wrote: Hi Andrew, From: Roger Tseng Realtek USB memstick host driver provides memstick host support based on the Realtek USB card reader MFD driver. Signed-off-by: Roger Tseng --- drivers/memstick/host/Kconfig | 10 + drivers/memstick/host/Makefi