From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sebastian Andrzej Siewior Subject: Re: musb: dsps: make it work with two instances Date: Mon, 08 Jul 2013 10:28:12 +0200 Message-ID: <51DA781C.1020701@linutronix.de> References: <1373031178-8871-1-git-send-email-bigeasy@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ezequiel Garcia Cc: Felipe Balbi , george.cherian-l0cyMroinI0@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, b-cousson-l0cyMroinI0@public.gmane.org List-Id: linux-omap@vger.kernel.org On 07/06/2013 11:39 PM, Ezequiel Garcia wrote: > Hi Sebastian, Hello Ezequiel, > On Fri, Jul 5, 2013 at 10:32 AM, Sebastian Andrzej Siewior > wrote: >> This enables the two musb instances on am335x to work. > > I like a lot the idea of splitting the DT representation of the two > USB instances. The DT binding > looks much better this way. > > After some minor DT tweaking on the current patchset, > I've managed to detect an USB mass storage device in the > second instance (host / usb1) using a Beaglebone black board. Forgot that. Will update the .dts > However, after I unplug the device, it's not recognized > when I replug it. Maybe you can take a look at this; > i'll do some more testings and see what I can come up with. I've seen that. If I remember correctly this isn't happening if runtime-pm is enabled. However if I reload the driver it works again. I will look at this. > Also, FWIW, I think that having a separate USB phy for am35xx would be > much better. George Cherian said that he is writing a phy driver. So my phy is just temporary but I think he keep the device nodes just remove the nop-phy code. > > Nice job! Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html