[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-21 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 03:33:14PM +0200, Daniel Vetter wrote: > On Fri, Apr 20, 2012 at 03:10:05PM +0200, Sascha Hauer wrote: > > (BTW each driver in drm has this layer somewhere in it. If I had hidden > > it in imx specific functions I probably wouldn't have raised any > > questions, but I don't

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-21 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 03:33:14PM +0200, Daniel Vetter wrote: > On Fri, Apr 20, 2012 at 03:10:05PM +0200, Sascha Hauer wrote: > > (BTW each driver in drm has this layer somewhere in it. If I had hidden > > it in imx specific functions I probably wouldn't have raised any > > questions, but I don't

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 03:25:58PM +0100, Mark Brown wrote: > On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > > * Dave Airlie wrote: > > > I get the feeling the drm can just be a virtual platform device of > > > some sort, then it reads the device tree and binds all the informatio

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Mark Brown wrote: > On Fri, Apr 20, 2012 at 04:49:43PM +0200, Thierry Reding wrote: > > * Mark Brown wrote: > > > > This sounds an awful lot like how ASoC hangs together... > > > What in particular sounds awful? > > Nothing - "an awful" is an English idiom for "very". I know =) But it has a s

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Mark Brown wrote: > On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > > * Dave Airlie wrote: > > > I get the feeling the drm can just be a virtual platform device of > > > some sort, then it reads the device tree and binds all the information > > > on what crtc/encoders are availa

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 05:15:18PM +0200, Sascha Hauer wrote: > On Fri, Apr 20, 2012 at 03:25:58PM +0100, Mark Brown wrote: > > This sounds an awful lot like how ASoC hangs together... > Very much, yes. In ASoC and DRM we both have several physical devices spread > around the SoC which form a log

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 04:49:43PM +0200, Thierry Reding wrote: > * Mark Brown wrote: > > This sounds an awful lot like how ASoC hangs together... > What in particular sounds awful? Nothing - "an awful" is an English idiom for "very". -- next part -- A non-text attachment

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Daniel Vetter
On Fri, Apr 20, 2012 at 03:10:05PM +0200, Sascha Hauer wrote: > (BTW each driver in drm has this layer somewhere in it. If I had hidden > it in imx specific functions I probably wouldn't have raised any > questions, but I don't want to go that way) That's _exactly_ what you should be doing. And on

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > * Dave Airlie wrote: > > I get the feeling the drm can just be a virtual platform device of > > some sort, then it reads the device tree and binds all the information > > on what crtc/encoders are available, > That's pretty much wha

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > * Dave Airlie wrote: > > I get the feeling the drm can just be a virtual platform device of > > some sort, then it reads the device tree and binds all the information > > on what crtc/encoders are available, > > That's pretty much w

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
[Added some embedded graphics maintainers to Cc who might be interested in this] On Fri, Apr 20, 2012 at 11:02:02AM +0100, Dave Airlie wrote: > On Wed, Apr 11, 2012 at 4:33 PM, Sascha Hauer > wrote: > > This patch adds support for creating simple drm devices. The > > basic idea of this patch is

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Dave Airlie wrote: > I get the feeling the drm can just be a virtual platform device of > some sort, then it reads the device tree and binds all the information > on what crtc/encoders are available, That's pretty much what I've come up with in the second round of Tegra DRM patches. Basically di

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Dave Airlie
On Wed, Apr 11, 2012 at 4:33 PM, Sascha Hauer wrote: > This patch adds support for creating simple drm devices. The > basic idea of this patch is that drm drivers using the sdrm layer > no longer have to implement a full drm device but instead only > register crtcs, encoders and connectors with th

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 04:49:43PM +0200, Thierry Reding wrote: > * Mark Brown wrote: > > This sounds an awful lot like how ASoC hangs together... > What in particular sounds awful? Nothing - "an awful" is an English idiom for "very". signature.asc Description: Digital signature __

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 05:15:18PM +0200, Sascha Hauer wrote: > On Fri, Apr 20, 2012 at 03:25:58PM +0100, Mark Brown wrote: > > This sounds an awful lot like how ASoC hangs together... > Very much, yes. In ASoC and DRM we both have several physical devices spread > around the SoC which form a log

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Mark Brown
On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > * Dave Airlie wrote: > > I get the feeling the drm can just be a virtual platform device of > > some sort, then it reads the device tree and binds all the information > > on what crtc/encoders are available, > That's pretty much wha

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 03:25:58PM +0100, Mark Brown wrote: > On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > > * Dave Airlie wrote: > > > I get the feeling the drm can just be a virtual platform device of > > > some sort, then it reads the device tree and binds all the informatio

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Mark Brown wrote: > On Fri, Apr 20, 2012 at 04:49:43PM +0200, Thierry Reding wrote: > > * Mark Brown wrote: > > > > This sounds an awful lot like how ASoC hangs together... > > > What in particular sounds awful? > > Nothing - "an awful" is an English idiom for "very". I know =) But it has a s

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Mark Brown wrote: > On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > > * Dave Airlie wrote: > > > I get the feeling the drm can just be a virtual platform device of > > > some sort, then it reads the device tree and binds all the information > > > on what crtc/encoders are availa

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Daniel Vetter
On Fri, Apr 20, 2012 at 03:10:05PM +0200, Sascha Hauer wrote: > (BTW each driver in drm has this layer somewhere in it. If I had hidden > it in imx specific functions I probably wouldn't have raised any > questions, but I don't want to go that way) That's _exactly_ what you should be doing. And on

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
On Fri, Apr 20, 2012 at 02:38:43PM +0200, Thierry Reding wrote: > * Dave Airlie wrote: > > I get the feeling the drm can just be a virtual platform device of > > some sort, then it reads the device tree and binds all the information > > on what crtc/encoders are available, > > That's pretty much w

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Sascha Hauer
[Added some embedded graphics maintainers to Cc who might be interested in this] On Fri, Apr 20, 2012 at 11:02:02AM +0100, Dave Airlie wrote: > On Wed, Apr 11, 2012 at 4:33 PM, Sascha Hauer wrote: > > This patch adds support for creating simple drm devices. The > > basic idea of this patch is tha

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Thierry Reding
* Dave Airlie wrote: > I get the feeling the drm can just be a virtual platform device of > some sort, then it reads the device tree and binds all the information > on what crtc/encoders are available, That's pretty much what I've come up with in the second round of Tegra DRM patches. Basically di

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-20 Thread Dave Airlie
On Wed, Apr 11, 2012 at 4:33 PM, Sascha Hauer wrote: > This patch adds support for creating simple drm devices. The > basic idea of this patch is that drm drivers using the sdrm layer > no longer have to implement a full drm device but instead only > register crtcs, encoders and connectors with th

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-12 Thread Sascha Hauer
On Wed, Apr 11, 2012 at 09:22:47PM +0100, Alan Cox wrote: > > +static int sdrm_suspend(struct drm_device *drm, pm_message_t state) > > +{ > > + /* TODO */ > > + > > + return 0; > > +} > > + > > +static int sdrm_resume(struct drm_device *drm) > > +{ > > + /* TODO */ > > + > > + return 0; > >

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-12 Thread Sascha Hauer
On Wed, Apr 11, 2012 at 09:22:47PM +0100, Alan Cox wrote: > > +static int sdrm_suspend(struct drm_device *drm, pm_message_t state) > > +{ > > + /* TODO */ > > + > > + return 0; > > +} > > + > > +static int sdrm_resume(struct drm_device *drm) > > +{ > > + /* TODO */ > > + > > + return 0; > >

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-11 Thread Alan Cox
> +static int sdrm_suspend(struct drm_device *drm, pm_message_t state) > +{ > + /* TODO */ > + > + return 0; > +} > + > +static int sdrm_resume(struct drm_device *drm) > +{ > + /* TODO */ > + > + return 0; > +} These probably need to call into the sdrm device specific handling. >

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-11 Thread Sascha Hauer
This patch adds support for creating simple drm devices. The basic idea of this patch is that drm drivers using the sdrm layer no longer have to implement a full drm device but instead only register crtcs, encoders and connectors with the sdrm layer. The sdrm layer then registers a drm device with

Re: [PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-11 Thread Alan Cox
> +static int sdrm_suspend(struct drm_device *drm, pm_message_t state) > +{ > + /* TODO */ > + > + return 0; > +} > + > +static int sdrm_resume(struct drm_device *drm) > +{ > + /* TODO */ > + > + return 0; > +} These probably need to call into the sdrm device specific handling. >

[PATCH 3/7] DRM: add sdrm layer for general embedded system support

2012-04-11 Thread Sascha Hauer
This patch adds support for creating simple drm devices. The basic idea of this patch is that drm drivers using the sdrm layer no longer have to implement a full drm device but instead only register crtcs, encoders and connectors with the sdrm layer. The sdrm layer then registers a drm device with