Re: [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 right direction, but

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

2014-09-22 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, Thank you for

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

2014-09-22 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 15, 2014 at

Re: [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 laurent.pinch...@ideasonboard.com 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

Re: [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 the bridge

Re: [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 Ajay Kumar

Re: [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 laurent.pinch...@ideasonboard.com 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

Re: [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

Re: [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 jav...@dowhile0.org wrote: Hello Ajay, On Thu, Jul 31, 2014 at 12:58 PM, Thierry Reding thierry.red...@gmail.com wrote: On Wed, Jul 30, 2014 at 09:33:28PM +0530, Ajay kumar wrote: On Wed, Jul 30, 2014 at 8:38 PM, Thierry

Re: [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 ajayn...@gmail.com 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 Yes,

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

2014-08-22 Thread Javier Martinez Canillas
Hello Ajay, On Thu, Jul 31, 2014 at 12:58 PM, Thierry Reding thierry.red...@gmail.com 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 thierry.red...@gmail.com wrote: [...] I think it should even be possible to do this in

Re: [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 thierry.red...@gmail.com 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

Re: [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 devices

Re: [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 thierry.red...@gmail.com 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

Re: [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 thierry.red...@gmail.com 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

Re: [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 thierry.red...@gmail.com 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 thierry.red...@gmail.com wrote: On Sat, Jul 26, 2014 at 12:52:08AM +0530, Ajay Kumar wrote: This patch

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

2014-07-25 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