Re: [Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-20 Thread Dhinakaran Pandiyan
On Fri, 2018-07-20 at 10:38 -0700, Rodrigo Vivi wrote: > On Thu, Jul 19, 2018 at 06:52:00PM -0700, Dhinakaran Pandiyan wrote: > > > > On Thu, 2018-07-19 at 17:31 -0700, Rodrigo Vivi wrote: > > > > > > First of all don't try to read dpcd if PSR is not even supported. > > > > > > But also, if read

Re: [Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-20 Thread Rodrigo Vivi
On Thu, Jul 19, 2018 at 06:52:00PM -0700, Dhinakaran Pandiyan wrote: > On Thu, 2018-07-19 at 17:31 -0700, Rodrigo Vivi wrote: > > First of all don't try to read dpcd if PSR is not even supported. > > > > But also, if read failed return -EIO instead of reporting via a > > backchannel. > > > > v2:

Re: [Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-19 Thread Dhinakaran Pandiyan
On Thu, 2018-07-19 at 17:31 -0700, Rodrigo Vivi wrote: > First of all don't try to read dpcd if PSR is not even supported. > > But also, if read failed return -EIO instead of reporting via a > backchannel. > > v2: fix dev_priv: At this level m->private is the connector. (CI/DK) > don't conver

[Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-19 Thread Rodrigo Vivi
First of all don't try to read dpcd if PSR is not even supported. But also, if read failed return -EIO instead of reporting via a backchannel. v2: fix dev_priv: At this level m->private is the connector. (CI/DK) don't convert dpcd read errors to EIO. (DK) Fixes: 5b7b30864d1d ("drm/i915/psr:

[Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-19 Thread Rodrigo Vivi
First of all don't try to read dpcd if PSR is not even supported. But also, if read failed return -EIO instead of reporting via a backchannel. Fixes: 5b7b30864d1d ("drm/i915/psr: Split sink status into a separate debugfs node") Cc: Chris Wilson Cc: Dhinakaran Pandiyan Cc: José Roberto de Souza

[Intel-gfx] [PATCH] drm/i915: Fix psr sink status report.

2018-07-18 Thread Rodrigo Vivi
First of all don't try to read dpcd if PSR is not even supported. But also, if read failed return -EIO instead of reporting via a backchannel. Fixes: 5b7b30864d1d ("drm/i915/psr: Split sink status into a separate debugfs node") Cc: Chris Wilson Cc: Dhinakaran Pandiyan Cc: José Roberto de Souza