[PATCH v7 1/5] drm: Read DP branch device HW revision

2016-08-11 Thread Jim Bride
On Thu, Aug 11, 2016 at 11:22:15AM +0300, Ville Syrjälä wrote: > On Thu, Aug 11, 2016 at 11:14:43AM +0300, Mika Kahola wrote: > > On Thu, 2016-08-11 at 10:10 +0300, Ville Syrjälä wrote: > > > On Mon, Aug 08, 2016 at 04:00:26PM +0300, Mika Kahola wrote: > > > > HW revision is mandatory field for

[Intel-gfx] [PATCH v2 1/4] drm: add picture aspect ratio flags

2016-10-13 Thread Jim Bride
atio, and help in setting right VIC field in avi > infoframes. > > Signed-off-by: Shashank Sharma The changes make sense according to the relevant specs, and although I don't have the HW to test them the code looks right. Reviewed-by: Jim Bride > > V2: Addressed revie

[Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer

2016-10-13 Thread Jim Bride
On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > - 64:27 > - 256:135 > > This patch: > - Adds new DRM flags for to represent these new aspect ratios. > - Adds new cases to handle these aspect ratios while converting > from

[Intel-gfx] [PATCH v2 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-13 Thread Jim Bride
Sean: > - Fix spellings/typo > - No need to handle aspect ratio none > - Add a break, for default case too > > Signed-off-by: Shashank Sharma > Signed-off-by: Lin, Jia > Signed-off-by: Akashdeep Sharma Reviewed-by: Jim Bride > > Cc: Daniel

[Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer

2016-10-18 Thread Jim Bride
On Mon, Oct 17, 2016 at 08:02:49AM +0200, Daniel Vetter wrote: > On Thu, Oct 13, 2016 at 10:28:14AM -0700, Jim Bride wrote: > > On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > > > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > > &

[PATCH v8 06/12] drm/i915: Cleanup DisplayPort AUX channel initialization

2016-09-07 Thread Jim Bride
On Wed, Aug 17, 2016 at 01:49:42PM +0300, Mika Kahola wrote: > Let's remove reference to "struct intel_connector *connector" > in intel_dp_aux_init() function as it is no longer required. > > Signed-off-by: Mika Kahola Reviewed-by: Jim Bride > --- > dr

[PATCH v8 07/12] drm/i915: Read DP branch device HW revision

2016-09-07 Thread Jim Bride
move dependency to drm_dp_helper but instead parse > DPCD and print HW revision info to dmesg (Ville) > > Signed-off-by: Mika Kahola Reviewed-by: Jim Bride > --- > drivers/gpu/drm/i915/intel_dp.c | 21 + > include/drm/drm_dp_helper.h | 1 + > 2 fi

[PATCH v8 08/12] drm/i915: Read DP branch device SW revision

2016-09-07 Thread Jim Bride
ision info to dmesg (Ville) > > Signed-off-by: Mika Kahola With the commit message change requested above, this is: Reviewed-by: Jim Bride > --- > drivers/gpu/drm/i915/intel_dp.c | 20 > include/drm/drm_dp_helper.h | 1 + > 2 files changed, 21 insertio

[PATCH v8 09/12] Check pixel rate for DP to VGA dongle

2016-09-07 Thread Jim Bride
pixel clock (Ville) > v5: Use of intel_dp->downstream_ports to read out port capabilities. > Code restructuring (Ville) > v6: Move DP branch device check to drm_dp_helper.c (Daniel) > v7: Cleanup as suggested by Ville > > Signed-off-by: Mika Kahola Reviewed-by: Jim

[PATCH v8 10/12] drm/i915: Update bits per component for display info

2016-09-07 Thread Jim Bride
On Wed, Aug 17, 2016 at 01:49:47PM +0300, Mika Kahola wrote: > DisplayPort branch device may define max supported bits per > component. Update display info based on this value if bpc > is defined. > > v2: cleanup to match the drm_dp_helper.c patches introduced > earlier in this series > v3: Fi

[PATCH v8 11/12] drm: Add DP branch device info on debugfs

2016-09-07 Thread Jim Bride
On Wed, Aug 17, 2016 at 01:49:48PM +0300, Mika Kahola wrote: > Read DisplayPort branch device info from through debugfs > interface. > > v2: use drm_dp_helper routines to collect data > v3: cleanup to match the drm_dp_helper.c patches introduced > earlier in this series > v4: move DP branch de

[PATCH v8 12/12] drm/i915: Check TMDS clock DP to HDMI dongle

2016-09-07 Thread Jim Bride
On Wed, Aug 17, 2016 at 01:49:49PM +0300, Mika Kahola wrote: > Respect max TMDS clock frequency from DPCD for active > DP to HDMI adapters. > > Signed-off-by: Mika Kahola Reviewed-by: Jim Bride > --- > drivers/gpu/drm/i915/intel_drv.h | 3 +++ > drivers/gpu/drm/i

[PATCH v9 08/12] drm/i915: Read DP branch device SW revision

2016-09-09 Thread Jim Bride
niel) > v3: remove dependency to drm_dp_helper but instead parse > DPCD and print SW revision info to dmesg (Ville) > v4: commit message fix (Jim Bride) > > Signed-off-by: Mika Kahola Reviewed-by: Jim Bride > --- > drivers/gpu/drm/i915/intel_dp.c | 20

[PATCH v9 10/12] drm/i915: Update bits per component for display info

2016-09-09 Thread Jim Bride
n this series > v3: Fill bpc for connector's display info in separate > drm_dp_helper function (Daniel) > v4: remove updating bpc for display info as it may be overridden > when parsing EDID. Instead, check bpc for DP branch device > during compute_config > v5: Inde

[PATCH v9 11/12] drm: Add DP branch device info on debugfs

2016-09-09 Thread Jim Bride
t; v4: move DP branch device info to function 'intel_dp_branch_device_info()' > v5: initial step to move debugging info from intel_dp. to drm_dp_helper.c > (Daniel) > v6: read hw and sw revision without using specific drm_dp_helper routines > v7: indentation fixes (Jim Bride) &g

[PATCH 02/10] drm/i915/psr: program vsc header for psr2

2017-01-04 Thread Jim Bride
_SU_Y_COORDINATE_REQUIRED > - Change DPRX_FEATURE_ENUMERATION_LIST to DP_DPRX_* > - Change VSC_SDP_EXT_FOR_COLORIMETRY_SUPPORTED to DP_VSC_* > > Cc: Rodrigo Vivi > Cc: Jim Bride > Signed-off-by: Vathsala Nagaraju > Signed-off-by: Patil Deepti Reviewed-by: Jim Bride > --- > d

[PATCH 05/10] drm/i915/psr: enable ALPM for psr2

2017-01-05 Thread Jim Bride
On Mon, Jan 02, 2017 at 05:00:58PM +0530, vathsala nagaraju wrote: > As per edp1.4 spec , alpm is required for psr2 operation as it's > used for all psr2 main link power down management and alpm enable > bit must be set for psr2 operation. > > Cc: Rodrigo Vivi > Cc: Jim

[PATCH 03/10] drm/i915/psr: fix blank screen issue for psr2

2017-01-05 Thread Jim Bride
, > as suggested by rodrigo and jim. > > v2: (Vivi Rodrigo) > - Rename hsw_enable_source_psr* to intel_enable_source_psr* > > Cc: Rodrigo Vivi > Cc: Jim Bride > Signed-off-by: Vathsala Nagaraju > Signed-off-by: Patil Deepti The new naming is much better! Reviewed-by:

[PATCH 07/10] drm/i915/psr: set PSR_MASK bits for deep sleep

2017-01-05 Thread Jim Bride
set 1 , if not set, > flickering is observed on psr2 panel. > > v2: (Ilia Mirkin) > - Remove duplicate bit definition 25:27 > > Cc: Rodrigo Vivi > Cc: Jim Bride > Signed-off-by: Vathsala Nagaraju > Signed-off-by: Patil Deepti Reviewed-by: Jim Bride >

[PATCH v6 03/10] drm: Helper to read max clock rate

2016-07-25 Thread Jim Bride
detailed port cap info is not available. > Replace if-else ladder with switch-case (Ville) Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > drivers/gpu/drm/drm_dp_helper.c | 33 + > include/drm/drm_dp_helper.h | 2

[PATCH v6 01/10] drm: Add missing DP downstream port types

2016-07-25 Thread Jim Bride
On Wed, Jul 06, 2016 at 02:04:45PM +0300, Mika Kahola wrote: > Add missing DisplayPort downstream port types. The introduced > new port types are DP++ and Wireless. Looks good relative to the DP spec. Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > include/d

[PATCH v6 02/10] drm: Drop VGA from bpc definitions

2016-07-25 Thread Jim Bride
On Wed, Jul 06, 2016 at 02:04:46PM +0300, Mika Kahola wrote: > Drop "VGA" from bits per component definitions as these > are also used by other standards such as DVI, HDMI, > DP++. Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > include/drm/drm_d

[PATCH v6 07/10] drm: Read DP branch device SW revision

2016-07-25 Thread Jim Bride
On Wed, Jul 06, 2016 at 02:04:51PM +0300, Mika Kahola wrote: > SW revision is mandatory field for DisplayPort branch > devices. This is defined in DPCD register field 0x50A. Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > drivers/gpu/drm/dr

[PATCH v6 04/10] drm: Helper to read max bits per component

2016-07-25 Thread Jim Bride
e) Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > drivers/gpu/drm/drm_dp_helper.c | 42 > + > include/drm/drm_dp_helper.h | 2 ++ > 2 files changed, 44 insertions(+) > > diff --git a/drivers/gpu/drm/drm_dp_

[PATCH v6 05/10] drm: Read DP branch device id

2016-07-25 Thread Jim Bride
On Wed, Jul 06, 2016 at 02:04:49PM +0300, Mika Kahola wrote: > Read DisplayPort branch device id string. Reviewed-by: Jim Bride > > Signed-off-by: Mika Kahola > --- > drivers/gpu/drm/drm_dp_helper.c | 12 > include/drm/drm_dp_helper.h | 2 ++ >

[PATCH 1/3] drm/edid: Add drm_edid_get_monitor_name()

2016-04-05 Thread Jim Bride
-off-by: Jim Bride --- drivers/gpu/drm/drm_edid.c | 28 include/drm/drm_crtc.h | 1 + 2 files changed, 29 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 558ef9f..fc69a46 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b

[PATCH 2/3] drm/dp/mst: Enhance DP MST debugfs output

2016-04-05 Thread Jim Bride
Add some additional information (input vs. output port, sink associated with VC, peer device type, max number of VCs supported) and ensure that any embedded '\0' characters in a branch device's devid string are not written to debugfs. cc: dri-devel at lists.freedesktop.org Sig

[Intel-gfx] [PATCH 1/3] drm/edid: Add drm_edid_get_monitor_name()

2016-04-06 Thread Jim Bride
On Wed, Apr 06, 2016 at 11:35:44AM +0300, Jani Nikula wrote: > On Tue, 05 Apr 2016, Jim Bride wrote: > > In order to include monitor name information in debugfs > > output we needed to add a function that would extract the > > monitor name from the EDID, and that function ne

[PATCH v2 1/3] drm/edid: Add drm_edid_get_monitor_name()

2016-04-07 Thread Jim Bride
() and drm_edid_to_eld() use a common helper function to extract the monitor name from the edid. [Jani] + rebase. cc: dri-devel at lists.freedesktop.org Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_edid.c | 50 +- include/drm/drm_crtc.h

[PATCH v2 2/3] drm/dp/mst: Enhance DP MST debugfs output

2016-04-07 Thread Jim Bride
_name() call to reflect new signature. cc: dri-devel at lists.freedesktop.org Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_dp_mst_topology.c | 35 ++- 1 file changed, 30 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/

[PATCH v3 2/3] drm/dp/mst: Enhance DP MST debugfs output

2016-04-11 Thread Jim Bride
_name() call to reflect new signature. v3: Minor changes suggested by Jani + rebase. cc: dri-devel at lists.freedesktop.org cc: Jani Nikula Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_dp_mst_topology.c | 29 - 1 file changed, 24 insertions(+), 5 deletions(-) diff

[PATCH v3 1/3] drm/edid: Add drm_edid_get_monitor_name()

2016-04-11 Thread Jim Bride
() and drm_edid_to_eld() use a common helper function to extract the monitor name from the edid. [Jani] + rebase. v3: Minor changes suggested by Jani + rebase. cc: dri-devel at lists.freedesktop.org cc: Jani Nikula Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_edid.c | 51

[PATCH v4 2/3] drm/dp/mst: Enhance DP MST debugfs output

2016-04-14 Thread Jim Bride
_name() call to reflect new signature. v3: Minor changes suggested by Jani + rebase. v4: Rebase cc: dri-devel at lists.freedesktop.org cc: Jani Nikula Reviewed-by: Jani Nikula Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_dp_mst_topology.c | 29 - 1 file changed

[PATCH v4 1/3] drm/edid: Add drm_edid_get_monitor_name()

2016-04-14 Thread Jim Bride
Nikula Signed-off-by: Jim Bride --- drivers/gpu/drm/drm_edid.c | 51 ++ include/drm/drm_crtc.h | 2 ++ 2 files changed, 44 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 558ef9f..96b181a