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

G, Manjunath Kondaiah manjugk at ti.com
Thu Jul 28 00:50:57 EDT 2011


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 <manjugk at ti.com> 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.

The bootlog @ http://paste.ubuntu.com/653527/

-M




More information about the linux-arm-kernel mailing list