Skip to contentSkip to conversation

drm/i915: fix get digital port issue in intel_audio

PATCH In review [email protected] 5 participants 7 messages Dec 23, 2015 – Jan 6, 2016

Subsystems (2)
DRM drivers Intel i915
lore
Share & tools
lore

[email protected]

author
#0 lore raw

PATCH 1/2 drm/i915: fix get digital port issue in intel_audio

From: Libin Yang <[email protected]>

For DP MST, use enc_to_mst(encoder)->primary to get intel_digital_port, instead of using enc_to_dig_port(encoder).

Signed-off-by Libin Yang <[email protected]>
---
 drivers/gpu/drm/i915/intel_audio.c | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)
drivers/gpu/drm/i915/intel_audio.c +17 -4
index 31f6d21..431487a0 100644
@@ -187,6 +187,16 @@ static bool intel_eld_uptodate(struct drm_connector *connector,
 	return true;
 }
 
+static struct intel_digital_port *
+intel_encoder_to_dig_port(struct intel_encoder *intel_encoder)
+{
+	struct drm_encoder *encoder = &intel_encoder->base;
+
+	if (intel_encoder->type == INTEL_OUTPUT_DP_MST)
+		return enc_to_mst(encoder)->primary;
+	return enc_to_dig_port(encoder);
+}
+
 static void g4x_audio_codec_disable(struct intel_encoder *encoder)
 {
 	struct drm_i915_private *dev_priv = encoder->base.dev->dev_private;

@@ -286,7 +296,7 @@ static void hsw_audio_codec_enable(struct drm_connector *connector,
 	struct i915_audio_component *acomp = dev_priv->audio_component;
 	const uint8_t *eld = connector->eld;
 	struct intel_digital_port *intel_dig_port =
-		enc_to_dig_port(&encoder->base);
+		intel_encoder_to_dig_port(encoder);
 	enum port port = intel_dig_port->port;
 	uint32_t tmp;
 	int len, i;

@@ -500,7 +510,8 @@ void intel_audio_codec_enable(struct intel_encoder *intel_encoder)
 	struct drm_device *dev = encoder->dev;
 	struct drm_i915_private *dev_priv = dev->dev_private;
 	struct i915_audio_component *acomp = dev_priv->audio_component;
-	struct intel_digital_port *intel_dig_port = enc_to_dig_port(encoder);
+	struct intel_digital_port *intel_dig_port =
+		intel_encoder_to_dig_port(intel_encoder);
 	enum port port = intel_dig_port->port;
 
 	connector = drm_select_eld(encoder);

@@ -546,7 +557,8 @@ void intel_audio_codec_disable(struct intel_encoder *intel_encoder)
 	struct drm_device *dev = encoder->dev;
 	struct drm_i915_private *dev_priv = dev->dev_private;
 	struct i915_audio_component *acomp = dev_priv->audio_component;
-	struct intel_digital_port *intel_dig_port = enc_to_dig_port(encoder);
+	struct intel_digital_port *intel_dig_port =
+		intel_encoder_to_dig_port(intel_encoder);
 	enum port port = intel_dig_port->port;
 
 	if (dev_priv->display.audio_codec_disable)

@@ -724,7 +736,8 @@ static int i915_audio_component_get_eld(struct device *dev, int port,
 	/* intel_encoder might be NULL for DP MST */
 	if (intel_encoder) {
 		ret = 0;
-		intel_dig_port = enc_to_dig_port(&intel_encoder->base);
+		intel_dig_port =
+			intel_encoder_to_dig_port(intel_encoder);
 		*enabled = intel_dig_port->audio_connector != NULL;
 		if (*enabled) {
 			eld = intel_dig_port->audio_connector->eld;
Signature · 6 lines
1.9.1

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
author
#1 lore raw

PATCH 2/2 drm/i915: add dp mst judgement in hsw_audio_codec_enable

From: Libin Yang <[email protected]>

hsw platforms supports DP MST while ilk doesn't.
This patch fixes the bug.

Signed-off-by Libin Yang <[email protected]>
---
 drivers/gpu/drm/i915/intel_audio.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)
drivers/gpu/drm/i915/intel_audio.c +9 -7
index 431487a0..f46fa7b 100644
@@ -255,6 +255,11 @@ static void g4x_audio_codec_enable(struct drm_connector *connector,
 	I915_WRITE(G4X_AUD_CNTL_ST, tmp);
 }
 
+/* pipe type is DP SST or DP MST */
+#define intel_pipe_is_dp(intel_crtc) \
+	(intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) || \
+	 intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+
 static void hsw_audio_codec_disable(struct intel_encoder *encoder)
 {
 	struct drm_i915_private *dev_priv = encoder->base.dev->dev_private;

@@ -272,8 +277,7 @@ static void hsw_audio_codec_disable(struct intel_encoder *encoder)
 	tmp |= AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_UPPER_N_MASK;
 	tmp &= ~AUD_CONFIG_LOWER_N_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	I915_WRITE(HSW_AUD_CFG(pipe), tmp);
 

@@ -339,7 +343,7 @@ static void hsw_audio_codec_enable(struct drm_connector *connector,
 	tmp = I915_READ(HSW_AUD_CFG(pipe));
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT))
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -486,8 +490,7 @@ static void ilk_audio_codec_enable(struct drm_connector *connector,
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -526,8 +529,7 @@ void intel_audio_codec_enable(struct intel_encoder *intel_encoder)
 
 	/* ELD Conn_Type */
 	connector->eld[5] &= ~(3 << 2);
-	if (intel_pipe_has_type(crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_is_dp(crtc))
 		connector->eld[5] |= (1 << 2);
 
 	connector->eld[6] = drm_av_sync_delay(connector, adjusted_mode) / 2;
Signature · 6 lines
1.9.1

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
#2 lore raw

Re: [PATCH 2/2] drm/i915: add dp mst judgement in hsw_audio_codec_enable

Quoted 15 lines · On Wed, Dec 23, 2015 at 02:50:47PM +0800, [email protected] wrote:

From: Libin Yang <[email protected]>

hsw platforms supports DP MST while ilk doesn't.
This patch fixes the bug.

Signed-off-by Libin Yang <[email protected]>
---
 drivers/gpu/drm/i915/intel_audio.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)
drivers/gpu/drm/i915/intel_audio.c +0 -0
index 431487a0..f46fa7b 100644

drivers/gpu/drm/i915/intel_audio.c static void g4x_audio_codec_enable(struct drm_connector *connector, · line 262

Quoted 8 lines
drivers/gpu/drm/i915/intel_audio.c +5 -0
@@ -255,6 +255,11 @@ static void g4x_audio_codec_enable(struct drm_connector *connector,
 	I915_WRITE(G4X_AUD_CNTL_ST, tmp);
 }
 
+/* pipe type is DP SST or DP MST */
+#define intel_pipe_is_dp(intel_crtc) \
+	(intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) || \
+	 intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+

crtc->config->has_dp_encoder should be good enough for all platforms.

Though I have some plans to eliminate it, but that doesn't mean we can't use it in the meantime.

Quoted 44 lines
drivers/gpu/drm/i915/intel_audio.c +4 -7
 static void hsw_audio_codec_disable(struct intel_encoder *encoder)
 {
 	struct drm_i915_private *dev_priv = encoder->base.dev->dev_private;

@@ -272,8 +277,7 @@ static void hsw_audio_codec_disable(struct intel_encoder *encoder)
 	tmp |= AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_UPPER_N_MASK;
 	tmp &= ~AUD_CONFIG_LOWER_N_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	I915_WRITE(HSW_AUD_CFG(pipe), tmp);
 

@@ -339,7 +343,7 @@ static void hsw_audio_codec_enable(struct drm_connector *connector,
 	tmp = I915_READ(HSW_AUD_CFG(pipe));
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT))
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -486,8 +490,7 @@ static void ilk_audio_codec_enable(struct drm_connector *connector,
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -526,8 +529,7 @@ void intel_audio_codec_enable(struct intel_encoder *intel_encoder)
 
 	/* ELD Conn_Type */
 	connector->eld[5] &= ~(3 << 2);
-	if (intel_pipe_has_type(crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_is_dp(crtc))
 		connector->eld[5] |= (1 << 2);
 
 	connector->eld[6] = drm_av_sync_delay(connector, adjusted_mode) / 2;
Signature · 1 line
1.9.1
Signature · 6 lines
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
#3 lore raw

Re: [PATCH 2/2] drm/i915: add dp mst judgement in hsw_audio_codec_enable

Hi Ville,

Quoted 35 lines

-----Original Message-----
From: Ville Syrjälä [mailto:[email protected]]
Sent: Tuesday, January 05, 2016 12:03 AM
To: [email protected]
Cc: [email protected]; [email protected]; [email protected]; [email protected]; Vetter, Daniel; Yang, Libin
Subject: Re: [PATCH 2/2] drm/i915: add dp mst judgement in hsw_audio_codec_enable

Review

Quoted 11 lines · On Wed, Dec 23, 2015 at 02:50:47PM +0800, [email protected] wrote:

From: Libin Yang <[email protected]>

hsw platforms supports DP MST while ilk doesn't.
This patch fixes the bug.

Signed-off-by Libin Yang <[email protected]>
---
 drivers/gpu/drm/i915/intel_audio.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)
(quoted diff) +0 -0

b/drivers/gpu/drm/i915/intel_audio.c

drivers/gpu/drm/i915/intel_audio.c static void g4x_audio_codec_enable(struct

Quoted 4 lines

index 431487a0..f46fa7b 100644

drivers/gpu/drm/i915/intel_audio.c +0 -0
@@ -255,6 +255,11 @@ static void g4x_audio_codec_enable(struct

drm_connector *connector,

drivers/gpu/drm/i915/intel_audio.c static void g4x_audio_codec_enable(struct drm_connector *connector,

Quoted 6 lines
drivers/gpu/drm/i915/intel_audio.c +3 -0
 	I915_WRITE(G4X_AUD_CNTL_ST, tmp);
 }
 
+/* pipe type is DP SST or DP MST */
+#define intel_pipe_is_dp(intel_crtc) \
+	(intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT)

|| \

Quoted 7 lines

drivers/gpu/drm/i915/intel_audio.c static void g4x_audio_codec_enable(struct drm_connector *connector, · line 262

Quoted 2 lines
drivers/gpu/drm/i915/intel_audio.c +2 -0
+	 intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))
+

crtc->config->has_dp_encoder should be good enough for all platforms.

Though I have some plans to eliminate it, but that doesn't mean we can't use it in the meantime.

Thanks. I have verified your method and it works. I will send the refined patches.

Regards,
Libin

Quoted 51 lines

drivers/gpu/drm/i915/intel_audio.c static void hsw_audio_codec_disable(struct

Quoted 6 lines
drivers/gpu/drm/i915/intel_audio.c +0 -0
 static void hsw_audio_codec_disable(struct intel_encoder *encoder)
 {
 	struct drm_i915_private *dev_priv = encoder->base.dev-

dev_private;

drivers/gpu/drm/i915/intel_audio.c +0 -0
@@ -272,8 +277,7 @@ static void hsw_audio_codec_disable(struct

intel_encoder *encoder)

drivers/gpu/drm/i915/intel_audio.c static void hsw_audio_codec_disable(struct intel_encoder *encoder)

Quoted 4 lines
drivers/gpu/drm/i915/intel_audio.c +0 -1
 	tmp |= AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_UPPER_N_MASK;
 	tmp &= ~AUD_CONFIG_LOWER_N_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT)

||
> -	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))

drivers/gpu/drm/i915/intel_audio.c static void hsw_audio_codec_enable(struct

Quoted 5 lines
drivers/gpu/drm/i915/intel_audio.c +1 -0
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	I915_WRITE(HSW_AUD_CFG(pipe), tmp);
 

@@ -339,7 +343,7 @@ static void hsw_audio_codec_enable(struct

drm_connector *connector,

drivers/gpu/drm/i915/intel_audio.c static void hsw_audio_codec_enable(struct drm_connector *connector,

Quoted 4 lines
drivers/gpu/drm/i915/intel_audio.c +0 -1
 	tmp = I915_READ(HSW_AUD_CFG(pipe));
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc,

INTEL_OUTPUT_DISPLAYPORT))

drivers/gpu/drm/i915/intel_audio.c static void ilk_audio_codec_enable(struct

Quoted 5 lines
drivers/gpu/drm/i915/intel_audio.c +1 -0
+	if (intel_pipe_is_dp(intel_crtc))
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -486,8 +490,7 @@ static void ilk_audio_codec_enable(struct

drm_connector *connector,

drivers/gpu/drm/i915/intel_audio.c static void ilk_audio_codec_enable(struct drm_connector *connector,

Quoted 4 lines
drivers/gpu/drm/i915/intel_audio.c +0 -1
 	tmp &= ~AUD_CONFIG_N_VALUE_INDEX;
 	tmp &= ~AUD_CONFIG_N_PROG_ENABLE;
 	tmp &= ~AUD_CONFIG_PIXEL_CLOCK_HDMI_MASK;
-	if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DISPLAYPORT)

||
> -	    intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_DP_MST))

drivers/gpu/drm/i915/intel_audio.c static void ilk_audio_codec_enable(struct · line 493

Quoted 1 line
drivers/gpu/drm/i915/intel_audio.c +1 -0
+	if (intel_pipe_has_type(intel_crtc,

INTEL_OUTPUT_DISPLAYPORT))

drivers/gpu/drm/i915/intel_audio.c void intel_audio_codec_enable(struct

Quoted 4 lines
drivers/gpu/drm/i915/intel_audio.c +0 -0
 		tmp |= AUD_CONFIG_N_VALUE_INDEX;
 	else
 		tmp |= audio_config_hdmi_pixel_clock(adjusted_mode);

@@ -526,8 +529,7 @@ void intel_audio_codec_enable(struct

intel_encoder *intel_encoder)

Quoted 8 lines
drivers/gpu/drm/i915/intel_audio.c +1 -2
 	/* ELD Conn_Type */
 	connector->eld[5] &= ~(3 << 2);
-	if (intel_pipe_has_type(crtc, INTEL_OUTPUT_DISPLAYPORT) ||
-	    intel_pipe_has_type(crtc, INTEL_OUTPUT_DP_MST))
+	if (intel_pipe_is_dp(crtc))
 		connector->eld[5] |= (1 << 2);
Quoted 8 lines

drivers/gpu/drm/i915/intel_audio.c void intel_audio_codec_enable(struct · line 534

Quoted 1 line
drivers/gpu/drm/i915/intel_audio.c +0 -0
 	connector->eld[6] = drm_av_sync_delay(connector,

adjusted_mode) / 2;

Quoted 2 lines
drivers/gpu/drm/i915/intel_audio.c +0 -1
--

1.9.1

--
Ville Syrjälä
Intel OTC

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

CI
#4 lore raw

✗ warning: Fi.CI.BAT

In reply to [email protected]
== Summary ==

== Summary ==

Built on 7e671e69deffb88d60687dacffe6e34a5d046500 drm-intel-nightly: 2015y-12m-22d-13h-28m-34s UTC integration manifest

Test gem_storedw_loop:

        Subgroup basic-render:
                pass       -> DMESG-WARN (skl-i5k-2)
Test kms_flip:
        Subgroup basic-flip-vs-dpms:
                dmesg-warn -> PASS       (ilk-hp8440p)
        Subgroup basic-flip-vs-modeset:
                dmesg-warn -> PASS       (bsw-nuc-2)
                pass       -> DMESG-WARN (skl-i5k-2)
                dmesg-warn -> PASS       (byt-nuc)
                pass       -> DMESG-WARN (hsw-brixbox)
                pass       -> DMESG-WARN (bdw-nuci7)
                dmesg-warn -> PASS       (ilk-hp8440p)
        Subgroup basic-plain-flip:
                pass       -> DMESG-WARN (bdw-ultra)
Test kms_pipe_crc_basic:
        Subgroup read-crc-pipe-a-frame-sequence:
                dmesg-warn -> PASS       (byt-nuc)
        Subgroup read-crc-pipe-b:
                dmesg-warn -> PASS       (skl-i5k-2)
        Subgroup read-crc-pipe-b-frame-sequence:
                pass       -> DMESG-WARN (byt-nuc)
Test kms_psr_sink_crc:
        Subgroup psr_basic:
                dmesg-warn -> PASS       (bdw-ultra)
bdw-nuci7        total:132  pass:121  dwarn:2   dfail:0   fail:0   skip:9  
bdw-ultra        total:132  pass:124  dwarn:2   dfail:0   fail:0   skip:6  
bsw-nuc-2        total:135  pass:114  dwarn:1   dfail:0   fail:0   skip:20 
byt-nuc          total:135  pass:119  dwarn:3   dfail:0   fail:0   skip:13 
hsw-brixbox      total:135  pass:126  dwarn:2   dfail:0   fail:0   skip:7  
hsw-gt2          total:135  pass:130  dwarn:1   dfail:0   fail:0   skip:4  
hsw-xps12        total:132  pass:125  dwarn:3   dfail:0   fail:0   skip:4  
ilk-hp8440p      total:135  pass:100  dwarn:0   dfail:0   fail:0   skip:35 
ivb-t430s        total:135  pass:127  dwarn:2   dfail:0   fail:0   skip:6  
skl-i5k-2        total:135  pass:123  dwarn:4   dfail:0   fail:0   skip:8  
skl-i7k-2        total:135  pass:124  dwarn:3   dfail:0   fail:0   skip:8  
snb-dellxps      total:135  pass:121  dwarn:2   dfail:0   fail:0   skip:12 
snb-x220t        total:135  pass:122  dwarn:1   dfail:0   fail:1   skip:11

Results at /archive/results/CI_IGT_test/Patchwork_799/

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

author
#5 lore raw

Re: ✗ warning: Fi.CI.BAT

Sorry to interrupt.

I got this email sent to me. It seems one of my patch causes this issue? Does anyone know how to find which patch causes this issue from the below message? Thanks.

Best Regards,
Libin

Quoted 44 lines · Patchwork

== Summary ==

Built on 7e671e69deffb88d60687dacffe6e34a5d046500 drm-intel-nightly: 2015y-12m-22d-13h-28m-34s UTC integration manifest

Test gem_storedw_loop:

         Subgroup basic-render:
                 pass       -> DMESG-WARN (skl-i5k-2)
Test kms_flip:
         Subgroup basic-flip-vs-dpms:
                 dmesg-warn -> PASS       (ilk-hp8440p)
         Subgroup basic-flip-vs-modeset:
                 dmesg-warn -> PASS       (bsw-nuc-2)
                 pass       -> DMESG-WARN (skl-i5k-2)
                 dmesg-warn -> PASS       (byt-nuc)
                 pass       -> DMESG-WARN (hsw-brixbox)
                 pass       -> DMESG-WARN (bdw-nuci7)
                 dmesg-warn -> PASS       (ilk-hp8440p)
         Subgroup basic-plain-flip:
                 pass       -> DMESG-WARN (bdw-ultra)
Test kms_pipe_crc_basic:
         Subgroup read-crc-pipe-a-frame-sequence:
                 dmesg-warn -> PASS       (byt-nuc)
         Subgroup read-crc-pipe-b:
                 dmesg-warn -> PASS       (skl-i5k-2)
         Subgroup read-crc-pipe-b-frame-sequence:
                 pass       -> DMESG-WARN (byt-nuc)
Test kms_psr_sink_crc:
         Subgroup psr_basic:
                 dmesg-warn -> PASS       (bdw-ultra)
bdw-nuci7        total:132  pass:121  dwarn:2   dfail:0   fail:0   skip:9
bdw-ultra        total:132  pass:124  dwarn:2   dfail:0   fail:0   skip:6
bsw-nuc-2        total:135  pass:114  dwarn:1   dfail:0   fail:0   skip:20
byt-nuc          total:135  pass:119  dwarn:3   dfail:0   fail:0   skip:13
hsw-brixbox      total:135  pass:126  dwarn:2   dfail:0   fail:0   skip:7
hsw-gt2          total:135  pass:130  dwarn:1   dfail:0   fail:0   skip:4
hsw-xps12        total:132  pass:125  dwarn:3   dfail:0   fail:0   skip:4
ilk-hp8440p      total:135  pass:100  dwarn:0   dfail:0   fail:0   skip:35
ivb-t430s        total:135  pass:127  dwarn:2   dfail:0   fail:0   skip:6
skl-i5k-2        total:135  pass:123  dwarn:4   dfail:0   fail:0   skip:8
skl-i7k-2        total:135  pass:124  dwarn:3   dfail:0   fail:0   skip:8
snb-dellxps      total:135  pass:121  dwarn:2   dfail:0   fail:0   skip:12
snb-x220t        total:135  pass:122  dwarn:1   dfail:0   fail:1   skip:11
Quoted 2 lines

Results at /archive/results/CI_IGT_test/Patchwork_799/

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

maintainer
#6 lore raw

Re: ✗ warning: Fi.CI.BAT

Quoted 5 lines · Libin Yang

Sorry to interrupt.

I got this email sent to me. It seems one of my patch causes this issue? Does anyone know how to find which patch causes this issue from the below message? Thanks.

Some of the tests don't give stable results yet. We're working on it. It doesn't look like your patch would change the results of any of those tests for real.

BR,
Jani.

Quoted 55 lines

Best Regards,
Libin

Quoted 44 lines · Patchwork

== Summary ==

Built on 7e671e69deffb88d60687dacffe6e34a5d046500 drm-intel-nightly: 2015y-12m-22d-13h-28m-34s UTC integration manifest

Test gem_storedw_loop:

         Subgroup basic-render:
                 pass       -> DMESG-WARN (skl-i5k-2)
Test kms_flip:
         Subgroup basic-flip-vs-dpms:
                 dmesg-warn -> PASS       (ilk-hp8440p)
         Subgroup basic-flip-vs-modeset:
                 dmesg-warn -> PASS       (bsw-nuc-2)
                 pass       -> DMESG-WARN (skl-i5k-2)
                 dmesg-warn -> PASS       (byt-nuc)
                 pass       -> DMESG-WARN (hsw-brixbox)
                 pass       -> DMESG-WARN (bdw-nuci7)
                 dmesg-warn -> PASS       (ilk-hp8440p)
         Subgroup basic-plain-flip:
                 pass       -> DMESG-WARN (bdw-ultra)
Test kms_pipe_crc_basic:
         Subgroup read-crc-pipe-a-frame-sequence:
                 dmesg-warn -> PASS       (byt-nuc)
         Subgroup read-crc-pipe-b:
                 dmesg-warn -> PASS       (skl-i5k-2)
         Subgroup read-crc-pipe-b-frame-sequence:
                 pass       -> DMESG-WARN (byt-nuc)
Test kms_psr_sink_crc:
         Subgroup psr_basic:
                 dmesg-warn -> PASS       (bdw-ultra)
bdw-nuci7        total:132  pass:121  dwarn:2   dfail:0   fail:0   skip:9
bdw-ultra        total:132  pass:124  dwarn:2   dfail:0   fail:0   skip:6
bsw-nuc-2        total:135  pass:114  dwarn:1   dfail:0   fail:0   skip:20
byt-nuc          total:135  pass:119  dwarn:3   dfail:0   fail:0   skip:13
hsw-brixbox      total:135  pass:126  dwarn:2   dfail:0   fail:0   skip:7
hsw-gt2          total:135  pass:130  dwarn:1   dfail:0   fail:0   skip:4
hsw-xps12        total:132  pass:125  dwarn:3   dfail:0   fail:0   skip:4
ilk-hp8440p      total:135  pass:100  dwarn:0   dfail:0   fail:0   skip:35
ivb-t430s        total:135  pass:127  dwarn:2   dfail:0   fail:0   skip:6
skl-i5k-2        total:135  pass:123  dwarn:4   dfail:0   fail:0   skip:8
skl-i7k-2        total:135  pass:124  dwarn:3   dfail:0   fail:0   skip:8
snb-dellxps      total:135  pass:121  dwarn:2   dfail:0   fail:0   skip:12
snb-x220t        total:135  pass:122  dwarn:1   dfail:0   fail:1   skip:11
Quoted 2 lines

Results at /archive/results/CI_IGT_test/Patchwork_799/

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Signature · 5 lines
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx