On Wed, May 05, 2010 at 05:56:11PM -0500, Dinh Nguyen wrote:
> This patch enables usb gadget for freescale mx51 babbage hw. It adds
> a kconfig condition for enabling either host or gadget on the OTG
> port.
I used a command line option for our boards which I find far more
convenient. Have a look at arch/arch/mach-mx2/mach-pca100.c how this is
done.
As a side effect you get rid of the 'defined but not used' warnings this
patch introduces and you don't have to put platform related kconfig
options to a totally unrelated place.
Sascha
>
> Signed-off-by: Dinh Nguyen <Dinh.Nguyen@xxxxxxxxxxxxx>
> ---
> arch/arm/mach-mx5/board-mx51_babbage.c | 11 +++++++++++
> arch/arm/mach-mx5/clock-mx51.c | 2 ++
> arch/arm/mach-mx5/devices.c | 12 ++++++++++++
> arch/arm/mach-mx5/devices.h | 1 +
> drivers/usb/host/Kconfig | 8 ++++++++
> 5 files changed, 34 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-mx5/board-mx51_babbage.c b/arch/arm/mach-mx5/board-mx51_babbage.c
> index 99f7ea9..fac8ce0 100644
> --- a/arch/arm/mach-mx5/board-mx51_babbage.c
> +++ b/arch/arm/mach-mx5/board-mx51_babbage.c
> @@ -15,6 +15,7 @@
> #include <linux/gpio.h>
> #include <linux/delay.h>
> #include <linux/io.h>
> +#include <linux/fsl_devices.h>
>
> #include <mach/common.h>
> #include <mach/hardware.h>
> @@ -179,6 +180,11 @@ static struct mxc_usbh_platform_data dr_utmi_config = {
> .flags = MXC_EHCI_INTERNAL_PHY,
> };
>
> +static struct fsl_usb2_platform_data usb_pdata = {
> + .operating_mode = FSL_USB2_DR_DEVICE,
> + .phy_mode = FSL_USB2_PHY_UTMI_WIDE,
> +};
> +
> static struct mxc_usbh_platform_data usbh1_config = {
> .init = initialize_usbh1_port,
> .portsc = MXC_EHCI_MODE_ULPI,
> @@ -197,7 +203,12 @@ static void __init mxc_board_init(void)
> mxc_init_imx_uart();
> platform_add_devices(devices, ARRAY_SIZE(devices));
>
> +#if defined(CONFIG_USB_EHCI_MXC_OTG)
> mxc_register_device(&mxc_usbdr_host_device, &dr_utmi_config);
> +#elif defined(CONFIG_USB_GADGET_FSL_USB2)
> + initialize_otg_port(NULL);
> + mxc_register_device(&mxc_usbdr_udc_device, &usb_pdata);
> +#endif
>
> gpio_usbh1_active();
> mxc_register_device(&mxc_usbh1_device, &usbh1_config);
> diff --git a/arch/arm/mach-mx5/clock-mx51.c b/arch/arm/mach-mx5/clock-mx51.c
> index 933c0d1..ff15424 100644
> --- a/arch/arm/mach-mx5/clock-mx51.c
> +++ b/arch/arm/mach-mx5/clock-mx51.c
> @@ -786,6 +786,8 @@ static struct clk_lookup lookups[] = {
> _REGISTER_CLOCK("mxc-ehci.0", "usb_ahb", ahb_clk)
> _REGISTER_CLOCK("mxc-ehci.1", "usb", usboh3_clk)
> _REGISTER_CLOCK("mxc-ehci.1", "usb_ahb", ahb_clk)
> + _REGISTER_CLOCK("fsl-usb2-udc", "usb", usboh3_clk)
> + _REGISTER_CLOCK("fsl-usb2-udc", "usb_ahb", ahb_clk)
> };
>
> static void clk_tree_init(void)
> diff --git a/arch/arm/mach-mx5/devices.c b/arch/arm/mach-mx5/devices.c
> index e6262f3..23850e6 100644
> --- a/arch/arm/mach-mx5/devices.c
> +++ b/arch/arm/mach-mx5/devices.c
> @@ -107,6 +107,18 @@ static struct resource usbotg_resources[] = {
> },
> };
>
> +/* OTG gadget device */
> +struct platform_device mxc_usbdr_udc_device = {
> + .name = "fsl-usb2-udc",
> + .id = -1,
> + .num_resources = ARRAY_SIZE(usbotg_resources),
> + .resource = usbotg_resources,
> + .dev = {
> + .dma_mask = &usb_dma_mask,
> + .coherent_dma_mask = DMA_BIT_MASK(32),
> + },
> +};
> +
> struct platform_device mxc_usbdr_host_device = {
> .name = "mxc-ehci",
> .id = 0,
> diff --git a/arch/arm/mach-mx5/devices.h b/arch/arm/mach-mx5/devices.h
> index 95c45f9..0494d6b 100644
> --- a/arch/arm/mach-mx5/devices.h
> +++ b/arch/arm/mach-mx5/devices.h
> @@ -4,3 +4,4 @@ extern struct platform_device mxc_uart_device2;
> extern struct platform_device mxc_fec_device;
> extern struct platform_device mxc_usbdr_host_device;
> extern struct platform_device mxc_usbh1_device;
> +extern struct platform_device mxc_usbdr_udc_device;
> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> index 8d3df03..da58a2e 100644
> --- a/drivers/usb/host/Kconfig
> +++ b/drivers/usb/host/Kconfig
> @@ -123,6 +123,14 @@ config USB_EHCI_MXC
> ---help---
> Variation of ARC USB block used in some Freescale chips.
>
> +config USB_EHCI_MXC_OTG
> + bool "Support for DR host port on Freescale controller"
> + depends on USB_EHCI_MXC && (ARCH_MX51)
> + ---help---
> + Enable support for the USB OTG port in HS/FS Host mode.
> + NOTE: Unless the OTG feature is enabled, EHCI and Gadget feature
> + for the OTG port should be mutually exclusive.
> +
> config USB_EHCI_HCD_PPC_OF
> bool "EHCI support for PPC USB controller on OF platform bus"
> depends on USB_EHCI_HCD && PPC_OF
> --
> 1.6.0.4
>
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
