On Fri, Jan 10, 2014 at 02:54:44PM +0000, Russell King - ARM Linux wrote:
> On Thu, Jan 02, 2014 at 07:10:55PM -0800, Greg Kroah-Hartman wrote:
> > On Thu, Jan 02, 2014 at 09:27:58PM +0000, Russell King wrote:
> > > Subsystems such as ALSA, DRM and others require a single card-level
> > > device structure to represent a subsystem.  However, firmware tends to
> > > describe the individual devices and the connections between them.
> > > 
> > > Therefore, we need a way to gather up the individual component devices
> > > together, and indicate when we have all the component devices.
> > > 
> > > We do this in DT by providing a "superdevice" node which specifies
> > > the components, eg:
> > > 
> > >   imx-drm {
> > >           compatible = "fsl,drm";
> > >           crtcs = <&ipu1>;
> > >           connectors = <&hdmi>;
> > >   };
> > > 
> > > The superdevice is declared into the component support, along with the
> > > subcomponents.  The superdevice receives callbacks to locate the
> > > subcomponents, and identify when all components are present.  At this
> > > point, we bind the superdevice, which causes the appropriate subsystem
> > > to be initialised in the conventional way.
> > > 
> > > When any of the components or superdevice are removed from the system,
> > > we unbind the superdevice, thereby taking the subsystem down.
> > 
> > This sounds a lot like the "containers" code that Rafael just submitted
> > and I acked for 3.14.  Look at the lkml post:
> >     Subject: [PATCH 2/2] ACPI / hotplug / driver core: Handle containers in 
> > a special way
> >     Message-ID: <1991202.gilw172...@vostro.rjw.lan>
> > 
> > And see if that could possibly be used instead?
> 
> Greg,
> 
> Not sure if you saw the outcome to your comment above.  My conclusion
> was:
> 
> "Yes, I'm coming to that conclusion as well.  It looks like your "containers"
> aren't about collecting up several individual component devices into one
> super-device and probing the appropriate subsystem when all components are
> known.
> 
> "Confused why Greg is pointing me at your patches."

Ah, sorry, I missed that in my "catch up on 2 weeks of email" flood.

> Does this mean you're happy with the patch?

Well, I will not object to it on the grounds of it being a duplicate of
Rafael's work now :)

I'll be glad to consider it on its own, when you feel the series is
ready to be submitted.

thanks,

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to