Re: [PATCH 1/1] omap-hsmmc.c: re-organize dma line population
* Madhvapathi Sriram [100511 02:26]:
> @@ -255,6 +256,52 @@ int __init omap_mmc_add(const char *name, int id,
> unsigned long base,
> res[0].flags = IORESOURCE_MEM;
> res[1].start = res[1].end = irq;
> res[1].flags = IORESOURCE_IRQ;
> + /* Populate DMA lines based on the instance used. Rx first,Tx next*/
> + switch (id) {
> + case 0:
> + res[2].start = OMAP24XX_DMA_MMC1_RX;
> + res[2].end = OMAP24XX_DMA_MMC1_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP24XX_DMA_MMC1_TX;
> + res[3].end = OMAP24XX_DMA_MMC1_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 1:
> + res[2].start = OMAP24XX_DMA_MMC2_RX;
> + res[2].end = OMAP24XX_DMA_MMC2_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP24XX_DMA_MMC2_TX;
> + res[3].end = OMAP24XX_DMA_MMC2_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 2:
> + res[2].start = OMAP34XX_DMA_MMC3_RX;
> + res[2].end = OMAP34XX_DMA_MMC3_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP34XX_DMA_MMC3_TX;
> + res[3].end = OMAP34XX_DMA_MMC3_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 3:
> + res[2].start = OMAP44XX_DMA_MMC4_RX;
> + res[2].end = OMAP44XX_DMA_MMC4_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP44XX_DMA_MMC4_TX;
> + res[3].end = OMAP44XX_DMA_MMC4_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 4:
> + res[2].start = OMAP44XX_DMA_MMC5_RX;
> + res[2].end = OMAP44XX_DMA_MMC5_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP44XX_DMA_MMC5_TX;
> + res[3].end = OMAP44XX_DMA_MMC5_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + default:
> + ret = -ENODEV;
> + goto fail;
> + }
To me it seems like this should go into mach-omap2/hsmmc.c instead
of plat-omap/devices.c.
Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/1] omap-hsmmc.c: re-organize dma line population
Madhvapathi Sriram wrote:
> - This moves the dma line population from omap_hsmmc.c to plat-omap/devices.c
> - This is done to keep platform specific code inside the platform directory
> rather than the host driver
> - This makes the porting work across OMAP platforms easier
>
> Signed-off-by: Madhvapathi Sriram
> ---
> arch/arm/plat-omap/devices.c | 49
> -
> drivers/mmc/host/omap_hsmmc.c | 35 +++-
> 2 files changed, 57 insertions(+), 27 deletions(-)
>
> diff --git a/arch/arm/plat-omap/devices.c b/arch/arm/plat-omap/devices.c
> index 95677d1..a62b148 100644
> --- a/arch/arm/plat-omap/devices.c
> +++ b/arch/arm/plat-omap/devices.c
> @@ -30,6 +30,7 @@
> #include
> #include
> #include
> +#include
>
> #if defined(CONFIG_OMAP_DSP) || defined(CONFIG_OMAP_DSP_MODULE)
>
> @@ -232,7 +233,7 @@ static inline void omap_init_mcpdm(void) {}
> #if defined(CONFIG_MMC_OMAP) || defined(CONFIG_MMC_OMAP_MODULE) || \
> defined(CONFIG_MMC_OMAP_HS) || defined(CONFIG_MMC_OMAP_HS_MODULE)
>
> -#define OMAP_MMC_NR_RES 2
> +#define OMAP_MMC_NR_RES 4
>
> /*
> * Register MMC devices. Called from mach-omap1 and mach-omap2 device init.
> @@ -255,6 +256,52 @@ int __init omap_mmc_add(const char *name, int id,
> unsigned long base,
> res[0].flags = IORESOURCE_MEM;
> res[1].start = res[1].end = irq;
> res[1].flags = IORESOURCE_IRQ;
> + /* Populate DMA lines based on the instance used. Rx first,Tx next*/
> + switch (id) {
> + case 0:
> + res[2].start = OMAP24XX_DMA_MMC1_RX;
> + res[2].end = OMAP24XX_DMA_MMC1_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP24XX_DMA_MMC1_TX;
> + res[3].end = OMAP24XX_DMA_MMC1_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 1:
> + res[2].start = OMAP24XX_DMA_MMC2_RX;
> + res[2].end = OMAP24XX_DMA_MMC2_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP24XX_DMA_MMC2_TX;
> + res[3].end = OMAP24XX_DMA_MMC2_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 2:
> + res[2].start = OMAP34XX_DMA_MMC3_RX;
> + res[2].end = OMAP34XX_DMA_MMC3_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP34XX_DMA_MMC3_TX;
> + res[3].end = OMAP34XX_DMA_MMC3_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 3:
> + res[2].start = OMAP44XX_DMA_MMC4_RX;
> + res[2].end = OMAP44XX_DMA_MMC4_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP44XX_DMA_MMC4_TX;
> + res[3].end = OMAP44XX_DMA_MMC4_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 4:
> + res[2].start = OMAP44XX_DMA_MMC5_RX;
> + res[2].end = OMAP44XX_DMA_MMC5_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP44XX_DMA_MMC5_TX;
> + res[3].end = OMAP44XX_DMA_MMC5_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + default:
> + ret = -ENODEV;
> + goto fail;
> + }
>
> ret = platform_device_add_resources(pdev, res, ARRAY_SIZE(res));
> if (ret == 0)
> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
> index e9caf69..61d30ab 100644
> --- a/drivers/mmc/host/omap_hsmmc.c
> +++ b/drivers/mmc/host/omap_hsmmc.c
> @@ -2096,32 +2096,15 @@ static int __init omap_hsmmc_probe(struct
> platform_device *pdev)
>
> omap_hsmmc_conf_bus_power(host);
>
> - /* Select DMA lines */
> - switch (host->id) {
> - case OMAP_MMC1_DEVID:
> - host->dma_line_tx = OMAP24XX_DMA_MMC1_TX;
> - host->dma_line_rx = OMAP24XX_DMA_MMC1_RX;
> - break;
> - case OMAP_MMC2_DEVID:
> - host->dma_line_tx = OMAP24XX_DMA_MMC2_TX;
> - host->dma_line_rx = OMAP24XX_DMA_MMC2_RX;
> - break;
> - case OMAP_MMC3_DEVID:
> - host->dma_line_tx = OMAP34XX_DMA_MMC3_TX;
> - host->dma_line_rx = OMAP34XX_DMA_MMC3_RX;
> - break;
> - case OMAP_MMC4_DEVID:
> - host->dma_line_tx = OMAP44XX_DMA_MMC4_TX;
> - host->dma_line_rx = OMAP44XX_DMA_MMC4_RX;
> - break;
> - case OMAP_MMC5_DEVID:
> - host->dma_line_tx = OMAP44XX_DMA_MMC5_TX;
> - host->dma_line_rx = OMAP44XX_DMA_MMC5_RX;
> - break;
> - default:
> - dev_err(mmc_dev(host->mmc), "Invalid MMC id\n");
> - goto err_irq;
> - }
> + res = platform_get_resource(pdev, IO
Re: [PATCH 1/1] omap-hsmmc.c: re-organize dma line population
On Tue, May 11, 2010 at 11:32 AM, Madhvapathi Sriram wrote:
> - This moves the dma line population from omap_hsmmc.c to plat-omap/devices.c
> - This is done to keep platform specific code inside the platform directory
> rather than the host driver
> - This makes the porting work across OMAP platforms easier
>
> Signed-off-by: Madhvapathi Sriram
> ---
> arch/arm/plat-omap/devices.c | 49
> -
> drivers/mmc/host/omap_hsmmc.c | 35 +++-
> 2 files changed, 57 insertions(+), 27 deletions(-)
>
> diff --git a/arch/arm/plat-omap/devices.c b/arch/arm/plat-omap/devices.c
> index 95677d1..a62b148 100644
> --- a/arch/arm/plat-omap/devices.c
> +++ b/arch/arm/plat-omap/devices.c
> @@ -30,6 +30,7 @@
> #include
> #include
> #include
> +#include
>
> #if defined(CONFIG_OMAP_DSP) || defined(CONFIG_OMAP_DSP_MODULE)
>
> @@ -232,7 +233,7 @@ static inline void omap_init_mcpdm(void) {}
> #if defined(CONFIG_MMC_OMAP) || defined(CONFIG_MMC_OMAP_MODULE) || \
> defined(CONFIG_MMC_OMAP_HS) || defined(CONFIG_MMC_OMAP_HS_MODULE)
>
> -#define OMAP_MMC_NR_RES 2
> +#define OMAP_MMC_NR_RES 4
>
> /*
> * Register MMC devices. Called from mach-omap1 and mach-omap2 device init.
> @@ -255,6 +256,52 @@ int __init omap_mmc_add(const char *name, int id,
> unsigned long base,
> res[0].flags = IORESOURCE_MEM;
> res[1].start = res[1].end = irq;
> res[1].flags = IORESOURCE_IRQ;
You can just set .flags right here, no need to repeat in each switch case:
> + res[2].flags = IORESOURCE_DMA;
> + res[3].flags = IORESOURCE_DMA;
> + /* Populate DMA lines based on the instance used. Rx first,Tx next*/
> + switch (id) {
> + case 0:
> + res[2].start = OMAP24XX_DMA_MMC1_RX;
> + res[2].end = OMAP24XX_DMA_MMC1_RX;
> + res[2].flags = IORESOURCE_DMA;
> + res[3].start = OMAP24XX_DMA_MMC1_TX;
> + res[3].end = OMAP24XX_DMA_MMC1_TX;
> + res[3].flags = IORESOURCE_DMA;
> + break;
> + case 1:
> + res[2].start = OMAP24XX_DMA_MMC2_RX;
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 1/1] omap-hsmmc.c: re-organize dma line population
- This moves the dma line population from omap_hsmmc.c to plat-omap/devices.c
- This is done to keep platform specific code inside the platform directory
rather than the host driver
- This makes the porting work across OMAP platforms easier
Signed-off-by: Madhvapathi Sriram
---
arch/arm/plat-omap/devices.c | 49 -
drivers/mmc/host/omap_hsmmc.c | 35 +++-
2 files changed, 57 insertions(+), 27 deletions(-)
diff --git a/arch/arm/plat-omap/devices.c b/arch/arm/plat-omap/devices.c
index 95677d1..a62b148 100644
--- a/arch/arm/plat-omap/devices.c
+++ b/arch/arm/plat-omap/devices.c
@@ -30,6 +30,7 @@
#include
#include
#include
+#include
#ifdefined(CONFIG_OMAP_DSP) || defined(CONFIG_OMAP_DSP_MODULE)
@@ -232,7 +233,7 @@ static inline void omap_init_mcpdm(void) {}
#if defined(CONFIG_MMC_OMAP) || defined(CONFIG_MMC_OMAP_MODULE) || \
defined(CONFIG_MMC_OMAP_HS) || defined(CONFIG_MMC_OMAP_HS_MODULE)
-#define OMAP_MMC_NR_RES2
+#define OMAP_MMC_NR_RES4
/*
* Register MMC devices. Called from mach-omap1 and mach-omap2 device init.
@@ -255,6 +256,52 @@ int __init omap_mmc_add(const char *name, int id, unsigned
long base,
res[0].flags = IORESOURCE_MEM;
res[1].start = res[1].end = irq;
res[1].flags = IORESOURCE_IRQ;
+ /* Populate DMA lines based on the instance used. Rx first,Tx next*/
+ switch (id) {
+ case 0:
+ res[2].start = OMAP24XX_DMA_MMC1_RX;
+ res[2].end = OMAP24XX_DMA_MMC1_RX;
+ res[2].flags = IORESOURCE_DMA;
+ res[3].start = OMAP24XX_DMA_MMC1_TX;
+ res[3].end = OMAP24XX_DMA_MMC1_TX;
+ res[3].flags = IORESOURCE_DMA;
+ break;
+ case 1:
+ res[2].start = OMAP24XX_DMA_MMC2_RX;
+ res[2].end = OMAP24XX_DMA_MMC2_RX;
+ res[2].flags = IORESOURCE_DMA;
+ res[3].start = OMAP24XX_DMA_MMC2_TX;
+ res[3].end = OMAP24XX_DMA_MMC2_TX;
+ res[3].flags = IORESOURCE_DMA;
+ break;
+ case 2:
+ res[2].start = OMAP34XX_DMA_MMC3_RX;
+ res[2].end = OMAP34XX_DMA_MMC3_RX;
+ res[2].flags = IORESOURCE_DMA;
+ res[3].start = OMAP34XX_DMA_MMC3_TX;
+ res[3].end = OMAP34XX_DMA_MMC3_TX;
+ res[3].flags = IORESOURCE_DMA;
+ break;
+ case 3:
+ res[2].start = OMAP44XX_DMA_MMC4_RX;
+ res[2].end = OMAP44XX_DMA_MMC4_RX;
+ res[2].flags = IORESOURCE_DMA;
+ res[3].start = OMAP44XX_DMA_MMC4_TX;
+ res[3].end = OMAP44XX_DMA_MMC4_TX;
+ res[3].flags = IORESOURCE_DMA;
+ break;
+ case 4:
+ res[2].start = OMAP44XX_DMA_MMC5_RX;
+ res[2].end = OMAP44XX_DMA_MMC5_RX;
+ res[2].flags = IORESOURCE_DMA;
+ res[3].start = OMAP44XX_DMA_MMC5_TX;
+ res[3].end = OMAP44XX_DMA_MMC5_TX;
+ res[3].flags = IORESOURCE_DMA;
+ break;
+ default:
+ ret = -ENODEV;
+ goto fail;
+ }
ret = platform_device_add_resources(pdev, res, ARRAY_SIZE(res));
if (ret == 0)
diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
index e9caf69..61d30ab 100644
--- a/drivers/mmc/host/omap_hsmmc.c
+++ b/drivers/mmc/host/omap_hsmmc.c
@@ -2096,32 +2096,15 @@ static int __init omap_hsmmc_probe(struct
platform_device *pdev)
omap_hsmmc_conf_bus_power(host);
- /* Select DMA lines */
- switch (host->id) {
- case OMAP_MMC1_DEVID:
- host->dma_line_tx = OMAP24XX_DMA_MMC1_TX;
- host->dma_line_rx = OMAP24XX_DMA_MMC1_RX;
- break;
- case OMAP_MMC2_DEVID:
- host->dma_line_tx = OMAP24XX_DMA_MMC2_TX;
- host->dma_line_rx = OMAP24XX_DMA_MMC2_RX;
- break;
- case OMAP_MMC3_DEVID:
- host->dma_line_tx = OMAP34XX_DMA_MMC3_TX;
- host->dma_line_rx = OMAP34XX_DMA_MMC3_RX;
- break;
- case OMAP_MMC4_DEVID:
- host->dma_line_tx = OMAP44XX_DMA_MMC4_TX;
- host->dma_line_rx = OMAP44XX_DMA_MMC4_RX;
- break;
- case OMAP_MMC5_DEVID:
- host->dma_line_tx = OMAP44XX_DMA_MMC5_TX;
- host->dma_line_rx = OMAP44XX_DMA_MMC5_RX;
- break;
- default:
- dev_err(mmc_dev(host->mmc), "Invalid MMC id\n");
- goto err_irq;
- }
+ res = platform_get_resource(pdev, IORESOURCE_DMA, 0);
+ if (!res)
+ goto err1;
+ host->dma_line_rx = res->start;
+
+ res = platform_get_resource(pdev, IORESOURCE_DMA, 1);
+ if (!res)
+ goto err1;
+ host->dma_line_tx = res->star
