public inbox for intel-xe@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers
@ 2026-03-04 14:40 Jani Nikula
  2026-03-04 14:40 ` [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization Jani Nikula
                   ` (11 more replies)
  0 siblings, 12 replies; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

A bunch of cleanup to share step enum across drivers.

Jani Nikula (8):
  drm/i915/dmc: simplify stepping info initialization
  drm/i915/display: add step name in display runtime info
  drm/i915/dmc: use step name from runtime info
  drm/xe/compat: remove intel_step_name macro
  drm/intel: add shared step.h and switch i915 to use it
  drm/xe/step: switch to the shared step definitions with i915
  drm/xe/step: switch from enum xe_step to intel_step naming
  drm/i915/display: switch to including common step file directly

 drivers/gpu/drm/i915/display/intel_cdclk.c    |  2 +-
 drivers/gpu/drm/i915/display/intel_ddi.c      |  2 +-
 .../drm/i915/display/intel_display_device.c   | 30 +++++++--
 .../drm/i915/display/intel_display_device.h   |  1 +
 .../drm/i915/display/intel_display_power.c    |  2 +-
 drivers/gpu/drm/i915/display/intel_dmc.c      | 19 +++---
 drivers/gpu/drm/i915/display/intel_dp_mst.c   |  2 +-
 drivers/gpu/drm/i915/display/intel_dpll_mgr.c |  2 +-
 drivers/gpu/drm/i915/display/intel_fbc.c      |  2 +-
 drivers/gpu/drm/i915/display/intel_flipq.c    |  2 +-
 drivers/gpu/drm/i915/display/intel_hdcp.c     |  2 +-
 drivers/gpu/drm/i915/display/intel_pmdemand.c |  2 +-
 drivers/gpu/drm/i915/display/intel_psr.c      |  2 +-
 .../drm/i915/display/skl_universal_plane.c    |  2 +-
 drivers/gpu/drm/i915/intel_step.h             | 57 +----------------
 .../drm/xe/compat-i915-headers/intel_step.h   | 14 -----
 drivers/gpu/drm/xe/tests/xe_pci.c             |  4 +-
 drivers/gpu/drm/xe/xe_step.c                  |  2 +-
 drivers/gpu/drm/xe/xe_step.h                  |  4 +-
 drivers/gpu/drm/xe/xe_step_types.h            | 61 +-----------------
 include/drm/intel/step.h                      | 62 +++++++++++++++++++
 21 files changed, 117 insertions(+), 159 deletions(-)
 delete mode 100644 drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
 create mode 100644 include/drm/intel/step.h

-- 
2.47.3


