Thread (1 message) 1 message, 1 author, 2011-07-29

[RFC/PATCH 0/7] decouple platform_device from omap_device

From: Kevin Hilman <hidden>
Date: 2011-07-29 23:59:38
Also in: linux-devicetree, linux-omap

"G, Manjunath Kondaiah" [off-list ref] writes:
On Wed, Jul 27, 2011 at 02:45:33PM -0700, Hilman, Kevin wrote:
quoted
Hi Manjunath,

On Wed, Jul 27, 2011 at 7:04 AM, G, Manjunath Kondaiah [off-list ref] wrote:
quoted
Kevin,

On Thu, Jul 21, 2011 at 04:52:10PM -0700, Kevin Hilman wrote:
quoted
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help