[GIT PULL] MMC fixes for 3.4-rc4

2012-04-20 Thread Chris Ball
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git tags/mmc-fixes-for-3.4-rc4 to receive MMC fixes for 3.4-rc4. There are no merge conflicts, and the patches have been tested in linux-next. Thanks. The following changes since commit 3b422e9c2c020a1137349c

Re: [PATCH 5/8] mmc: omap_hsmmc: release correct resource

2012-04-20 Thread Chris Ball
Hi, On Fri, Apr 20 2012, Chris Ball wrote: > On Fri, Apr 20 2012, Tony Lindgren wrote: >> * Russell King [120418 03:15]: >>> res can be one of several resources, as this variable is re-used several >>> times during probe. This can cause the wrong resource parameters to be >>> passed to release_m

Re: [PATCH 5/8] mmc: omap_hsmmc: release correct resource

2012-04-20 Thread Chris Ball
Hi, On Fri, Apr 20 2012, Tony Lindgren wrote: > * Russell King [120418 03:15]: >> res can be one of several resources, as this variable is re-used several >> times during probe. This can cause the wrong resource parameters to be >> passed to release_mem_region(). >> >> Get the original memory r

Re: [PATCH 5/8] mmc: omap_hsmmc: release correct resource

2012-04-20 Thread Tony Lindgren
* Russell King [120418 03:15]: > res can be one of several resources, as this variable is re-used several > times during probe. This can cause the wrong resource parameters to be > passed to release_mem_region(). > > Get the original memory resource before calling release_mem_region(). > > Sign

Re: [PATCH 1/8] ARM: OMAP: fix DMA vs memory ordering

