linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected
@ 2010-05-25  1:13 Pinkava J.
  2010-05-25  1:22 ` [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35 Pinkava J.
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Pinkava J. @ 2010-05-25  1:13 UTC (permalink / raw)
  To: linux-arm-kernel


Signed-off-by: Jiri Pinkava <jiri.pinkava@vscht.cz>
---
 drivers/mmc/host/s3cmci.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mmc/host/s3cmci.c b/drivers/mmc/host/s3cmci.c
index 2fdf768..4426a4b 100644
--- a/drivers/mmc/host/s3cmci.c
+++ b/drivers/mmc/host/s3cmci.c
@@ -1600,7 +1600,7 @@ static int __devinit s3cmci_probe(struct platform_device
*pdev)
 	host->pio_active 	= XFER_NONE;

 #ifdef CONFIG_MMC_S3C_PIODMA
-	host->dodma		= host->pdata->dma;
+	host->dodma		= host->pdata->use_dma;
 #endif

 	host->mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35
  2010-05-25  1:13 [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Pinkava J.
@ 2010-05-25  1:22 ` Pinkava J.
  2010-05-25  8:09   ` Pinkava J.
  2010-05-25  4:38 ` [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Ben Dooks
  2010-05-25  9:29 ` Sergei Shtylyov
  2 siblings, 1 reply; 5+ messages in thread
From: Pinkava J. @ 2010-05-25  1:22 UTC (permalink / raw)
  To: linux-arm-kernel

Exchange on and off state, MMC is working even if "powered off", but when comes
to write it causes random failures, most frequently zeroes first/all sector(s).

Signed-off-by: Jiri Pinkava <jiri.pinkava@vscht.cz>
---
 arch/arm/mach-s3c2410/mach-n30.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-s3c2410/mach-n30.c b/arch/arm/mach-s3c2410/mach-n30.c
index 41f299d..6c3456c 100644
--- a/arch/arm/mach-s3c2410/mach-n30.c
+++ b/arch/arm/mach-s3c2410/mach-n30.c
@@ -357,11 +357,11 @@ static void n30_sdi_set_power(unsigned char power_mode,
unsigned short vdd)
 	switch (power_mode) {
 	case MMC_POWER_ON:
 	case MMC_POWER_UP:
-		gpio_set_value(S3C2410_GPG(4), 1);
+		gpio_set_value(S3C2410_GPG(4), 0);
 		break;
 	case MMC_POWER_OFF:
 	default:
-		gpio_set_value(S3C2410_GPG(4), 0);
+		gpio_set_value(S3C2410_GPG(4), 1);
 		break;
 	}
 }
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected
  2010-05-25  1:13 [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Pinkava J.
  2010-05-25  1:22 ` [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35 Pinkava J.
@ 2010-05-25  4:38 ` Ben Dooks
  2010-05-25  9:29 ` Sergei Shtylyov
  2 siblings, 0 replies; 5+ messages in thread
From: Ben Dooks @ 2010-05-25  4:38 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, May 25, 2010 at 03:13:39AM +0200, Pinkava J. wrote:
> 
> Signed-off-by: Jiri Pinkava <jiri.pinkava@vscht.cz>
Acked-by: Ben Dooks <ben-linux@fluff.org>

please send to the linux mmc list and/or akpm for inclusion.

> ---
>  drivers/mmc/host/s3cmci.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/mmc/host/s3cmci.c b/drivers/mmc/host/s3cmci.c
> index 2fdf768..4426a4b 100644
> --- a/drivers/mmc/host/s3cmci.c
> +++ b/drivers/mmc/host/s3cmci.c
> @@ -1600,7 +1600,7 @@ static int __devinit s3cmci_probe(struct platform_device
> *pdev)
>  	host->pio_active 	= XFER_NONE;
> 
>  #ifdef CONFIG_MMC_S3C_PIODMA
> -	host->dodma		= host->pdata->dma;
> +	host->dodma		= host->pdata->use_dma;
>  #endif
> 
>  	host->mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -- 
> 1.7.1
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

-- 
-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35
  2010-05-25  1:22 ` [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35 Pinkava J.
@ 2010-05-25  8:09   ` Pinkava J.
  0 siblings, 0 replies; 5+ messages in thread
From: Pinkava J. @ 2010-05-25  8:09 UTC (permalink / raw)
  To: linux-arm-kernel

Dne 25.5.2010 03:22, Pinkava J. napsal(a):
> Exchange on and off state, MMC is working even if "powered off", but when comes
> to write it causes random failures, most frequently zeroes first/all sector(s).
Sorry for mess, this is idependend on previous patch and should be included.

(Or removed patch named:
ARM: n30: Add support for power on/off on Acer n30 / Acer n35 MMC card reader
)


> 
> Signed-off-by: Jiri Pinkava <jiri.pinkava@vscht.cz>
> ---
>  arch/arm/mach-s3c2410/mach-n30.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-s3c2410/mach-n30.c b/arch/arm/mach-s3c2410/mach-n30.c
> index 41f299d..6c3456c 100644
> --- a/arch/arm/mach-s3c2410/mach-n30.c
> +++ b/arch/arm/mach-s3c2410/mach-n30.c
> @@ -357,11 +357,11 @@ static void n30_sdi_set_power(unsigned char power_mode,
> unsigned short vdd)
>  	switch (power_mode) {
>  	case MMC_POWER_ON:
>  	case MMC_POWER_UP:
> -		gpio_set_value(S3C2410_GPG(4), 1);
> +		gpio_set_value(S3C2410_GPG(4), 0);
>  		break;
>  	case MMC_POWER_OFF:
>  	default:
> -		gpio_set_value(S3C2410_GPG(4), 0);
> +		gpio_set_value(S3C2410_GPG(4), 1);
>  		break;
>  	}
>  }

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected
  2010-05-25  1:13 [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Pinkava J.
  2010-05-25  1:22 ` [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35 Pinkava J.
  2010-05-25  4:38 ` [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Ben Dooks
@ 2010-05-25  9:29 ` Sergei Shtylyov
  2 siblings, 0 replies; 5+ messages in thread
From: Sergei Shtylyov @ 2010-05-25  9:29 UTC (permalink / raw)
  To: linux-arm-kernel

Hello.

Pinkava J. wrote:

> Signed-off-by: Jiri Pinkava <jiri.pinkava@vscht.cz>
> ---
>  drivers/mmc/host/s3cmci.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

> diff --git a/drivers/mmc/host/s3cmci.c b/drivers/mmc/host/s3cmci.c
> index 2fdf768..4426a4b 100644
> --- a/drivers/mmc/host/s3cmci.c
> +++ b/drivers/mmc/host/s3cmci.c
> @@ -1600,7 +1600,7 @@ static int __devinit s3cmci_probe(struct platform_device
> *pdev)

    Your patch is line wrapped. Should be easy to fix however...

WBR, Sergei

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-05-25  9:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-25  1:13 [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Pinkava J.
2010-05-25  1:22 ` [PATCH 2/2] ARM: Fix MMC power management on Acer n30 / n35 Pinkava J.
2010-05-25  8:09   ` Pinkava J.
2010-05-25  4:38 ` [PATCH 1/2] ARM: SAMSUNG: MMC: Fix build error when both DMA and PIO mode selected Ben Dooks
2010-05-25  9:29 ` Sergei Shtylyov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).