Hi,<br><br><div class="gmail_quote">On Thu, Jul 26, 2012 at 8:57 PM, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Saturday 21 July 2012, Vivek Gautam wrote:<br>
> @@ -28,6 +29,11 @@ static struct exynos4_ohci_platdata smdk5250_ohci_pdata = {<br>
>         .phy_exit = s5p_usb_phy_exit,<br>
>  };<br>
><br>
> +static struct s5p_ehci_platdata smdk5250_ehci_pdata = {<br>
> +       .phy_init = s5p_usb_phy_init,<br>
> +       .phy_exit = s5p_usb_phy_exit,<br>
> +};<br>
> +<br>
>  /*<br>
>   * The following lookup table is used to override device names when devices<br>
>   * are registered from device tree. This is temporarily added to enable<br>
<br>
</div>I'm a bit confused of the purpose of setting the function pointers here<br>
when the pointers are always to the same s5p_usb_phy_init/s5p_usb_phy_exit<br>
identifiers, but you have different definitions of those functions<br>
depending on which platforms you are building for.<br>
<br>
What is the idea here? Are you going to unify the three definitions of<br>
s5p_usb_phy_init, or is that function going away eventually when we<br>
get to the point of building exynos/s5p/s3c64xx into the same kernel<br>
binary?<br>
<span><font color="#888888"><br></font></span></blockquote><div>Should we be differentiating here between the calls at runtime for different SOCs.</div><div>In that case, yes i think we need a way here to have a solution here so that the function call</div>
<div>goes differently for different soc while in runtime. I will work on this and get back asap.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888">        Arnd<br>

</font></span><div><div><br>
_______________________________________________<br>
devicetree-discuss mailing list<br>
<a href="mailto:devicetree-discuss@lists.ozlabs.org" target="_blank">devicetree-discuss@lists.ozlabs.org</a><br>
<a href="https://lists.ozlabs.org/listinfo/devicetree-discuss" target="_blank">https://lists.ozlabs.org/listinfo/devicetree-discuss</a></div></div></blockquote><div>Best regards</div><div>Vivek </div></div><br>