<br><br><div class="gmail_quote">2012/6/11 Jon Hunter <span dir="ltr"><<a href="mailto:jon-hunter@ti.com" target="_blank">jon-hunter@ti.com</a>></span><br><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im"><br>
On 06/11/2012 09:00 AM, Zumeng Chen wrote:<br>
> A typo fix for this cosmetic change and mute a failed message from<br>
> a unnecessary setting of some parent clk for usbhs_omap on OMAP3EVM.<br>
><br>
> Signed-off-by: Zumeng Chen <<a href="mailto:zumeng.chen@gmail.com">zumeng.chen@gmail.com</a>><br>
> ---<br>
>  drivers/mfd/omap-usb-host.c |    4 +++-<br>
>  1 files changed, 3 insertions(+), 1 deletions(-)<br>
><br>
> diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c<br>
> index 7e96bb2..9aaaf3c 100644<br>
> --- a/drivers/mfd/omap-usb-host.c<br>
> +++ b/drivers/mfd/omap-usb-host.c<br>
> @@ -698,8 +698,9 @@ static int __devinit usbhs_omap_probe(struct platform_device *pdev)<br>
>               goto err_usbtll_p2_fck;<br>
>       }<br>
><br>
> +#ifndef CONFIG_MACH_OMAP3EVM<br>
> +     /* for OMAP3 , the clk set parent fails */<br>
>       if (is_ehci_phy_mode(pdata->port_mode[0])) {<br>
> -             /* for OMAP3 , the clk set paretn fails */<br>
>               ret = clk_set_parent(omap->utmi_p1_fck,<br>
>                                       omap->xclk60mhsp1_ck);<br>
>               if (ret != 0)<br>
<br>
</div>This begs the question, why is port_mode[0] set to ehci phy mode if this<br>
is failing? Something does not seem right here but this does not look<br>
like the right fix.<br></blockquote><div>Actually, for omap3530evm, there is only port-mode[1] is valid, so port_mode[0]<br>will be skipped by "if". And it will report "xclk60mhsp2_ck set parent failed"<br>
<br>But why I bracket both of them, because, both xclk60mhsp2_ck and xclk60mhsp1_ck<br>have dummy_ck clk for omap3530evm. So we can skip them directly.<br><br>Regards,<br>Zumeng<br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">Jon<br>
</font></span></blockquote></div><br>