From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Kemnade Subject: Re: [PATCHv2] usb: musb: Fix unbalanced platform_disable Date: Tue, 20 Sep 2016 07:04:33 +0200 Message-ID: <20160920070433.7dada289@aktux> References: <20160912153947.k4gnggur6usyujii@atomide.com> <14745757.ahNt4Zeunj@avalon> <20160918151901.k7go65s4jauldxcl@atomide.com> <20160919075941.4022b3f3@aktux> <20160919160250.itstpdk2rqw3zhzi@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160919160250.itstpdk2rqw3zhzi-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Tony Lindgren , Laurent Pinchart , Bin Liu , Greg Kroah-Hartman , Kishon Vijay Abraham I , linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-omap@vger.kernel.org On Mon, 19 Sep 2016 09:02:50 -0700 Tony Lindgren wrote: > * Andreas Kemnade [160918 23:00]: > > On Sun, 18 Sep 2016 08:19:02 -0700 > > Tony Lindgren wrote: > > > > > * Laurent Pinchart [160918 > > > 05:13]: > > > > > > > > FYI, while this patch allows me to boot my Panda board with NFS > > > > over usbnet, it only works with cold boots. A warm reboot > > > > results in the following warning, and no ethernet traffic going > > > > through. The USB device is detected by the host though. > > > > > > Yeah I noticed too that we still have issues. For example doing > > > rmmod of omap2430 with gadget configured and connected will > > > produce a hardirq-safe hardirq-unsafe lock order error. That also > > > happens with reboot with gadget configured and connected. > > > > > hmm, well, there is a musb_platform_disable() in musb_remove() > > which is simply superfluous... > > Some days ago we had a locking problem with musb_start() and moved > > it out of the locked area. Maybe we could do also something similar > > here. > > Well I don't think we can do that safely at this point because we > have unpaired calls to musb_start() and musb_stop(). So trying to > make musb_platform_enable/disable() paired right now will just lead > into mystery breakage in various use cases. > I am primarily talking about doing things like the patch usb: musb: Fix locking errors for host only mode 2c5575401e34de3d2f did for musb_start(), for musb_stop() also. Regards, Andreas -- 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