[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Ville Syrjälä
On Wed, Oct 26, 2016 at 02:11:00PM +0100, Chris Wilson wrote: > On Wed, Oct 26, 2016 at 02:17:16PM +0300, Ville Syrjälä wrote: > > On Wed, Oct 26, 2016 at 12:51:41PM +0300, Jani Nikula wrote: > > > On Wed, 26 Oct 2016, Chris Wilson wrote: > > > > On Wed, Oct 26, 2016 at 07:52:26AM +0200, Daniel

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Chris Wilson
On Wed, Oct 26, 2016 at 04:15:39PM +0300, Ville Syrjälä wrote: > On Wed, Oct 26, 2016 at 02:11:00PM +0100, Chris Wilson wrote: > > On Wed, Oct 26, 2016 at 02:17:16PM +0300, Ville Syrjälä wrote: > > > On Wed, Oct 26, 2016 at 12:51:41PM +0300, Jani Nikula wrote: > > > > On Wed, 26 Oct 2016,

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Ville Syrjälä
On Wed, Oct 26, 2016 at 12:51:41PM +0300, Jani Nikula wrote: > On Wed, 26 Oct 2016, Chris Wilson wrote: > > On Wed, Oct 26, 2016 at 07:52:26AM +0200, Daniel Vetter wrote: > >> I'd go further and just always create this as one of the standard > >> properties (and always attach it to the connector,

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Chris Wilson
On Wed, Oct 26, 2016 at 02:17:16PM +0300, Ville Syrjälä wrote: > On Wed, Oct 26, 2016 at 12:51:41PM +0300, Jani Nikula wrote: > > On Wed, 26 Oct 2016, Chris Wilson wrote: > > > On Wed, Oct 26, 2016 at 07:52:26AM +0200, Daniel Vetter wrote: > > >> I'd go further and just always create this as

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Jani Nikula
On Wed, 26 Oct 2016, Chris Wilson wrote: > On Wed, Oct 26, 2016 at 07:52:26AM +0200, Daniel Vetter wrote: >> I'd go further and just always create this as one of the standard >> properties (and always attach it to the connector, like edid), and only >> expose helpers to set the link status to

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Chris Wilson
On Wed, Oct 26, 2016 at 07:52:26AM +0200, Daniel Vetter wrote: > I'd go further and just always create this as one of the standard > properties (and always attach it to the connector, like edid), and only > expose helpers to set the link status to good or bad. One of the sketches for this idea

[Intel-gfx] [PATCH 1/2] drm: Add a new connector property for link status

2016-10-26 Thread Daniel Vetter
On Tue, Oct 25, 2016 at 06:16:34PM -0700, Manasi Navare wrote: > A new optional connector property is added for keeping > track of whether the link is good (link training passed) or > link is bad (link training failed). If the link status property > is Bad, then userspace should fire off a new