Hi Stefano,

Cc'd Govindraj.

On 02/05/12 16:57, Stefano Babic wrote:
> From: Ilya Yanok <ya...@emcraft.com>
> 
> This patch adds support for the HTKW mcx AM3517-based board.
> Serial, Ethernet, NAND, MMC, RTC, EHCI USB host and both
> NAND and MMC SPLs are supported.
> 
> Signed-off-by: Ilya Yanok <ya...@emcraft.com>
> Signed-off-by: Stefano Babic <sba...@denx.de>
> CC: Tom Rini <tom.r...@gmail.com>
> Cc: Detlev Zundel <d...@denx.de>

[...]

> diff --git a/board/htkw/mcx/mcx.c b/board/htkw/mcx/mcx.c
> new file mode 100644
> index 0000000..dcc8707
> --- /dev/null
> +++ b/board/htkw/mcx/mcx.c

[...]

> +#ifdef CONFIG_USB_EHCI
> +static struct omap_usbhs_board_data usbhs_bdata = {
> +     .port_mode[0] = OMAP_EHCI_PORT_MODE_PHY,
> +     .port_mode[1] = OMAP_EHCI_PORT_MODE_PHY,
> +     .port_mode[2] = OMAP_EHCI_PORT_MODE_PHY,

This looks wrong, as port3 is TLL only, so it cannot have a PHY,
or am I missing something?

> +};
> +
> +int ehci_hcd_init(void)
> +{
> +     int ret;
> +
> +     ret = omap_ehci_hcd_init(&usbhs_bdata);
> +     if (ret < 0)
> +             return ret;
> +
> +     return 0;
> +}

return omap_ehci_hcd_init(&usbhs_bdata);
?

> +
> +int ehci_hcd_stop(void)
> +{
> +     int ret;
> +
> +     ret = omap_ehci_hcd_stop();
> +     return ret;
> +}

same here:
return omap_ehci_hcd_stop();
?

[...]


-- 
Regards,
Igor.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to