2012-04-20 Thread Tony Lindgren
* Russell King [120418 03:14]: > Using coherent DMA memory with the OMAP DMA engine results in > unpredictable behaviour due to memory ordering issues; as things stand, > there is no guarantee that data written to coherent DMA memory will be > visible to the DMA hardware. > > This is because the

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Tony Lindgren
* Russell King - ARM Linux [120420 15:13]: > On Fri, Apr 20, 2012 at 09:43:07AM -0700, Tony Lindgren wrote: > > * Russell King - ARM Linux [120420 08:41]: > > > --- a/drivers/dma/omap-dma.c > > > +++ b/drivers/dma/omap-dma.c > > > @@ -83,11 +84,11 @@ static void omap_dma_start_sg(struct omap_chan

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Russell King - ARM Linux
On Fri, Apr 20, 2012 at 09:43:07AM -0700, Tony Lindgren wrote: > * Russell King - ARM Linux [120420 08:41]: > > --- a/drivers/dma/omap-dma.c > > +++ b/drivers/dma/omap-dma.c > > @@ -83,11 +84,11 @@ static void omap_dma_start_sg(struct omap_chan *c, > > struct omap_desc *d, > > struct omap_sg

Re: [PATCH/RFC 5/5] sh: ecovec: switch MMC power control to regulator

2012-04-20 Thread Guennadi Liakhovetski
(this patch should have been 5/5, as you have guessed:-)) On Fri, 20 Apr 2012, Mark Brown wrote: > On Fri, Apr 20, 2012 at 06:59:41PM +0200, Guennadi Liakhovetski wrote: > > > +static struct gpio_regulator_state mmcif_power_states[] = { > > + { .value = 330, .gpios = 0 }, > > +}; > > It d

Possible to trigger card insert/remove event from user space?

2012-04-20 Thread Steve Sakoman
I am working with a system that has a wifi module connected via SDIO (8686 using the libertas_sdio driver). This is a hard connection, not socketed -- the hw is soldered to the board. To reduce power I would like to be able to unload the driver module and power down the wifi hw (via a GPIO contro

Re: [PATCH] mmc: core: Do not pre-claim host in suspend

2012-04-20 Thread Sujit Reddy Thumma
Hi Ulf, > Since SDIO drivers may want to do some SDIO operations > in their suspend callback functions, we must not keep > the host claimed when calling them. > > Daniel Drake reported that libertas_sdio encountered > a deadlock in it's suspend function. > > Signed-off-by: Ulf Hansson > --- Sorr

Re: [PATCH/RFC] sh: ecovec: switch MMC power control to regulator

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 06:59:41PM +0200, Guennadi Liakhovetski wrote: > +static struct gpio_regulator_state mmcif_power_states[] = { > + { .value = 330, .gpios = 0 }, > +}; It doesn't make much odds but if you've only got one voltage the fixed voltage regulator is a little more idiomatic

[PATCH 4/5] mmc: sh_mmcif: add regulator support

2012-04-20 Thread Guennadi Liakhovetski
Add regulator support to the sh_mmcif driver, but also preserve the current power-callback. Signed-off-by: Guennadi Liakhovetski --- drivers/mmc/host/sh_mmcif.c | 48 -- 1 files changed, 41 insertions(+), 7 deletions(-) diff --git a/drivers/mmc/host/sh_

[PATCH 3/5] mmc: sh_mmcif: remove unused .down_pwr() callback

2012-04-20 Thread Guennadi Liakhovetski
Signed-off-by: Guennadi Liakhovetski --- include/linux/mmc/sh_mmcif.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/include/linux/mmc/sh_mmcif.h b/include/linux/mmc/sh_mmcif.h index 05f0e3d..cb84340 100644 --- a/include/linux/mmc/sh_mmcif.h +++ b/include/linux/mmc/sh_mm

[PATCH 2/5] sh: ecovec: remove unused .down_pwr() MMCIF callback

2012-04-20 Thread Guennadi Liakhovetski
.down_pwr() on ecovec was equivalent to .set_pwr(0). Now that .down_pwr() is no longer used by the driver, it can be removed. Signed-off-by: Guennadi Liakhovetski --- arch/sh/boards/mach-ecovec24/setup.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/arch/sh/boards

[PATCH/RFC] sh: ecovec: switch MMC power control to regulator

2012-04-20 Thread Guennadi Liakhovetski
Power on the CN12 SD/MMC slot on ecovec is controlled by a GPIO, which makes it possible to use the gpio-regulator driver to produce a primitive voltage regulator. Signed-off-by: Guennadi Liakhovetski --- As mentioned in the introductory mail, this patch is mostly an implementation example. a

[PATCH 1/5] mmc: sh_mmcif: remove redundant .down_pwr() callback

2012-04-20 Thread Guennadi Liakhovetski
>From the original version of sh_mmcif the .set_pwr() callback has only been used to turn the card's power on, and the .down_pwr() callback has been used to turn it off. .set_pwr() can be used for both these tasks, which is also how it is implemented by the only user of this API: the SH7724 ecovec

[PATCH 0/5] mmc: sh_mmcif: add regulator support

2012-04-20 Thread Guennadi Liakhovetski
Hi all The first 3 patches in this series are preparatory, patches 2 and 3 can be pushed with later kernel versions to simplify dependencies. Patch 4 actually adds regulator support. Patch 5 is not (necessarily) for inclusion. It's just an implementation example for ecovec. Since ecovec's CN12

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Tony Lindgren
* Russell King - ARM Linux [120420 08:41]: > > So, with this patch plus my original patch to omap's mmc host driver, this > should result in something which works without all the overhead of drivers > supplying the port information. Please confirm, and I'll merge this into > the original omap-dm

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Tony Lindgren
* Russell King - ARM Linux [120420 08:41]: > --- a/drivers/dma/omap-dma.c > +++ b/drivers/dma/omap-dma.c > @@ -83,11 +84,11 @@ static void omap_dma_start_sg(struct omap_chan *c, struct > omap_desc *d, > struct omap_sg *sg = d->sg + idx; > > if (d->dir == DMA_DEV_TO_MEM) > -

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Russell King - ARM Linux
On Fri, Apr 20, 2012 at 08:26:25AM -0700, Tony Lindgren wrote: > * Russell King - ARM Linux [120420 08:15]: > > On Thu, Apr 19, 2012 at 11:07:42AM -0700, Tony Lindgren wrote: > > > There's some information in the omap5912 trm on pages 717 and 718: > > > > > > http://www.ti.com/litv/pdf/spru742 >

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Tony Lindgren
* Russell King - ARM Linux [120420 08:15]: > On Thu, Apr 19, 2012 at 11:07:42AM -0700, Tony Lindgren wrote: > > * Russell King - ARM Linux [120419 10:46]: > > > On Wed, Apr 18, 2012 at 06:39:14PM -0700, Tony Lindgren wrote: > > > > Cool, you almost got it. Got it working for n800 and 770 with the

Re: [PATCH 8/8] ARM: omap_hsmmc: remove platform data dma_mask and initialization

2012-04-20 Thread Russell King - ARM Linux
On Thu, Apr 19, 2012 at 11:07:42AM -0700, Tony Lindgren wrote: > * Russell King - ARM Linux [120419 10:46]: > > On Wed, Apr 18, 2012 at 06:39:14PM -0700, Tony Lindgren wrote: > > > Cool, you almost got it. Got it working for n800 and 770 with the > > > following > > > patch. Only extremely light

Re: [PATCH] mmc: core: Do not pre-claim host in suspend

2012-04-20 Thread Chris Ball
Hi Ulf, On Thu, Apr 19 2012, Ulf Hansson wrote: > Since SDIO drivers may want to do some SDIO operations > in their suspend callback functions, we must not keep > the host claimed when calling them. > > Daniel Drake reported that libertas_sdio encountered > a deadlock in it's suspend function. > >

Re: [PATCH] mmc: core: Do not pre-claim host in suspend

2012-04-20 Thread Daniel Drake
On Fri, Apr 20, 2012 at 2:55 AM, Ulf Hansson wrote: > Thanks Daniel for testing, could we add your "Tested-by" to this patch then? Tested-by: Daniel Drake Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More ma

Re: [PATCH] MMC-4.5 Power OFF Notify rework

2012-04-20 Thread Ulf Hansson
On 04/20/2012 01:33 PM, Girish K S wrote: On 19 April 2012 18:11, Girish K S wrote: This is a rework of the existing POWER OFF NOTIFY patch. The current problem with the patch comes from the ambiguity on the usage of POWER OFF NOTIFY together with SLEEP and misunderstanding on the usage of MMC_

Re: [PATCH] MMC-4.5 Power OFF Notify rework

2012-04-20 Thread Girish K S
On 19 April 2012 18:11, Girish K S wrote: > This is a rework of the existing POWER OFF NOTIFY patch. The current problem > with the patch comes from the ambiguity on the usage of POWER OFF NOTIFY > together with SLEEP and misunderstanding on the usage of MMC_POWER_OFF > power_mode from mmc_set_ios

Re: [PATCH] mmc-utils: add the way to enable boot-partition

2012-04-20 Thread Chris Ball
Hi, On Fri, Apr 20 2012, Giuseppe CAVALLARO wrote: > On 4/19/2012 7:19 PM, Chris Ball wrote: >> Hi Giuseppe, >> >> On Thu, Apr 19 2012, Giuseppe CAVALLARO wrote: >>> This adds a new option to enable the boot from eMMC >>> by writing to the 179 ext_csd register. >>> >>> The example below shows as

Re: [PATCH] mmc: core: Do not pre-claim host in suspend

2012-04-20 Thread Ulf Hansson
On 04/19/2012 05:48 PM, Daniel Drake wrote: On Thu, Apr 19, 2012 at 4:02 AM, Vitaly Wool wrote: I think it looks like a hack. Can you provide a better description of where this deadlock actually happens? If libertas_sdio claims host as a part of its suspend operation from a context different f