On Thu, Dec 2, 2010 at 07:57, Kanigeri, Hari <h-kanigeri2@xxxxxx> wrote: > On Wed, Dec 1, 2010 at 8:10 PM, Kanigeri, Hari <h-kanigeri2@xxxxxx> wrote: >> On Wed, Dec 1, 2010 at 12:04 AM, Varadarajan, Charulatha <charu@xxxxxx> wrote: >>> David, >> >>> >>>> + if (!oh) >>>> + goto error; >>>> + >>>> + data->base = oh->_mpu_rt_va; >>> >>> not required. Make use of platform_get APIs in probe to extract the >>> base, dma and irq info using pdev. >> >> Not sure about using platform_get APIs. I think one has to use >> omap_hwmod_get_mpu_rt_va to get the address, which internally returns >> oh-<_mpu_rt_va. > small correction... omap_device_get_rt_va and not omap_hwmod_get_mpu_rt_va. To get the base address & irq, you need not have to use omap_device_get_rt_va and pass it as pdata and then use it during probe. Instead in probe, you may do something like the following: static int __devinit dev_probe (*pdev) { struct resource *res; void __iomem *base; u16 irq; .... .... res = platform_get_resource(pdev, IORESOURCE_MEM, 0); if (unlikely(!res)) { .... return -ENODEV; } base = ioremap(res->start, resource_size(res)); if (base) { .... return -ENOMEM; } res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); if (unlikely(!res)) { .... return -ENODEV; } irq = res->start; .... .... } -V Charulatha -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html