<br><br><div class="gmail_quote">On 22 January 2013 09:09, Alan Stern <span dir="ltr"><<a href="mailto:stern@rowland.harvard.edu" target="_blank">stern@rowland.harvard.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Mon, 21 Jan 2013, Arnd Bergmann wrote:<br>
<br>
> Alan, one comment about your version: You keep maintaining the<br>
> '#if IS_ENABLED' list in the main driver, which I think can actually<br>
> get removed now.<br>
<br>
</div>I considered doing that.  The benefit of keeping it is that it warns<br>
about configs where the main driver is built but can never be used.<br>
<br>
Still, you're right that its purpose is becoming less important.  I had<br>
planned to remove it when all the glue driver had been converted, but<br>
in fact it could be removed at any time.<br>
<div class="im"><br>
>  Since the base driver can be built independent of<br>
> the presence of platform glue drivers, there is no need to forbid<br>
> it any more, and the #if block will cause merge conflicts for each<br>
> patch that converts or adds another platform.<br>
<br>
</div>If the patches are done independently.  There are likely to be some<br>
context conflicts anyway.<br>
<div class="im"><br>
>  I think we can actually<br>
> get the same results by turning the Kconfig logic around and making<br>
> the platform glue drivers 'select USB_EHCI_HCD' than depending on<br>
> it.<br>
<br>
</div>That's a good idea.  The Kconfig changes would have to be done<br>
carefully to make sure that the dependency on USB_ARCH_HAS_EHCI still<br>
applies to all the platform drivers.<br>
<br>
Manjunath, would you like to send a patch to do this?<br>
<span class="HOEnZb"><font color="#888888"><br>
Alan Stern<br>
<br>
</font></span></blockquote></div>Ya sure,could you explain little bit  briefly about the Kconfig changes.<div><br></div><div>Thanks</div><div>MAnjunath Goudar  </div>