^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  8:18   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 2/8] drm/i915/display: add step name in display runtime info Jani Nikula
                   ` (10 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Having intel_get_stepping_info() return the pointer that was passed in
isn't necessary. Just use a pointer to the local variable instead.

The initialization to ** didn't make a difference, because it was always
overridden.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dmc.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
index c3b411259a0c..4e763f2f17c7 100644
--- a/drivers/gpu/drm/i915/display/intel_dmc.c
+++ b/drivers/gpu/drm/i915/display/intel_dmc.c
@@ -418,15 +418,12 @@ bool intel_dmc_has_payload(struct intel_display *display)
 	return has_dmc_id_fw(display, DMC_FW_MAIN);
 }
 
-static const struct stepping_info *
-intel_get_stepping_info(struct intel_display *display,
-			struct stepping_info *si)
+static void initialize_stepping_info(struct intel_display *display, struct stepping_info *si)
 {
 	const char *step_name = intel_step_name(INTEL_DISPLAY_STEP(display));
 
 	si->stepping = step_name[0];
 	si->substepping = step_name[1];
-	return si;
 }
 
 static void gen9_set_dc_state_debugmask(struct intel_display *display)
@@ -1274,8 +1271,7 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
 	struct intel_css_header *css_header;
 	struct intel_package_header *package_header;
 	struct intel_dmc_header_base *dmc_header;
-	struct stepping_info display_info = { '*', '*'};
-	const struct stepping_info *si = intel_get_stepping_info(display, &display_info);
+	struct stepping_info si = {};
 	enum intel_dmc_id dmc_id;
 	u32 readcount = 0;
 	u32 r, offset;
@@ -1283,6 +1279,8 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
 	if (!fw)
 		return -EINVAL;
 
+	initialize_stepping_info(display, &si);
+
 	/* Extract CSS Header information */
 	css_header = (struct intel_css_header *)fw->data;
 	r = parse_dmc_fw_css(dmc, css_header, fw->size);
@@ -1293,7 +1291,7 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
 
 	/* Extract Package Header information */
 	package_header = (struct intel_package_header *)&fw->data[readcount];
-	r = parse_dmc_fw_package(dmc, package_header, si, fw->size - readcount);
+	r = parse_dmc_fw_package(dmc, package_header, &si, fw->size - readcount);
 	if (!r)
 		return -EINVAL;
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 2/8] drm/i915/display: add step name in display runtime info
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
  2026-03-04 14:40 ` [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  8:24   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 3/8] drm/i915/dmc: use step name from " Jani Nikula
                   ` (9 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Initialize the stepping name in display runtime info. This avoids having
to use intel_step_name().

For display device info print at boot, debugfs and snapshot this changes
the unknown step name from ** to N/A, which is more user friendly
anyway.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/i915/display/intel_display_device.c   | 28 +++++++++++++++++--
 .../drm/i915/display/intel_display_device.h   |  1 +
 2 files changed, 26 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display_device.c b/drivers/gpu/drm/i915/display/intel_display_device.c
index 0241603470a2..f716e9a51532 100644
--- a/drivers/gpu/drm/i915/display/intel_display_device.c
+++ b/drivers/gpu/drm/i915/display/intel_display_device.c
@@ -1653,6 +1653,28 @@ static void display_platforms_or(struct intel_display_platforms *dst,
 	bitmap_or(dst->bitmap, dst->bitmap, src->bitmap, display_platforms_num_bits());
 }
 
+#define __STEP_NAME(name) [STEP_##name] = #name,
+
+static void initialize_step(struct intel_display *display, enum intel_step step)
+{
+	static const char step_names[][3] = {
+		STEP_NAME_LIST(__STEP_NAME)
+	};
+
+	DISPLAY_RUNTIME_INFO(display)->step = step;
+
+	/* Step name will remain an empty string if not applicable */
+	if (step >= 0 && step < ARRAY_SIZE(step_names))
+		strscpy(DISPLAY_RUNTIME_INFO(display)->step_name, step_names[step]);
+}
+
+#undef __STEP_NAME
+
+static const char *step_name(const struct intel_display_runtime_info *runtime)
+{
+	return strlen(runtime->step_name) ? runtime->step_name : "N/A";
+}
+
 struct intel_display *intel_display_device_probe(struct pci_dev *pdev,
 						 const struct intel_display_parent_interface *parent)
 {
@@ -1730,14 +1752,14 @@ struct intel_display *intel_display_device_probe(struct pci_dev *pdev,
 					  subdesc ? &subdesc->step_info : NULL);
 	}
 
-	DISPLAY_RUNTIME_INFO(display)->step = step;
+	initialize_step(display, step);
 
 	drm_info(display->drm, "Found %s%s%s (device ID %04x) %s display version %u.%02u stepping %s\n",
 		 desc->name, subdesc ? "/" : "", subdesc ? subdesc->name : "",
 		 pdev->device, display->platform.dgfx ? "discrete" : "integrated",
 		 DISPLAY_RUNTIME_INFO(display)->ip.ver,
 		 DISPLAY_RUNTIME_INFO(display)->ip.rel,
-		 step != STEP_NONE ? intel_step_name(step) : "N/A");
+		 step_name(DISPLAY_RUNTIME_INFO(display)));
 
 	return display;
 
@@ -1953,7 +1975,7 @@ void intel_display_device_info_print(const struct intel_display_device_info *inf
 		drm_printf(p, "display version: %u\n",
 			   runtime->ip.ver);
 
-	drm_printf(p, "display stepping: %s\n", intel_step_name(runtime->step));
+	drm_printf(p, "display stepping: %s\n", step_name(runtime));
 
 #define PRINT_FLAG(name) drm_printf(p, "%s: %s\n", #name, str_yes_no(info->name))
 	DEV_INFO_DISPLAY_FOR_EACH_FLAG(PRINT_FLAG);
diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h b/drivers/gpu/drm/i915/display/intel_display_device.h
index e84c190dcc4f..1170ac346615 100644
--- a/drivers/gpu/drm/i915/display/intel_display_device.h
+++ b/drivers/gpu/drm/i915/display/intel_display_device.h
@@ -287,6 +287,7 @@ struct intel_display_runtime_info {
 		u16 step; /* hardware */
 	} ip;
 	int step; /* symbolic */
+	char step_name[3]; /* empty string if not applicable */
 
 	u32 rawclk_freq;
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 3/8] drm/i915/dmc: use step name from runtime info
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
  2026-03-04 14:40 ` [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization Jani Nikula
  2026-03-04 14:40 ` [PATCH 2/8] drm/i915/display: add step name in display runtime info Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  8:35   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 4/8] drm/xe/compat: remove intel_step_name macro Jani Nikula
                   ` (8 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Now that the step name is in runtime info, switch to using it instead of
intel_step_name().

The ** are only relevant for DMC, so make their use explicit.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dmc.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
index 4e763f2f17c7..b250d41dc6a1 100644
--- a/drivers/gpu/drm/i915/display/intel_dmc.c
+++ b/drivers/gpu/drm/i915/display/intel_dmc.c
@@ -39,7 +39,6 @@
 #include "intel_dmc.h"
 #include "intel_dmc_regs.h"
 #include "intel_flipq.h"
-#include "intel_step.h"
 
 /**
  * DOC: DMC Firmware Support
@@ -420,10 +419,10 @@ bool intel_dmc_has_payload(struct intel_display *display)
 
 static void initialize_stepping_info(struct intel_display *display, struct stepping_info *si)
 {
-	const char *step_name = intel_step_name(INTEL_DISPLAY_STEP(display));
+	const char *step_name = DISPLAY_RUNTIME_INFO(display)->step_name;
 
-	si->stepping = step_name[0];
-	si->substepping = step_name[1];
+	si->stepping = step_name[0] ?: '*';
+	si->substepping = step_name[1] ?: '*';
 }
 
 static void gen9_set_dc_state_debugmask(struct intel_display *display)
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 4/8] drm/xe/compat: remove intel_step_name macro
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (2 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 3/8] drm/i915/dmc: use step name from " Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  9:07   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it Jani Nikula
                   ` (7 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

As there are no more compat users left for intel_step_name(), remove the
macro and use the more direct include for the enumerations.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/xe/compat-i915-headers/intel_step.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
index 2cf13a572ab0..0eabe2866f5f 100644
--- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
+++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
@@ -6,9 +6,8 @@
 #ifndef __INTEL_STEP_H__
 #define __INTEL_STEP_H__
 
-#include "xe_step.h"
+#include "xe_step_types.h"
 
 #define intel_step xe_step
-#define intel_step_name xe_step_name
 
 #endif /* __INTEL_STEP_H__ */
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (3 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 4/8] drm/xe/compat: remove intel_step_name macro Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  9:09   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915 Jani Nikula
                   ` (6 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

As the first step towards using shared definitions for step name
enumerations, add shared include/drm/intel/step.h and switch i915 to use
it.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/intel_step.h | 57 +---------------------------
 include/drm/intel/step.h          | 62 +++++++++++++++++++++++++++++++
 2 files changed, 64 insertions(+), 55 deletions(-)
 create mode 100644 include/drm/intel/step.h

diff --git a/drivers/gpu/drm/i915/intel_step.h b/drivers/gpu/drm/i915/intel_step.h
index 22f1d6905160..2ca36eae4b5a 100644
--- a/drivers/gpu/drm/i915/intel_step.h
+++ b/drivers/gpu/drm/i915/intel_step.h
@@ -8,6 +8,8 @@
 
 #include <linux/types.h>
 
+#include <drm/intel/step.h>
+
 struct drm_i915_private;
 
 struct intel_step_info {
@@ -19,61 +21,6 @@ struct intel_step_info {
 	u8 media_step;
 };
 
-#define STEP_ENUM_VAL(name)  STEP_##name,
-
-#define STEP_NAME_LIST(func)		\
-	func(A0)			\
-	func(A1)			\
-	func(A2)			\
-	func(A3)			\
-	func(B0)			\
-	func(B1)			\
-	func(B2)			\
-	func(B3)			\
-	func(C0)			\
-	func(C1)			\
-	func(C2)			\
-	func(C3)			\
-	func(D0)			\
-	func(D1)			\
-	func(D2)			\
-	func(D3)			\
-	func(E0)			\
-	func(E1)			\
-	func(E2)			\
-	func(E3)			\
-	func(F0)			\
-	func(F1)			\
-	func(F2)			\
-	func(F3)			\
-	func(G0)			\
-	func(G1)			\
-	func(G2)			\
-	func(G3)			\
-	func(H0)			\
-	func(H1)			\
-	func(H2)			\
-	func(H3)			\
-	func(I0)			\
-	func(I1)			\
-	func(I2)			\
-	func(I3)			\
-	func(J0)			\
-	func(J1)			\
-	func(J2)			\
-	func(J3)
-
-/*
- * Symbolic steppings that do not match the hardware. These are valid both as gt
- * and display steppings as symbolic names.
- */
-enum intel_step {
-	STEP_NONE = 0,
-	STEP_NAME_LIST(STEP_ENUM_VAL)
-	STEP_FUTURE,
-	STEP_FOREVER,
-};
-
 void intel_step_init(struct drm_i915_private *i915);
 const char *intel_step_name(enum intel_step step);
 
diff --git a/include/drm/intel/step.h b/include/drm/intel/step.h
new file mode 100644
index 000000000000..4de7520109bc
--- /dev/null
+++ b/include/drm/intel/step.h
@@ -0,0 +1,62 @@
+/* SPDX-License-Identifier: MIT */
+/* Copyright © 2026 Intel Corporation */
+
+#ifndef __STEP_H__
+#define __STEP_H__
+
+#define STEP_ENUM_VAL(name)  STEP_##name,
+
+#define STEP_NAME_LIST(func)		\
+	func(A0)			\
+	func(A1)			\
+	func(A2)			\
+	func(A3)			\
+	func(B0)			\
+	func(B1)			\
+	func(B2)			\
+	func(B3)			\
+	func(C0)			\
+	func(C1)			\
+	func(C2)			\
+	func(C3)			\
+	func(D0)			\
+	func(D1)			\
+	func(D2)			\
+	func(D3)			\
+	func(E0)			\
+	func(E1)			\
+	func(E2)			\
+	func(E3)			\
+	func(F0)			\
+	func(F1)			\
+	func(F2)			\
+	func(F3)			\
+	func(G0)			\
+	func(G1)			\
+	func(G2)			\
+	func(G3)			\
+	func(H0)			\
+	func(H1)			\
+	func(H2)			\
+	func(H3)			\
+	func(I0)			\
+	func(I1)			\
+	func(I2)			\
+	func(I3)			\
+	func(J0)			\
+	func(J1)			\
+	func(J2)			\
+	func(J3)
+
+/*
+ * Symbolic steppings that do not match the hardware. These are valid both as gt
+ * and display steppings as symbolic names.
+ */
+enum intel_step {
+	STEP_NONE = 0,
+	STEP_NAME_LIST(STEP_ENUM_VAL)
+	STEP_FUTURE,
+	STEP_FOREVER,
+};
+
+#endif /* __STEP_H__ */
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (4 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  9:10   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming Jani Nikula
                   ` (5 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Use the shared stepping enums from include/drm/intel/step.h.

For now, define xe_step as intel_step to avoid mass renames at the same
time. For compat, we can remove the reverse macro.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/xe/compat-i915-headers/intel_step.h   |  4 +-
 drivers/gpu/drm/xe/xe_step_types.h            | 63 ++-----------------
 2 files changed, 5 insertions(+), 62 deletions(-)

diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
index 0eabe2866f5f..cb55a659856b 100644
--- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
+++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
@@ -6,8 +6,6 @@
 #ifndef __INTEL_STEP_H__
 #define __INTEL_STEP_H__
 
-#include "xe_step_types.h"
-
-#define intel_step xe_step
+#include <drm/intel/step.h>
 
 #endif /* __INTEL_STEP_H__ */
diff --git a/drivers/gpu/drm/xe/xe_step_types.h b/drivers/gpu/drm/xe/xe_step_types.h
index d978cc2512f2..bc3ed5a8282a 100644
--- a/drivers/gpu/drm/xe/xe_step_types.h
+++ b/drivers/gpu/drm/xe/xe_step_types.h
@@ -8,69 +8,14 @@
 
 #include <linux/types.h>
 
+#include <drm/intel/step.h>
+
+#define xe_step intel_step
+
 struct xe_step_info {
 	u8 graphics;
 	u8 media;
 	u8 basedie;
 };
 
-#define STEP_ENUM_VAL(name)  STEP_##name,
-
-/*
- * Always define four minor steppings 0-3 for each stepping to match GMD ID
- * spacing of values. See xe_step_gmdid_get().
- */
-#define STEP_NAME_LIST(func)		\
-	func(A0)			\
-	func(A1)			\
-	func(A2)			\
-	func(A3)			\
-	func(B0)			\
-	func(B1)			\
-	func(B2)			\
-	func(B3)			\
-	func(C0)			\
-	func(C1)			\
-	func(C2)			\
-	func(C3)			\
-	func(D0)			\
-	func(D1)			\
-	func(D2)			\
-	func(D3)			\
-	func(E0)			\
-	func(E1)			\
-	func(E2)			\
-	func(E3)			\
-	func(F0)			\
-	func(F1)			\
-	func(F2)			\
-	func(F3)			\
-	func(G0)			\
-	func(G1)			\
-	func(G2)			\
-	func(G3)			\
-	func(H0)			\
-	func(H1)			\
-	func(H2)			\
-	func(H3)			\
-	func(I0)			\
-	func(I1)			\
-	func(I2)			\
-	func(I3)			\
-	func(J0)			\
-	func(J1)			\
-	func(J2)			\
-	func(J3)
-
-/*
- * Symbolic steppings that do not match the hardware. These are valid both as gt
- * and display steppings as symbolic names.
- */
-enum xe_step {
-	STEP_NONE = 0,
-	STEP_NAME_LIST(STEP_ENUM_VAL)
-	STEP_FUTURE,
-	STEP_FOREVER,
-};
-
 #endif
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (5 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915 Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  9:11   ` Luca Coelho
  2026-03-04 14:40 ` [PATCH 8/8] drm/i915/display: switch to including common step file directly Jani Nikula
                   ` (4 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Remove the xe_step macro, and use the enum intel_step name directly.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/xe/tests/xe_pci.c  | 4 ++--
 drivers/gpu/drm/xe/xe_step.c       | 2 +-
 drivers/gpu/drm/xe/xe_step.h       | 4 ++--
 drivers/gpu/drm/xe/xe_step_types.h | 2 --
 4 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index f3179b31f13e..860409c579f8 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -131,12 +131,12 @@ static const char *subplatform_prefix(enum xe_subplatform s)
 	return s == XE_SUBPLATFORM_NONE ? "" : " ";
 }
 
-static const char *step_prefix(enum xe_step step)
+static const char *step_prefix(enum intel_step step)
 {
 	return step == STEP_NONE ? "" : " ";
 }
 
-static const char *step_name(enum xe_step step)
+static const char *step_name(enum intel_step step)
 {
 	return step == STEP_NONE ? "" : xe_step_name(step);
 }
diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
index 2860986f82f7..e021da35b03c 100644
--- a/drivers/gpu/drm/xe/xe_step.c
+++ b/drivers/gpu/drm/xe/xe_step.c
@@ -248,7 +248,7 @@ struct xe_step_info xe_step_gmdid_get(struct xe_device *xe,
 	case STEP_##name:	\
 		return #name;
 
-const char *xe_step_name(enum xe_step step)
+const char *xe_step_name(enum intel_step step)
 {
 	switch (step) {
 	STEP_NAME_LIST(STEP_NAME_CASE);
diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
index 686cb59200c2..6fd680c1b45e 100644
--- a/drivers/gpu/drm/xe/xe_step.h
+++ b/drivers/gpu/drm/xe/xe_step.h
@@ -16,8 +16,8 @@ struct xe_step_info xe_step_pre_gmdid_get(struct xe_device *xe);
 struct xe_step_info xe_step_gmdid_get(struct xe_device *xe,
 				      u32 graphics_gmdid_revid,
 				      u32 media_gmdid_revid);
-static inline u32 xe_step_to_gmdid(enum xe_step step) { return step - STEP_A0; }
+static inline u32 xe_step_to_gmdid(enum intel_step step) { return step - STEP_A0; }
 
-const char *xe_step_name(enum xe_step step);
+const char *xe_step_name(enum intel_step step);
 
 #endif
diff --git a/drivers/gpu/drm/xe/xe_step_types.h b/drivers/gpu/drm/xe/xe_step_types.h
index bc3ed5a8282a..bd9a7a44a67f 100644
--- a/drivers/gpu/drm/xe/xe_step_types.h
+++ b/drivers/gpu/drm/xe/xe_step_types.h
@@ -10,8 +10,6 @@
 
 #include <drm/intel/step.h>
 
-#define xe_step intel_step
-
 struct xe_step_info {
 	u8 graphics;
 	u8 media;
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 8/8] drm/i915/display: switch to including common step file directly
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (6 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming Jani Nikula
@ 2026-03-04 14:40 ` Jani Nikula
  2026-03-09  9:12   ` Luca Coelho
  2026-03-05 19:23 ` ✗ CI.checkpatch: warning for drm/{i915,xe}: sort out step enums between the drivers Patchwork
                   ` (3 subsequent siblings)
  11 siblings, 1 reply; 22+ messages in thread
From: Jani Nikula @ 2026-03-04 14:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Instead of using the proxy intel_step.h in display, just include the
common step file directly where needed. This allows us to remove the
compat intel_step.h header.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_cdclk.c          |  2 +-
 drivers/gpu/drm/i915/display/intel_ddi.c            |  2 +-
 drivers/gpu/drm/i915/display/intel_display_device.c |  2 +-
 drivers/gpu/drm/i915/display/intel_display_power.c  |  2 +-
 drivers/gpu/drm/i915/display/intel_dp_mst.c         |  2 +-
 drivers/gpu/drm/i915/display/intel_dpll_mgr.c       |  2 +-
 drivers/gpu/drm/i915/display/intel_fbc.c            |  2 +-
 drivers/gpu/drm/i915/display/intel_flipq.c          |  2 +-
 drivers/gpu/drm/i915/display/intel_hdcp.c           |  2 +-
 drivers/gpu/drm/i915/display/intel_pmdemand.c       |  2 +-
 drivers/gpu/drm/i915/display/intel_psr.c            |  2 +-
 drivers/gpu/drm/i915/display/skl_universal_plane.c  |  2 +-
 drivers/gpu/drm/xe/compat-i915-headers/intel_step.h | 11 -----------
 13 files changed, 12 insertions(+), 23 deletions(-)
 delete mode 100644 drivers/gpu/drm/xe/compat-i915-headers/intel_step.h

diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
index 38331e899519..fcee3b1bfdcc 100644
--- a/drivers/gpu/drm/i915/display/intel_cdclk.c
+++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
@@ -28,6 +28,7 @@
 #include <drm/drm_fixed.h>
 #include <drm/drm_print.h>
 #include <drm/intel/intel_pcode_regs.h>
+#include <drm/intel/step.h>
 
 #include "hsw_ips.h"
 #include "intel_atomic.h"
@@ -46,7 +47,6 @@
 #include "intel_pci_config.h"
 #include "intel_plane.h"
 #include "intel_psr.h"
-#include "intel_step.h"
 #include "intel_vdsc.h"
 #include "skl_watermark.h"
 #include "skl_watermark_regs.h"
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 94ae583e907f..b4c84528ce5c 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -33,6 +33,7 @@
 #include <drm/display/drm_scdc_helper.h>
 #include <drm/drm_print.h>
 #include <drm/drm_privacy_screen_consumer.h>
+#include <drm/intel/step.h>
 
 #include "icl_dsi.h"
 #include "intel_alpm.h"
@@ -80,7 +81,6 @@
 #include "intel_psr.h"
 #include "intel_quirks.h"
 #include "intel_snps_phy.h"
-#include "intel_step.h"
 #include "intel_tc.h"
 #include "intel_vdsc.h"
 #include "intel_vdsc_regs.h"
diff --git a/drivers/gpu/drm/i915/display/intel_display_device.c b/drivers/gpu/drm/i915/display/intel_display_device.c
index f716e9a51532..361a2fca58a3 100644
--- a/drivers/gpu/drm/i915/display/intel_display_device.c
+++ b/drivers/gpu/drm/i915/display/intel_display_device.c
@@ -9,6 +9,7 @@
 #include <drm/drm_drv.h>
 #include <drm/drm_print.h>
 #include <drm/intel/pciids.h>
+#include <drm/intel/step.h>
 
 #include "intel_cx0_phy_regs.h"
 #include "intel_de.h"
@@ -20,7 +21,6 @@
 #include "intel_display_regs.h"
 #include "intel_display_types.h"
 #include "intel_fbc.h"
-#include "intel_step.h"
 
 __diag_push();
 __diag_ignore_all("-Woverride-init", "Allow field initialization overrides for display info");
diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
index 755935dcfe23..fa76a3ed6559 100644
--- a/drivers/gpu/drm/i915/display/intel_display_power.c
+++ b/drivers/gpu/drm/i915/display/intel_display_power.c
@@ -8,6 +8,7 @@
 
 #include <drm/drm_print.h>
 #include <drm/intel/intel_pcode_regs.h>
+#include <drm/intel/step.h>
 
 #include "intel_backlight_regs.h"
 #include "intel_cdclk.h"
@@ -29,7 +30,6 @@
 #include "intel_pmdemand.h"
 #include "intel_pps_regs.h"
 #include "intel_snps_phy.h"
-#include "intel_step.h"
 #include "skl_watermark.h"
 #include "skl_watermark_regs.h"
 #include "vlv_sideband.h"
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c
index 0bb39f9bd74e..262fd2ed278e 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
@@ -32,6 +32,7 @@
 #include <drm/drm_fixed.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
+#include <drm/intel/step.h>
 
 #include "intel_atomic.h"
 #include "intel_audio.h"
@@ -55,7 +56,6 @@
 #include "intel_link_bw.h"
 #include "intel_pfit.h"
 #include "intel_psr.h"
-#include "intel_step.h"
 #include "intel_vdsc.h"
 #include "intel_vrr.h"
 #include "skl_scaler.h"
diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
index f35a9252f4e1..908a90804881 100644
--- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
+++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
@@ -25,6 +25,7 @@
 #include <linux/string_helpers.h>
 
 #include <drm/drm_print.h>
+#include <drm/intel/step.h>
 
 #include "bxt_dpio_phy_regs.h"
 #include "intel_cx0_phy.h"
@@ -41,7 +42,6 @@
 #include "intel_lt_phy.h"
 #include "intel_mg_phy_regs.h"
 #include "intel_pch_refclk.h"
-#include "intel_step.h"
 #include "intel_tc.h"
 
 /**
diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
index 3e9b3e532499..a9de9fe6fa28 100644
--- a/drivers/gpu/drm/i915/display/intel_fbc.c
+++ b/drivers/gpu/drm/i915/display/intel_fbc.c
@@ -44,6 +44,7 @@
 #include <drm/drm_blend.h>
 #include <drm/drm_fourcc.h>
 #include <drm/drm_print.h>
+#include <drm/intel/step.h>
 
 #include "i9xx_plane_regs.h"
 #include "intel_de.h"
@@ -58,7 +59,6 @@
 #include "intel_fbc_regs.h"
 #include "intel_frontbuffer.h"
 #include "intel_parent.h"
-#include "intel_step.h"
 
 #define for_each_fbc_id(__display, __fbc_id) \
 	for ((__fbc_id) = INTEL_FBC_A; (__fbc_id) < I915_MAX_FBCS; (__fbc_id)++) \
diff --git a/drivers/gpu/drm/i915/display/intel_flipq.c b/drivers/gpu/drm/i915/display/intel_flipq.c
index 1e9550cb66a3..d2c69a666f0b 100644
--- a/drivers/gpu/drm/i915/display/intel_flipq.c
+++ b/drivers/gpu/drm/i915/display/intel_flipq.c
@@ -6,6 +6,7 @@
 #include <linux/pci.h>
 
 #include <drm/drm_print.h>
+#include <drm/intel/step.h>
 
 #include "intel_crtc.h"
 #include "intel_de.h"
@@ -16,7 +17,6 @@
 #include "intel_dmc_regs.h"
 #include "intel_dsb.h"
 #include "intel_flipq.h"
-#include "intel_step.h"
 #include "intel_vblank.h"
 #include "intel_vrr.h"
 
diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
index 116d2e0af878..6e86d97f8ea9 100644
--- a/drivers/gpu/drm/i915/display/intel_hdcp.c
+++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
@@ -18,6 +18,7 @@
 #include <drm/drm_print.h>
 #include <drm/intel/i915_component.h>
 #include <drm/intel/intel_pcode_regs.h>
+#include <drm/intel/step.h>
 
 #include "intel_connector.h"
 #include "intel_de.h"
@@ -33,7 +34,6 @@
 #include "intel_hdcp_regs.h"
 #include "intel_hdcp_shim.h"
 #include "intel_parent.h"
-#include "intel_step.h"
 
 #define USE_HDCP_GSC(__display)		(DISPLAY_VER(__display) >= 14)
 
diff --git a/drivers/gpu/drm/i915/display/intel_pmdemand.c b/drivers/gpu/drm/i915/display/intel_pmdemand.c
index f3db55710010..a41ff04a183c 100644
--- a/drivers/gpu/drm/i915/display/intel_pmdemand.c
+++ b/drivers/gpu/drm/i915/display/intel_pmdemand.c
@@ -6,6 +6,7 @@
 #include <linux/bitops.h>
 
 #include <drm/drm_print.h>
+#include <drm/intel/step.h>
 
 #include "intel_atomic.h"
 #include "intel_bw.h"
@@ -16,7 +17,6 @@
 #include "intel_display_trace.h"
 #include "intel_display_utils.h"
 #include "intel_pmdemand.h"
-#include "intel_step.h"
 #include "skl_watermark.h"
 
 struct pmdemand_params {
diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c
index 9296ca3a4ff4..a38d872dcb80 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -28,6 +28,7 @@
 #include <drm/drm_debugfs.h>
 #include <drm/drm_print.h>
 #include <drm/drm_vblank.h>
+#include <drm/intel/step.h>
 
 #include "intel_alpm.h"
 #include "intel_atomic.h"
@@ -49,7 +50,6 @@
 #include "intel_psr.h"
 #include "intel_psr_regs.h"
 #include "intel_snps_phy.h"
-#include "intel_step.h"
 #include "intel_vblank.h"
 #include "intel_vdsc.h"
 #include "intel_vrr.h"
diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c b/drivers/gpu/drm/i915/display/skl_universal_plane.c
index 677f1339b7f8..91e2950c3741 100644
--- a/drivers/gpu/drm/i915/display/skl_universal_plane.c
+++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c
@@ -8,6 +8,7 @@
 #include <drm/drm_damage_helper.h>
 #include <drm/drm_fourcc.h>
 #include <drm/drm_print.h>
+#include <drm/intel/step.h>
 
 #include "intel_bo.h"
 #include "intel_color.h"
@@ -24,7 +25,6 @@
 #include "intel_plane.h"
 #include "intel_psr.h"
 #include "intel_psr_regs.h"
-#include "intel_step.h"
 #include "skl_scaler.h"
 #include "skl_universal_plane.h"
 #include "skl_universal_plane_regs.h"
diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
deleted file mode 100644
index cb55a659856b..000000000000
--- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
+++ /dev/null
@@ -1,11 +0,0 @@
-/* SPDX-License-Identifier: MIT */
-/*
- * Copyright © 2023 Intel Corporation
- */
-
-#ifndef __INTEL_STEP_H__
-#define __INTEL_STEP_H__
-
-#include <drm/intel/step.h>
-
-#endif /* __INTEL_STEP_H__ */
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 22+ messages in thread

* ✗ CI.checkpatch: warning for drm/{i915,xe}: sort out step enums between the drivers
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (7 preceding siblings ...)
  2026-03-04 14:40 ` [PATCH 8/8] drm/i915/display: switch to including common step file directly Jani Nikula
@ 2026-03-05 19:23 ` Patchwork
  2026-03-05 19:25 ` ✓ CI.KUnit: success " Patchwork
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 22+ messages in thread
From: Patchwork @ 2026-03-05 19:23 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

== Series Details ==

Series: drm/{i915,xe}: sort out step enums between the drivers
URL   : https://patchwork.freedesktop.org/series/162594/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
1f57ba1afceae32108bd24770069f764d940a0e4
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit a2d5cea0d7772df2eba1a3672413a70260751555
Author: Jani Nikula <jani.nikula@intel.com>
Date:   Wed Mar 4 16:40:53 2026 +0200

    drm/i915/display: switch to including common step file directly
    
    Instead of using the proxy intel_step.h in display, just include the
    common step file directly where needed. This allows us to remove the
    compat intel_step.h header.
    
    Signed-off-by: Jani Nikula <jani.nikula@intel.com>
+ /mt/dim checkpatch ce63d46cc8fc3bb754efb93026b55acaa616cfa2 drm-intel
da088e028f72 drm/i915/dmc: simplify stepping info initialization
2ff0eb57f975 drm/i915/display: add step name in display runtime info
-:23: ERROR:BRACKET_SPACE: space prohibited before open square bracket '['
#23: FILE: drivers/gpu/drm/i915/display/intel_display_device.c:1656:
+#define __STEP_NAME(name) [STEP_##name] = #name,

total: 1 errors, 0 warnings, 0 checks, 59 lines checked
5397ff2c8c77 drm/i915/dmc: use step name from runtime info
a73fe4cd2e84 drm/xe/compat: remove intel_step_name macro
3a9c4e4e86ab drm/intel: add shared step.h and switch i915 to use it
-:88: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#88: 
new file mode 100644

-:101: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#101: FILE: include/drm/intel/step.h:9:
+#define STEP_NAME_LIST(func)		\
+	func(A0)			\
+	func(A1)			\
+	func(A2)			\
+	func(A3)			\
+	func(B0)			\
+	func(B1)			\
+	func(B2)			\
+	func(B3)			\
+	func(C0)			\
+	func(C1)			\
+	func(C2)			\
+	func(C3)			\
+	func(D0)			\
+	func(D1)			\
+	func(D2)			\
+	func(D3)			\
+	func(E0)			\
+	func(E1)			\
+	func(E2)			\
+	func(E3)			\
+	func(F0)			\
+	func(F1)			\
+	func(F2)			\
+	func(F3)			\
+	func(G0)			\
+	func(G1)			\
+	func(G2)			\
+	func(G3)			\
+	func(H0)			\
+	func(H1)			\
+	func(H2)			\
+	func(H3)			\
+	func(I0)			\
+	func(I1)			\
+	func(I2)			\
+	func(I3)			\
+	func(J0)			\
+	func(J1)			\
+	func(J2)			\
+	func(J3)

BUT SEE:

   do {} while (0) advice is over-stated in a few situations:

   The more obvious case is macros, like MODULE_PARM_DESC, invoked at
   file-scope, where C disallows code (it must be in functions).  See
   $exceptions if you have one to add by name.

   More troublesome is declarative macros used at top of new scope,
   like DECLARE_PER_CPU.  These might just compile with a do-while-0
   wrapper, but would be incorrect.  Most of these are handled by
   detecting struct,union,etc declaration primitives in $exceptions.

   Theres also macros called inside an if (block), which "return" an
   expression.  These cannot do-while, and need a ({}) wrapper.

   Enjoy this qualification while we work to improve our heuristics.

-:101: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'func' - possible side-effects?
#101: FILE: include/drm/intel/step.h:9:
+#define STEP_NAME_LIST(func)		\
+	func(A0)			\
+	func(A1)			\
+	func(A2)			\
+	func(A3)			\
+	func(B0)			\
+	func(B1)			\
+	func(B2)			\
+	func(B3)			\
+	func(C0)			\
+	func(C1)			\
+	func(C2)			\
+	func(C3)			\
+	func(D0)			\
+	func(D1)			\
+	func(D2)			\
+	func(D3)			\
+	func(E0)			\
+	func(E1)			\
+	func(E2)			\
+	func(E3)			\
+	func(F0)			\
+	func(F1)			\
+	func(F2)			\
+	func(F3)			\
+	func(G0)			\
+	func(G1)			\
+	func(G2)			\
+	func(G3)			\
+	func(H0)			\
+	func(H1)			\
+	func(H2)			\
+	func(H3)			\
+	func(I0)			\
+	func(I1)			\
+	func(I2)			\
+	func(I3)			\
+	func(J0)			\
+	func(J1)			\
+	func(J2)			\
+	func(J3)

total: 1 errors, 1 warnings, 1 checks, 131 lines checked
9ec471d1c16b drm/xe/step: switch to the shared step definitions with i915
2de46132e604 drm/xe/step: switch from enum xe_step to intel_step naming
a2d5cea0d777 drm/i915/display: switch to including common step file directly
-:254: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#254: 
deleted file mode 100644

total: 0 errors, 1 warnings, 0 checks, 168 lines checked



^ permalink raw reply	[flat|nested] 22+ messages in thread

* ✓ CI.KUnit: success for drm/{i915,xe}: sort out step enums between the drivers
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (8 preceding siblings ...)
  2026-03-05 19:23 ` ✗ CI.checkpatch: warning for drm/{i915,xe}: sort out step enums between the drivers Patchwork
@ 2026-03-05 19:25 ` Patchwork
  2026-03-05 20:11 ` ✓ Xe.CI.BAT: " Patchwork
  2026-03-06 11:35 ` ✗ Xe.CI.FULL: failure " Patchwork
  11 siblings, 0 replies; 22+ messages in thread
From: Patchwork @ 2026-03-05 19:25 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

== Series Details ==

Series: drm/{i915,xe}: sort out step enums between the drivers
URL   : https://patchwork.freedesktop.org/series/162594/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[19:23:40] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[19:23:45] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[19:24:23] Starting KUnit Kernel (1/1)...
[19:24:23] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[19:24:23] ================== guc_buf (11 subtests) ===================
[19:24:23] [PASSED] test_smallest
[19:24:23] [PASSED] test_largest
[19:24:23] [PASSED] test_granular
[19:24:23] [PASSED] test_unique
[19:24:23] [PASSED] test_overlap
[19:24:23] [PASSED] test_reusable
[19:24:23] [PASSED] test_too_big
[19:24:23] [PASSED] test_flush
[19:24:23] [PASSED] test_lookup
[19:24:23] [PASSED] test_data
[19:24:23] [PASSED] test_class
[19:24:23] ===================== [PASSED] guc_buf =====================
[19:24:23] =================== guc_dbm (7 subtests) ===================
[19:24:23] [PASSED] test_empty
[19:24:23] [PASSED] test_default
[19:24:23] ======================== test_size  ========================
[19:24:23] [PASSED] 4
[19:24:23] [PASSED] 8
[19:24:23] [PASSED] 32
[19:24:23] [PASSED] 256
[19:24:23] ==================== [PASSED] test_size ====================
[19:24:23] ======================= test_reuse  ========================
[19:24:23] [PASSED] 4
[19:24:23] [PASSED] 8
[19:24:23] [PASSED] 32
[19:24:23] [PASSED] 256
[19:24:23] =================== [PASSED] test_reuse ====================
[19:24:23] =================== test_range_overlap  ====================
[19:24:23] [PASSED] 4
[19:24:23] [PASSED] 8
[19:24:23] [PASSED] 32
[19:24:23] [PASSED] 256
[19:24:23] =============== [PASSED] test_range_overlap ================
[19:24:23] =================== test_range_compact  ====================
[19:24:23] [PASSED] 4
[19:24:23] [PASSED] 8
[19:24:23] [PASSED] 32
[19:24:23] [PASSED] 256
[19:24:23] =============== [PASSED] test_range_compact ================
[19:24:23] ==================== test_range_spare  =====================
[19:24:23] [PASSED] 4
[19:24:23] [PASSED] 8
[19:24:23] [PASSED] 32
[19:24:23] [PASSED] 256
[19:24:23] ================ [PASSED] test_range_spare =================
[19:24:23] ===================== [PASSED] guc_dbm =====================
[19:24:23] =================== guc_idm (6 subtests) ===================
[19:24:23] [PASSED] bad_init
[19:24:23] [PASSED] no_init
[19:24:23] [PASSED] init_fini
[19:24:23] [PASSED] check_used
[19:24:23] [PASSED] check_quota
[19:24:23] [PASSED] check_all
[19:24:23] ===================== [PASSED] guc_idm =====================
[19:24:23] ================== no_relay (3 subtests) ===================
[19:24:23] [PASSED] xe_drops_guc2pf_if_not_ready
[19:24:23] [PASSED] xe_drops_guc2vf_if_not_ready
[19:24:23] [PASSED] xe_rejects_send_if_not_ready
[19:24:23] ==================== [PASSED] no_relay =====================
[19:24:23] ================== pf_relay (14 subtests) ==================
[19:24:23] [PASSED] pf_rejects_guc2pf_too_short
[19:24:23] [PASSED] pf_rejects_guc2pf_too_long
[19:24:23] [PASSED] pf_rejects_guc2pf_no_payload
[19:24:23] [PASSED] pf_fails_no_payload
[19:24:23] [PASSED] pf_fails_bad_origin
[19:24:23] [PASSED] pf_fails_bad_type
[19:24:23] [PASSED] pf_txn_reports_error
[19:24:23] [PASSED] pf_txn_sends_pf2guc
[19:24:23] [PASSED] pf_sends_pf2guc
[19:24:23] [SKIPPED] pf_loopback_nop
[19:24:23] [SKIPPED] pf_loopback_echo
[19:24:23] [SKIPPED] pf_loopback_fail
[19:24:23] [SKIPPED] pf_loopback_busy
[19:24:23] [SKIPPED] pf_loopback_retry
[19:24:23] ==================== [PASSED] pf_relay =====================
[19:24:23] ================== vf_relay (3 subtests) ===================
[19:24:23] [PASSED] vf_rejects_guc2vf_too_short
[19:24:23] [PASSED] vf_rejects_guc2vf_too_long
[19:24:23] [PASSED] vf_rejects_guc2vf_no_payload
[19:24:23] ==================== [PASSED] vf_relay =====================
[19:24:23] ================ pf_gt_config (9 subtests) =================
[19:24:23] [PASSED] fair_contexts_1vf
[19:24:23] [PASSED] fair_doorbells_1vf
[19:24:23] [PASSED] fair_ggtt_1vf
[19:24:23] ====================== fair_vram_1vf  ======================
[19:24:23] [PASSED] 3.50 GiB
[19:24:23] [PASSED] 11.5 GiB
[19:24:23] [PASSED] 15.5 GiB
[19:24:23] [PASSED] 31.5 GiB
[19:24:23] [PASSED] 63.5 GiB
[19:24:23] [PASSED] 1.91 GiB
[19:24:23] ================== [PASSED] fair_vram_1vf ==================
[19:24:23] ================ fair_vram_1vf_admin_only  =================
[19:24:23] [PASSED] 3.50 GiB
[19:24:23] [PASSED] 11.5 GiB
[19:24:23] [PASSED] 15.5 GiB
[19:24:23] [PASSED] 31.5 GiB
[19:24:23] [PASSED] 63.5 GiB
[19:24:23] [PASSED] 1.91 GiB
[19:24:23] ============ [PASSED] fair_vram_1vf_admin_only =============
[19:24:23] ====================== fair_contexts  ======================
[19:24:23] [PASSED] 1 VF
[19:24:23] [PASSED] 2 VFs
[19:24:23] [PASSED] 3 VFs
[19:24:23] [PASSED] 4 VFs
[19:24:23] [PASSED] 5 VFs
[19:24:23] [PASSED] 6 VFs
[19:24:23] [PASSED] 7 VFs
[19:24:23] [PASSED] 8 VFs
[19:24:23] [PASSED] 9 VFs
[19:24:23] [PASSED] 10 VFs
[19:24:23] [PASSED] 11 VFs
[19:24:23] [PASSED] 12 VFs
[19:24:23] [PASSED] 13 VFs
[19:24:23] [PASSED] 14 VFs
[19:24:23] [PASSED] 15 VFs
[19:24:23] [PASSED] 16 VFs
[19:24:23] [PASSED] 17 VFs
[19:24:23] [PASSED] 18 VFs
[19:24:23] [PASSED] 19 VFs
[19:24:23] [PASSED] 20 VFs
[19:24:23] [PASSED] 21 VFs
[19:24:23] [PASSED] 22 VFs
[19:24:23] [PASSED] 23 VFs
[19:24:23] [PASSED] 24 VFs
[19:24:23] [PASSED] 25 VFs
[19:24:23] [PASSED] 26 VFs
[19:24:23] [PASSED] 27 VFs
[19:24:23] [PASSED] 28 VFs
[19:24:23] [PASSED] 29 VFs
[19:24:23] [PASSED] 30 VFs
[19:24:23] [PASSED] 31 VFs
[19:24:23] [PASSED] 32 VFs
[19:24:23] [PASSED] 33 VFs
[19:24:23] [PASSED] 34 VFs
[19:24:23] [PASSED] 35 VFs
[19:24:23] [PASSED] 36 VFs
[19:24:23] [PASSED] 37 VFs
[19:24:23] [PASSED] 38 VFs
[19:24:23] [PASSED] 39 VFs
[19:24:23] [PASSED] 40 VFs
[19:24:23] [PASSED] 41 VFs
[19:24:23] [PASSED] 42 VFs
[19:24:23] [PASSED] 43 VFs
[19:24:23] [PASSED] 44 VFs
[19:24:23] [PASSED] 45 VFs
[19:24:23] [PASSED] 46 VFs
[19:24:23] [PASSED] 47 VFs
[19:24:23] [PASSED] 48 VFs
[19:24:23] [PASSED] 49 VFs
[19:24:23] [PASSED] 50 VFs
[19:24:23] [PASSED] 51 VFs
[19:24:23] [PASSED] 52 VFs
[19:24:23] [PASSED] 53 VFs
[19:24:23] [PASSED] 54 VFs
[19:24:23] [PASSED] 55 VFs
[19:24:23] [PASSED] 56 VFs
[19:24:23] [PASSED] 57 VFs
[19:24:23] [PASSED] 58 VFs
[19:24:23] [PASSED] 59 VFs
[19:24:23] [PASSED] 60 VFs
[19:24:23] [PASSED] 61 VFs
[19:24:23] [PASSED] 62 VFs
[19:24:23] [PASSED] 63 VFs
[19:24:23] ================== [PASSED] fair_contexts ==================
[19:24:23] ===================== fair_doorbells  ======================
[19:24:23] [PASSED] 1 VF
[19:24:23] [PASSED] 2 VFs
[19:24:23] [PASSED] 3 VFs
[19:24:23] [PASSED] 4 VFs
[19:24:23] [PASSED] 5 VFs
[19:24:23] [PASSED] 6 VFs
[19:24:23] [PASSED] 7 VFs
[19:24:23] [PASSED] 8 VFs
[19:24:23] [PASSED] 9 VFs
[19:24:23] [PASSED] 10 VFs
[19:24:23] [PASSED] 11 VFs
[19:24:23] [PASSED] 12 VFs
[19:24:23] [PASSED] 13 VFs
[19:24:23] [PASSED] 14 VFs
[19:24:23] [PASSED] 15 VFs
[19:24:23] [PASSED] 16 VFs
[19:24:23] [PASSED] 17 VFs
[19:24:23] [PASSED] 18 VFs
[19:24:23] [PASSED] 19 VFs
[19:24:23] [PASSED] 20 VFs
[19:24:23] [PASSED] 21 VFs
[19:24:23] [PASSED] 22 VFs
[19:24:23] [PASSED] 23 VFs
[19:24:23] [PASSED] 24 VFs
[19:24:23] [PASSED] 25 VFs
[19:24:23] [PASSED] 26 VFs
[19:24:23] [PASSED] 27 VFs
[19:24:23] [PASSED] 28 VFs
[19:24:23] [PASSED] 29 VFs
[19:24:23] [PASSED] 30 VFs
[19:24:23] [PASSED] 31 VFs
[19:24:23] [PASSED] 32 VFs
[19:24:23] [PASSED] 33 VFs
[19:24:23] [PASSED] 34 VFs
[19:24:23] [PASSED] 35 VFs
[19:24:23] [PASSED] 36 VFs
[19:24:23] [PASSED] 37 VFs
[19:24:23] [PASSED] 38 VFs
[19:24:23] [PASSED] 39 VFs
[19:24:23] [PASSED] 40 VFs
[19:24:23] [PASSED] 41 VFs
[19:24:23] [PASSED] 42 VFs
[19:24:23] [PASSED] 43 VFs
[19:24:23] [PASSED] 44 VFs
[19:24:23] [PASSED] 45 VFs
[19:24:23] [PASSED] 46 VFs
[19:24:23] [PASSED] 47 VFs
[19:24:23] [PASSED] 48 VFs
[19:24:23] [PASSED] 49 VFs
[19:24:23] [PASSED] 50 VFs
[19:24:23] [PASSED] 51 VFs
[19:24:23] [PASSED] 52 VFs
[19:24:23] [PASSED] 53 VFs
[19:24:23] [PASSED] 54 VFs
[19:24:23] [PASSED] 55 VFs
[19:24:23] [PASSED] 56 VFs
[19:24:23] [PASSED] 57 VFs
[19:24:23] [PASSED] 58 VFs
[19:24:23] [PASSED] 59 VFs
[19:24:23] [PASSED] 60 VFs
[19:24:23] [PASSED] 61 VFs
[19:24:23] [PASSED] 62 VFs
[19:24:23] [PASSED] 63 VFs
[19:24:23] ================= [PASSED] fair_doorbells ==================
[19:24:23] ======================== fair_ggtt  ========================
[19:24:23] [PASSED] 1 VF
[19:24:23] [PASSED] 2 VFs
[19:24:23] [PASSED] 3 VFs
[19:24:23] [PASSED] 4 VFs
[19:24:23] [PASSED] 5 VFs
[19:24:23] [PASSED] 6 VFs
[19:24:23] [PASSED] 7 VFs
[19:24:23] [PASSED] 8 VFs
[19:24:23] [PASSED] 9 VFs
[19:24:23] [PASSED] 10 VFs
[19:24:23] [PASSED] 11 VFs
[19:24:23] [PASSED] 12 VFs
[19:24:23] [PASSED] 13 VFs
[19:24:23] [PASSED] 14 VFs
[19:24:23] [PASSED] 15 VFs
[19:24:23] [PASSED] 16 VFs
[19:24:23] [PASSED] 17 VFs
[19:24:23] [PASSED] 18 VFs
[19:24:23] [PASSED] 19 VFs
[19:24:23] [PASSED] 20 VFs
[19:24:23] [PASSED] 21 VFs
[19:24:23] [PASSED] 22 VFs
[19:24:23] [PASSED] 23 VFs
[19:24:23] [PASSED] 24 VFs
[19:24:23] [PASSED] 25 VFs
[19:24:23] [PASSED] 26 VFs
[19:24:23] [PASSED] 27 VFs
[19:24:23] [PASSED] 28 VFs
[19:24:23] [PASSED] 29 VFs
[19:24:23] [PASSED] 30 VFs
[19:24:23] [PASSED] 31 VFs
[19:24:23] [PASSED] 32 VFs
[19:24:23] [PASSED] 33 VFs
[19:24:23] [PASSED] 34 VFs
[19:24:23] [PASSED] 35 VFs
[19:24:23] [PASSED] 36 VFs
[19:24:23] [PASSED] 37 VFs
[19:24:23] [PASSED] 38 VFs
[19:24:23] [PASSED] 39 VFs
[19:24:23] [PASSED] 40 VFs
[19:24:23] [PASSED] 41 VFs
[19:24:23] [PASSED] 42 VFs
[19:24:23] [PASSED] 43 VFs
[19:24:23] [PASSED] 44 VFs
[19:24:23] [PASSED] 45 VFs
[19:24:23] [PASSED] 46 VFs
[19:24:23] [PASSED] 47 VFs
[19:24:23] [PASSED] 48 VFs
[19:24:23] [PASSED] 49 VFs
[19:24:23] [PASSED] 50 VFs
[19:24:23] [PASSED] 51 VFs
[19:24:23] [PASSED] 52 VFs
[19:24:23] [PASSED] 53 VFs
[19:24:23] [PASSED] 54 VFs
[19:24:23] [PASSED] 55 VFs
[19:24:23] [PASSED] 56 VFs
[19:24:23] [PASSED] 57 VFs
[19:24:23] [PASSED] 58 VFs
[19:24:23] [PASSED] 59 VFs
[19:24:23] [PASSED] 60 VFs
[19:24:23] [PASSED] 61 VFs
[19:24:23] [PASSED] 62 VFs
[19:24:23] [PASSED] 63 VFs
[19:24:23] ==================== [PASSED] fair_ggtt ====================
[19:24:23] ======================== fair_vram  ========================
[19:24:23] [PASSED] 1 VF
[19:24:23] [PASSED] 2 VFs
[19:24:23] [PASSED] 3 VFs
[19:24:23] [PASSED] 4 VFs
[19:24:23] [PASSED] 5 VFs
[19:24:23] [PASSED] 6 VFs
[19:24:23] [PASSED] 7 VFs
[19:24:23] [PASSED] 8 VFs
[19:24:23] [PASSED] 9 VFs
[19:24:23] [PASSED] 10 VFs
[19:24:23] [PASSED] 11 VFs
[19:24:23] [PASSED] 12 VFs
[19:24:23] [PASSED] 13 VFs
[19:24:23] [PASSED] 14 VFs
[19:24:23] [PASSED] 15 VFs
[19:24:23] [PASSED] 16 VFs
[19:24:23] [PASSED] 17 VFs
[19:24:23] [PASSED] 18 VFs
[19:24:23] [PASSED] 19 VFs
[19:24:23] [PASSED] 20 VFs
[19:24:23] [PASSED] 21 VFs
[19:24:23] [PASSED] 22 VFs
[19:24:23] [PASSED] 23 VFs
[19:24:23] [PASSED] 24 VFs
[19:24:23] [PASSED] 25 VFs
[19:24:23] [PASSED] 26 VFs
[19:24:23] [PASSED] 27 VFs
[19:24:23] [PASSED] 28 VFs
[19:24:23] [PASSED] 29 VFs
[19:24:23] [PASSED] 30 VFs
[19:24:23] [PASSED] 31 VFs
[19:24:23] [PASSED] 32 VFs
[19:24:23] [PASSED] 33 VFs
[19:24:23] [PASSED] 34 VFs
[19:24:23] [PASSED] 35 VFs
[19:24:23] [PASSED] 36 VFs
[19:24:23] [PASSED] 37 VFs
[19:24:23] [PASSED] 38 VFs
[19:24:23] [PASSED] 39 VFs
[19:24:23] [PASSED] 40 VFs
[19:24:23] [PASSED] 41 VFs
[19:24:23] [PASSED] 42 VFs
[19:24:23] [PASSED] 43 VFs
[19:24:23] [PASSED] 44 VFs
[19:24:23] [PASSED] 45 VFs
[19:24:23] [PASSED] 46 VFs
[19:24:23] [PASSED] 47 VFs
[19:24:23] [PASSED] 48 VFs
[19:24:23] [PASSED] 49 VFs
[19:24:23] [PASSED] 50 VFs
[19:24:23] [PASSED] 51 VFs
[19:24:23] [PASSED] 52 VFs
[19:24:23] [PASSED] 53 VFs
[19:24:23] [PASSED] 54 VFs
[19:24:23] [PASSED] 55 VFs
[19:24:23] [PASSED] 56 VFs
[19:24:23] [PASSED] 57 VFs
[19:24:23] [PASSED] 58 VFs
[19:24:23] [PASSED] 59 VFs
[19:24:23] [PASSED] 60 VFs
[19:24:23] [PASSED] 61 VFs
[19:24:23] [PASSED] 62 VFs
[19:24:23] [PASSED] 63 VFs
[19:24:23] ==================== [PASSED] fair_vram ====================
[19:24:23] ================== [PASSED] pf_gt_config ===================
[19:24:23] ===================== lmtt (1 subtest) =====================
[19:24:23] ======================== test_ops  =========================
[19:24:23] [PASSED] 2-level
[19:24:23] [PASSED] multi-level
[19:24:23] ==================== [PASSED] test_ops =====================
[19:24:23] ====================== [PASSED] lmtt =======================
[19:24:23] ================= pf_service (11 subtests) =================
[19:24:23] [PASSED] pf_negotiate_any
[19:24:23] [PASSED] pf_negotiate_base_match
[19:24:23] [PASSED] pf_negotiate_base_newer
[19:24:23] [PASSED] pf_negotiate_base_next
[19:24:23] [SKIPPED] pf_negotiate_base_older
[19:24:23] [PASSED] pf_negotiate_base_prev
[19:24:23] [PASSED] pf_negotiate_latest_match
[19:24:23] [PASSED] pf_negotiate_latest_newer
[19:24:23] [PASSED] pf_negotiate_latest_next
[19:24:23] [SKIPPED] pf_negotiate_latest_older
[19:24:23] [SKIPPED] pf_negotiate_latest_prev
[19:24:23] =================== [PASSED] pf_service ====================
[19:24:23] ================= xe_guc_g2g (2 subtests) ==================
[19:24:23] ============== xe_live_guc_g2g_kunit_default  ==============
[19:24:23] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[19:24:23] ============== xe_live_guc_g2g_kunit_allmem  ===============
[19:24:23] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[19:24:23] =================== [SKIPPED] xe_guc_g2g ===================
[19:24:23] =================== xe_mocs (2 subtests) ===================
[19:24:23] ================ xe_live_mocs_kernel_kunit  ================
[19:24:23] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[19:24:23] ================ xe_live_mocs_reset_kunit  =================
[19:24:23] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[19:24:23] ==================== [SKIPPED] xe_mocs =====================
[19:24:23] ================= xe_migrate (2 subtests) ==================
[19:24:23] ================= xe_migrate_sanity_kunit  =================
[19:24:23] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[19:24:23] ================== xe_validate_ccs_kunit  ==================
[19:24:23] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[19:24:23] =================== [SKIPPED] xe_migrate ===================
[19:24:23] ================== xe_dma_buf (1 subtest) ==================
[19:24:23] ==================== xe_dma_buf_kunit  =====================
[19:24:23] ================ [SKIPPED] xe_dma_buf_kunit ================
[19:24:23] =================== [SKIPPED] xe_dma_buf ===================
[19:24:23] ================= xe_bo_shrink (1 subtest) =================
[19:24:23] =================== xe_bo_shrink_kunit  ====================
[19:24:23] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[19:24:23] ================== [SKIPPED] xe_bo_shrink ==================
[19:24:23] ==================== xe_bo (2 subtests) ====================
[19:24:23] ================== xe_ccs_migrate_kunit  ===================
[19:24:23] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[19:24:23] ==================== xe_bo_evict_kunit  ====================
[19:24:23] =============== [SKIPPED] xe_bo_evict_kunit ================
[19:24:23] ===================== [SKIPPED] xe_bo ======================
[19:24:23] ==================== args (13 subtests) ====================
[19:24:23] [PASSED] count_args_test
[19:24:23] [PASSED] call_args_example
[19:24:23] [PASSED] call_args_test
[19:24:23] [PASSED] drop_first_arg_example
[19:24:23] [PASSED] drop_first_arg_test
[19:24:23] [PASSED] first_arg_example
[19:24:23] [PASSED] first_arg_test
[19:24:23] [PASSED] last_arg_example
[19:24:23] [PASSED] last_arg_test
[19:24:23] [PASSED] pick_arg_example
[19:24:23] [PASSED] if_args_example
[19:24:23] [PASSED] if_args_test
[19:24:23] [PASSED] sep_comma_example
[19:24:23] ====================== [PASSED] args =======================
[19:24:23] =================== xe_pci (3 subtests) ====================
[19:24:23] ==================== check_graphics_ip  ====================
[19:24:23] [PASSED] 12.00 Xe_LP
[19:24:23] [PASSED] 12.10 Xe_LP+
[19:24:23] [PASSED] 12.55 Xe_HPG
[19:24:23] [PASSED] 12.60 Xe_HPC
[19:24:23] [PASSED] 12.70 Xe_LPG
[19:24:23] [PASSED] 12.71 Xe_LPG
[19:24:23] [PASSED] 12.74 Xe_LPG+
[19:24:23] [PASSED] 20.01 Xe2_HPG
[19:24:23] [PASSED] 20.02 Xe2_HPG
[19:24:23] [PASSED] 20.04 Xe2_LPG
[19:24:23] [PASSED] 30.00 Xe3_LPG
[19:24:23] [PASSED] 30.01 Xe3_LPG
[19:24:23] [PASSED] 30.03 Xe3_LPG
[19:24:23] [PASSED] 30.04 Xe3_LPG
[19:24:23] [PASSED] 30.05 Xe3_LPG
[19:24:23] [PASSED] 35.10 Xe3p_LPG
[19:24:23] [PASSED] 35.11 Xe3p_XPC
[19:24:23] ================ [PASSED] check_graphics_ip ================
[19:24:23] ===================== check_media_ip  ======================
[19:24:23] [PASSED] 12.00 Xe_M
[19:24:23] [PASSED] 12.55 Xe_HPM
[19:24:23] [PASSED] 13.00 Xe_LPM+
[19:24:23] [PASSED] 13.01 Xe2_HPM
[19:24:23] [PASSED] 20.00 Xe2_LPM
[19:24:23] [PASSED] 30.00 Xe3_LPM
[19:24:23] [PASSED] 30.02 Xe3_LPM
[19:24:23] [PASSED] 35.00 Xe3p_LPM
[19:24:23] [PASSED] 35.03 Xe3p_HPM
[19:24:23] ================= [PASSED] check_media_ip ==================
[19:24:23] =================== check_platform_desc  ===================
[19:24:23] [PASSED] 0x9A60 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A68 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A70 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A40 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A49 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A59 (TIGERLAKE)
[19:24:23] [PASSED] 0x9A78 (TIGERLAKE)
[19:24:23] [PASSED] 0x9AC0 (TIGERLAKE)
[19:24:23] [PASSED] 0x9AC9 (TIGERLAKE)
[19:24:23] [PASSED] 0x9AD9 (TIGERLAKE)
[19:24:23] [PASSED] 0x9AF8 (TIGERLAKE)
[19:24:23] [PASSED] 0x4C80 (ROCKETLAKE)
[19:24:23] [PASSED] 0x4C8A (ROCKETLAKE)
[19:24:23] [PASSED] 0x4C8B (ROCKETLAKE)
[19:24:23] [PASSED] 0x4C8C (ROCKETLAKE)
[19:24:23] [PASSED] 0x4C90 (ROCKETLAKE)
[19:24:23] [PASSED] 0x4C9A (ROCKETLAKE)
[19:24:23] [PASSED] 0x4680 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4682 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4688 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x468A (ALDERLAKE_S)
[19:24:23] [PASSED] 0x468B (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4690 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4692 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4693 (ALDERLAKE_S)
[19:24:23] [PASSED] 0x46A0 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46A1 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46A2 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46A3 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46A6 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46A8 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46AA (ALDERLAKE_P)
[19:24:23] [PASSED] 0x462A (ALDERLAKE_P)
[19:24:23] [PASSED] 0x4626 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x4628 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46B0 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46B1 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46B2 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46B3 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46C0 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46C1 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46C2 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46C3 (ALDERLAKE_P)
[19:24:23] [PASSED] 0x46D0 (ALDERLAKE_N)
[19:24:23] [PASSED] 0x46D1 (ALDERLAKE_N)
[19:24:23] [PASSED] 0x46D2 (ALDERLAKE_N)
[19:24:23] [PASSED] 0x46D3 (ALDERLAKE_N)
[19:24:23] [PASSED] 0x46D4 (ALDERLAKE_N)
[19:24:23] [PASSED] 0xA721 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7A1 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7A9 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7AC (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7AD (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA720 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7A0 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7A8 (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7AA (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA7AB (ALDERLAKE_P)
[19:24:23] [PASSED] 0xA780 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA781 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA782 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA783 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA788 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA789 (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA78A (ALDERLAKE_S)
[19:24:23] [PASSED] 0xA78B (ALDERLAKE_S)
[19:24:23] [PASSED] 0x4905 (DG1)
[19:24:23] [PASSED] 0x4906 (DG1)
[19:24:23] [PASSED] 0x4907 (DG1)
[19:24:23] [PASSED] 0x4908 (DG1)
[19:24:23] [PASSED] 0x4909 (DG1)
[19:24:23] [PASSED] 0x56C0 (DG2)
[19:24:23] [PASSED] 0x56C2 (DG2)
[19:24:23] [PASSED] 0x56C1 (DG2)
[19:24:23] [PASSED] 0x7D51 (METEORLAKE)
[19:24:23] [PASSED] 0x7DD1 (METEORLAKE)
[19:24:23] [PASSED] 0x7D41 (METEORLAKE)
[19:24:23] [PASSED] 0x7D67 (METEORLAKE)
[19:24:23] [PASSED] 0xB640 (METEORLAKE)
[19:24:23] [PASSED] 0x56A0 (DG2)
[19:24:23] [PASSED] 0x56A1 (DG2)
[19:24:23] [PASSED] 0x56A2 (DG2)
[19:24:23] [PASSED] 0x56BE (DG2)
[19:24:23] [PASSED] 0x56BF (DG2)
[19:24:23] [PASSED] 0x5690 (DG2)
[19:24:23] [PASSED] 0x5691 (DG2)
[19:24:23] [PASSED] 0x5692 (DG2)
[19:24:23] [PASSED] 0x56A5 (DG2)
[19:24:23] [PASSED] 0x56A6 (DG2)
[19:24:23] [PASSED] 0x56B0 (DG2)
[19:24:23] [PASSED] 0x56B1 (DG2)
[19:24:23] [PASSED] 0x56BA (DG2)
[19:24:23] [PASSED] 0x56BB (DG2)
[19:24:23] [PASSED] 0x56BC (DG2)
[19:24:23] [PASSED] 0x56BD (DG2)
[19:24:23] [PASSED] 0x5693 (DG2)
[19:24:23] [PASSED] 0x5694 (DG2)
[19:24:23] [PASSED] 0x5695 (DG2)
[19:24:23] [PASSED] 0x56A3 (DG2)
[19:24:23] [PASSED] 0x56A4 (DG2)
[19:24:23] [PASSED] 0x56B2 (DG2)
[19:24:23] [PASSED] 0x56B3 (DG2)
[19:24:23] [PASSED] 0x5696 (DG2)
[19:24:23] [PASSED] 0x5697 (DG2)
[19:24:23] [PASSED] 0xB69 (PVC)
[19:24:23] [PASSED] 0xB6E (PVC)
[19:24:23] [PASSED] 0xBD4 (PVC)
[19:24:23] [PASSED] 0xBD5 (PVC)
[19:24:23] [PASSED] 0xBD6 (PVC)
[19:24:23] [PASSED] 0xBD7 (PVC)
[19:24:23] [PASSED] 0xBD8 (PVC)
[19:24:23] [PASSED] 0xBD9 (PVC)
[19:24:23] [PASSED] 0xBDA (PVC)
[19:24:23] [PASSED] 0xBDB (PVC)
[19:24:23] [PASSED] 0xBE0 (PVC)
[19:24:23] [PASSED] 0xBE1 (PVC)
[19:24:23] [PASSED] 0xBE5 (PVC)
[19:24:23] [PASSED] 0x7D40 (METEORLAKE)
[19:24:23] [PASSED] 0x7D45 (METEORLAKE)
[19:24:23] [PASSED] 0x7D55 (METEORLAKE)
[19:24:23] [PASSED] 0x7D60 (METEORLAKE)
[19:24:23] [PASSED] 0x7DD5 (METEORLAKE)
[19:24:23] [PASSED] 0x6420 (LUNARLAKE)
[19:24:23] [PASSED] 0x64A0 (LUNARLAKE)
[19:24:23] [PASSED] 0x64B0 (LUNARLAKE)
[19:24:23] [PASSED] 0xE202 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE209 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE20B (BATTLEMAGE)
[19:24:23] [PASSED] 0xE20C (BATTLEMAGE)
[19:24:23] [PASSED] 0xE20D (BATTLEMAGE)
[19:24:23] [PASSED] 0xE210 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE211 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE212 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE216 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE220 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE221 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE222 (BATTLEMAGE)
[19:24:23] [PASSED] 0xE223 (BATTLEMAGE)
[19:24:23] [PASSED] 0xB080 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB081 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB082 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB083 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB084 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB085 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB086 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB087 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB08F (PANTHERLAKE)
[19:24:23] [PASSED] 0xB090 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB0A0 (PANTHERLAKE)
[19:24:23] [PASSED] 0xB0B0 (PANTHERLAKE)
[19:24:23] [PASSED] 0xFD80 (PANTHERLAKE)
[19:24:23] [PASSED] 0xFD81 (PANTHERLAKE)
[19:24:23] [PASSED] 0xD740 (NOVALAKE_S)
[19:24:23] [PASSED] 0xD741 (NOVALAKE_S)
[19:24:23] [PASSED] 0xD742 (NOVALAKE_S)
[19:24:23] [PASSED] 0xD743 (NOVALAKE_S)
[19:24:23] [PASSED] 0xD744 (NOVALAKE_S)
[19:24:23] [PASSED] 0xD745 (NOVALAKE_S)
[19:24:23] [PASSED] 0x674C (CRESCENTISLAND)
[19:24:23] [PASSED] 0xD750 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD751 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD752 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD753 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD754 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD755 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD756 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD757 (NOVALAKE_P)
[19:24:23] [PASSED] 0xD75F (NOVALAKE_P)
[19:24:23] =============== [PASSED] check_platform_desc ===============
[19:24:23] ===================== [PASSED] xe_pci ======================
[19:24:23] =================== xe_rtp (2 subtests) ====================
[19:24:23] =============== xe_rtp_process_to_sr_tests  ================
[19:24:23] [PASSED] coalesce-same-reg
[19:24:23] [PASSED] no-match-no-add
[19:24:23] [PASSED] match-or
[19:24:23] [PASSED] match-or-xfail
[19:24:23] [PASSED] no-match-no-add-multiple-rules
[19:24:23] [PASSED] two-regs-two-entries
[19:24:23] [PASSED] clr-one-set-other
[19:24:23] [PASSED] set-field
[19:24:23] [PASSED] conflict-duplicate
stty: 'standard input': Inappropriate ioctl for device
[19:24:23] [PASSED] conflict-not-disjoint
[19:24:23] [PASSED] conflict-reg-type
[19:24:23] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[19:24:23] ================== xe_rtp_process_tests  ===================
[19:24:23] [PASSED] active1
[19:24:23] [PASSED] active2
[19:24:23] [PASSED] active-inactive
[19:24:23] [PASSED] inactive-active
[19:24:23] [PASSED] inactive-1st_or_active-inactive
[19:24:23] [PASSED] inactive-2nd_or_active-inactive
[19:24:23] [PASSED] inactive-last_or_active-inactive
[19:24:23] [PASSED] inactive-no_or_active-inactive
[19:24:23] ============== [PASSED] xe_rtp_process_tests ===============
[19:24:23] ===================== [PASSED] xe_rtp ======================
[19:24:23] ==================== xe_wa (1 subtest) =====================
[19:24:23] ======================== xe_wa_gt  =========================
[19:24:23] [PASSED] TIGERLAKE B0
[19:24:23] [PASSED] DG1 A0
[19:24:23] [PASSED] DG1 B0
[19:24:23] [PASSED] ALDERLAKE_S A0
[19:24:23] [PASSED] ALDERLAKE_S B0
[19:24:23] [PASSED] ALDERLAKE_S C0
[19:24:23] [PASSED] ALDERLAKE_S D0
[19:24:23] [PASSED] ALDERLAKE_P A0
[19:24:23] [PASSED] ALDERLAKE_P B0
[19:24:23] [PASSED] ALDERLAKE_P C0
[19:24:23] [PASSED] ALDERLAKE_S RPLS D0
[19:24:23] [PASSED] ALDERLAKE_P RPLU E0
[19:24:23] [PASSED] DG2 G10 C0
[19:24:23] [PASSED] DG2 G11 B1
[19:24:23] [PASSED] DG2 G12 A1
[19:24:23] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[19:24:23] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[19:24:23] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[19:24:23] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[19:24:23] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[19:24:23] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[19:24:23] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[19:24:23] ==================== [PASSED] xe_wa_gt =====================
[19:24:23] ====================== [PASSED] xe_wa ======================
[19:24:23] ============================================================
[19:24:23] Testing complete. Ran 597 tests: passed: 579, skipped: 18
[19:24:24] Elapsed time: 43.460s total, 4.759s configuring, 38.034s building, 0.614s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[19:24:24] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[19:24:25] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[19:24:54] Starting KUnit Kernel (1/1)...
[19:24:54] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[19:24:54] ============ drm_test_pick_cmdline (2 subtests) ============
[19:24:54] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[19:24:54] =============== drm_test_pick_cmdline_named  ===============
[19:24:54] [PASSED] NTSC
[19:24:54] [PASSED] NTSC-J
[19:24:54] [PASSED] PAL
[19:24:54] [PASSED] PAL-M
[19:24:54] =========== [PASSED] drm_test_pick_cmdline_named ===========
[19:24:54] ============== [PASSED] drm_test_pick_cmdline ==============
[19:24:54] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[19:24:54] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[19:24:54] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[19:24:54] =========== drm_validate_clone_mode (2 subtests) ===========
[19:24:54] ============== drm_test_check_in_clone_mode  ===============
[19:24:54] [PASSED] in_clone_mode
[19:24:54] [PASSED] not_in_clone_mode
[19:24:54] ========== [PASSED] drm_test_check_in_clone_mode ===========
[19:24:54] =============== drm_test_check_valid_clones  ===============
[19:24:54] [PASSED] not_in_clone_mode
[19:24:54] [PASSED] valid_clone
[19:24:54] [PASSED] invalid_clone
[19:24:54] =========== [PASSED] drm_test_check_valid_clones ===========
[19:24:54] ============= [PASSED] drm_validate_clone_mode =============
[19:24:54] ============= drm_validate_modeset (1 subtest) =============
[19:24:54] [PASSED] drm_test_check_connector_changed_modeset
[19:24:54] ============== [PASSED] drm_validate_modeset ===============
[19:24:54] ====== drm_test_bridge_get_current_state (2 subtests) ======
[19:24:54] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[19:24:54] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[19:24:54] ======== [PASSED] drm_test_bridge_get_current_state ========
[19:24:54] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[19:24:54] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[19:24:54] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[19:24:54] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[19:24:54] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[19:24:54] ============== drm_bridge_alloc (2 subtests) ===============
[19:24:54] [PASSED] drm_test_drm_bridge_alloc_basic
[19:24:54] [PASSED] drm_test_drm_bridge_alloc_get_put
[19:24:54] ================ [PASSED] drm_bridge_alloc =================
[19:24:54] ============= drm_cmdline_parser (40 subtests) =============
[19:24:54] [PASSED] drm_test_cmdline_force_d_only
[19:24:54] [PASSED] drm_test_cmdline_force_D_only_dvi
[19:24:54] [PASSED] drm_test_cmdline_force_D_only_hdmi
[19:24:54] [PASSED] drm_test_cmdline_force_D_only_not_digital
[19:24:54] [PASSED] drm_test_cmdline_force_e_only
[19:24:54] [PASSED] drm_test_cmdline_res
[19:24:54] [PASSED] drm_test_cmdline_res_vesa
[19:24:54] [PASSED] drm_test_cmdline_res_vesa_rblank
[19:24:54] [PASSED] drm_test_cmdline_res_rblank
[19:24:54] [PASSED] drm_test_cmdline_res_bpp
[19:24:54] [PASSED] drm_test_cmdline_res_refresh
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[19:24:54] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[19:24:54] [PASSED] drm_test_cmdline_res_margins_force_on
[19:24:54] [PASSED] drm_test_cmdline_res_vesa_margins
[19:24:54] [PASSED] drm_test_cmdline_name
[19:24:54] [PASSED] drm_test_cmdline_name_bpp
[19:24:54] [PASSED] drm_test_cmdline_name_option
[19:24:54] [PASSED] drm_test_cmdline_name_bpp_option
[19:24:54] [PASSED] drm_test_cmdline_rotate_0
[19:24:54] [PASSED] drm_test_cmdline_rotate_90
[19:24:54] [PASSED] drm_test_cmdline_rotate_180
[19:24:54] [PASSED] drm_test_cmdline_rotate_270
[19:24:54] [PASSED] drm_test_cmdline_hmirror
[19:24:54] [PASSED] drm_test_cmdline_vmirror
[19:24:54] [PASSED] drm_test_cmdline_margin_options
[19:24:54] [PASSED] drm_test_cmdline_multiple_options
[19:24:54] [PASSED] drm_test_cmdline_bpp_extra_and_option
[19:24:54] [PASSED] drm_test_cmdline_extra_and_option
[19:24:54] [PASSED] drm_test_cmdline_freestanding_options
[19:24:54] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[19:24:54] [PASSED] drm_test_cmdline_panel_orientation
[19:24:54] ================ drm_test_cmdline_invalid  =================
[19:24:54] [PASSED] margin_only
[19:24:54] [PASSED] interlace_only
[19:24:54] [PASSED] res_missing_x
[19:24:54] [PASSED] res_missing_y
[19:24:54] [PASSED] res_bad_y
[19:24:54] [PASSED] res_missing_y_bpp
[19:24:54] [PASSED] res_bad_bpp
[19:24:54] [PASSED] res_bad_refresh
[19:24:54] [PASSED] res_bpp_refresh_force_on_off
[19:24:54] [PASSED] res_invalid_mode
[19:24:54] [PASSED] res_bpp_wrong_place_mode
[19:24:54] [PASSED] name_bpp_refresh
[19:24:54] [PASSED] name_refresh
[19:24:54] [PASSED] name_refresh_wrong_mode
[19:24:54] [PASSED] name_refresh_invalid_mode
[19:24:54] [PASSED] rotate_multiple
[19:24:54] [PASSED] rotate_invalid_val
[19:24:54] [PASSED] rotate_truncated
[19:24:54] [PASSED] invalid_option
[19:24:54] [PASSED] invalid_tv_option
[19:24:54] [PASSED] truncated_tv_option
[19:24:54] ============ [PASSED] drm_test_cmdline_invalid =============
[19:24:54] =============== drm_test_cmdline_tv_options  ===============
[19:24:54] [PASSED] NTSC
[19:24:54] [PASSED] NTSC_443
[19:24:54] [PASSED] NTSC_J
[19:24:54] [PASSED] PAL
[19:24:54] [PASSED] PAL_M
[19:24:54] [PASSED] PAL_N
[19:24:54] [PASSED] SECAM
[19:24:54] [PASSED] MONO_525
[19:24:54] [PASSED] MONO_625
[19:24:54] =========== [PASSED] drm_test_cmdline_tv_options ===========
[19:24:54] =============== [PASSED] drm_cmdline_parser ================
[19:24:54] ========== drmm_connector_hdmi_init (20 subtests) ==========
[19:24:54] [PASSED] drm_test_connector_hdmi_init_valid
[19:24:54] [PASSED] drm_test_connector_hdmi_init_bpc_8
[19:24:54] [PASSED] drm_test_connector_hdmi_init_bpc_10
[19:24:54] [PASSED] drm_test_connector_hdmi_init_bpc_12
[19:24:54] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[19:24:54] [PASSED] drm_test_connector_hdmi_init_bpc_null
[19:24:54] [PASSED] drm_test_connector_hdmi_init_formats_empty
[19:24:54] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[19:24:54] === drm_test_connector_hdmi_init_formats_yuv420_allowed  ===
[19:24:54] [PASSED] supported_formats=0x9 yuv420_allowed=1
[19:24:54] [PASSED] supported_formats=0x9 yuv420_allowed=0
[19:24:54] [PASSED] supported_formats=0x3 yuv420_allowed=1
[19:24:54] [PASSED] supported_formats=0x3 yuv420_allowed=0
[19:24:54] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[19:24:54] [PASSED] drm_test_connector_hdmi_init_null_ddc
[19:24:54] [PASSED] drm_test_connector_hdmi_init_null_product
[19:24:54] [PASSED] drm_test_connector_hdmi_init_null_vendor
[19:24:54] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[19:24:54] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[19:24:54] [PASSED] drm_test_connector_hdmi_init_product_valid
[19:24:54] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[19:24:54] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[19:24:54] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[19:24:54] ========= drm_test_connector_hdmi_init_type_valid  =========
[19:24:54] [PASSED] HDMI-A
[19:24:54] [PASSED] HDMI-B
[19:24:54] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[19:24:54] ======== drm_test_connector_hdmi_init_type_invalid  ========
[19:24:54] [PASSED] Unknown
[19:24:54] [PASSED] VGA
[19:24:54] [PASSED] DVI-I
[19:24:54] [PASSED] DVI-D
[19:24:54] [PASSED] DVI-A
[19:24:54] [PASSED] Composite
[19:24:54] [PASSED] SVIDEO
[19:24:54] [PASSED] LVDS
[19:24:54] [PASSED] Component
[19:24:54] [PASSED] DIN
[19:24:54] [PASSED] DP
[19:24:54] [PASSED] TV
[19:24:54] [PASSED] eDP
[19:24:54] [PASSED] Virtual
[19:24:54] [PASSED] DSI
[19:24:54] [PASSED] DPI
[19:24:54] [PASSED] Writeback
[19:24:54] [PASSED] SPI
[19:24:54] [PASSED] USB
[19:24:54] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[19:24:54] ============ [PASSED] drmm_connector_hdmi_init =============
[19:24:54] ============= drmm_connector_init (3 subtests) =============
[19:24:54] [PASSED] drm_test_drmm_connector_init
[19:24:54] [PASSED] drm_test_drmm_connector_init_null_ddc
[19:24:54] ========= drm_test_drmm_connector_init_type_valid  =========
[19:24:54] [PASSED] Unknown
[19:24:54] [PASSED] VGA
[19:24:54] [PASSED] DVI-I
[19:24:54] [PASSED] DVI-D
[19:24:54] [PASSED] DVI-A
[19:24:54] [PASSED] Composite
[19:24:54] [PASSED] SVIDEO
[19:24:54] [PASSED] LVDS
[19:24:54] [PASSED] Component
[19:24:54] [PASSED] DIN
[19:24:54] [PASSED] DP
[19:24:54] [PASSED] HDMI-A
[19:24:54] [PASSED] HDMI-B
[19:24:54] [PASSED] TV
[19:24:54] [PASSED] eDP
[19:24:54] [PASSED] Virtual
[19:24:54] [PASSED] DSI
[19:24:54] [PASSED] DPI
[19:24:54] [PASSED] Writeback
[19:24:54] [PASSED] SPI
[19:24:54] [PASSED] USB
[19:24:54] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[19:24:54] =============== [PASSED] drmm_connector_init ===============
[19:24:54] ========= drm_connector_dynamic_init (6 subtests) ==========
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_init
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_init_properties
[19:24:54] ===== drm_test_drm_connector_dynamic_init_type_valid  ======
[19:24:54] [PASSED] Unknown
[19:24:54] [PASSED] VGA
[19:24:54] [PASSED] DVI-I
[19:24:54] [PASSED] DVI-D
[19:24:54] [PASSED] DVI-A
[19:24:54] [PASSED] Composite
[19:24:54] [PASSED] SVIDEO
[19:24:54] [PASSED] LVDS
[19:24:54] [PASSED] Component
[19:24:54] [PASSED] DIN
[19:24:54] [PASSED] DP
[19:24:54] [PASSED] HDMI-A
[19:24:54] [PASSED] HDMI-B
[19:24:54] [PASSED] TV
[19:24:54] [PASSED] eDP
[19:24:54] [PASSED] Virtual
[19:24:54] [PASSED] DSI
[19:24:54] [PASSED] DPI
[19:24:54] [PASSED] Writeback
[19:24:54] [PASSED] SPI
[19:24:54] [PASSED] USB
[19:24:54] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[19:24:54] ======== drm_test_drm_connector_dynamic_init_name  =========
[19:24:54] [PASSED] Unknown
[19:24:54] [PASSED] VGA
[19:24:54] [PASSED] DVI-I
[19:24:54] [PASSED] DVI-D
[19:24:54] [PASSED] DVI-A
[19:24:54] [PASSED] Composite
[19:24:54] [PASSED] SVIDEO
[19:24:54] [PASSED] LVDS
[19:24:54] [PASSED] Component
[19:24:54] [PASSED] DIN
[19:24:54] [PASSED] DP
[19:24:54] [PASSED] HDMI-A
[19:24:54] [PASSED] HDMI-B
[19:24:54] [PASSED] TV
[19:24:54] [PASSED] eDP
[19:24:54] [PASSED] Virtual
[19:24:54] [PASSED] DSI
[19:24:54] [PASSED] DPI
[19:24:54] [PASSED] Writeback
[19:24:54] [PASSED] SPI
[19:24:54] [PASSED] USB
[19:24:54] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[19:24:54] =========== [PASSED] drm_connector_dynamic_init ============
[19:24:54] ==== drm_connector_dynamic_register_early (4 subtests) =====
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[19:24:54] ====== [PASSED] drm_connector_dynamic_register_early =======
[19:24:54] ======= drm_connector_dynamic_register (7 subtests) ========
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[19:24:54] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[19:24:54] ========= [PASSED] drm_connector_dynamic_register ==========
[19:24:54] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[19:24:54] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[19:24:54] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[19:24:54] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[19:24:54] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[19:24:54] ========== drm_test_get_tv_mode_from_name_valid  ===========
[19:24:54] [PASSED] NTSC
[19:24:54] [PASSED] NTSC-443
[19:24:54] [PASSED] NTSC-J
[19:24:54] [PASSED] PAL
[19:24:54] [PASSED] PAL-M
[19:24:54] [PASSED] PAL-N
[19:24:54] [PASSED] SECAM
[19:24:54] [PASSED] Mono
[19:24:54] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[19:24:54] [PASSED] drm_test_get_tv_mode_from_name_truncated
[19:24:54] ============ [PASSED] drm_get_tv_mode_from_name ============
[19:24:54] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[19:24:54] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[19:24:54] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid  =
[19:24:54] [PASSED] VIC 96
[19:24:54] [PASSED] VIC 97
[19:24:54] [PASSED] VIC 101
[19:24:54] [PASSED] VIC 102
[19:24:54] [PASSED] VIC 106
[19:24:54] [PASSED] VIC 107
[19:24:54] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[19:24:54] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[19:24:54] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[19:24:54] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[19:24:54] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[19:24:54] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[19:24:54] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[19:24:54] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[19:24:54] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name  ====
[19:24:54] [PASSED] Automatic
[19:24:54] [PASSED] Full
[19:24:54] [PASSED] Limited 16:235
[19:24:54] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[19:24:54] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[19:24:54] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[19:24:54] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[19:24:54] === drm_test_drm_hdmi_connector_get_output_format_name  ====
[19:24:54] [PASSED] RGB
[19:24:54] [PASSED] YUV 4:2:0
[19:24:54] [PASSED] YUV 4:2:2
[19:24:54] [PASSED] YUV 4:4:4
[19:24:54] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[19:24:54] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[19:24:54] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[19:24:54] ============= drm_damage_helper (21 subtests) ==============
[19:24:54] [PASSED] drm_test_damage_iter_no_damage
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_src_moved
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_not_visible
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[19:24:54] [PASSED] drm_test_damage_iter_no_damage_no_fb
[19:24:54] [PASSED] drm_test_damage_iter_simple_damage
[19:24:54] [PASSED] drm_test_damage_iter_single_damage
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_outside_src
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_src_moved
[19:24:54] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[19:24:54] [PASSED] drm_test_damage_iter_damage
[19:24:54] [PASSED] drm_test_damage_iter_damage_one_intersect
[19:24:54] [PASSED] drm_test_damage_iter_damage_one_outside
[19:24:54] [PASSED] drm_test_damage_iter_damage_src_moved
[19:24:54] [PASSED] drm_test_damage_iter_damage_not_visible
[19:24:54] ================ [PASSED] drm_damage_helper ================
[19:24:54] ============== drm_dp_mst_helper (3 subtests) ==============
[19:24:54] ============== drm_test_dp_mst_calc_pbn_mode  ==============
[19:24:54] [PASSED] Clock 154000 BPP 30 DSC disabled
[19:24:54] [PASSED] Clock 234000 BPP 30 DSC disabled
[19:24:54] [PASSED] Clock 297000 BPP 24 DSC disabled
[19:24:54] [PASSED] Clock 332880 BPP 24 DSC enabled
[19:24:54] [PASSED] Clock 324540 BPP 24 DSC enabled
[19:24:54] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[19:24:54] ============== drm_test_dp_mst_calc_pbn_div  ===============
[19:24:54] [PASSED] Link rate 2000000 lane count 4
[19:24:54] [PASSED] Link rate 2000000 lane count 2
[19:24:54] [PASSED] Link rate 2000000 lane count 1
[19:24:54] [PASSED] Link rate 1350000 lane count 4
[19:24:54] [PASSED] Link rate 1350000 lane count 2
[19:24:54] [PASSED] Link rate 1350000 lane count 1
[19:24:54] [PASSED] Link rate 1000000 lane count 4
[19:24:54] [PASSED] Link rate 1000000 lane count 2
[19:24:54] [PASSED] Link rate 1000000 lane count 1
[19:24:54] [PASSED] Link rate 810000 lane count 4
[19:24:54] [PASSED] Link rate 810000 lane count 2
[19:24:54] [PASSED] Link rate 810000 lane count 1
[19:24:54] [PASSED] Link rate 540000 lane count 4
[19:24:54] [PASSED] Link rate 540000 lane count 2
[19:24:54] [PASSED] Link rate 540000 lane count 1
[19:24:54] [PASSED] Link rate 270000 lane count 4
[19:24:54] [PASSED] Link rate 270000 lane count 2
[19:24:54] [PASSED] Link rate 270000 lane count 1
[19:24:54] [PASSED] Link rate 162000 lane count 4
[19:24:54] [PASSED] Link rate 162000 lane count 2
[19:24:54] [PASSED] Link rate 162000 lane count 1
[19:24:54] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[19:24:54] ========= drm_test_dp_mst_sideband_msg_req_decode  =========
[19:24:54] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[19:24:54] [PASSED] DP_POWER_UP_PHY with port number
[19:24:54] [PASSED] DP_POWER_DOWN_PHY with port number
[19:24:54] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[19:24:54] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[19:24:54] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[19:24:54] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[19:24:54] [PASSED] DP_QUERY_PAYLOAD with port number
[19:24:54] [PASSED] DP_QUERY_PAYLOAD with VCPI
[19:24:54] [PASSED] DP_REMOTE_DPCD_READ with port number
[19:24:54] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[19:24:54] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[19:24:54] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[19:24:54] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[19:24:54] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[19:24:54] [PASSED] DP_REMOTE_I2C_READ with port number
[19:24:54] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[19:24:54] [PASSED] DP_REMOTE_I2C_READ with transactions array
[19:24:54] [PASSED] DP_REMOTE_I2C_WRITE with port number
[19:24:54] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[19:24:54] [PASSED] DP_REMOTE_I2C_WRITE with data array
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[19:24:54] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[19:24:54] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[19:24:54] ================ [PASSED] drm_dp_mst_helper ================
[19:24:54] ================== drm_exec (7 subtests) ===================
[19:24:54] [PASSED] sanitycheck
[19:24:54] [PASSED] test_lock
[19:24:54] [PASSED] test_lock_unlock
[19:24:54] [PASSED] test_duplicates
[19:24:54] [PASSED] test_prepare
[19:24:54] [PASSED] test_prepare_array
[19:24:54] [PASSED] test_multiple_loops
[19:24:54] ==================== [PASSED] drm_exec =====================
[19:24:54] =========== drm_format_helper_test (17 subtests) ===========
[19:24:54] ============== drm_test_fb_xrgb8888_to_gray8  ==============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[19:24:54] ============= drm_test_fb_xrgb8888_to_rgb332  ==============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[19:24:54] ============= drm_test_fb_xrgb8888_to_rgb565  ==============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[19:24:54] ============ drm_test_fb_xrgb8888_to_xrgb1555  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[19:24:54] ============ drm_test_fb_xrgb8888_to_argb1555  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[19:24:54] ============ drm_test_fb_xrgb8888_to_rgba5551  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[19:24:54] ============= drm_test_fb_xrgb8888_to_rgb888  ==============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[19:24:54] ============= drm_test_fb_xrgb8888_to_bgr888  ==============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[19:24:54] ============ drm_test_fb_xrgb8888_to_argb8888  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[19:24:54] =========== drm_test_fb_xrgb8888_to_xrgb2101010  ===========
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[19:24:54] =========== drm_test_fb_xrgb8888_to_argb2101010  ===========
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[19:24:54] ============== drm_test_fb_xrgb8888_to_mono  ===============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[19:24:54] ==================== drm_test_fb_swab  =====================
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ================ [PASSED] drm_test_fb_swab =================
[19:24:54] ============ drm_test_fb_xrgb8888_to_xbgr8888  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[19:24:54] ============ drm_test_fb_xrgb8888_to_abgr8888  =============
[19:24:54] [PASSED] single_pixel_source_buffer
[19:24:54] [PASSED] single_pixel_clip_rectangle
[19:24:54] [PASSED] well_known_colors
[19:24:54] [PASSED] destination_pitch
[19:24:54] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[19:24:54] ================= drm_test_fb_clip_offset  =================
[19:24:54] [PASSED] pass through
[19:24:54] [PASSED] horizontal offset
[19:24:54] [PASSED] vertical offset
[19:24:54] [PASSED] horizontal and vertical offset
[19:24:54] [PASSED] horizontal offset (custom pitch)
[19:24:54] [PASSED] vertical offset (custom pitch)
[19:24:54] [PASSED] horizontal and vertical offset (custom pitch)
[19:24:54] ============= [PASSED] drm_test_fb_clip_offset =============
[19:24:54] =================== drm_test_fb_memcpy  ====================
[19:24:54] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[19:24:54] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[19:24:54] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[19:24:54] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[19:24:54] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[19:24:54] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[19:24:54] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[19:24:54] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[19:24:54] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[19:24:54] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[19:24:54] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[19:24:54] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[19:24:54] =============== [PASSED] drm_test_fb_memcpy ================
[19:24:54] ============= [PASSED] drm_format_helper_test ==============
[19:24:54] ================= drm_format (18 subtests) =================
[19:24:54] [PASSED] drm_test_format_block_width_invalid
[19:24:54] [PASSED] drm_test_format_block_width_one_plane
[19:24:54] [PASSED] drm_test_format_block_width_two_plane
[19:24:54] [PASSED] drm_test_format_block_width_three_plane
[19:24:54] [PASSED] drm_test_format_block_width_tiled
[19:24:54] [PASSED] drm_test_format_block_height_invalid
[19:24:54] [PASSED] drm_test_format_block_height_one_plane
[19:24:54] [PASSED] drm_test_format_block_height_two_plane
[19:24:54] [PASSED] drm_test_format_block_height_three_plane
[19:24:54] [PASSED] drm_test_format_block_height_tiled
[19:24:54] [PASSED] drm_test_format_min_pitch_invalid
[19:24:54] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[19:24:54] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[19:24:54] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[19:24:54] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[19:24:54] [PASSED] drm_test_format_min_pitch_two_plane
[19:24:54] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[19:24:54] [PASSED] drm_test_format_min_pitch_tiled
[19:24:54] =================== [PASSED] drm_format ====================
[19:24:54] ============== drm_framebuffer (10 subtests) ===============
[19:24:54] ========== drm_test_framebuffer_check_src_coords  ==========
[19:24:54] [PASSED] Success: source fits into fb
[19:24:54] [PASSED] Fail: overflowing fb with x-axis coordinate
[19:24:54] [PASSED] Fail: overflowing fb with y-axis coordinate
[19:24:54] [PASSED] Fail: overflowing fb with source width
[19:24:54] [PASSED] Fail: overflowing fb with source height
[19:24:54] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[19:24:54] [PASSED] drm_test_framebuffer_cleanup
[19:24:54] =============== drm_test_framebuffer_create  ===============
[19:24:54] [PASSED] ABGR8888 normal sizes
[19:24:54] [PASSED] ABGR8888 max sizes
[19:24:54] [PASSED] ABGR8888 pitch greater than min required
[19:24:54] [PASSED] ABGR8888 pitch less than min required
[19:24:54] [PASSED] ABGR8888 Invalid width
[19:24:54] [PASSED] ABGR8888 Invalid buffer handle
[19:24:54] [PASSED] No pixel format
[19:24:54] [PASSED] ABGR8888 Width 0
[19:24:54] [PASSED] ABGR8888 Height 0
[19:24:54] [PASSED] ABGR8888 Out of bound height * pitch combination
[19:24:54] [PASSED] ABGR8888 Large buffer offset
[19:24:54] [PASSED] ABGR8888 Buffer offset for inexistent plane
[19:24:54] [PASSED] ABGR8888 Invalid flag
[19:24:54] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[19:24:54] [PASSED] ABGR8888 Valid buffer modifier
[19:24:54] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[19:24:54] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] NV12 Normal sizes
[19:24:54] [PASSED] NV12 Max sizes
[19:24:54] [PASSED] NV12 Invalid pitch
[19:24:54] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[19:24:54] [PASSED] NV12 different  modifier per-plane
[19:24:54] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[19:24:54] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] NV12 Modifier for inexistent plane
[19:24:54] [PASSED] NV12 Handle for inexistent plane
[19:24:54] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[19:24:54] [PASSED] YVU420 Normal sizes
[19:24:54] [PASSED] YVU420 Max sizes
[19:24:54] [PASSED] YVU420 Invalid pitch
[19:24:54] [PASSED] YVU420 Different pitches
[19:24:54] [PASSED] YVU420 Different buffer offsets/pitches
[19:24:54] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[19:24:54] [PASSED] YVU420 Valid modifier
[19:24:54] [PASSED] YVU420 Different modifiers per plane
[19:24:54] [PASSED] YVU420 Modifier for inexistent plane
[19:24:54] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[19:24:54] [PASSED] X0L2 Normal sizes
[19:24:54] [PASSED] X0L2 Max sizes
[19:24:54] [PASSED] X0L2 Invalid pitch
[19:24:54] [PASSED] X0L2 Pitch greater than minimum required
[19:24:54] [PASSED] X0L2 Handle for inexistent plane
[19:24:54] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[19:24:54] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[19:24:54] [PASSED] X0L2 Valid modifier
[19:24:54] [PASSED] X0L2 Modifier for inexistent plane
[19:24:54] =========== [PASSED] drm_test_framebuffer_create ===========
[19:24:54] [PASSED] drm_test_framebuffer_free
[19:24:54] [PASSED] drm_test_framebuffer_init
[19:24:54] [PASSED] drm_test_framebuffer_init_bad_format
[19:24:54] [PASSED] drm_test_framebuffer_init_dev_mismatch
[19:24:54] [PASSED] drm_test_framebuffer_lookup
[19:24:54] [PASSED] drm_test_framebuffer_lookup_inexistent
[19:24:54] [PASSED] drm_test_framebuffer_modifiers_not_supported
[19:24:54] ================= [PASSED] drm_framebuffer =================
[19:24:54] ================ drm_gem_shmem (8 subtests) ================
[19:24:54] [PASSED] drm_gem_shmem_test_obj_create
[19:24:54] [PASSED] drm_gem_shmem_test_obj_create_private
[19:24:54] [PASSED] drm_gem_shmem_test_pin_pages
[19:24:54] [PASSED] drm_gem_shmem_test_vmap
[19:24:54] [PASSED] drm_gem_shmem_test_get_sg_table
[19:24:54] [PASSED] drm_gem_shmem_test_get_pages_sgt
[19:24:54] [PASSED] drm_gem_shmem_test_madvise
[19:24:54] [PASSED] drm_gem_shmem_test_purge
[19:24:54] ================== [PASSED] drm_gem_shmem ==================
[19:24:54] === drm_atomic_helper_connector_hdmi_check (27 subtests) ===
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[19:24:54] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420  =======
[19:24:54] [PASSED] Automatic
[19:24:54] [PASSED] Full
[19:24:54] [PASSED] Limited 16:235
[19:24:54] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[19:24:54] [PASSED] drm_test_check_disable_connector
[19:24:54] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[19:24:54] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[19:24:54] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[19:24:54] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[19:24:54] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[19:24:54] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[19:24:54] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[19:24:54] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[19:24:54] [PASSED] drm_test_check_output_bpc_dvi
[19:24:54] [PASSED] drm_test_check_output_bpc_format_vic_1
[19:24:54] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[19:24:54] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[19:24:54] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[19:24:54] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[19:24:54] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[19:24:54] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[19:24:54] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[19:24:54] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[19:24:54] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[19:24:54] [PASSED] drm_test_check_broadcast_rgb_value
[19:24:54] [PASSED] drm_test_check_bpc_8_value
[19:24:54] [PASSED] drm_test_check_bpc_10_value
[19:24:54] [PASSED] drm_test_check_bpc_12_value
[19:24:54] [PASSED] drm_test_check_format_value
[19:24:54] [PASSED] drm_test_check_tmds_char_value
[19:24:54] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[19:24:54] = drm_atomic_helper_connector_hdmi_mode_valid (4 subtests) =
[19:24:54] [PASSED] drm_test_check_mode_valid
[19:24:54] [PASSED] drm_test_check_mode_valid_reject
[19:24:54] [PASSED] drm_test_check_mode_valid_reject_rate
[19:24:54] [PASSED] drm_test_check_mode_valid_reject_max_clock
[19:24:54] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[19:24:54] = drm_atomic_helper_connector_hdmi_infoframes (5 subtests) =
[19:24:54] [PASSED] drm_test_check_infoframes
[19:24:54] [PASSED] drm_test_check_reject_avi_infoframe
[19:24:54] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_8
[19:24:54] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_10
[19:24:54] [PASSED] drm_test_check_reject_audio_infoframe
[19:24:54] === [PASSED] drm_atomic_helper_connector_hdmi_infoframes ===
[19:24:54] ================= drm_managed (2 subtests) =================
[19:24:54] [PASSED] drm_test_managed_release_action
[19:24:54] [PASSED] drm_test_managed_run_action
[19:24:54] =================== [PASSED] drm_managed ===================
[19:24:54] =================== drm_mm (6 subtests) ====================
[19:24:54] [PASSED] drm_test_mm_init
[19:24:54] [PASSED] drm_test_mm_debug
[19:24:54] [PASSED] drm_test_mm_align32
[19:24:54] [PASSED] drm_test_mm_align64
[19:24:54] [PASSED] drm_test_mm_lowest
[19:24:54] [PASSED] drm_test_mm_highest
[19:24:54] ===================== [PASSED] drm_mm ======================
[19:24:54] ============= drm_modes_analog_tv (5 subtests) =============
[19:24:54] [PASSED] drm_test_modes_analog_tv_mono_576i
[19:24:54] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[19:24:54] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[19:24:54] [PASSED] drm_test_modes_analog_tv_pal_576i
[19:24:54] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[19:24:54] =============== [PASSED] drm_modes_analog_tv ===============
[19:24:54] ============== drm_plane_helper (2 subtests) ===============
[19:24:54] =============== drm_test_check_plane_state  ================
[19:24:54] [PASSED] clipping_simple
[19:24:54] [PASSED] clipping_rotate_reflect
[19:24:54] [PASSED] positioning_simple
[19:24:54] [PASSED] upscaling
[19:24:54] [PASSED] downscaling
[19:24:54] [PASSED] rounding1
[19:24:54] [PASSED] rounding2
[19:24:54] [PASSED] rounding3
[19:24:54] [PASSED] rounding4
[19:24:54] =========== [PASSED] drm_test_check_plane_state ============
[19:24:54] =========== drm_test_check_invalid_plane_state  ============
[19:24:54] [PASSED] positioning_invalid
[19:24:54] [PASSED] upscaling_invalid
[19:24:54] [PASSED] downscaling_invalid
[19:24:54] ======= [PASSED] drm_test_check_invalid_plane_state ========
[19:24:54] ================ [PASSED] drm_plane_helper =================
[19:24:54] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[19:24:54] ====== drm_test_connector_helper_tv_get_modes_check  =======
[19:24:54] [PASSED] None
[19:24:54] [PASSED] PAL
[19:24:54] [PASSED] NTSC
[19:24:54] [PASSED] Both, NTSC Default
[19:24:54] [PASSED] Both, PAL Default
[19:24:54] [PASSED] Both, NTSC Default, with PAL on command-line
[19:24:54] [PASSED] Both, PAL Default, with NTSC on command-line
[19:24:54] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[19:24:54] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[19:24:54] ================== drm_rect (9 subtests) ===================
[19:24:54] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[19:24:54] [PASSED] drm_test_rect_clip_scaled_not_clipped
[19:24:54] [PASSED] drm_test_rect_clip_scaled_clipped
[19:24:54] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[19:24:54] ================= drm_test_rect_intersect  =================
[19:24:54] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[19:24:54] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[19:24:54] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[19:24:54] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[19:24:54] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[19:24:54] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[19:24:54] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[19:24:54] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[19:24:54] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[19:24:54] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[19:24:54] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[19:24:54] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[19:24:54] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[19:24:54] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[19:24:54] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[19:24:54] ============= [PASSED] drm_test_rect_intersect =============
[19:24:54] ================ drm_test_rect_calc_hscale  ================
[19:24:54] [PASSED] normal use
[19:24:54] [PASSED] out of max range
[19:24:54] [PASSED] out of min range
[19:24:54] [PASSED] zero dst
[19:24:54] [PASSED] negative src
[19:24:54] [PASSED] negative dst
[19:24:54] ============ [PASSED] drm_test_rect_calc_hscale ============
[19:24:54] ================ drm_test_rect_calc_vscale  ================
[19:24:54] [PASSED] normal use
[19:24:54] [PASSED] out of max range
[19:24:54] [PASSED] out of min range
[19:24:54] [PASSED] zero dst
[19:24:54] [PASSED] negative src
[19:24:54] [PASSED] negative dst
stty: 'standard input': Inappropriate ioctl for device
[19:24:54] ============ [PASSED] drm_test_rect_calc_vscale ============
[19:24:54] ================== drm_test_rect_rotate  ===================
[19:24:54] [PASSED] reflect-x
[19:24:54] [PASSED] reflect-y
[19:24:54] [PASSED] rotate-0
[19:24:54] [PASSED] rotate-90
[19:24:54] [PASSED] rotate-180
[19:24:54] [PASSED] rotate-270
[19:24:54] ============== [PASSED] drm_test_rect_rotate ===============
[19:24:54] ================ drm_test_rect_rotate_inv  =================
[19:24:54] [PASSED] reflect-x
[19:24:54] [PASSED] reflect-y
[19:24:54] [PASSED] rotate-0
[19:24:54] [PASSED] rotate-90
[19:24:54] [PASSED] rotate-180
[19:24:54] [PASSED] rotate-270
[19:24:54] ============ [PASSED] drm_test_rect_rotate_inv =============
[19:24:54] ==================== [PASSED] drm_rect =====================
[19:24:54] ============ drm_sysfb_modeset_test (1 subtest) ============
[19:24:54] ============ drm_test_sysfb_build_fourcc_list  =============
[19:24:54] [PASSED] no native formats
[19:24:54] [PASSED] XRGB8888 as native format
[19:24:54] [PASSED] remove duplicates
[19:24:54] [PASSED] convert alpha formats
[19:24:54] [PASSED] random formats
[19:24:54] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[19:24:54] ============= [PASSED] drm_sysfb_modeset_test ==============
[19:24:54] ================== drm_fixp (2 subtests) ===================
[19:24:54] [PASSED] drm_test_int2fixp
[19:24:54] [PASSED] drm_test_sm2fixp
[19:24:54] ==================== [PASSED] drm_fixp =====================
[19:24:54] ============================================================
[19:24:54] Testing complete. Ran 621 tests: passed: 621
[19:24:54] Elapsed time: 30.872s total, 1.663s configuring, 29.042s building, 0.118s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[19:24:55] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[19:24:56] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[19:25:05] Starting KUnit Kernel (1/1)...
[19:25:05] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[19:25:06] ================= ttm_device (5 subtests) ==================
[19:25:06] [PASSED] ttm_device_init_basic
[19:25:06] [PASSED] ttm_device_init_multiple
[19:25:06] [PASSED] ttm_device_fini_basic
[19:25:06] [PASSED] ttm_device_init_no_vma_man
[19:25:06] ================== ttm_device_init_pools  ==================
[19:25:06] [PASSED] No DMA allocations, no DMA32 required
[19:25:06] [PASSED] DMA allocations, DMA32 required
[19:25:06] [PASSED] No DMA allocations, DMA32 required
[19:25:06] [PASSED] DMA allocations, no DMA32 required
[19:25:06] ============== [PASSED] ttm_device_init_pools ==============
[19:25:06] =================== [PASSED] ttm_device ====================
[19:25:06] ================== ttm_pool (8 subtests) ===================
[19:25:06] ================== ttm_pool_alloc_basic  ===================
[19:25:06] [PASSED] One page
[19:25:06] [PASSED] More than one page
[19:25:06] [PASSED] Above the allocation limit
[19:25:06] [PASSED] One page, with coherent DMA mappings enabled
[19:25:06] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[19:25:06] ============== [PASSED] ttm_pool_alloc_basic ===============
[19:25:06] ============== ttm_pool_alloc_basic_dma_addr  ==============
[19:25:06] [PASSED] One page
[19:25:06] [PASSED] More than one page
[19:25:06] [PASSED] Above the allocation limit
[19:25:06] [PASSED] One page, with coherent DMA mappings enabled
[19:25:06] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[19:25:06] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[19:25:06] [PASSED] ttm_pool_alloc_order_caching_match
[19:25:06] [PASSED] ttm_pool_alloc_caching_mismatch
[19:25:06] [PASSED] ttm_pool_alloc_order_mismatch
[19:25:06] [PASSED] ttm_pool_free_dma_alloc
[19:25:06] [PASSED] ttm_pool_free_no_dma_alloc
[19:25:06] [PASSED] ttm_pool_fini_basic
[19:25:06] ==================== [PASSED] ttm_pool =====================
[19:25:06] ================ ttm_resource (8 subtests) =================
[19:25:06] ================= ttm_resource_init_basic  =================
[19:25:06] [PASSED] Init resource in TTM_PL_SYSTEM
[19:25:06] [PASSED] Init resource in TTM_PL_VRAM
[19:25:06] [PASSED] Init resource in a private placement
[19:25:06] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[19:25:06] ============= [PASSED] ttm_resource_init_basic =============
[19:25:06] [PASSED] ttm_resource_init_pinned
[19:25:06] [PASSED] ttm_resource_fini_basic
[19:25:06] [PASSED] ttm_resource_manager_init_basic
[19:25:06] [PASSED] ttm_resource_manager_usage_basic
[19:25:06] [PASSED] ttm_resource_manager_set_used_basic
[19:25:06] [PASSED] ttm_sys_man_alloc_basic
[19:25:06] [PASSED] ttm_sys_man_free_basic
[19:25:06] ================== [PASSED] ttm_resource ===================
[19:25:06] =================== ttm_tt (15 subtests) ===================
[19:25:06] ==================== ttm_tt_init_basic  ====================
[19:25:06] [PASSED] Page-aligned size
[19:25:06] [PASSED] Extra pages requested
[19:25:06] ================ [PASSED] ttm_tt_init_basic ================
[19:25:06] [PASSED] ttm_tt_init_misaligned
[19:25:06] [PASSED] ttm_tt_fini_basic
[19:25:06] [PASSED] ttm_tt_fini_sg
[19:25:06] [PASSED] ttm_tt_fini_shmem
[19:25:06] [PASSED] ttm_tt_create_basic
[19:25:06] [PASSED] ttm_tt_create_invalid_bo_type
[19:25:06] [PASSED] ttm_tt_create_ttm_exists
[19:25:06] [PASSED] ttm_tt_create_failed
[19:25:06] [PASSED] ttm_tt_destroy_basic
[19:25:06] [PASSED] ttm_tt_populate_null_ttm
[19:25:06] [PASSED] ttm_tt_populate_populated_ttm
[19:25:06] [PASSED] ttm_tt_unpopulate_basic
[19:25:06] [PASSED] ttm_tt_unpopulate_empty_ttm
[19:25:06] [PASSED] ttm_tt_swapin_basic
[19:25:06] ===================== [PASSED] ttm_tt ======================
[19:25:06] =================== ttm_bo (14 subtests) ===================
[19:25:06] =========== ttm_bo_reserve_optimistic_no_ticket  ===========
[19:25:06] [PASSED] Cannot be interrupted and sleeps
[19:25:06] [PASSED] Cannot be interrupted, locks straight away
[19:25:06] [PASSED] Can be interrupted, sleeps
[19:25:06] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[19:25:06] [PASSED] ttm_bo_reserve_locked_no_sleep
[19:25:06] [PASSED] ttm_bo_reserve_no_wait_ticket
[19:25:06] [PASSED] ttm_bo_reserve_double_resv
[19:25:06] [PASSED] ttm_bo_reserve_interrupted
[19:25:06] [PASSED] ttm_bo_reserve_deadlock
[19:25:06] [PASSED] ttm_bo_unreserve_basic
[19:25:06] [PASSED] ttm_bo_unreserve_pinned
[19:25:06] [PASSED] ttm_bo_unreserve_bulk
[19:25:06] [PASSED] ttm_bo_fini_basic
[19:25:06] [PASSED] ttm_bo_fini_shared_resv
[19:25:06] [PASSED] ttm_bo_pin_basic
[19:25:06] [PASSED] ttm_bo_pin_unpin_resource
[19:25:06] [PASSED] ttm_bo_multiple_pin_one_unpin
[19:25:06] ===================== [PASSED] ttm_bo ======================
[19:25:06] ============== ttm_bo_validate (21 subtests) ===============
[19:25:06] ============== ttm_bo_init_reserved_sys_man  ===============
[19:25:06] [PASSED] Buffer object for userspace
[19:25:06] [PASSED] Kernel buffer object
[19:25:06] [PASSED] Shared buffer object
[19:25:06] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[19:25:06] ============== ttm_bo_init_reserved_mock_man  ==============
[19:25:06] [PASSED] Buffer object for userspace
[19:25:06] [PASSED] Kernel buffer object
[19:25:06] [PASSED] Shared buffer object
[19:25:06] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[19:25:06] [PASSED] ttm_bo_init_reserved_resv
[19:25:06] ================== ttm_bo_validate_basic  ==================
[19:25:06] [PASSED] Buffer object for userspace
[19:25:06] [PASSED] Kernel buffer object
[19:25:06] [PASSED] Shared buffer object
[19:25:06] ============== [PASSED] ttm_bo_validate_basic ==============
[19:25:06] [PASSED] ttm_bo_validate_invalid_placement
[19:25:06] ============= ttm_bo_validate_same_placement  ==============
[19:25:06] [PASSED] System manager
[19:25:06] [PASSED] VRAM manager
[19:25:06] ========= [PASSED] ttm_bo_validate_same_placement ==========
[19:25:06] [PASSED] ttm_bo_validate_failed_alloc
[19:25:06] [PASSED] ttm_bo_validate_pinned
[19:25:06] [PASSED] ttm_bo_validate_busy_placement
[19:25:06] ================ ttm_bo_validate_multihop  =================
[19:25:06] [PASSED] Buffer object for userspace
[19:25:06] [PASSED] Kernel buffer object
[19:25:06] [PASSED] Shared buffer object
[19:25:06] ============ [PASSED] ttm_bo_validate_multihop =============
[19:25:06] ========== ttm_bo_validate_no_placement_signaled  ==========
[19:25:06] [PASSED] Buffer object in system domain, no page vector
[19:25:06] [PASSED] Buffer object in system domain with an existing page vector
[19:25:06] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[19:25:06] ======== ttm_bo_validate_no_placement_not_signaled  ========
[19:25:06] [PASSED] Buffer object for userspace
[19:25:06] [PASSED] Kernel buffer object
[19:25:06] [PASSED] Shared buffer object
[19:25:06] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[19:25:06] [PASSED] ttm_bo_validate_move_fence_signaled
[19:25:06] ========= ttm_bo_validate_move_fence_not_signaled  =========
[19:25:06] [PASSED] Waits for GPU
[19:25:06] [PASSED] Tries to lock straight away
[19:25:06] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[19:25:06] [PASSED] ttm_bo_validate_happy_evict
[19:25:06] [PASSED] ttm_bo_validate_all_pinned_evict
[19:25:06] [PASSED] ttm_bo_validate_allowed_only_evict
[19:25:06] [PASSED] ttm_bo_validate_deleted_evict
[19:25:06] [PASSED] ttm_bo_validate_busy_domain_evict
[19:25:06] [PASSED] ttm_bo_validate_evict_gutting
[19:25:06] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[19:25:06] ================= [PASSED] ttm_bo_validate =================
[19:25:06] ============================================================
[19:25:06] Testing complete. Ran 101 tests: passed: 101
[19:25:06] Elapsed time: 11.068s total, 1.656s configuring, 9.195s building, 0.186s running

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel



^ permalink raw reply	[flat|nested] 22+ messages in thread

* ✓ Xe.CI.BAT: success for drm/{i915,xe}: sort out step enums between the drivers
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (9 preceding siblings ...)
  2026-03-05 19:25 ` ✓ CI.KUnit: success " Patchwork
@ 2026-03-05 20:11 ` Patchwork
  2026-03-06 11:35 ` ✗ Xe.CI.FULL: failure " Patchwork
  11 siblings, 0 replies; 22+ messages in thread
From: Patchwork @ 2026-03-05 20:11 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

[-- Attachment #1: Type: text/plain, Size: 6627 bytes --]

== Series Details ==

Series: drm/{i915,xe}: sort out step enums between the drivers
URL   : https://patchwork.freedesktop.org/series/162594/
State : success

== Summary ==

CI Bug Log - changes from xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb_BAT -> xe-pw-162594v1_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (13 -> 14)
------------------------------

  Additional (1): bat-atsm-2 

Known issues
------------

  Here are the changes found in xe-pw-162594v1_BAT that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@fbdev@write:
    - bat-atsm-2:         NOTRUN -> [SKIP][1] ([Intel XE#2134]) +4 other tests skip
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@fbdev@write.html

  * igt@kms_addfb_basic@addfb25-yf-tiled-legacy:
    - bat-atsm-2:         NOTRUN -> [SKIP][2] ([i915#6077]) +30 other tests skip
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_addfb_basic@addfb25-yf-tiled-legacy.html

  * igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
    - bat-atsm-2:         NOTRUN -> [SKIP][3] ([Intel XE#1024] / [Intel XE#782] / [Intel XE#947]) +5 other tests skip
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html

  * igt@kms_dsc@dsc-basic:
    - bat-atsm-2:         NOTRUN -> [SKIP][4] ([Intel XE#1024] / [Intel XE#784] / [Intel XE#947])
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_dsc@dsc-basic.html

  * igt@kms_frontbuffer_tracking@basic:
    - bat-atsm-2:         NOTRUN -> [SKIP][5] ([Intel XE#1024] / [Intel XE#783] / [Intel XE#947])
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_frontbuffer_tracking@basic.html

  * igt@kms_hdmi_inject@inject-audio:
    - bat-atsm-2:         NOTRUN -> [SKIP][6] ([Intel XE#540]) +3 other tests skip
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_hdmi_inject@inject-audio.html

  * igt@kms_pipe_crc_basic@nonblocking-crc:
    - bat-atsm-2:         NOTRUN -> [SKIP][7] ([Intel XE#829] / [i915#1836]) +6 other tests skip
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_pipe_crc_basic@nonblocking-crc.html

  * igt@kms_prop_blob@basic:
    - bat-atsm-2:         NOTRUN -> [SKIP][8] ([Intel XE#780])
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_prop_blob@basic.html

  * igt@kms_psr@psr-primary-page-flip:
    - bat-atsm-2:         NOTRUN -> [SKIP][9] ([Intel XE#1024] / [Intel XE#947]) +6 other tests skip
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@kms_psr@psr-primary-page-flip.html

  * igt@xe_exec_fault_mode@twice-userptr-prefetch:
    - bat-atsm-2:         NOTRUN -> [SKIP][10] ([Intel XE#288]) +32 other tests skip
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_exec_fault_mode@twice-userptr-prefetch.html

  * igt@xe_huc_copy@huc_copy:
    - bat-atsm-2:         NOTRUN -> [SKIP][11] ([Intel XE#255])
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_huc_copy@huc_copy.html

  * igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit:
    - bat-bmg-1:          NOTRUN -> [SKIP][12] ([Intel XE#2229])
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-bmg-1/igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit.html

  * igt@xe_live_ktest@xe_migrate@xe_validate_ccs_kunit:
    - bat-atsm-2:         NOTRUN -> [SKIP][13] ([Intel XE#2229])
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_live_ktest@xe_migrate@xe_validate_ccs_kunit.html

  * igt@xe_pat@pat-index-xe2:
    - bat-atsm-2:         NOTRUN -> [SKIP][14] ([Intel XE#977])
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_pat@pat-index-xe2.html

  * igt@xe_pat@pat-index-xehpc:
    - bat-atsm-2:         NOTRUN -> [SKIP][15] ([Intel XE#2838] / [Intel XE#979])
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_pat@pat-index-xehpc.html

  * igt@xe_pat@pat-index-xelpg:
    - bat-atsm-2:         NOTRUN -> [SKIP][16] ([Intel XE#979])
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-atsm-2/igt@xe_pat@pat-index-xelpg.html

  
#### Possible fixes ####

  * igt@core_hotunplug@unbind-rebind:
    - bat-bmg-1:          [ABORT][17] ([Intel XE#7249]) -> [PASS][18]
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/bat-bmg-1/igt@core_hotunplug@unbind-rebind.html
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/bat-bmg-1/igt@core_hotunplug@unbind-rebind.html

  
  [Intel XE#1024]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1024
  [Intel XE#2134]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2134
  [Intel XE#2229]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2229
  [Intel XE#255]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/255
  [Intel XE#2838]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2838
  [Intel XE#288]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/288
  [Intel XE#540]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/540
  [Intel XE#7249]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7249
  [Intel XE#780]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/780
  [Intel XE#782]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/782
  [Intel XE#783]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/783
  [Intel XE#784]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/784
  [Intel XE#829]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/829
  [Intel XE#947]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/947
  [Intel XE#977]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/977
  [Intel XE#979]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/979
  [i915#1836]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1836
  [i915#6077]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6077


Build changes
-------------

  * Linux: xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb -> xe-pw-162594v1

  IGT_8781: 911c75d8ee24b6d42d31e76bc60149a9350f1190 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb: d5f492dbc8c65c78ee639fa06821eeb568c547cb
  xe-pw-162594v1: 162594v1

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/index.html

[-- Attachment #2: Type: text/html, Size: 8124 bytes --]

^ permalink raw reply	[flat|nested] 22+ messages in thread

* ✗ Xe.CI.FULL: failure for drm/{i915,xe}: sort out step enums between the drivers
  2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
                   ` (10 preceding siblings ...)
  2026-03-05 20:11 ` ✓ Xe.CI.BAT: " Patchwork
@ 2026-03-06 11:35 ` Patchwork
  11 siblings, 0 replies; 22+ messages in thread
From: Patchwork @ 2026-03-06 11:35 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

[-- Attachment #1: Type: text/plain, Size: 13039 bytes --]

== Series Details ==

Series: drm/{i915,xe}: sort out step enums between the drivers
URL   : https://patchwork.freedesktop.org/series/162594/
State : failure

== Summary ==

CI Bug Log - changes from xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb_FULL -> xe-pw-162594v1_FULL
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-162594v1_FULL absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-162594v1_FULL, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Participating hosts (2 -> 2)
------------------------------

  No changes in participating hosts

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in xe-pw-162594v1_FULL:

### IGT changes ###

#### Possible regressions ####

  * igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-d-dp-2:
    - shard-bmg:          [PASS][1] -> [INCOMPLETE][2]
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-9/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-d-dp-2.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-6/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-d-dp-2.html

  * igt@kms_cursor_legacy@flip-vs-cursor-legacy:
    - shard-bmg:          [PASS][3] -> [FAIL][4]
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-2/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-4/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html

  * igt@kms_vblank@ts-continuation-dpms-suspend@pipe-d-dp-2:
    - shard-bmg:          NOTRUN -> [INCOMPLETE][5] +1 other test incomplete
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_vblank@ts-continuation-dpms-suspend@pipe-d-dp-2.html

  * igt@xe_vm@large-binds-536870912:
    - shard-bmg:          [PASS][6] -> [ABORT][7]
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-1/igt@xe_vm@large-binds-536870912.html
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@xe_vm@large-binds-536870912.html

  
Known issues
------------

  Here are the changes found in xe-pw-162594v1_FULL that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
    - shard-bmg:          NOTRUN -> [SKIP][8] ([Intel XE#2233])
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html

  * igt@kms_big_fb@y-tiled-32bpp-rotate-0:
    - shard-bmg:          NOTRUN -> [SKIP][9] ([Intel XE#1124])
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html

  * igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs:
    - shard-bmg:          [PASS][10] -> [INCOMPLETE][11] ([Intel XE#7084])
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-9/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-6/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html

  * igt@kms_chamelium_hpd@hdmi-hpd-after-suspend:
    - shard-bmg:          NOTRUN -> [SKIP][12] ([Intel XE#2252])
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@kms_chamelium_hpd@hdmi-hpd-after-suspend.html

  * igt@kms_frontbuffer_tracking@drrs-2p-rte:
    - shard-bmg:          NOTRUN -> [SKIP][13] ([Intel XE#2311]) +1 other test skip
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_frontbuffer_tracking@drrs-2p-rte.html

  * igt@kms_frontbuffer_tracking@drrs-argb161616f-draw-mmap-wc:
    - shard-bmg:          NOTRUN -> [SKIP][14] ([Intel XE#7061] / [Intel XE#7356])
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_frontbuffer_tracking@drrs-argb161616f-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-render:
    - shard-bmg:          NOTRUN -> [SKIP][15] ([Intel XE#4141])
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-render.html

  * igt@kms_frontbuffer_tracking@psr-2p-primscrn-indfb-msflip-blt:
    - shard-bmg:          NOTRUN -> [SKIP][16] ([Intel XE#2313]) +2 other tests skip
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@kms_frontbuffer_tracking@psr-2p-primscrn-indfb-msflip-blt.html

  * igt@kms_pm_dc@dc5-retention-flops:
    - shard-bmg:          NOTRUN -> [SKIP][17] ([Intel XE#3309] / [Intel XE#7368])
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@kms_pm_dc@dc5-retention-flops.html

  * igt@kms_psr2_sf@fbc-psr2-overlay-plane-update-continuous-sf:
    - shard-bmg:          NOTRUN -> [SKIP][18] ([Intel XE#1489])
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_psr2_sf@fbc-psr2-overlay-plane-update-continuous-sf.html

  * igt@kms_psr@fbc-psr2-primary-blt:
    - shard-bmg:          NOTRUN -> [SKIP][19] ([Intel XE#2234] / [Intel XE#2850]) +1 other test skip
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_psr@fbc-psr2-primary-blt.html

  * igt@kms_sharpness_filter@invalid-filter-with-plane:
    - shard-bmg:          NOTRUN -> [SKIP][20] ([Intel XE#6503])
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_sharpness_filter@invalid-filter-with-plane.html

  * igt@kms_vrr@cmrr@pipe-a-edp-1:
    - shard-lnl:          [PASS][21] -> [FAIL][22] ([Intel XE#4459]) +1 other test fail
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-lnl-3/igt@kms_vrr@cmrr@pipe-a-edp-1.html
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-lnl-3/igt@kms_vrr@cmrr@pipe-a-edp-1.html

  * igt@xe_eudebug_online@writes-caching-sram-bb-vram-target-vram:
    - shard-bmg:          NOTRUN -> [SKIP][23] ([Intel XE#4837] / [Intel XE#6665])
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@xe_eudebug_online@writes-caching-sram-bb-vram-target-vram.html

  * igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-userptr:
    - shard-bmg:          NOTRUN -> [SKIP][24] ([Intel XE#2322] / [Intel XE#7372]) +1 other test skip
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-userptr.html

  * igt@xe_exec_fault_mode@many-multi-queue-userptr-invalidate-race-prefetch:
    - shard-bmg:          NOTRUN -> [SKIP][25] ([Intel XE#7136]) +1 other test skip
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@xe_exec_fault_mode@many-multi-queue-userptr-invalidate-race-prefetch.html

  * igt@xe_exec_multi_queue@two-queues-preempt-mode-dyn-priority-smem:
    - shard-bmg:          NOTRUN -> [SKIP][26] ([Intel XE#6874]) +2 other tests skip
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-7/igt@xe_exec_multi_queue@two-queues-preempt-mode-dyn-priority-smem.html

  * igt@xe_exec_system_allocator@threads-many-large-mmap-remap-dontunmap-eocheck:
    - shard-bmg:          [PASS][27] -> [DMESG-FAIL][28] ([Intel XE#5213] / [Intel XE#6652])
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-1/igt@xe_exec_system_allocator@threads-many-large-mmap-remap-dontunmap-eocheck.html
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@xe_exec_system_allocator@threads-many-large-mmap-remap-dontunmap-eocheck.html

  * igt@xe_exec_threads@threads-multi-queue-mixed-fd-userptr-rebind:
    - shard-bmg:          NOTRUN -> [SKIP][29] ([Intel XE#7138]) +1 other test skip
   [29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@xe_exec_threads@threads-multi-queue-mixed-fd-userptr-rebind.html

  
#### Possible fixes ####

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
    - shard-lnl:          [FAIL][30] ([Intel XE#301]) -> [PASS][31] +1 other test pass
   [30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-lnl-2/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
   [31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-lnl-6/igt@kms_flip@flip-vs-expired-vblank-interruptible.html

  * igt@kms_flip@flip-vs-modeset-vs-hang@a-dp2:
    - shard-bmg:          [ABORT][32] -> [PASS][33] +1 other test pass
   [32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-2/igt@kms_flip@flip-vs-modeset-vs-hang@a-dp2.html
   [33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-2/igt@kms_flip@flip-vs-modeset-vs-hang@a-dp2.html

  
#### Warnings ####

  * igt@kms_tiled_display@basic-test-pattern:
    - shard-bmg:          [FAIL][34] ([Intel XE#1729] / [Intel XE#7424]) -> [SKIP][35] ([Intel XE#2426] / [Intel XE#5848])
   [34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-6/igt@kms_tiled_display@basic-test-pattern.html
   [35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-1/igt@kms_tiled_display@basic-test-pattern.html

  * igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv:
    - shard-bmg:          [ABORT][36] ([Intel XE#5466]) -> [ABORT][37] ([Intel XE#5466] / [Intel XE#6652])
   [36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb/shard-bmg-2/igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv.html
   [37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/shard-bmg-4/igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv.html

  
  [Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
  [Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
  [Intel XE#1729]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1729
  [Intel XE#2233]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2233
  [Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
  [Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
  [Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
  [Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
  [Intel XE#2322]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2322
  [Intel XE#2426]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2426
  [Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#3309]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3309
  [Intel XE#4141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4141
  [Intel XE#4459]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4459
  [Intel XE#4837]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4837
  [Intel XE#5213]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5213
  [Intel XE#5466]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5466
  [Intel XE#5848]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5848
  [Intel XE#6503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6503
  [Intel XE#6652]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6652
  [Intel XE#6665]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6665
  [Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
  [Intel XE#7061]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7061
  [Intel XE#7084]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7084
  [Intel XE#7136]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7136
  [Intel XE#7138]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7138
  [Intel XE#7356]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7356
  [Intel XE#7368]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7368
  [Intel XE#7372]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7372
  [Intel XE#7424]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7424


Build changes
-------------

  * Linux: xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb -> xe-pw-162594v1

  IGT_8781: 911c75d8ee24b6d42d31e76bc60149a9350f1190 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-4663-d5f492dbc8c65c78ee639fa06821eeb568c547cb: d5f492dbc8c65c78ee639fa06821eeb568c547cb
  xe-pw-162594v1: 162594v1

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-162594v1/index.html

[-- Attachment #2: Type: text/html, Size: 14331 bytes --]

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization
  2026-03-04 14:40 ` [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization Jani Nikula
@ 2026-03-09  8:18   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  8:18 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Having intel_get_stepping_info() return the pointer that was passed in
> isn't necessary. Just use a pointer to the local variable instead.
> 
> The initialization to ** didn't make a difference, because it was always
> overridden.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/i915/display/intel_dmc.c | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
> index c3b411259a0c..4e763f2f17c7 100644
> --- a/drivers/gpu/drm/i915/display/intel_dmc.c
> +++ b/drivers/gpu/drm/i915/display/intel_dmc.c
> @@ -418,15 +418,12 @@ bool intel_dmc_has_payload(struct intel_display *display)
>  	return has_dmc_id_fw(display, DMC_FW_MAIN);
>  }
>  
> -static const struct stepping_info *
> -intel_get_stepping_info(struct intel_display *display,
> -			struct stepping_info *si)
> +static void initialize_stepping_info(struct intel_display *display, struct stepping_info *si)
>  {
>  	const char *step_name = intel_step_name(INTEL_DISPLAY_STEP(display));
>  
>  	si->stepping = step_name[0];
>  	si->substepping = step_name[1];
> -	return si;
>  }
>  
>  static void gen9_set_dc_state_debugmask(struct intel_display *display)
> @@ -1274,8 +1271,7 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
>  	struct intel_css_header *css_header;
>  	struct intel_package_header *package_header;
>  	struct intel_dmc_header_base *dmc_header;
> -	struct stepping_info display_info = { '*', '*'};
> -	const struct stepping_info *si = intel_get_stepping_info(display, &display_info);
> +	struct stepping_info si = {};
>  	enum intel_dmc_id dmc_id;
>  	u32 readcount = 0;
>  	u32 r, offset;
> @@ -1283,6 +1279,8 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
>  	if (!fw)
>  		return -EINVAL;
>  
> +	initialize_stepping_info(display, &si);
> +
>  	/* Extract CSS Header information */
>  	css_header = (struct intel_css_header *)fw->data;
>  	r = parse_dmc_fw_css(dmc, css_header, fw->size);
> @@ -1293,7 +1291,7 @@ static int parse_dmc_fw(struct intel_dmc *dmc, const struct firmware *fw)
>  
>  	/* Extract Package Header information */
>  	package_header = (struct intel_package_header *)&fw->data[readcount];
> -	r = parse_dmc_fw_package(dmc, package_header, si, fw->size - readcount);
> +	r = parse_dmc_fw_package(dmc, package_header, &si, fw->size - readcount);
>  	if (!r)
>  		return -EINVAL;
>  

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/8] drm/i915/display: add step name in display runtime info
  2026-03-04 14:40 ` [PATCH 2/8] drm/i915/display: add step name in display runtime info Jani Nikula
@ 2026-03-09  8:24   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  8:24 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Initialize the stepping name in display runtime info. This avoids having
> to use intel_step_name().
> 
> For display device info print at boot, debugfs and snapshot this changes
> the unknown step name from ** to N/A, which is more user friendly
> anyway.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  .../drm/i915/display/intel_display_device.c   | 28 +++++++++++++++++--
>  .../drm/i915/display/intel_display_device.h   |  1 +
>  2 files changed, 26 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display_device.c b/drivers/gpu/drm/i915/display/intel_display_device.c
> index 0241603470a2..f716e9a51532 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.c
> @@ -1653,6 +1653,28 @@ static void display_platforms_or(struct intel_display_platforms *dst,
>  	bitmap_or(dst->bitmap, dst->bitmap, src->bitmap, display_platforms_num_bits());
>  }
>  
> +#define __STEP_NAME(name) [STEP_##name] = #name,
> +
> +static void initialize_step(struct intel_display *display, enum intel_step step)
> +{
> +	static const char step_names[][3] = {
> +		STEP_NAME_LIST(__STEP_NAME)
> +	};
> +
> +	DISPLAY_RUNTIME_INFO(display)->step = step;
> +
> +	/* Step name will remain an empty string if not applicable */
> +	if (step >= 0 && step < ARRAY_SIZE(step_names))
> +		strscpy(DISPLAY_RUNTIME_INFO(display)->step_name, step_names[step]);
> +}
> +
> +#undef __STEP_NAME
> +
> +static const char *step_name(const struct intel_display_runtime_info *runtime)
> +{
> +	return strlen(runtime->step_name) ? runtime->step_name : "N/A";
> +}
> +
>  struct intel_display *intel_display_device_probe(struct pci_dev *pdev,
>  						 const struct intel_display_parent_interface *parent)
>  {
> @@ -1730,14 +1752,14 @@ struct intel_display *intel_display_device_probe(struct pci_dev *pdev,
>  					  subdesc ? &subdesc->step_info : NULL);
>  	}
>  
> -	DISPLAY_RUNTIME_INFO(display)->step = step;
> +	initialize_step(display, step);
>  
>  	drm_info(display->drm, "Found %s%s%s (device ID %04x) %s display version %u.%02u stepping %s\n",
>  		 desc->name, subdesc ? "/" : "", subdesc ? subdesc->name : "",
>  		 pdev->device, display->platform.dgfx ? "discrete" : "integrated",
>  		 DISPLAY_RUNTIME_INFO(display)->ip.ver,
>  		 DISPLAY_RUNTIME_INFO(display)->ip.rel,
> -		 step != STEP_NONE ? intel_step_name(step) : "N/A");
> +		 step_name(DISPLAY_RUNTIME_INFO(display)));
>  
>  	return display;
>  
> @@ -1953,7 +1975,7 @@ void intel_display_device_info_print(const struct intel_display_device_info *inf
>  		drm_printf(p, "display version: %u\n",
>  			   runtime->ip.ver);
>  
> -	drm_printf(p, "display stepping: %s\n", intel_step_name(runtime->step));
> +	drm_printf(p, "display stepping: %s\n", step_name(runtime));
>  
>  #define PRINT_FLAG(name) drm_printf(p, "%s: %s\n", #name, str_yes_no(info->name))
>  	DEV_INFO_DISPLAY_FOR_EACH_FLAG(PRINT_FLAG);
> diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h b/drivers/gpu/drm/i915/display/intel_display_device.h
> index e84c190dcc4f..1170ac346615 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.h
> @@ -287,6 +287,7 @@ struct intel_display_runtime_info {
>  		u16 step; /* hardware */
>  	} ip;
>  	int step; /* symbolic */
> +	char step_name[3]; /* empty string if not applicable */
>  
>  	u32 rawclk_freq;
>  

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 3/8] drm/i915/dmc: use step name from runtime info
  2026-03-04 14:40 ` [PATCH 3/8] drm/i915/dmc: use step name from " Jani Nikula
@ 2026-03-09  8:35   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  8:35 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Now that the step name is in runtime info, switch to using it instead of
> intel_step_name().
> 
> The ** are only relevant for DMC, so make their use explicit.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/i915/display/intel_dmc.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
> index 4e763f2f17c7..b250d41dc6a1 100644
> --- a/drivers/gpu/drm/i915/display/intel_dmc.c
> +++ b/drivers/gpu/drm/i915/display/intel_dmc.c
> @@ -39,7 +39,6 @@
>  #include "intel_dmc.h"
>  #include "intel_dmc_regs.h"
>  #include "intel_flipq.h"
> -#include "intel_step.h"
>  
>  /**
>   * DOC: DMC Firmware Support
> @@ -420,10 +419,10 @@ bool intel_dmc_has_payload(struct intel_display *display)
>  
>  static void initialize_stepping_info(struct intel_display *display, struct stepping_info *si)
>  {
> -	const char *step_name = intel_step_name(INTEL_DISPLAY_STEP(display));
> +	const char *step_name = DISPLAY_RUNTIME_INFO(display)->step_name;
>  
> -	si->stepping = step_name[0];
> -	si->substepping = step_name[1];
> +	si->stepping = step_name[0] ?: '*';
> +	si->substepping = step_name[1] ?: '*';
>  }
>  
>  static void gen9_set_dc_state_debugmask(struct intel_display *display)

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 4/8] drm/xe/compat: remove intel_step_name macro
  2026-03-04 14:40 ` [PATCH 4/8] drm/xe/compat: remove intel_step_name macro Jani Nikula
@ 2026-03-09  9:07   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:07 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> As there are no more compat users left for intel_step_name(), remove the
> macro and use the more direct include for the enumerations.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/xe/compat-i915-headers/intel_step.h | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> index 2cf13a572ab0..0eabe2866f5f 100644
> --- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> +++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> @@ -6,9 +6,8 @@
>  #ifndef __INTEL_STEP_H__
>  #define __INTEL_STEP_H__
>  
> -#include "xe_step.h"
> +#include "xe_step_types.h"
>  
>  #define intel_step xe_step
> -#define intel_step_name xe_step_name
>  
>  #endif /* __INTEL_STEP_H__ */

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it
  2026-03-04 14:40 ` [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it Jani Nikula
@ 2026-03-09  9:09   ` Luca Coelho
  2026-03-09  9:16     ` Luca Coelho
  0 siblings, 1 reply; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:09 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> As the first step towards using shared definitions for step name
> enumerations, add shared include/drm/intel/step.h and switch i915 to use
> it.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/i915/intel_step.h | 57 +---------------------------
>  include/drm/intel/step.h          | 62 +++++++++++++++++++++++++++++++
>  2 files changed, 64 insertions(+), 55 deletions(-)
>  create mode 100644 include/drm/intel/step.h
> 
> diff --git a/drivers/gpu/drm/i915/intel_step.h b/drivers/gpu/drm/i915/intel_step.h
> index 22f1d6905160..2ca36eae4b5a 100644
> --- a/drivers/gpu/drm/i915/intel_step.h
> +++ b/drivers/gpu/drm/i915/intel_step.h
> @@ -8,6 +8,8 @@
>  
>  #include <linux/types.h>
>  
> +#include <drm/intel/step.h>
> +
>  struct drm_i915_private;
>  
>  struct intel_step_info {
> @@ -19,61 +21,6 @@ struct intel_step_info {
>  	u8 media_step;
>  };
>  
> -#define STEP_ENUM_VAL(name)  STEP_##name,
> -
> -#define STEP_NAME_LIST(func)		\
> -	func(A0)			\
> -	func(A1)			\
> -	func(A2)			\
> -	func(A3)			\
> -	func(B0)			\
> -	func(B1)			\
> -	func(B2)			\
> -	func(B3)			\
> -	func(C0)			\
> -	func(C1)			\
> -	func(C2)			\
> -	func(C3)			\
> -	func(D0)			\
> -	func(D1)			\
> -	func(D2)			\
> -	func(D3)			\
> -	func(E0)			\
> -	func(E1)			\
> -	func(E2)			\
> -	func(E3)			\
> -	func(F0)			\
> -	func(F1)			\
> -	func(F2)			\
> -	func(F3)			\
> -	func(G0)			\
> -	func(G1)			\
> -	func(G2)			\
> -	func(G3)			\
> -	func(H0)			\
> -	func(H1)			\
> -	func(H2)			\
> -	func(H3)			\
> -	func(I0)			\
> -	func(I1)			\
> -	func(I2)			\
> -	func(I3)			\
> -	func(J0)			\
> -	func(J1)			\
> -	func(J2)			\
> -	func(J3)
> -
> -/*
> - * Symbolic steppings that do not match the hardware. These are valid both as gt
> - * and display steppings as symbolic names.
> - */
> -enum intel_step {
> -	STEP_NONE = 0,
> -	STEP_NAME_LIST(STEP_ENUM_VAL)
> -	STEP_FUTURE,
> -	STEP_FOREVER,
> -};
> -
>  void intel_step_init(struct drm_i915_private *i915);
>  const char *intel_step_name(enum intel_step step);
>  
> diff --git a/include/drm/intel/step.h b/include/drm/intel/step.h
> new file mode 100644
> index 000000000000..4de7520109bc
> --- /dev/null
> +++ b/include/drm/intel/step.h
> @@ -0,0 +1,62 @@
> +/* SPDX-License-Identifier: MIT */
> +/* Copyright © 2026 Intel Corporation */
> +
> +#ifndef __STEP_H__
> +#define __STEP_H__
> +
> +#define STEP_ENUM_VAL(name)  STEP_##name,
> +
> +#define STEP_NAME_LIST(func)		\
> +	func(A0)			\
> +	func(A1)			\
> +	func(A2)			\
> +	func(A3)			\
> +	func(B0)			\
> +	func(B1)			\
> +	func(B2)			\
> +	func(B3)			\
> +	func(C0)			\
> +	func(C1)			\
> +	func(C2)			\
> +	func(C3)			\
> +	func(D0)			\
> +	func(D1)			\
> +	func(D2)			\
> +	func(D3)			\
> +	func(E0)			\
> +	func(E1)			\
> +	func(E2)			\
> +	func(E3)			\
> +	func(F0)			\
> +	func(F1)			\
> +	func(F2)			\
> +	func(F3)			\
> +	func(G0)			\
> +	func(G1)			\
> +	func(G2)			\
> +	func(G3)			\
> +	func(H0)			\
> +	func(H1)			\
> +	func(H2)			\
> +	func(H3)			\
> +	func(I0)			\
> +	func(I1)			\
> +	func(I2)			\
> +	func(I3)			\
> +	func(J0)			\
> +	func(J1)			\
> +	func(J2)			\
> +	func(J3)
> +
> +/*
> + * Symbolic steppings that do not match the hardware. These are valid both as gt
> + * and display steppings as symbolic names.
> + */
> +enum intel_step {
> +	STEP_NONE = 0,
> +	STEP_NAME_LIST(STEP_ENUM_VAL)
> +	STEP_FUTURE,
> +	STEP_FOREVER,
> +};
> +
> +#endif /* __STEP_H__ */

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915
  2026-03-04 14:40 ` [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915 Jani Nikula
@ 2026-03-09  9:10   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:10 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Use the shared stepping enums from include/drm/intel/step.h.
> 
> For now, define xe_step as intel_step to avoid mass renames at the same
> time. For compat, we can remove the reverse macro.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  .../drm/xe/compat-i915-headers/intel_step.h   |  4 +-
>  drivers/gpu/drm/xe/xe_step_types.h            | 63 ++-----------------
>  2 files changed, 5 insertions(+), 62 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> index 0eabe2866f5f..cb55a659856b 100644
> --- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> +++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> @@ -6,8 +6,6 @@
>  #ifndef __INTEL_STEP_H__
>  #define __INTEL_STEP_H__
>  
> -#include "xe_step_types.h"
> -
> -#define intel_step xe_step
> +#include <drm/intel/step.h>
>  
>  #endif /* __INTEL_STEP_H__ */
> diff --git a/drivers/gpu/drm/xe/xe_step_types.h b/drivers/gpu/drm/xe/xe_step_types.h
> index d978cc2512f2..bc3ed5a8282a 100644
> --- a/drivers/gpu/drm/xe/xe_step_types.h
> +++ b/drivers/gpu/drm/xe/xe_step_types.h
> @@ -8,69 +8,14 @@
>  
>  #include <linux/types.h>
>  
> +#include <drm/intel/step.h>
> +
> +#define xe_step intel_step
> +
>  struct xe_step_info {
>  	u8 graphics;
>  	u8 media;
>  	u8 basedie;
>  };
>  
> -#define STEP_ENUM_VAL(name)  STEP_##name,
> -
> -/*
> - * Always define four minor steppings 0-3 for each stepping to match GMD ID
> - * spacing of values. See xe_step_gmdid_get().
> - */
> -#define STEP_NAME_LIST(func)		\
> -	func(A0)			\
> -	func(A1)			\
> -	func(A2)			\
> -	func(A3)			\
> -	func(B0)			\
> -	func(B1)			\
> -	func(B2)			\
> -	func(B3)			\
> -	func(C0)			\
> -	func(C1)			\
> -	func(C2)			\
> -	func(C3)			\
> -	func(D0)			\
> -	func(D1)			\
> -	func(D2)			\
> -	func(D3)			\
> -	func(E0)			\
> -	func(E1)			\
> -	func(E2)			\
> -	func(E3)			\
> -	func(F0)			\
> -	func(F1)			\
> -	func(F2)			\
> -	func(F3)			\
> -	func(G0)			\
> -	func(G1)			\
> -	func(G2)			\
> -	func(G3)			\
> -	func(H0)			\
> -	func(H1)			\
> -	func(H2)			\
> -	func(H3)			\
> -	func(I0)			\
> -	func(I1)			\
> -	func(I2)			\
> -	func(I3)			\
> -	func(J0)			\
> -	func(J1)			\
> -	func(J2)			\
> -	func(J3)
> -
> -/*
> - * Symbolic steppings that do not match the hardware. These are valid both as gt
> - * and display steppings as symbolic names.
> - */
> -enum xe_step {
> -	STEP_NONE = 0,
> -	STEP_NAME_LIST(STEP_ENUM_VAL)
> -	STEP_FUTURE,
> -	STEP_FOREVER,
> -};
> -
>  #endif

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming
  2026-03-04 14:40 ` [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming Jani Nikula
@ 2026-03-09  9:11   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:11 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Remove the xe_step macro, and use the enum intel_step name directly.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/xe/tests/xe_pci.c  | 4 ++--
>  drivers/gpu/drm/xe/xe_step.c       | 2 +-
>  drivers/gpu/drm/xe/xe_step.h       | 4 ++--
>  drivers/gpu/drm/xe/xe_step_types.h | 2 --
>  4 files changed, 5 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index f3179b31f13e..860409c579f8 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -131,12 +131,12 @@ static const char *subplatform_prefix(enum xe_subplatform s)
>  	return s == XE_SUBPLATFORM_NONE ? "" : " ";
>  }
>  
> -static const char *step_prefix(enum xe_step step)
> +static const char *step_prefix(enum intel_step step)
>  {
>  	return step == STEP_NONE ? "" : " ";
>  }
>  
> -static const char *step_name(enum xe_step step)
> +static const char *step_name(enum intel_step step)
>  {
>  	return step == STEP_NONE ? "" : xe_step_name(step);
>  }
> diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
> index 2860986f82f7..e021da35b03c 100644
> --- a/drivers/gpu/drm/xe/xe_step.c
> +++ b/drivers/gpu/drm/xe/xe_step.c
> @@ -248,7 +248,7 @@ struct xe_step_info xe_step_gmdid_get(struct xe_device *xe,
>  	case STEP_##name:	\
>  		return #name;
>  
> -const char *xe_step_name(enum xe_step step)
> +const char *xe_step_name(enum intel_step step)
>  {
>  	switch (step) {
>  	STEP_NAME_LIST(STEP_NAME_CASE);
> diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
> index 686cb59200c2..6fd680c1b45e 100644
> --- a/drivers/gpu/drm/xe/xe_step.h
> +++ b/drivers/gpu/drm/xe/xe_step.h
> @@ -16,8 +16,8 @@ struct xe_step_info xe_step_pre_gmdid_get(struct xe_device *xe);
>  struct xe_step_info xe_step_gmdid_get(struct xe_device *xe,
>  				      u32 graphics_gmdid_revid,
>  				      u32 media_gmdid_revid);
> -static inline u32 xe_step_to_gmdid(enum xe_step step) { return step - STEP_A0; }
> +static inline u32 xe_step_to_gmdid(enum intel_step step) { return step - STEP_A0; }
>  
> -const char *xe_step_name(enum xe_step step);
> +const char *xe_step_name(enum intel_step step);
>  
>  #endif
> diff --git a/drivers/gpu/drm/xe/xe_step_types.h b/drivers/gpu/drm/xe/xe_step_types.h
> index bc3ed5a8282a..bd9a7a44a67f 100644
> --- a/drivers/gpu/drm/xe/xe_step_types.h
> +++ b/drivers/gpu/drm/xe/xe_step_types.h
> @@ -10,8 +10,6 @@
>  
>  #include <drm/intel/step.h>
>  
> -#define xe_step intel_step
> -
>  struct xe_step_info {
>  	u8 graphics;
>  	u8 media;

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 8/8] drm/i915/display: switch to including common step file directly
  2026-03-04 14:40 ` [PATCH 8/8] drm/i915/display: switch to including common step file directly Jani Nikula
@ 2026-03-09  9:12   ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:12 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> Instead of using the proxy intel_step.h in display, just include the
> common step file directly where needed. This allows us to remove the
> compat intel_step.h header.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>

--
Cheers,
Luca.


>  drivers/gpu/drm/i915/display/intel_cdclk.c          |  2 +-
>  drivers/gpu/drm/i915/display/intel_ddi.c            |  2 +-
>  drivers/gpu/drm/i915/display/intel_display_device.c |  2 +-
>  drivers/gpu/drm/i915/display/intel_display_power.c  |  2 +-
>  drivers/gpu/drm/i915/display/intel_dp_mst.c         |  2 +-
>  drivers/gpu/drm/i915/display/intel_dpll_mgr.c       |  2 +-
>  drivers/gpu/drm/i915/display/intel_fbc.c            |  2 +-
>  drivers/gpu/drm/i915/display/intel_flipq.c          |  2 +-
>  drivers/gpu/drm/i915/display/intel_hdcp.c           |  2 +-
>  drivers/gpu/drm/i915/display/intel_pmdemand.c       |  2 +-
>  drivers/gpu/drm/i915/display/intel_psr.c            |  2 +-
>  drivers/gpu/drm/i915/display/skl_universal_plane.c  |  2 +-
>  drivers/gpu/drm/xe/compat-i915-headers/intel_step.h | 11 -----------
>  13 files changed, 12 insertions(+), 23 deletions(-)
>  delete mode 100644 drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> index 38331e899519..fcee3b1bfdcc 100644
> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> @@ -28,6 +28,7 @@
>  #include <drm/drm_fixed.h>
>  #include <drm/drm_print.h>
>  #include <drm/intel/intel_pcode_regs.h>
> +#include <drm/intel/step.h>
>  
>  #include "hsw_ips.h"
>  #include "intel_atomic.h"
> @@ -46,7 +47,6 @@
>  #include "intel_pci_config.h"
>  #include "intel_plane.h"
>  #include "intel_psr.h"
> -#include "intel_step.h"
>  #include "intel_vdsc.h"
>  #include "skl_watermark.h"
>  #include "skl_watermark_regs.h"
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> index 94ae583e907f..b4c84528ce5c 100644
> --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> @@ -33,6 +33,7 @@
>  #include <drm/display/drm_scdc_helper.h>
>  #include <drm/drm_print.h>
>  #include <drm/drm_privacy_screen_consumer.h>
> +#include <drm/intel/step.h>
>  
>  #include "icl_dsi.h"
>  #include "intel_alpm.h"
> @@ -80,7 +81,6 @@
>  #include "intel_psr.h"
>  #include "intel_quirks.h"
>  #include "intel_snps_phy.h"
> -#include "intel_step.h"
>  #include "intel_tc.h"
>  #include "intel_vdsc.h"
>  #include "intel_vdsc_regs.h"
> diff --git a/drivers/gpu/drm/i915/display/intel_display_device.c b/drivers/gpu/drm/i915/display/intel_display_device.c
> index f716e9a51532..361a2fca58a3 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.c
> @@ -9,6 +9,7 @@
>  #include <drm/drm_drv.h>
>  #include <drm/drm_print.h>
>  #include <drm/intel/pciids.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_cx0_phy_regs.h"
>  #include "intel_de.h"
> @@ -20,7 +21,6 @@
>  #include "intel_display_regs.h"
>  #include "intel_display_types.h"
>  #include "intel_fbc.h"
> -#include "intel_step.h"
>  
>  __diag_push();
>  __diag_ignore_all("-Woverride-init", "Allow field initialization overrides for display info");
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
> index 755935dcfe23..fa76a3ed6559 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c
> @@ -8,6 +8,7 @@
>  
>  #include <drm/drm_print.h>
>  #include <drm/intel/intel_pcode_regs.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_backlight_regs.h"
>  #include "intel_cdclk.h"
> @@ -29,7 +30,6 @@
>  #include "intel_pmdemand.h"
>  #include "intel_pps_regs.h"
>  #include "intel_snps_phy.h"
> -#include "intel_step.h"
>  #include "skl_watermark.h"
>  #include "skl_watermark_regs.h"
>  #include "vlv_sideband.h"
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> index 0bb39f9bd74e..262fd2ed278e 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> @@ -32,6 +32,7 @@
>  #include <drm/drm_fixed.h>
>  #include <drm/drm_print.h>
>  #include <drm/drm_probe_helper.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_atomic.h"
>  #include "intel_audio.h"
> @@ -55,7 +56,6 @@
>  #include "intel_link_bw.h"
>  #include "intel_pfit.h"
>  #include "intel_psr.h"
> -#include "intel_step.h"
>  #include "intel_vdsc.h"
>  #include "intel_vrr.h"
>  #include "skl_scaler.h"
> diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> index f35a9252f4e1..908a90804881 100644
> --- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> +++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
> @@ -25,6 +25,7 @@
>  #include <linux/string_helpers.h>
>  
>  #include <drm/drm_print.h>
> +#include <drm/intel/step.h>
>  
>  #include "bxt_dpio_phy_regs.h"
>  #include "intel_cx0_phy.h"
> @@ -41,7 +42,6 @@
>  #include "intel_lt_phy.h"
>  #include "intel_mg_phy_regs.h"
>  #include "intel_pch_refclk.h"
> -#include "intel_step.h"
>  #include "intel_tc.h"
>  
>  /**
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 3e9b3e532499..a9de9fe6fa28 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -44,6 +44,7 @@
>  #include <drm/drm_blend.h>
>  #include <drm/drm_fourcc.h>
>  #include <drm/drm_print.h>
> +#include <drm/intel/step.h>
>  
>  #include "i9xx_plane_regs.h"
>  #include "intel_de.h"
> @@ -58,7 +59,6 @@
>  #include "intel_fbc_regs.h"
>  #include "intel_frontbuffer.h"
>  #include "intel_parent.h"
> -#include "intel_step.h"
>  
>  #define for_each_fbc_id(__display, __fbc_id) \
>  	for ((__fbc_id) = INTEL_FBC_A; (__fbc_id) < I915_MAX_FBCS; (__fbc_id)++) \
> diff --git a/drivers/gpu/drm/i915/display/intel_flipq.c b/drivers/gpu/drm/i915/display/intel_flipq.c
> index 1e9550cb66a3..d2c69a666f0b 100644
> --- a/drivers/gpu/drm/i915/display/intel_flipq.c
> +++ b/drivers/gpu/drm/i915/display/intel_flipq.c
> @@ -6,6 +6,7 @@
>  #include <linux/pci.h>
>  
>  #include <drm/drm_print.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_crtc.h"
>  #include "intel_de.h"
> @@ -16,7 +17,6 @@
>  #include "intel_dmc_regs.h"
>  #include "intel_dsb.h"
>  #include "intel_flipq.h"
> -#include "intel_step.h"
>  #include "intel_vblank.h"
>  #include "intel_vrr.h"
>  
> diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
> index 116d2e0af878..6e86d97f8ea9 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdcp.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
> @@ -18,6 +18,7 @@
>  #include <drm/drm_print.h>
>  #include <drm/intel/i915_component.h>
>  #include <drm/intel/intel_pcode_regs.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_connector.h"
>  #include "intel_de.h"
> @@ -33,7 +34,6 @@
>  #include "intel_hdcp_regs.h"
>  #include "intel_hdcp_shim.h"
>  #include "intel_parent.h"
> -#include "intel_step.h"
>  
>  #define USE_HDCP_GSC(__display)		(DISPLAY_VER(__display) >= 14)
>  
> diff --git a/drivers/gpu/drm/i915/display/intel_pmdemand.c b/drivers/gpu/drm/i915/display/intel_pmdemand.c
> index f3db55710010..a41ff04a183c 100644
> --- a/drivers/gpu/drm/i915/display/intel_pmdemand.c
> +++ b/drivers/gpu/drm/i915/display/intel_pmdemand.c
> @@ -6,6 +6,7 @@
>  #include <linux/bitops.h>
>  
>  #include <drm/drm_print.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_atomic.h"
>  #include "intel_bw.h"
> @@ -16,7 +17,6 @@
>  #include "intel_display_trace.h"
>  #include "intel_display_utils.h"
>  #include "intel_pmdemand.h"
> -#include "intel_step.h"
>  #include "skl_watermark.h"
>  
>  struct pmdemand_params {
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c
> index 9296ca3a4ff4..a38d872dcb80 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -28,6 +28,7 @@
>  #include <drm/drm_debugfs.h>
>  #include <drm/drm_print.h>
>  #include <drm/drm_vblank.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_alpm.h"
>  #include "intel_atomic.h"
> @@ -49,7 +50,6 @@
>  #include "intel_psr.h"
>  #include "intel_psr_regs.h"
>  #include "intel_snps_phy.h"
> -#include "intel_step.h"
>  #include "intel_vblank.h"
>  #include "intel_vdsc.h"
>  #include "intel_vrr.h"
> diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> index 677f1339b7f8..91e2950c3741 100644
> --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c
> +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> @@ -8,6 +8,7 @@
>  #include <drm/drm_damage_helper.h>
>  #include <drm/drm_fourcc.h>
>  #include <drm/drm_print.h>
> +#include <drm/intel/step.h>
>  
>  #include "intel_bo.h"
>  #include "intel_color.h"
> @@ -24,7 +25,6 @@
>  #include "intel_plane.h"
>  #include "intel_psr.h"
>  #include "intel_psr_regs.h"
> -#include "intel_step.h"
>  #include "skl_scaler.h"
>  #include "skl_universal_plane.h"
>  #include "skl_universal_plane_regs.h"
> diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> deleted file mode 100644
> index cb55a659856b..000000000000
> --- a/drivers/gpu/drm/xe/compat-i915-headers/intel_step.h
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -/* SPDX-License-Identifier: MIT */
> -/*
> - * Copyright © 2023 Intel Corporation
> - */
> -
> -#ifndef __INTEL_STEP_H__
> -#define __INTEL_STEP_H__
> -
> -#include <drm/intel/step.h>
> -
> -#endif /* __INTEL_STEP_H__ */

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it
  2026-03-09  9:09   ` Luca Coelho
@ 2026-03-09  9:16     ` Luca Coelho
  0 siblings, 0 replies; 22+ messages in thread
From: Luca Coelho @ 2026-03-09  9:16 UTC (permalink / raw)
  To: Jani Nikula, intel-gfx, intel-xe

On Mon, 2026-03-09 at 11:09 +0200, Luca Coelho wrote:
> On Wed, 2026-03-04 at 16:40 +0200, Jani Nikula wrote:
> > As the first step towards using shared definitions for step name
> > enumerations, add shared include/drm/intel/step.h and switch i915 to use
> > it.
> > 
> > Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> > ---
> 
> Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
> 
> --
> Cheers,
> Luca.
> 
> 
> >  drivers/gpu/drm/i915/intel_step.h | 57 +---------------------------
> >  include/drm/intel/step.h          | 62 +++++++++++++++++++++++++++++++
> >  2 files changed, 64 insertions(+), 55 deletions(-)
> >  create mode 100644 include/drm/intel/step.h
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_step.h b/drivers/gpu/drm/i915/intel_step.h
> > index 22f1d6905160..2ca36eae4b5a 100644
> > --- a/drivers/gpu/drm/i915/intel_step.h
> > +++ b/drivers/gpu/drm/i915/intel_step.h
> > @@ -8,6 +8,8 @@
> >  
> >  #include <linux/types.h>
> >  
> > +#include <drm/intel/step.h>
> > +
> >  struct drm_i915_private;
> >  
> >  struct intel_step_info {
> > @@ -19,61 +21,6 @@ struct intel_step_info {
> >  	u8 media_step;
> >  };
> >  
> > -#define STEP_ENUM_VAL(name)  STEP_##name,
> > -
> > -#define STEP_NAME_LIST(func)		\
> > -	func(A0)			\
> > -	func(A1)			\
> > -	func(A2)			\
> > -	func(A3)			\
> > -	func(B0)			\
> > -	func(B1)			\
> > -	func(B2)			\
> > -	func(B3)			\
> > -	func(C0)			\
> > -	func(C1)			\
> > -	func(C2)			\
> > -	func(C3)			\
> > -	func(D0)			\
> > -	func(D1)			\
> > -	func(D2)			\
> > -	func(D3)			\
> > -	func(E0)			\
> > -	func(E1)			\
> > -	func(E2)			\
> > -	func(E3)			\
> > -	func(F0)			\
> > -	func(F1)			\
> > -	func(F2)			\
> > -	func(F3)			\
> > -	func(G0)			\
> > -	func(G1)			\
> > -	func(G2)			\
> > -	func(G3)			\
> > -	func(H0)			\
> > -	func(H1)			\
> > -	func(H2)			\
> > -	func(H3)			\
> > -	func(I0)			\
> > -	func(I1)			\
> > -	func(I2)			\
> > -	func(I3)			\
> > -	func(J0)			\
> > -	func(J1)			\
> > -	func(J2)			\
> > -	func(J3)
> > -
> > -/*
> > - * Symbolic steppings that do not match the hardware. These are valid both as gt
> > - * and display steppings as symbolic names.
> > - */
> > -enum intel_step {
> > -	STEP_NONE = 0,
> > -	STEP_NAME_LIST(STEP_ENUM_VAL)
> > -	STEP_FUTURE,
> > -	STEP_FOREVER,
> > -};
> > -
> >  void intel_step_init(struct drm_i915_private *i915);
> >  const char *intel_step_name(enum intel_step step);
> >  
> > diff --git a/include/drm/intel/step.h b/include/drm/intel/step.h
> > new file mode 100644
> > index 000000000000..4de7520109bc
> > --- /dev/null
> > +++ b/include/drm/intel/step.h
> > @@ -0,0 +1,62 @@
> > +/* SPDX-License-Identifier: MIT */
> > +/* Copyright © 2026 Intel Corporation */
> > +
> > +#ifndef __STEP_H__
> > +#define __STEP_H__
> > +
> > +#define STEP_ENUM_VAL(name)  STEP_##name,
> > +
> > +#define STEP_NAME_LIST(func)		\
> > +	func(A0)			\
> > +	func(A1)			\
> > +	func(A2)			\
> > +	func(A3)			\
> > +	func(B0)			\
> > +	func(B1)			\
> > +	func(B2)			\
> > +	func(B3)			\
> > +	func(C0)			\
> > +	func(C1)			\
> > +	func(C2)			\
> > +	func(C3)			\
> > +	func(D0)			\
> > +	func(D1)			\
> > +	func(D2)			\
> > +	func(D3)			\
> > +	func(E0)			\
> > +	func(E1)			\
> > +	func(E2)			\
> > +	func(E3)			\
> > +	func(F0)			\
> > +	func(F1)			\
> > +	func(F2)			\
> > +	func(F3)			\
> > +	func(G0)			\
> > +	func(G1)			\
> > +	func(G2)			\
> > +	func(G3)			\
> > +	func(H0)			\
> > +	func(H1)			\
> > +	func(H2)			\
> > +	func(H3)			\
> > +	func(I0)			\
> > +	func(I1)			\
> > +	func(I2)			\
> > +	func(I3)			\
> > +	func(J0)			\
> > +	func(J1)			\
> > +	func(J2)			\
> > +	func(J3)
> > +
> > +/*
> > + * Symbolic steppings that do not match the hardware. These are valid both as gt
> > + * and display steppings as symbolic names.
> > + */
> > +enum intel_step {

Just a small thing that came to my mind, intel_step sounds a bit too
generic at this level, I think?

This is specifically a drm/intel enum, but there surely are other Intel
steps out there?

This doesn't change my r-b, but just something to consider.

--
Cheers,
Luca.


> > +	STEP_NONE = 0,
> > +	STEP_NAME_LIST(STEP_ENUM_VAL)
> > +	STEP_FUTURE,
> > +	STEP_FOREVER,
> > +};
> > +
> > +#endif /* __STEP_H__ */

^ permalink raw reply	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2026-03-09  9:16 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-04 14:40 [PATCH 0/8] drm/{i915,xe}: sort out step enums between the drivers Jani Nikula
2026-03-04 14:40 ` [PATCH 1/8] drm/i915/dmc: simplify stepping info initialization Jani Nikula
2026-03-09  8:18   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 2/8] drm/i915/display: add step name in display runtime info Jani Nikula
2026-03-09  8:24   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 3/8] drm/i915/dmc: use step name from " Jani Nikula
2026-03-09  8:35   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 4/8] drm/xe/compat: remove intel_step_name macro Jani Nikula
2026-03-09  9:07   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 5/8] drm/intel: add shared step.h and switch i915 to use it Jani Nikula
2026-03-09  9:09   ` Luca Coelho
2026-03-09  9:16     ` Luca Coelho
2026-03-04 14:40 ` [PATCH 6/8] drm/xe/step: switch to the shared step definitions with i915 Jani Nikula
2026-03-09  9:10   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 7/8] drm/xe/step: switch from enum xe_step to intel_step naming Jani Nikula
2026-03-09  9:11   ` Luca Coelho
2026-03-04 14:40 ` [PATCH 8/8] drm/i915/display: switch to including common step file directly Jani Nikula
2026-03-09  9:12   ` Luca Coelho
2026-03-05 19:23 ` ✗ CI.checkpatch: warning for drm/{i915,xe}: sort out step enums between the drivers Patchwork
2026-03-05 19:25 ` ✓ CI.KUnit: success " Patchwork
2026-03-05 20:11 ` ✓ Xe.CI.BAT: " Patchwork
2026-03-06 11:35 ` ✗ Xe.CI.FULL: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox