Re: [PATCH 3/3 v2] musb: AM35x: Workaround for fifo read issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 20, 2010 at 11:41:14AM +0200, ext Gadiyar, Anand wrote:
It would not scale very well - we already have multi-omap builds
that select support for multiple boards. If the AM35x boards are
part of such builds, then mutually exclusive config options won't
work - already n8x0 MUSB is exclusive with 243x/omap3.

If it is possible to detect the AM35x at runtime, then we could
handle this well. Also, a similar set of changes will be needed for
the DMA code as well (right now we can pick only one of the DMA
engines at a time).

it's time to split out platform code from musb core. I was thinking of having omap2430.c blackfin.c tusb6010.c davinci.c be a platform_device that instantiates musb_hdrc platform_device. It would also ioremap() the area and pass the gotten/enabled clock to musb. Then we can have all of the platforms enabled since the board files would pass down the platform_device for the platform code. Something like:

arch/arm/mach-omap2/usb-musb.c:

static struct platform_device omap_musb = {
	.dev	= {
		.name	= "omap-hsusbotg",	/* using the IP block name */
	},
	[..]
};

[..]

	platform_device_register(&omap_musb);

drivers/usb/musb/omap2430.c:

static struct musb_platform_data omap_musb_data;

static int __init omap_musb_probe(struct platform_device *pdev)
{
	clk = clk_get(&pdev->dev, "ick");
	base = ioremap(res->start, resource_size(res));

	omap_musb_data.clk = clk;
	omap_musb_data.base = base;

	[... all other necessary fields, like mode, etc]

	musb_pdev = platform_device_alloc("musb_hdrc", -1);
	musb_pdev->dev.parent = &pdev->dev;
	platform_device_add_data(musb_pdev, &omap_musb_data);
	platform_device_add(musb_pdev);

	return 0;
}

static int __exit omap_musb_remove(struct platform_device *pdev)
{
	clk_put(clk);
	iounmap(base);

	platform_device_del(musb_pdev);

	return 0;
}

[...]

io functions could also be passed through this platform_device so that we remove the insane amounts of ifdefs. Also context save/restore could be done a per-platform basis. Since the platform-code would be a parent to musb_hdrc, platform-code's suspend would be called before musb_hdrc suspend, then we would save context at that point already.

Any other ifdefferry on anything would be easier to remove with this approach and it also means we can have one musb_hdrc.ko for all (well arm-based only) boards and be sure it would work. We would just need to put something similar down for dma engines.

--
balbi

DefectiveByDesign.org
--
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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux