Andreas, Tony, Thank you for your help. Unfortunately, I tried changing compatible = "ti,omap4-hsmmc"; to compatible = "ti,am33xx-hsmmc"; in the dtb, but the modifications didn't change the end result. When I boot with the module in place I end up in initramfs and when I insert the module into a booted system I get errors at the console and the error -bash: /bin/dmesg: Input/output error when I try and run dmesg. That said, ls is working now and I am able to list the contents of a directory which I couldn't do before. I don't have the idle state defined the dts, but I've used this module with an iMX6 Ultralite board (with the non-open source Broadcom drivers in NXP's git) and I didn't need to define that state there. I'll take a look at PINCTRL_STATE_IDLE in the driver to see if I learn anything. Any other suggestions? -----Original Message----- From: Tony Lindgren [mailto:tony@xxxxxxxxxxx] Sent: Friday, June 10, 2016 7:48 AM To: Andreas Fenkart <afenkart@xxxxxxxxx> Cc: Michael Eskowitz <MichaelE@xxxxxxxxxxxxxxx>; linux-omap <linux-omap@xxxxxxxxxxxxxxx> Subject: Re: Bringing up the BRCM43362 on Beaglebone Black * Andreas Fenkart <afenkart@xxxxxxxxx> [160610 04:40]: > me again, > > 2016-06-10 13:20 GMT+02:00 Andreas Fenkart <afenkart@xxxxxxxxx>: > > > >> The omap_hsmmc.c driver supports configuring two interrupts, and > >> two named pinctrl states. See PINCTRL_STATE_IDLE in the driver. > >> > >> You need to define the dat1 GPIO mux mode as the PINCTRL_STATE_IDLE > >> in the dts file, and add that as the second interrupt using the > >> interrupts-extended binding. > > > > Actually it should work without this. The throughput would be lame > > though, since it would fallback to polling mode. > > > > http://www.mjmwired.net/kernel/Documentation/devicetree/bindings/mmc > > /ti-omap-hsmmc.txt#61 OK > just realized that am33xx.dtsi, declares the mmc ports to be > compatible with "ti,omap4-hsmmc" > > mmc1: mmc@48060000 { > compatible = "ti,omap4-hsmmc"; > > That way the omap_hsmmc driver will never emit a warning for missing > pinctrl states, and NOT fallback to polling. That would explain the > problems that Michael observed. > > This probably should be changed to: > > --- a/arch/arm/boot/dts/am33xx.dtsi > +++ b/arch/arm/boot/dts/am33xx.dtsi > @@ -357,7 +357,7 @@ > }; > > mmc1: mmc@48060000 { > - compatible = "ti,omap4-hsmmc"; > + compatible = "ti,am33xx-hsmmc"; > ti,hwmods = "mmc1"; > ti,dual-volt; > ti,needs-special-reset; > > Guess this makes sense for all platforms based on am33xx and points > people into the right direction quicker. OK care to post a separate patch for that? Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
