From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Fri, 29 Jul 2011 16:59:38 -0700 Subject: [RFC/PATCH 0/7] decouple platform_device from omap_device In-Reply-To: <20110728045057.GA31297@manju-desktop> (Manjunath Kondaiah G.'s message of "Thu, 28 Jul 2011 10:20:57 +0530") References: <1311292338-11830-1-git-send-email-khilman@ti.com> <20110727140457.GA8558@manju-desktop> <20110728045057.GA31297@manju-desktop> Message-ID: <87hb64ocyd.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org "G, Manjunath Kondaiah" writes: > On Wed, Jul 27, 2011 at 02:45:33PM -0700, Hilman, Kevin wrote: >> Hi Manjunath, >> >> On Wed, Jul 27, 2011 at 7:04 AM, G, Manjunath Kondaiah wrote: >> > Kevin, >> > >> > On Thu, Jul 21, 2011 at 04:52:10PM -0700, Kevin Hilman wrote: >> >> Here's a first whack, proof-of-concept on how we could start to >> >> decouple the platform_device from an omap_device. >> >> >> >> The main RFC is in the last patch, and everything leading up to it are >> >> misc. omap_device cleanups that make the last patch cleaner and >> >> clearer. ?It's really the last patch that does the decoupling. >> >> >> >> This will be necessary if we're going to decouple the platform_device >> >> creation from the omap_device/omap_hwmod creation etc. ?This patch >> >> leaves them both done in omap_device_build(), but shows that they can >> >> be decoupled. >> > Can you pls mention baseline used for these patches? I tried applying on >> > latest mainline, v3.0 and ?git://git.pwsan.com/linux-2.6 prcm-devel-3.1 >> >> Oops, sorry. I forgot to mention it. >> >> Due to some misc. dependencies (mainly on Beagle board file), I've >> temporarily based it on the for-next branch of the arm-soc tree[1] >> since that has everything already queued for the next merge window. I >> also based it on the omap_device patch I posted which changes the pr_* >> prints to dev_*. >> >> For convenience, I've pushed this series to the 'wip/od-devres' branch >> of my tree[2]. >> >> Sorry for the confusion, >> >> Kevin >> >> [1] git://git.kernel.org/pub/scm/linux/kernel/git/arm/linux-arm-soc.git >> >> [2] git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git > > This series results in boot crash. However, it boots fine without this > series. I have not tried to debug but it looks like from boot log, > the res structure has invalid entries in mmc probe. As mentioned by > Grant in 7/7, the scope of devres is not lifetime hence it needs to be > fixed. This limitation was also clearly described in the changelog of patch 7/7. However, I don't think that problem should cause a problem on boot, only a from the driver core (which, suprisingly, I don't see in your bootlog.) I suspect your crash is because you're not also testing with the MMC runtime PM series that is merging for v3.1. Kevin