From: Tony Lindgren <tony@atomide.com> To: Alan Stern <stern@rowland.harvard.edu> Cc: Grygorii Strashko <grygorii.strashko@ti.com>, Felipe Balbi <balbi@ti.com>, gregkh@linuxfoundation.org, linux-omap@vger.kernel.org, "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>, Bin Liu <b-liu@ti.com>, Heikki Krogerus <heikki.krogerus@linux.intel.com>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Lee Jones <lee.jones@linaro.org>, linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Linux PM list <linux-pm@vger.kernel.org> Subject: Re: MUSB peripheral DMA regression caused by driver core runtime PM change Date: Fri, 23 Oct 2015 13:36:21 -0700 [thread overview] Message-ID: <20151023203620.GY3078@atomide.com> (raw) In-Reply-To: <Pine.LNX.4.44L0.1510231629430.1644-100000@iolanthe.rowland.org> * Alan Stern <stern@rowland.harvard.edu> [151023 13:34]: > On Fri, 23 Oct 2015, Tony Lindgren wrote: > > > > Thus the sequence of events should be: > > > > > > Allocate the musb device; > > > Runtime-enable the omap2430 (since it is now safe to do so); > > > Runtime-enable the musb and declare it irq_safe (this will > > > automatically runtime-resume the omap2430); > > > Register the musb. > > > > > > If things are done this way, no special action needs to be taken. > > > > Yes good point, that requires changing the init for the whole > > drivers/musb though. > > This will have to be done anyway, since the way it is now (if I > understand correctly), the musb is registered and made available to > userspace before its parent is operational (i.e., at full power). Yes I agree. > > Also, we should reorganize the whole musb and make > > the platform glue and musb core drivers completely separate using a > > shared interrupt where needed. > > > > For the regression for the -rc series? Do you see any better > > alternatives to what I posted? > > No. OK thanks, Tony
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren) To: linux-arm-kernel@lists.infradead.org Subject: MUSB peripheral DMA regression caused by driver core runtime PM change Date: Fri, 23 Oct 2015 13:36:21 -0700 [thread overview] Message-ID: <20151023203620.GY3078@atomide.com> (raw) In-Reply-To: <Pine.LNX.4.44L0.1510231629430.1644-100000@iolanthe.rowland.org> * Alan Stern <stern@rowland.harvard.edu> [151023 13:34]: > On Fri, 23 Oct 2015, Tony Lindgren wrote: > > > > Thus the sequence of events should be: > > > > > > Allocate the musb device; > > > Runtime-enable the omap2430 (since it is now safe to do so); > > > Runtime-enable the musb and declare it irq_safe (this will > > > automatically runtime-resume the omap2430); > > > Register the musb. > > > > > > If things are done this way, no special action needs to be taken. > > > > Yes good point, that requires changing the init for the whole > > drivers/musb though. > > This will have to be done anyway, since the way it is now (if I > understand correctly), the musb is registered and made available to > userspace before its parent is operational (i.e., at full power). Yes I agree. > > Also, we should reorganize the whole musb and make > > the platform glue and musb core drivers completely separate using a > > shared interrupt where needed. > > > > For the regression for the -rc series? Do you see any better > > alternatives to what I posted? > > No. OK thanks, Tony
next prev parent reply other threads:[~2015-10-23 20:36 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-10-21 23:41 MUSB peripheral DMA regression caused by driver core runtime PM change Tony Lindgren 2015-10-21 23:41 ` Tony Lindgren [not found] ` <20151021234134.GQ3078-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2015-10-22 18:02 ` Tony Lindgren 2015-10-22 18:02 ` Tony Lindgren [not found] ` <20151022180216.GT3078-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2015-10-22 23:01 ` Tony Lindgren 2015-10-22 23:01 ` Tony Lindgren [not found] ` <20151022230133.GV3078-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2015-10-23 12:50 ` Grygorii Strashko 2015-10-23 12:50 ` Grygorii Strashko [not found] ` <562A2D0D.3060104-l0cyMroinI0@public.gmane.org> 2015-10-23 16:43 ` Tony Lindgren 2015-10-23 16:43 ` Tony Lindgren [not found] ` <20151023163957.GW3078-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2015-10-23 16:48 ` Felipe Balbi 2015-10-23 16:48 ` Felipe Balbi [not found] ` <877fmdcyzi.fsf-HgARHv6XitJaoMGHk7MhZQC/G2K4zDHf@public.gmane.org> 2015-10-23 17:58 ` Grygorii Strashko 2015-10-23 17:58 ` Grygorii Strashko [not found] ` <562A7549.3070400-l0cyMroinI0@public.gmane.org> 2015-10-23 18:27 ` Alan Stern 2015-10-23 18:27 ` Alan Stern [not found] ` <Pine.LNX.4.44L0.1510231406200.1644-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org> 2015-10-23 19:20 ` Tony Lindgren 2015-10-23 19:20 ` Tony Lindgren 2015-10-23 20:33 ` Alan Stern 2015-10-23 20:33 ` Alan Stern 2015-10-23 20:36 ` Tony Lindgren [this message] 2015-10-23 20:36 ` Tony Lindgren 2015-10-28 17:14 ` Tony Lindgren 2015-10-28 17:14 ` Tony Lindgren [not found] ` <20151028171441.GE3078-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2015-10-28 17:16 ` Felipe Balbi 2015-10-28 17:16 ` Felipe Balbi
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20151023203620.GY3078@atomide.com \ --to=tony@atomide.com \ --cc=andriy.shevchenko@linux.intel.com \ --cc=b-liu@ti.com \ --cc=balbi@ti.com \ --cc=gregkh@linuxfoundation.org \ --cc=grygorii.strashko@ti.com \ --cc=heikki.krogerus@linux.intel.com \ --cc=lee.jones@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=rafael.j.wysocki@intel.com \ --cc=stern@rowland.harvard.edu \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.