Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-03-01 Thread Daniel Vetter
On Thu, Mar 02, 2017 at 02:30:09AM +0200, Laurent Pinchart wrote: > Hi Daniel, > > On Wednesday 04 Jan 2017 17:13:23 Laurent Pinchart wrote: > > On Wednesday 04 Jan 2017 15:58:25 Daniel Vetter wrote: > > > On Wed, Jan 04, 2017 at 04:33:57PM +0200, Laurent Pinchart wrote: > > >> On Wednesday 04 Jan

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-03-01 Thread Laurent Pinchart
Hi Daniel, On Wednesday 04 Jan 2017 17:13:23 Laurent Pinchart wrote: > On Wednesday 04 Jan 2017 15:58:25 Daniel Vetter wrote: > > On Wed, Jan 04, 2017 at 04:33:57PM +0200, Laurent Pinchart wrote: > >> On Wednesday 04 Jan 2017 14:51:48 Daniel Vetter wrote: > >>> Hm, something like drm_bridge_panel_

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Laurent Pinchart
Hi Daniel, On Wednesday 04 Jan 2017 15:58:25 Daniel Vetter wrote: > On Wed, Jan 04, 2017 at 04:33:57PM +0200, Laurent Pinchart wrote: > > On Wednesday 04 Jan 2017 14:51:48 Daniel Vetter wrote: > >> Hm, something like drm_bridge_panel_bridge_init(dev, panel) should be > >> enough, or not? My idea i

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Daniel Vetter
On Wed, Jan 04, 2017 at 04:33:57PM +0200, Laurent Pinchart wrote: > On Wednesday 04 Jan 2017 14:51:48 Daniel Vetter wrote: > > Hm, something like drm_bridge_panel_bridge_init(dev, panel) should be > > enough, or not? My idea is to use this for the case where the only > > thing in dt is the panel, w

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Laurent Pinchart
Hi Daniel, On Wednesday 04 Jan 2017 14:51:48 Daniel Vetter wrote: > On Wed, Jan 4, 2017 at 2:08 PM, Laurent Pinchart wrote: > > On Wednesday 04 Jan 2017 09:18:18 Daniel Vetter wrote: > >> On Tue, Nov 29, 2016 at 10:57:07PM +0200, Laurent Pinchart wrote: > >>> On Tuesday 29 Nov 2016 10:54:09 Daniel

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Daniel Vetter
On Wed, Jan 4, 2017 at 2:08 PM, Laurent Pinchart wrote: > Hi Daniel, > > On Wednesday 04 Jan 2017 09:18:18 Daniel Vetter wrote: >> On Tue, Nov 29, 2016 at 10:57:07PM +0200, Laurent Pinchart wrote: >> > On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: >> >> On Tue, Nov 29, 2016 at 11:04:36AM +0

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Laurent Pinchart
Hi Daniel, On Wednesday 04 Jan 2017 09:18:18 Daniel Vetter wrote: > On Tue, Nov 29, 2016 at 10:57:07PM +0200, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: > >> On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > >>> The LVDS encoder driver is a D

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-04 Thread Daniel Vetter
On Tue, Nov 29, 2016 at 10:57:07PM +0200, Laurent Pinchart wrote: > Hi Daniel, > > On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: > > On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > > > The LVDS encoder driver is a DRM bridge driver that supports the > > > parallel to LVD

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2017-01-03 Thread Laurent Pinchart
Hi Daniel, On Tuesday 29 Nov 2016 22:57:07 Laurent Pinchart wrote: > On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: > > On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > >> The LVDS encoder driver is a DRM bridge driver that supports the > >> parallel to LVDS encoders that

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2016-11-29 Thread Laurent Pinchart
Hi Daniel, On Tuesday 29 Nov 2016 10:54:09 Daniel Vetter wrote: > On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > > The LVDS encoder driver is a DRM bridge driver that supports the > > parallel to LVDS encoders that don't require any configuration. The > > driver thus doesn't i

Re: [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2016-11-29 Thread Daniel Vetter
On Tue, Nov 29, 2016 at 11:04:36AM +0200, Laurent Pinchart wrote: > The LVDS encoder driver is a DRM bridge driver that supports the > parallel to LVDS encoders that don't require any configuration. The > driver thus doesn't interact with the device, but creates an LVDS > connector for the panel an

[PATCH v3 06/13] drm: bridge: Add LVDS encoder driver

2016-11-29 Thread Laurent Pinchart
The LVDS encoder driver is a DRM bridge driver that supports the parallel to LVDS encoders that don't require any configuration. The driver thus doesn't interact with the device, but creates an LVDS connector for the panel and exposes its size and timing based on information retrieved from DT. Sig