[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 4:35 PM, Laurent Pinchart wrote: > Hi Archit, > > On Wednesday 30 Nov 2016 16:30:53 Archit Taneja wrote: >> On 11/30/2016 03:53 PM, Laurent Pinchart wrote: >>> On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: On 11/29/2016 11:27 PM, Laurent Pinchart wrote: > On

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 03:53 PM, Laurent Pinchart wrote: > Hi Archit, > > On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: >> On 11/29/2016 11:27 PM, Laurent Pinchart wrote: >>> On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > Instead

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Vincent ABRIOU
On 11/29/2016 10:04 AM, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the encoder and

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Laurent Pinchart
Hi Archit, On Wednesday 30 Nov 2016 16:30:53 Archit Taneja wrote: > On 11/30/2016 03:53 PM, Laurent Pinchart wrote: > > On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: > >> On 11/29/2016 11:27 PM, Laurent Pinchart wrote: > >>> On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: > On

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Laurent Pinchart
Hi Archit, On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: > On 11/29/2016 11:27 PM, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: > >> On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > >>> Instead of linking encoders and bridges in every driver (and

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/29/2016 11:27 PM, Laurent Pinchart wrote: > Hi Archit, > > On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: >> On 11/29/2016 02:34 PM, Laurent Pinchart wrote: >>> Instead of linking encoders and bridges in every driver (and getting it >>> wrong half of the time, as many drivers forget

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Boris Brezillon
On Tue, 29 Nov 2016 11:04:33 +0200 Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Laurent Pinchart
Hi Daniel, On Tuesday 29 Nov 2016 20:02:01 Laurent Pinchart wrote: > On Tuesday 29 Nov 2016 11:05:27 Daniel Vetter wrote: > > On Tue, Nov 29, 2016 at 11:43:19AM +0200, Laurent Pinchart wrote: > >> On Tuesday 29 Nov 2016 10:35:24 Daniel Vetter wrote: > >>> On Tue, Nov 29, 2016 at 11:04:33AM +0200,

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Laurent Pinchart
Hi Daniel, On Tuesday 29 Nov 2016 11:05:27 Daniel Vetter wrote: > On Tue, Nov 29, 2016 at 11:43:19AM +0200, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 10:35:24 Daniel Vetter wrote: > >> On Tue, Nov 29, 2016 at 11:04:33AM +0200, Laurent Pinchart wrote: > >>> Instead of linking encoders and

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Laurent Pinchart
Hi Archit, On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: > On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > > Instead of linking encoders and bridges in every driver (and getting it > > wrong half of the time, as many drivers forget to set the drm_bridge > > encoder pointer), do so in

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Archit Taneja
On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the encoder and

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Laurent Pinchart
Hi Daniel, On Tuesday 29 Nov 2016 10:35:24 Daniel Vetter wrote: > On Tue, Nov 29, 2016 at 11:04:33AM +0200, Laurent Pinchart wrote: > > Instead of linking encoders and bridges in every driver (and getting it > > wrong half of the time, as many drivers forget to set the drm_bridge > > encoder

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Daniel Vetter
On Tue, Nov 29, 2016 at 11:43:19AM +0200, Laurent Pinchart wrote: > Hi Daniel, > > On Tuesday 29 Nov 2016 10:35:24 Daniel Vetter wrote: > > On Tue, Nov 29, 2016 at 11:04:33AM +0200, Laurent Pinchart wrote: > > > Instead of linking encoders and bridges in every driver (and getting it > > > wrong

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Laurent Pinchart
Instead of linking encoders and bridges in every driver (and getting it wrong half of the time, as many drivers forget to set the drm_bridge encoder pointer), do so in core code. The drm_bridge_attach() function needs the encoder and optional previous bridge to perform that task, update all the

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Daniel Vetter
On Tue, Nov 29, 2016 at 11:04:33AM +0200, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Stefan Agner
On 2016-11-29 01:04, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the encoder and optional