[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-23 Thread Thierry Reding
On Tue, Sep 23, 2014 at 03:29:13AM +0300, Laurent Pinchart wrote: > Hi Thierry, > > On Monday 22 September 2014 09:40:38 Thierry Reding wrote: > > On Wed, Sep 17, 2014 at 12:27:13PM +0300, Laurent Pinchart wrote: > > > On Wednesday 17 September 2014 14:37:30 Ajay kumar wrote: > > > > On Mon, Sep

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-23 Thread Laurent Pinchart
Hi Thierry, On Monday 22 September 2014 09:40:38 Thierry Reding wrote: > On Wed, Sep 17, 2014 at 12:27:13PM +0300, Laurent Pinchart wrote: > > On Wednesday 17 September 2014 14:37:30 Ajay kumar wrote: > > > On Mon, Sep 15, 2014 at 11:07 PM, Laurent Pinchart wrote: > > > > Hi Ajay, > > > > > > >

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-22 Thread Thierry Reding
On Wed, Sep 17, 2014 at 12:27:13PM +0300, Laurent Pinchart wrote: > Hi Ajay, > > On Wednesday 17 September 2014 14:37:30 Ajay kumar wrote: > > On Mon, Sep 15, 2014 at 11:07 PM, Laurent Pinchart wrote: > > > Hi Ajay, > > > > > > Thank you for the patch. > > > > > > I think we're moving in the

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-17 Thread Dave Airlie
>> >> Before the introduction of the component framework I would have said this is >> the way to go. Now, I think bridges should register themselves as components, >> and the DRM master driver should use the component framework to get a >> reference to the bridges it needs. > Well, I have modified

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-17 Thread Ajay kumar
On Wed, Sep 17, 2014 at 2:57 PM, Laurent Pinchart wrote: > Hi Ajay, > > On Wednesday 17 September 2014 14:37:30 Ajay kumar wrote: >> On Mon, Sep 15, 2014 at 11:07 PM, Laurent Pinchart wrote: >> > Hi Ajay, >> > >> > Thank you for the patch. >> > >> > I think we're moving in the right direction,

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-17 Thread Ajay kumar
Hi Laurent, On Mon, Sep 15, 2014 at 11:07 PM, Laurent Pinchart wrote: > Hi Ajay, > > Thank you for the patch. > > I think we're moving in the right direction, but we're not there yet. > > On Saturday 26 July 2014 00:52:08 Ajay Kumar wrote: >> This patch tries to seperate drm_bridge

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-17 Thread Laurent Pinchart
Hi Ajay, On Wednesday 17 September 2014 14:37:30 Ajay kumar wrote: > On Mon, Sep 15, 2014 at 11:07 PM, Laurent Pinchart wrote: > > Hi Ajay, > > > > Thank you for the patch. > > > > I think we're moving in the right direction, but we're not there yet. > > > > On Saturday 26 July 2014 00:52:08

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-09-15 Thread Laurent Pinchart
Hi Ajay, Thank you for the patch. I think we're moving in the right direction, but we're not there yet. On Saturday 26 July 2014 00:52:08 Ajay Kumar wrote: > This patch tries to seperate drm_bridge implementation > into 2 parts, a drm part and a non_drm part. > > A set of helper functions are

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-08-25 Thread Javier Martinez Canillas
Hello Ajay, On Mon, Aug 25, 2014 at 8:11 AM, Ajay kumar wrote: >> >> Do you plan to address Thierry's concerns and re-spin this patch? >> >> Same question for patches: >> >> "drm/bridge: Add i2c based driver for ptn3460 bridge" >> "drm/bridge: Add i2c based driver for ps8622/ps8625 bridge" >

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-08-25 Thread Ajay kumar
Hi Javier, On Sat, Aug 23, 2014 at 5:03 AM, Javier Martinez Canillas wrote: > Hello Ajay, > > On Thu, Jul 31, 2014 at 12:58 PM, Thierry Reding > wrote: >> On Wed, Jul 30, 2014 at 09:33:28PM +0530, Ajay kumar wrote: >>> On Wed, Jul 30, 2014 at 8:38 PM, Thierry Reding >> gmail.com> wrote: >>

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-08-23 Thread Javier Martinez Canillas
Hello Ajay, On Thu, Jul 31, 2014 at 12:58 PM, Thierry Reding wrote: > On Wed, Jul 30, 2014 at 09:33:28PM +0530, Ajay kumar wrote: >> On Wed, Jul 30, 2014 at 8:38 PM, Thierry Reding > gmail.com> wrote: > [...] >> > I think it should even be possible to do this in more separate steps. >> > For

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-31 Thread Thierry Reding
On Wed, Jul 30, 2014 at 09:33:28PM +0530, Ajay kumar wrote: > On Wed, Jul 30, 2014 at 8:38 PM, Thierry Reding > wrote: [...] > > I think it should even be possible to do this in more separate steps. > > For example you could add the new bridge infrastructure without touching > > any of the

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-30 Thread Ajay kumar
On Wed, Jul 30, 2014 at 8:38 PM, Thierry Reding wrote: > On Wed, Jul 30, 2014 at 08:01:44PM +0530, Ajay kumar wrote: >> On Wed, Jul 30, 2014 at 4:49 PM, Thierry Reding >> wrote: >> > On Sat, Jul 26, 2014 at 12:52:08AM +0530, Ajay Kumar wrote: >> >> This patch tries to seperate drm_bridge

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-30 Thread Ajay kumar
On Wed, Jul 30, 2014 at 4:49 PM, Thierry Reding wrote: > On Sat, Jul 26, 2014 at 12:52:08AM +0530, Ajay Kumar wrote: >> This patch tries to seperate drm_bridge implementation >> into 2 parts, a drm part and a non_drm part. >> >> A set of helper functions are defined in this patch to make >>

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-30 Thread Thierry Reding
On Wed, Jul 30, 2014 at 08:01:44PM +0530, Ajay kumar wrote: > On Wed, Jul 30, 2014 at 4:49 PM, Thierry Reding > wrote: > > On Sat, Jul 26, 2014 at 12:52:08AM +0530, Ajay Kumar wrote: > >> This patch tries to seperate drm_bridge implementation > >> into 2 parts, a drm part and a non_drm part. > >>

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-30 Thread Thierry Reding
On Sat, Jul 26, 2014 at 12:52:08AM +0530, Ajay Kumar wrote: > This patch tries to seperate drm_bridge implementation > into 2 parts, a drm part and a non_drm part. > > A set of helper functions are defined in this patch to make > bridge driver probe independent of the drm flow. > > The bridge

[PATCH V6 6/8] drm/bridge: Modify drm_bridge core to support driver model

2014-07-26 Thread Ajay Kumar
This patch tries to seperate drm_bridge implementation into 2 parts, a drm part and a non_drm part. A set of helper functions are defined in this patch to make bridge driver probe independent of the drm flow. The bridge devices register themselves on a lookup table when they get probed by