Hi Tony, 2014-11-18 16:42 GMT+01:00 Tony Lindgren <tony@xxxxxxxxxxx>: > * Enric Balletbo Serra <eballetbo@xxxxxxxxx> [141118 01:04]: >> Hi Tony, >> >> 2014-11-17 19:04 GMT+01:00 Tony Lindgren <tony@xxxxxxxxxxx>: >> > >> > Just tested v3.18-rc5 with beagleboard xm, and the host mode >> > enumerates the devices, then may fail with "insufficient bus power" >> > at least for the WLAN device I tried with. >> > >> >> Is this with following options ? >> >> CONFIG_USB_MUSB_DUAL_ROLE=y >> # CONFIG_USB_MUSB_HOST is not set >> # CONFIG_USB_MUSB_GADGET is not set >> >> Because with these options doesn't work for me. It only works if I >> don't use dual role mode >> >> CONFIG_USB_MUSB_HOST=y >> CONFIG_USB_MUSB_DUAL_ROLE is not set > > Weird. Yes I have CONFIG_USB_MUSB_DUAL_ROLE. AFAIK there's actually > no way to disable the role switching in MUSB hardware, the hardware > tries to do things on it's own anyways. So it's probably best to always > set it to CONFIG_USB_MUSB_DUAL_ROLE=y unless size of the code is an > issue. > >> > I noticed that this only works with CONFIG_USB_MUSB_HDRC and >> > CONFIG_TWL4030_USB built into the kernel. No luck so far with them >> > as loadable modules for some reason. Looks like also USB gadgets >> > fail with MUSB as modules. > > I found the core issue with loadable modules, we currently need to > have just one platform glue layer module compiled in. I have a > fix coming up for that.. > >> It's weird, for me fails in both cases, built-in and with loadable >> modules. Connecting the OTG Cable Adapter with a pendrive reports >> >> [ 51.462432] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x54/84; link 1 >> [ 51.470916] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> twl4030_usb_runtime_resume >> [ 51.487274] musb-hdrc musb-hdrc.0.auto: ID GND >> [ 52.480712] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x54/84; link 1 >> [ 53.489044] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x54/84; link 1 >> >> And then polls until I disconnect the OTG Cable Adapter with the pendrive. >> >> [ 71.488983] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x54/84; link 1 >> [ 71.778930] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x50/80; link 4 >> [ 71.787536] musb-hdrc musb-hdrc.0.auto: VBUS Disconnect >> [ 72.489044] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> HW_CONDITIONS 0x50/80; link 4 >> [ 74.088714] twl4030_usb 48070000.i2c:twl@48:twl4030-usb: >> twl4030_usb_runtime_suspend > > Looks like the twl4030-usb is producing interrupts alright. Maybe > check your MUSB configuration one more time. See the attached > patch, but note you need to only select the glue layer for the > CONFIG_USB_MUSB_OMAP2PLUS option and disable other glue layer > module options. > Checked again, and no luck. It's very weird because from the OTG point of view, OTG is exactly the same between Beagleboard-XM and IGEPv2. Can you confirm that you're using kernel 3.18-rc5 without other patches applied ? At this moment, I don't have a Beagleboard-XM to test, I'll try to get one because at this moment I'm a bit stuck with this problem. > Also, you may want to monitor the VBUS line in host mode. If the > VBUS does not come up fast enough, MUSB hardware will give up as > it's again trying to do things on its own. > >> > And on the 37xx EVM, I've never had any luck getting any twl4030_usb >> > interrupts for some reason. > > Turns out the 37xx EVM is using an isp1507 phy instead of the > twl4030 phy. The proper fix for that will be to start using the > usb-nop-xceiv or phy-ulpi when we have drivers/phy/ driver for > it. Right now trying to use it seems to fail with the following > error: > > HS USB OTG: no PHY configured > > Regards, > > Tony > Thanks, Enric > 8< ------------- > From: Tony Lindgren <tony@xxxxxxxxxxx> > Date: Mon, 17 Nov 2014 07:53:59 -0800 > Subject: [PATCH] ARM: omap2plus_defconfig: Enable USB as loadable modules > > NOTE: Currently only one MUSB glue layer can be selected > below because of the ifdefs, I'll do a fix for that. > > --- a/arch/arm/configs/omap2plus_defconfig > +++ b/arch/arm/configs/omap2plus_defconfig > @@ -270,17 +270,92 @@ CONFIG_SND_OMAP_SOC_OMAP_ABE_TWL6040=m > CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m > CONFIG_USB=y > CONFIG_USB_ANNOUNCE_NEW_DEVICES=y > +CONFIG_USB_DYNAMIC_MINORS=y > +CONFIG_USB_OTG=y > CONFIG_USB_MON=y > CONFIG_USB_WDM=y > CONFIG_USB_STORAGE=y > +CONFIG_USB_MUSB_HDRC=m > +CONFIG_USB_MUSB_TUSB6010=m > +CONFIG_USB_MUSB_OMAP2PLUS=m > +CONFIG_USB_MUSB_AM35X=m > +CONFIG_USB_MUSB_DSPS=m > CONFIG_USB_DWC3=m > -CONFIG_USB_TEST=y > -CONFIG_AM335X_PHY_USB=y > +CONFIG_USB_SERIAL=m > +CONFIG_USB_SERIAL_GENERIC=y > +CONFIG_USB_SERIAL_SIMPLE=m > +CONFIG_USB_SERIAL_FTDI_SIO=m > +CONFIG_USB_SERIAL_PL2303=m > +CONFIG_USB_EMI62=m > +CONFIG_USB_EMI26=m > +CONFIG_USB_ADUTUX=m > +CONFIG_USB_SEVSEG=m > +CONFIG_USB_RIO500=m > +CONFIG_USB_LEGOTOWER=m > +CONFIG_USB_LCD=m > +CONFIG_USB_LED=m > +CONFIG_USB_CYPRESS_CY7C63=m > +CONFIG_USB_CYTHERM=m > +CONFIG_USB_IDMOUSE=m > +CONFIG_USB_FTDI_ELAN=m > +CONFIG_USB_APPLEDISPLAY=m > +CONFIG_USB_SISUSBVGA=m > +CONFIG_USB_SISUSBVGA_CON=y > +CONFIG_USB_LD=m > +CONFIG_USB_TRANCEVIBRATOR=m > +CONFIG_USB_IOWARRIOR=m > +CONFIG_USB_TEST=m > +CONFIG_USB_EHSET_TEST_FIXTURE=m > +CONFIG_USB_ISIGHTFW=m > +CONFIG_USB_YUREX=m > +CONFIG_USB_EZUSB_FX2=m > +CONFIG_USB_HSIC_USB3503=m > +CONFIG_USB_LINK_LAYER_TEST=m > +CONFIG_NOP_USB_XCEIV=y > +CONFIG_AM335X_PHY_USB=m > +CONFIG_TWL6030_USB=m > +CONFIG_USB_GPIO_VBUS=m > +CONFIG_USB_ISP1301=m > +CONFIG_USB_ULPI=y > CONFIG_USB_GADGET=y > CONFIG_USB_GADGET_DEBUG=y > CONFIG_USB_GADGET_DEBUG_FILES=y > CONFIG_USB_GADGET_DEBUG_FS=y > +CONFIG_USB_CONFIGFS=m > +CONFIG_USB_CONFIGFS_SERIAL=y > +CONFIG_USB_CONFIGFS_ACM=y > +CONFIG_USB_CONFIGFS_OBEX=y > +CONFIG_USB_CONFIGFS_NCM=y > +CONFIG_USB_CONFIGFS_ECM=y > +CONFIG_USB_CONFIGFS_ECM_SUBSET=y > +CONFIG_USB_CONFIGFS_RNDIS=y > +CONFIG_USB_CONFIGFS_EEM=y > +CONFIG_USB_CONFIGFS_MASS_STORAGE=y > +CONFIG_USB_CONFIGFS_F_LB_SS=y > +CONFIG_USB_CONFIGFS_F_FS=y > CONFIG_USB_ZERO=m > +CONFIG_USB_ZERO_HNPTEST=y > +CONFIG_USB_AUDIO=m > +CONFIG_GADGET_UAC1=y > +CONFIG_USB_ETH=m > +CONFIG_USB_ETH_EEM=y > +CONFIG_USB_G_NCM=m > +CONFIG_USB_GADGETFS=m > +CONFIG_USB_FUNCTIONFS=m > +CONFIG_USB_FUNCTIONFS_ETH=y > +CONFIG_USB_FUNCTIONFS_RNDIS=y > +CONFIG_USB_FUNCTIONFS_GENERIC=y > +CONFIG_USB_MASS_STORAGE=m > +CONFIG_USB_G_SERIAL=m > +CONFIG_USB_MIDI_GADGET=m > +CONFIG_USB_G_PRINTER=m > +CONFIG_USB_CDC_COMPOSITE=m > +CONFIG_USB_G_ACM_MS=m > +CONFIG_USB_G_MULTI=m > +CONFIG_USB_G_MULTI_CDC=y > +CONFIG_USB_G_HID=m > +CONFIG_USB_G_DBGP=m > +CONFIG_USB_LED_TRIG=y > CONFIG_MMC=y > CONFIG_SDIO_UART=y > CONFIG_MMC_OMAP=y > @@ -310,6 +385,7 @@ CONFIG_PWM_TWL=y > CONFIG_PWM_TWL_LED=y > CONFIG_OMAP_USB2=y > CONFIG_TI_PIPE3=y > +CONFIG_TWL4030_USB=m > CONFIG_EXT2_FS=y > CONFIG_EXT3_FS=y > # CONFIG_EXT3_FS_XATTR is not set -- 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
