On Wed, Oct 20, 2010 at 10:46 AM, Roger Quadros <roger.quadros@xxxxxxxxx> wrote: > On 10/19/2010 05:33 PM, ext Felipe Contreras wrote: >> On Tue, Oct 19, 2010 at 4:40 PM, Roger Quadros<roger.quadros@xxxxxxxxx> >> Âwrote: >>>> >>>> @@ -843,6 +841,7 @@ config USB_CDC_COMPOSITE >>>> Âconfig USB_G_NOKIA >>>> Â Â Â Âtristate "Nokia composite gadget" >>>> Â Â Â Âdepends on PHONET >>>> + Â Â Â depends on USB_GADGET_MUSB_HDRC >>> >>> This is wrong. Is there a build problem or run-time problem without this? >> >> Try: >> CONFIG_USB_G_NOKIA=y >> CONFIG_USB_GADGET_MUSB_HDRC=n >> >> At least g_nokia doesn't seem to work correctly on my N900. >> > It won't work on N900 because N900 has MUSB USB gadget controller and it > needs to be selected somehow while building for N900. > > You should set CONFIG_USB_GADGET_MUSB_HDRC to y somewhere else while > building for N900 and not here because g_nokia is not MUSB specific. How do you expect people to know that? The Kconfig should define what USB_G_NOKIA needs to work, so that people can enable those things without go Googling hunting for a workable defconfig. Right now people can build g_nokia modules that don't work, and that shouldn't be allowed by the Kconfig. -- Felipe Contreras -- 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