* [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G
@ 2019-12-13 17:43 Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[] Ville Syrjala
` (6 more replies)
0 siblings, 7 replies; 11+ messages in thread
From: Ville Syrjala @ 2019-12-13 17:43 UTC (permalink / raw)
To: dri-devel; +Cc: Hans Verkuil, intel-gfx, Thomas Anderson
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Review feedback addressed. I considered changing the approach
based on Tom's comments but in the end decided that probably
better to go with this for now. We can massage it later if
required.
Cc: Hans Verkuil <hansverk@cisco.com>
Cc: Manasi Navare <manasi.d.navare@intel.com>
Cc: Thomas Anderson <thomasanderson@google.com>
Ville Syrjälä (4):
drm/edid: Abstract away cea_edid_modes[]
drm/edid: Add CTA-861-G modes with VIC >= 193
drm/edid: Throw away the dummy VIC 0 cea mode
drm/edid: Make sure the CEA mode arrays have the correct amount of
modes
drivers/gpu/drm/drm_edid.c | 217 +++++++++++++++++++++++++++++++++----
1 file changed, 193 insertions(+), 24 deletions(-)
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[]
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
@ 2019-12-13 17:43 ` Ville Syrjala
2019-12-13 21:03 ` Tom Anderson
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 2/4] drm/edid: Add CTA-861-G modes with VIC >= 193 Ville Syrjala
` (5 subsequent siblings)
6 siblings, 1 reply; 11+ messages in thread
From: Ville Syrjala @ 2019-12-13 17:43 UTC (permalink / raw)
To: dri-devel; +Cc: Hans Verkuil, intel-gfx, Tom Anderson
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
We're going to need two cea mode tables (one for VICs < 128,
another one for VICs >= 193). To that end replace the direct
edid_cea_modes[] lookups with a function call. And we'll rename
the array to edid_cea_modes_0[] to indicate how it's to be
indexed.
v2: Fix typos (Tom)
Drop the pointless NULL checks in the loops (Tom)
Assign when declaring (Tom)
Improve the comment for cea_modes_*[] to indicate
that one should always use cea_mode_for_vic() (Tom)
Cc: Tom Anderson <thomasanderson@google.com>
Cc: Hans Verkuil <hansverk@cisco.com>
Cc: Manasi Navare <manasi.d.navare@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/drm_edid.c | 67 +++++++++++++++++++++++++-------------
1 file changed, 45 insertions(+), 22 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 5b33b7cfd645..00a543b9daab 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -710,12 +710,11 @@ static const struct minimode extra_modes[] = {
};
/*
- * Probably taken from CEA-861 spec.
- * This table is converted from xorg's hw/xfree86/modes/xf86EdidModes.c.
+ * From CEA/CTA-861 spec.
*
- * Index using the VIC.
+ * Do not access directly, instead always use cea_mode_for_vic().
*/
-static const struct drm_display_mode edid_cea_modes[] = {
+static const struct drm_display_mode edid_cea_modes_0[] = {
/* 0 - dummy, VICs start at 1 */
{ },
/* 1 - 640x480@60Hz 4:3 */
@@ -3071,6 +3070,25 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
return cea;
}
+static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
+{
+ if (!vic)
+ return NULL;
+ if (vic < ARRAY_SIZE(edid_cea_modes_0))
+ return &edid_cea_modes_0[vic];
+ return NULL;
+}
+
+static u8 cea_num_vics(void)
+{
+ return ARRAY_SIZE(edid_cea_modes_0);
+}
+
+static u8 cea_next_vic(u8 vic)
+{
+ return vic + 1;
+}
+
/*
* Calculate the alternate clock for the CEA mode
* (60Hz vs. 59.94Hz etc.)
@@ -3108,14 +3126,14 @@ cea_mode_alternate_timings(u8 vic, struct drm_display_mode *mode)
* get the other variants by simply increasing the
* vertical front porch length.
*/
- BUILD_BUG_ON(edid_cea_modes[8].vtotal != 262 ||
- edid_cea_modes[9].vtotal != 262 ||
- edid_cea_modes[12].vtotal != 262 ||
- edid_cea_modes[13].vtotal != 262 ||
- edid_cea_modes[23].vtotal != 312 ||
- edid_cea_modes[24].vtotal != 312 ||
- edid_cea_modes[27].vtotal != 312 ||
- edid_cea_modes[28].vtotal != 312);
+ BUILD_BUG_ON(cea_mode_for_vic(8)->vtotal != 262 ||
+ cea_mode_for_vic(9)->vtotal != 262 ||
+ cea_mode_for_vic(12)->vtotal != 262 ||
+ cea_mode_for_vic(13)->vtotal != 262 ||
+ cea_mode_for_vic(23)->vtotal != 312 ||
+ cea_mode_for_vic(24)->vtotal != 312 ||
+ cea_mode_for_vic(27)->vtotal != 312 ||
+ cea_mode_for_vic(28)->vtotal != 312);
if (((vic == 8 || vic == 9 ||
vic == 12 || vic == 13) && mode->vtotal < 263) ||
@@ -3143,8 +3161,8 @@ static u8 drm_match_cea_mode_clock_tolerance(const struct drm_display_mode *to_m
if (to_match->picture_aspect_ratio)
match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
- for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
- struct drm_display_mode cea_mode = edid_cea_modes[vic];
+ for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
+ struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
unsigned int clock1, clock2;
/* Check both 60Hz and 59.94Hz */
@@ -3182,8 +3200,8 @@ u8 drm_match_cea_mode(const struct drm_display_mode *to_match)
if (to_match->picture_aspect_ratio)
match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
- for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
- struct drm_display_mode cea_mode = edid_cea_modes[vic];
+ for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
+ struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
unsigned int clock1, clock2;
/* Check both 60Hz and 59.94Hz */
@@ -3206,12 +3224,17 @@ EXPORT_SYMBOL(drm_match_cea_mode);
static bool drm_valid_cea_vic(u8 vic)
{
- return vic > 0 && vic < ARRAY_SIZE(edid_cea_modes);
+ return cea_mode_for_vic(vic) != NULL;
}
static enum hdmi_picture_aspect drm_get_cea_aspect_ratio(const u8 video_code)
{
- return edid_cea_modes[video_code].picture_aspect_ratio;
+ const struct drm_display_mode *mode = cea_mode_for_vic(video_code);
+
+ if (mode)
+ return mode->picture_aspect_ratio;
+
+ return HDMI_PICTURE_ASPECT_NONE;
}
static enum hdmi_picture_aspect drm_get_hdmi_aspect_ratio(const u8 video_code)
@@ -3323,7 +3346,7 @@ add_alternate_cea_modes(struct drm_connector *connector, struct edid *edid)
unsigned int clock1, clock2;
if (drm_valid_cea_vic(vic)) {
- cea_mode = &edid_cea_modes[vic];
+ cea_mode = cea_mode_for_vic(vic);
clock2 = cea_mode_alternate_clock(cea_mode);
} else {
vic = drm_match_hdmi_mode(mode);
@@ -3398,7 +3421,7 @@ drm_display_mode_from_vic_index(struct drm_connector *connector,
if (!drm_valid_cea_vic(vic))
return NULL;
- newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
+ newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
if (!newmode)
return NULL;
@@ -3432,7 +3455,7 @@ static int do_y420vdb_modes(struct drm_connector *connector,
if (!drm_valid_cea_vic(vic))
continue;
- newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
+ newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
if (!newmode)
break;
bitmap_set(hdmi->y420_vdb_modes, vic, 1);
@@ -4001,7 +4024,7 @@ static void fixup_detailed_cea_mode_clock(struct drm_display_mode *mode)
vic = drm_match_cea_mode_clock_tolerance(mode, 5);
if (drm_valid_cea_vic(vic)) {
type = "CEA";
- cea_mode = &edid_cea_modes[vic];
+ cea_mode = cea_mode_for_vic(vic);
clock1 = cea_mode->clock;
clock2 = cea_mode_alternate_clock(cea_mode);
} else {
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v4 2/4] drm/edid: Add CTA-861-G modes with VIC >= 193
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[] Ville Syrjala
@ 2019-12-13 17:43 ` Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode Ville Syrjala
` (4 subsequent siblings)
6 siblings, 0 replies; 11+ messages in thread
From: Ville Syrjala @ 2019-12-13 17:43 UTC (permalink / raw)
To: dri-devel; +Cc: Hans Verkuil, intel-gfx, Thomas Anderson
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Add a second table to the cea modes with VIC >= 193.
v2: Improve the comment for cea_modes_*[] to indicate
that one should always use cea_mode_for_vic() (Tom)
Cc: Hans Verkuil <hansverk@cisco.com>
Reviewed-by: Manasi Navare <manasi.d.navare@intel.com>
Reviewed-by: Thomas Anderson <thomasanderson@google.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/drm_edid.c | 151 ++++++++++++++++++++++++++++++++++++-
1 file changed, 149 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 00a543b9daab..2787ad0ef881 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -1379,6 +1379,149 @@ static const struct drm_display_mode edid_cea_modes_0[] = {
.vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
};
+/*
+ * From CEA/CTA-861 spec.
+ *
+ * Do not access directly, instead always use cea_mode_for_vic().
+ */
+static const struct drm_display_mode edid_cea_modes_193[] = {
+ /* 193 - 5120x2160@120Hz 64:27 */
+ { DRM_MODE("5120x2160", DRM_MODE_TYPE_DRIVER, 1485000, 5120, 5284,
+ 5372, 5500, 0, 2160, 2168, 2178, 2250, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 194 - 7680x4320@24Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 10232,
+ 10408, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 24, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 195 - 7680x4320@25Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 10032,
+ 10208, 10800, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 25, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 196 - 7680x4320@30Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 8232,
+ 8408, 9000, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 30, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 197 - 7680x4320@48Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 10232,
+ 10408, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 48, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 198 - 7680x4320@50Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 10032,
+ 10208, 10800, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 199 - 7680x4320@60Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 8232,
+ 8408, 9000, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 200 - 7680x4320@100Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 4752000, 7680, 9792,
+ 9968, 10560, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 201 - 7680x4320@120Hz 16:9 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 4752000, 7680, 8032,
+ 8208, 8800, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
+ /* 202 - 7680x4320@24Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 10232,
+ 10408, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 24, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 203 - 7680x4320@25Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 10032,
+ 10208, 10800, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 25, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 204 - 7680x4320@30Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 1188000, 7680, 8232,
+ 8408, 9000, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 30, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 205 - 7680x4320@48Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 10232,
+ 10408, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 48, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 206 - 7680x4320@50Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 10032,
+ 10208, 10800, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 207 - 7680x4320@60Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 2376000, 7680, 8232,
+ 8408, 9000, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 208 - 7680x4320@100Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 4752000, 7680, 9792,
+ 9968, 10560, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 209 - 7680x4320@120Hz 64:27 */
+ { DRM_MODE("7680x4320", DRM_MODE_TYPE_DRIVER, 4752000, 7680, 8032,
+ 8208, 8800, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 210 - 10240x4320@24Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 1485000, 10240, 11732,
+ 11908, 12500, 0, 4320, 4336, 4356, 4950, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 24, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 211 - 10240x4320@25Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 1485000, 10240, 12732,
+ 12908, 13500, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 25, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 212 - 10240x4320@30Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 1485000, 10240, 10528,
+ 10704, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 30, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 213 - 10240x4320@48Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 2970000, 10240, 11732,
+ 11908, 12500, 0, 4320, 4336, 4356, 4950, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 48, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 214 - 10240x4320@50Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 2970000, 10240, 12732,
+ 12908, 13500, 0, 4320, 4336, 4356, 4400, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 215 - 10240x4320@60Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 2970000, 10240, 10528,
+ 10704, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 216 - 10240x4320@100Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 5940000, 10240, 12432,
+ 12608, 13200, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 217 - 10240x4320@120Hz 64:27 */
+ { DRM_MODE("10240x4320", DRM_MODE_TYPE_DRIVER, 5940000, 10240, 10528,
+ 10704, 11000, 0, 4320, 4336, 4356, 4500, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_64_27, },
+ /* 218 - 4096x2160@100Hz 256:135 */
+ { DRM_MODE("4096x2160", DRM_MODE_TYPE_DRIVER, 1188000, 4096, 4896,
+ 4984, 5280, 0, 2160, 2168, 2178, 2250, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_256_135, },
+ /* 219 - 4096x2160@120Hz 256:135 */
+ { DRM_MODE("4096x2160", DRM_MODE_TYPE_DRIVER, 1188000, 4096, 4184,
+ 4272, 4400, 0, 2160, 2168, 2178, 2250, 0,
+ DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC),
+ .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_256_135, },
+};
+
/*
* HDMI 1.4 4k modes. Index using the VIC.
*/
@@ -3076,17 +3219,21 @@ static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
return NULL;
if (vic < ARRAY_SIZE(edid_cea_modes_0))
return &edid_cea_modes_0[vic];
+ if (vic >= 193 && vic < 193 + ARRAY_SIZE(edid_cea_modes_193))
+ return &edid_cea_modes_193[vic - 193];
return NULL;
}
static u8 cea_num_vics(void)
{
- return ARRAY_SIZE(edid_cea_modes_0);
+ return 193 + ARRAY_SIZE(edid_cea_modes_193);
}
static u8 cea_next_vic(u8 vic)
{
- return vic + 1;
+ if (++vic == ARRAY_SIZE(edid_cea_modes_0))
+ vic = 193;
+ return vic;
}
/*
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[] Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 2/4] drm/edid: Add CTA-861-G modes with VIC >= 193 Ville Syrjala
@ 2019-12-13 17:43 ` Ville Syrjala
2019-12-13 21:05 ` Tom Anderson
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 4/4] drm/edid: Make sure the CEA mode arrays have the correct amount of modes Ville Syrjala
` (3 subsequent siblings)
6 siblings, 1 reply; 11+ messages in thread
From: Ville Syrjala @ 2019-12-13 17:43 UTC (permalink / raw)
To: dri-devel; +Cc: Hans Verkuil, intel-gfx, Tom Anderson
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Now that the cea mode handling is not 100% tied to the single
array the dummy VIC 0 mode is pretty much pointles. Throw it
out.
v2: Rebase
Cc: Tom Anderson <thomasanderson@google.com>
Cc: Hans Verkuil <hansverk@cisco.com>
Cc: Manasi Navare <manasi.d.navare@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/drm_edid.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 2787ad0ef881..8bc69da53c2e 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -714,9 +714,7 @@ static const struct minimode extra_modes[] = {
*
* Do not access directly, instead always use cea_mode_for_vic().
*/
-static const struct drm_display_mode edid_cea_modes_0[] = {
- /* 0 - dummy, VICs start at 1 */
- { },
+static const struct drm_display_mode edid_cea_modes_1[] = {
/* 1 - 640x480@60Hz 4:3 */
{ DRM_MODE("640x480", DRM_MODE_TYPE_DRIVER, 25175, 640, 656,
752, 800, 0, 480, 490, 492, 525, 0,
@@ -3215,10 +3213,8 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
{
- if (!vic)
- return NULL;
- if (vic < ARRAY_SIZE(edid_cea_modes_0))
- return &edid_cea_modes_0[vic];
+ if (vic >= 1 && vic < 1 + ARRAY_SIZE(edid_cea_modes_1))
+ return &edid_cea_modes_1[vic - 1];
if (vic >= 193 && vic < 193 + ARRAY_SIZE(edid_cea_modes_193))
return &edid_cea_modes_193[vic - 193];
return NULL;
@@ -3231,7 +3227,7 @@ static u8 cea_num_vics(void)
static u8 cea_next_vic(u8 vic)
{
- if (++vic == ARRAY_SIZE(edid_cea_modes_0))
+ if (++vic == 1 + ARRAY_SIZE(edid_cea_modes_1))
vic = 193;
return vic;
}
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v4 4/4] drm/edid: Make sure the CEA mode arrays have the correct amount of modes
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
` (2 preceding siblings ...)
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode Ville Syrjala
@ 2019-12-13 17:43 ` Ville Syrjala
2019-12-14 0:29 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/edid: Add new modes from CTA-861-G (rev3) Patchwork
` (2 subsequent siblings)
6 siblings, 0 replies; 11+ messages in thread
From: Ville Syrjala @ 2019-12-13 17:43 UTC (permalink / raw)
To: dri-devel; +Cc: Hans Verkuil, intel-gfx, Thomas Anderson
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
We depend on a specific relationship between the VIC number and the
index in the CEA mode arrays. Assert that the arrays have the expected
size to make sure we've not accidentally left holes in them.
v2: Pimp the BUILD_BUG_ON()s
v3: Fix typos (Manasi)
Cc: Hans Verkuil <hansverk@cisco.com>
Reviewed-by: Manasi Navare <manasi.d.navare@intel.com>
Reviewed-by: Thomas Anderson <thomasanderson@google.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/drm_edid.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 8bc69da53c2e..ec5b88120428 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -3213,6 +3213,9 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
{
+ BUILD_BUG_ON(1 + ARRAY_SIZE(edid_cea_modes_1) - 1 != 127);
+ BUILD_BUG_ON(193 + ARRAY_SIZE(edid_cea_modes_193) - 1 != 219);
+
if (vic >= 1 && vic < 1 + ARRAY_SIZE(edid_cea_modes_1))
return &edid_cea_modes_1[vic - 1];
if (vic >= 193 && vic < 193 + ARRAY_SIZE(edid_cea_modes_193))
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[]
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[] Ville Syrjala
@ 2019-12-13 21:03 ` Tom Anderson
2019-12-16 15:23 ` Ville Syrjälä
0 siblings, 1 reply; 11+ messages in thread
From: Tom Anderson @ 2019-12-13 21:03 UTC (permalink / raw)
To: Ville Syrjala; +Cc: Hans Verkuil, intel-gfx, dri-devel
Latest patch looks good to me, thanks for the changes!
Reviewed-by: Thomas Anderson <thomasanderson@google.com>
On Fri, Dec 13, 2019 at 07:43:45PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> We're going to need two cea mode tables (one for VICs < 128,
> another one for VICs >= 193). To that end replace the direct
> edid_cea_modes[] lookups with a function call. And we'll rename
> the array to edid_cea_modes_0[] to indicate how it's to be
> indexed.
>
> v2: Fix typos (Tom)
> Drop the pointless NULL checks in the loops (Tom)
> Assign when declaring (Tom)
> Improve the comment for cea_modes_*[] to indicate
> that one should always use cea_mode_for_vic() (Tom)
>
> Cc: Tom Anderson <thomasanderson@google.com>
> Cc: Hans Verkuil <hansverk@cisco.com>
> Cc: Manasi Navare <manasi.d.navare@intel.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/drm_edid.c | 67 +++++++++++++++++++++++++-------------
> 1 file changed, 45 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 5b33b7cfd645..00a543b9daab 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -710,12 +710,11 @@ static const struct minimode extra_modes[] = {
> };
>
> /*
> - * Probably taken from CEA-861 spec.
> - * This table is converted from xorg's hw/xfree86/modes/xf86EdidModes.c.
> + * From CEA/CTA-861 spec.
> *
> - * Index using the VIC.
> + * Do not access directly, instead always use cea_mode_for_vic().
> */
> -static const struct drm_display_mode edid_cea_modes[] = {
> +static const struct drm_display_mode edid_cea_modes_0[] = {
> /* 0 - dummy, VICs start at 1 */
> { },
> /* 1 - 640x480@60Hz 4:3 */
> @@ -3071,6 +3070,25 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
> return cea;
> }
>
> +static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
> +{
> + if (!vic)
> + return NULL;
> + if (vic < ARRAY_SIZE(edid_cea_modes_0))
> + return &edid_cea_modes_0[vic];
> + return NULL;
> +}
> +
> +static u8 cea_num_vics(void)
> +{
> + return ARRAY_SIZE(edid_cea_modes_0);
> +}
> +
> +static u8 cea_next_vic(u8 vic)
> +{
> + return vic + 1;
> +}
> +
> /*
> * Calculate the alternate clock for the CEA mode
> * (60Hz vs. 59.94Hz etc.)
> @@ -3108,14 +3126,14 @@ cea_mode_alternate_timings(u8 vic, struct drm_display_mode *mode)
> * get the other variants by simply increasing the
> * vertical front porch length.
> */
> - BUILD_BUG_ON(edid_cea_modes[8].vtotal != 262 ||
> - edid_cea_modes[9].vtotal != 262 ||
> - edid_cea_modes[12].vtotal != 262 ||
> - edid_cea_modes[13].vtotal != 262 ||
> - edid_cea_modes[23].vtotal != 312 ||
> - edid_cea_modes[24].vtotal != 312 ||
> - edid_cea_modes[27].vtotal != 312 ||
> - edid_cea_modes[28].vtotal != 312);
> + BUILD_BUG_ON(cea_mode_for_vic(8)->vtotal != 262 ||
> + cea_mode_for_vic(9)->vtotal != 262 ||
> + cea_mode_for_vic(12)->vtotal != 262 ||
> + cea_mode_for_vic(13)->vtotal != 262 ||
> + cea_mode_for_vic(23)->vtotal != 312 ||
> + cea_mode_for_vic(24)->vtotal != 312 ||
> + cea_mode_for_vic(27)->vtotal != 312 ||
> + cea_mode_for_vic(28)->vtotal != 312);
>
> if (((vic == 8 || vic == 9 ||
> vic == 12 || vic == 13) && mode->vtotal < 263) ||
> @@ -3143,8 +3161,8 @@ static u8 drm_match_cea_mode_clock_tolerance(const struct drm_display_mode *to_m
> if (to_match->picture_aspect_ratio)
> match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
>
> - for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
> - struct drm_display_mode cea_mode = edid_cea_modes[vic];
> + for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
> + struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
> unsigned int clock1, clock2;
>
> /* Check both 60Hz and 59.94Hz */
> @@ -3182,8 +3200,8 @@ u8 drm_match_cea_mode(const struct drm_display_mode *to_match)
> if (to_match->picture_aspect_ratio)
> match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
>
> - for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
> - struct drm_display_mode cea_mode = edid_cea_modes[vic];
> + for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
> + struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
> unsigned int clock1, clock2;
>
> /* Check both 60Hz and 59.94Hz */
> @@ -3206,12 +3224,17 @@ EXPORT_SYMBOL(drm_match_cea_mode);
>
> static bool drm_valid_cea_vic(u8 vic)
> {
> - return vic > 0 && vic < ARRAY_SIZE(edid_cea_modes);
> + return cea_mode_for_vic(vic) != NULL;
> }
>
> static enum hdmi_picture_aspect drm_get_cea_aspect_ratio(const u8 video_code)
> {
> - return edid_cea_modes[video_code].picture_aspect_ratio;
> + const struct drm_display_mode *mode = cea_mode_for_vic(video_code);
> +
> + if (mode)
> + return mode->picture_aspect_ratio;
> +
> + return HDMI_PICTURE_ASPECT_NONE;
> }
>
> static enum hdmi_picture_aspect drm_get_hdmi_aspect_ratio(const u8 video_code)
> @@ -3323,7 +3346,7 @@ add_alternate_cea_modes(struct drm_connector *connector, struct edid *edid)
> unsigned int clock1, clock2;
>
> if (drm_valid_cea_vic(vic)) {
> - cea_mode = &edid_cea_modes[vic];
> + cea_mode = cea_mode_for_vic(vic);
> clock2 = cea_mode_alternate_clock(cea_mode);
> } else {
> vic = drm_match_hdmi_mode(mode);
> @@ -3398,7 +3421,7 @@ drm_display_mode_from_vic_index(struct drm_connector *connector,
> if (!drm_valid_cea_vic(vic))
> return NULL;
>
> - newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
> + newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
> if (!newmode)
> return NULL;
>
> @@ -3432,7 +3455,7 @@ static int do_y420vdb_modes(struct drm_connector *connector,
> if (!drm_valid_cea_vic(vic))
> continue;
>
> - newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
> + newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
> if (!newmode)
> break;
> bitmap_set(hdmi->y420_vdb_modes, vic, 1);
> @@ -4001,7 +4024,7 @@ static void fixup_detailed_cea_mode_clock(struct drm_display_mode *mode)
> vic = drm_match_cea_mode_clock_tolerance(mode, 5);
> if (drm_valid_cea_vic(vic)) {
> type = "CEA";
> - cea_mode = &edid_cea_modes[vic];
> + cea_mode = cea_mode_for_vic(vic);
> clock1 = cea_mode->clock;
> clock2 = cea_mode_alternate_clock(cea_mode);
> } else {
> --
> 2.23.0
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode Ville Syrjala
@ 2019-12-13 21:05 ` Tom Anderson
0 siblings, 0 replies; 11+ messages in thread
From: Tom Anderson @ 2019-12-13 21:05 UTC (permalink / raw)
To: Ville Syrjala; +Cc: Hans Verkuil, intel-gfx, dri-devel
lgtm
Reviewed-by: Thomas Anderson <thomasanderson@google.com>
On Fri, Dec 13, 2019 at 07:43:47PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Now that the cea mode handling is not 100% tied to the single
> array the dummy VIC 0 mode is pretty much pointles. Throw it
> out.
>
> v2: Rebase
>
> Cc: Tom Anderson <thomasanderson@google.com>
> Cc: Hans Verkuil <hansverk@cisco.com>
> Cc: Manasi Navare <manasi.d.navare@intel.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/drm_edid.c | 12 ++++--------
> 1 file changed, 4 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 2787ad0ef881..8bc69da53c2e 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -714,9 +714,7 @@ static const struct minimode extra_modes[] = {
> *
> * Do not access directly, instead always use cea_mode_for_vic().
> */
> -static const struct drm_display_mode edid_cea_modes_0[] = {
> - /* 0 - dummy, VICs start at 1 */
> - { },
> +static const struct drm_display_mode edid_cea_modes_1[] = {
> /* 1 - 640x480@60Hz 4:3 */
> { DRM_MODE("640x480", DRM_MODE_TYPE_DRIVER, 25175, 640, 656,
> 752, 800, 0, 480, 490, 492, 525, 0,
> @@ -3215,10 +3213,8 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
>
> static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
> {
> - if (!vic)
> - return NULL;
> - if (vic < ARRAY_SIZE(edid_cea_modes_0))
> - return &edid_cea_modes_0[vic];
> + if (vic >= 1 && vic < 1 + ARRAY_SIZE(edid_cea_modes_1))
> + return &edid_cea_modes_1[vic - 1];
> if (vic >= 193 && vic < 193 + ARRAY_SIZE(edid_cea_modes_193))
> return &edid_cea_modes_193[vic - 193];
> return NULL;
> @@ -3231,7 +3227,7 @@ static u8 cea_num_vics(void)
>
> static u8 cea_next_vic(u8 vic)
> {
> - if (++vic == ARRAY_SIZE(edid_cea_modes_0))
> + if (++vic == 1 + ARRAY_SIZE(edid_cea_modes_1))
> vic = 193;
> return vic;
> }
> --
> 2.23.0
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/edid: Add new modes from CTA-861-G (rev3)
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
` (3 preceding siblings ...)
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 4/4] drm/edid: Make sure the CEA mode arrays have the correct amount of modes Ville Syrjala
@ 2019-12-14 0:29 ` Patchwork
2019-12-14 1:04 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2019-12-15 0:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-12-14 0:29 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
== Series Details ==
Series: drm/edid: Add new modes from CTA-861-G (rev3)
URL : https://patchwork.freedesktop.org/series/63554/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f65c5faeb690 drm/edid: Abstract away cea_edid_modes[]
-:122: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "cea_mode_for_vic"
#122: FILE: drivers/gpu/drm/drm_edid.c:3227:
+ return cea_mode_for_vic(vic) != NULL;
total: 0 errors, 0 warnings, 1 checks, 133 lines checked
1e2d560339b9 drm/edid: Add CTA-861-G modes with VIC >= 193
04407c04c8f5 drm/edid: Throw away the dummy VIC 0 cea mode
1eb2087450a1 drm/edid: Make sure the CEA mode arrays have the correct amount of modes
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/edid: Add new modes from CTA-861-G (rev3)
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
` (4 preceding siblings ...)
2019-12-14 0:29 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/edid: Add new modes from CTA-861-G (rev3) Patchwork
@ 2019-12-14 1:04 ` Patchwork
2019-12-15 0:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-12-14 1:04 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
== Series Details ==
Series: drm/edid: Add new modes from CTA-861-G (rev3)
URL : https://patchwork.freedesktop.org/series/63554/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7566 -> Patchwork_15754
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/index.html
Known issues
------------
Here are the changes found in Patchwork_15754 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_close_race@basic-threads:
- fi-byt-j1900: [PASS][1] -> [TIMEOUT][2] ([i915#816])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-byt-j1900/igt@gem_close_race@basic-threads.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-byt-j1900/igt@gem_close_race@basic-threads.html
#### Possible fixes ####
* igt@gem_close_race@basic-threads:
- fi-byt-n2820: [TIMEOUT][3] ([i915#816]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-byt-n2820/igt@gem_close_race@basic-threads.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-byt-n2820/igt@gem_close_race@basic-threads.html
* igt@gem_exec_gttfill@basic:
- {fi-tgl-u}: [INCOMPLETE][5] ([fdo#111593]) -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-tgl-u/igt@gem_exec_gttfill@basic.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-tgl-u/igt@gem_exec_gttfill@basic.html
* igt@gem_sync@basic-all:
- fi-tgl-y: [INCOMPLETE][7] ([i915#470]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-tgl-y/igt@gem_sync@basic-all.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-tgl-y/igt@gem_sync@basic-all.html
* igt@i915_selftest@live_blt:
- fi-ivb-3770: [DMESG-FAIL][9] ([i915#725]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-ivb-3770/igt@i915_selftest@live_blt.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-ivb-3770/igt@i915_selftest@live_blt.html
* igt@i915_selftest@live_gem_contexts:
- fi-skl-lmem: [INCOMPLETE][11] ([i915#424]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
- fi-hsw-peppy: [INCOMPLETE][13] ([i915#694]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-hsw-peppy/igt@i915_selftest@live_gem_contexts.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-hsw-peppy/igt@i915_selftest@live_gem_contexts.html
* igt@i915_selftest@live_requests:
- fi-hsw-4770: [INCOMPLETE][15] ([i915#773]) -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-hsw-4770/igt@i915_selftest@live_requests.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-hsw-4770/igt@i915_selftest@live_requests.html
#### Warnings ####
* igt@i915_pm_rpm@basic-rte:
- fi-kbl-guc: [SKIP][17] ([fdo#109271]) -> [FAIL][18] ([i915#579])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-kbl-guc/igt@i915_pm_rpm@basic-rte.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-kbl-guc/igt@i915_pm_rpm@basic-rte.html
* igt@kms_busy@basic-flip-pipe-b:
- fi-kbl-x1275: [DMESG-WARN][19] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][20] ([i915#62] / [i915#92]) +5 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][21] ([i915#62] / [i915#92]) -> [DMESG-WARN][22] ([i915#62] / [i915#92] / [i915#95]) +6 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/fi-kbl-x1275/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/fi-kbl-x1275/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
[i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424
[i915#470]: https://gitlab.freedesktop.org/drm/intel/issues/470
[i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694
[i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
[i915#773]: https://gitlab.freedesktop.org/drm/intel/issues/773
[i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816
[i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (54 -> 47)
------------------------------
Additional (1): fi-hsw-4770r
Missing (8): fi-icl-1065g7 fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7566 -> Patchwork_15754
CI-20190529: 20190529
CI_DRM_7566: 87c99602f2beb1b0ee7bdb3310bf12133f4d3f7f @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5349: 048f58513d8b8ec6bb307a939f0ac959bc0f0e10 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15754: 1eb2087450a1e802554fcd950feff6b55a145fcd @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
1eb2087450a1 drm/edid: Make sure the CEA mode arrays have the correct amount of modes
04407c04c8f5 drm/edid: Throw away the dummy VIC 0 cea mode
1e2d560339b9 drm/edid: Add CTA-861-G modes with VIC >= 193
f65c5faeb690 drm/edid: Abstract away cea_edid_modes[]
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/edid: Add new modes from CTA-861-G (rev3)
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
` (5 preceding siblings ...)
2019-12-14 1:04 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2019-12-15 0:13 ` Patchwork
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-12-15 0:13 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
== Series Details ==
Series: drm/edid: Add new modes from CTA-861-G (rev3)
URL : https://patchwork.freedesktop.org/series/63554/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7566_full -> Patchwork_15754_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15754_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15754_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15754_full:
### IGT changes ###
#### Possible regressions ####
* igt@runner@aborted:
- shard-apl: NOTRUN -> [FAIL][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-apl3/igt@runner@aborted.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* {igt@gen9_exec_parse@allowed-single}:
- shard-apl: [PASS][2] -> [DMESG-WARN][3]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-apl3/igt@gen9_exec_parse@allowed-single.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-apl3/igt@gen9_exec_parse@allowed-single.html
* {igt@gen9_exec_parse@basic-rejected-ctx-param}:
- shard-tglb: NOTRUN -> [SKIP][4] +3 similar issues
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb1/igt@gen9_exec_parse@basic-rejected-ctx-param.html
* {igt@gen9_exec_parse@unaligned-jump}:
- shard-iclb: NOTRUN -> [SKIP][5]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb3/igt@gen9_exec_parse@unaligned-jump.html
Known issues
------------
Here are the changes found in Patchwork_15754_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_eio@in-flight-suspend:
- shard-tglb: [PASS][6] -> [INCOMPLETE][7] ([i915#456] / [i915#460] / [i915#534])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb4/igt@gem_eio@in-flight-suspend.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb5/igt@gem_eio@in-flight-suspend.html
* igt@gem_eio@kms:
- shard-glk: [PASS][8] -> [INCOMPLETE][9] ([i915#58] / [k.org#198133])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-glk8/igt@gem_eio@kms.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-glk3/igt@gem_eio@kms.html
* igt@gem_exec_create@basic:
- shard-tglb: [PASS][10] -> [INCOMPLETE][11] ([fdo#111736])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb3/igt@gem_exec_create@basic.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb6/igt@gem_exec_create@basic.html
* igt@gem_exec_gttfill@basic:
- shard-tglb: [PASS][12] -> [INCOMPLETE][13] ([fdo#111593])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb7/igt@gem_exec_gttfill@basic.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb9/igt@gem_exec_gttfill@basic.html
* igt@gem_exec_schedule@independent-bsd:
- shard-iclb: [PASS][14] -> [SKIP][15] ([fdo#112146])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb8/igt@gem_exec_schedule@independent-bsd.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb1/igt@gem_exec_schedule@independent-bsd.html
* igt@gem_exec_schedule@preempt-contexts-bsd2:
- shard-iclb: [PASS][16] -> [SKIP][17] ([fdo#109276]) +9 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb2/igt@gem_exec_schedule@preempt-contexts-bsd2.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb6/igt@gem_exec_schedule@preempt-contexts-bsd2.html
* igt@gem_exec_schedule@preempt-queue-chain-render:
- shard-tglb: [PASS][18] -> [INCOMPLETE][19] ([fdo#111606] / [fdo#111677])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb1/igt@gem_exec_schedule@preempt-queue-chain-render.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb6/igt@gem_exec_schedule@preempt-queue-chain-render.html
* igt@gem_exec_schedule@preempt-queue-vebox:
- shard-tglb: [PASS][20] -> [INCOMPLETE][21] ([fdo#111677])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb4/igt@gem_exec_schedule@preempt-queue-vebox.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb6/igt@gem_exec_schedule@preempt-queue-vebox.html
* igt@gem_exec_schedule@smoketest-all:
- shard-tglb: [PASS][22] -> [INCOMPLETE][23] ([i915#463])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb9/igt@gem_exec_schedule@smoketest-all.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb4/igt@gem_exec_schedule@smoketest-all.html
* igt@gem_persistent_relocs@forked-interruptible-thrash-inactive:
- shard-snb: [PASS][24] -> [TIMEOUT][25] ([i915#530])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-snb4/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-snb4/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html
* igt@gem_userptr_blits@sync-unmap-cycles:
- shard-snb: [PASS][26] -> [DMESG-WARN][27] ([fdo#111870]) +1 similar issue
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-snb1/igt@gem_userptr_blits@sync-unmap-cycles.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-snb1/igt@gem_userptr_blits@sync-unmap-cycles.html
* igt@i915_selftest@mock_sanitycheck:
- shard-kbl: [PASS][28] -> [DMESG-WARN][29] ([i915#747])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-kbl7/igt@i915_selftest@mock_sanitycheck.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-kbl6/igt@i915_selftest@mock_sanitycheck.html
* igt@i915_suspend@debugfs-reader:
- shard-tglb: [PASS][30] -> [INCOMPLETE][31] ([i915#460])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb1/igt@i915_suspend@debugfs-reader.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb4/igt@i915_suspend@debugfs-reader.html
* igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding:
- shard-skl: [PASS][32] -> [FAIL][33] ([i915#54])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl9/igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl10/igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding.html
* igt@kms_cursor_crc@pipe-b-cursor-suspend:
- shard-tglb: [PASS][34] -> [INCOMPLETE][35] ([i915#456] / [i915#460])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb9/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb5/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
* igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled:
- shard-skl: [PASS][36] -> [FAIL][37] ([fdo#108145] / [i915#177] / [i915#52] / [i915#54])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl8/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl6/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl: [PASS][38] -> [FAIL][39] ([i915#79])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl2/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@flip-vs-suspend:
- shard-tglb: [PASS][40] -> [INCOMPLETE][41] ([i915#460] / [i915#516])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb6/igt@kms_flip@flip-vs-suspend.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb8/igt@kms_flip@flip-vs-suspend.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-msflip-blt:
- shard-tglb: [PASS][42] -> [FAIL][43] ([i915#49])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-msflip-blt.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb9/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-msflip-blt.html
* igt@kms_frontbuffer_tracking@fbc-suspend:
- shard-kbl: [PASS][44] -> [DMESG-WARN][45] ([i915#180]) +5 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-kbl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-kbl1/igt@kms_frontbuffer_tracking@fbc-suspend.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-gtt:
- shard-tglb: [PASS][46] -> [INCOMPLETE][47] ([i915#667])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb3/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-gtt.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb9/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-mmap-cpu:
- shard-skl: [PASS][48] -> [FAIL][49] ([i915#49])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl8/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-mmap-cpu.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl6/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-mmap-cpu.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-indfb-plflip-blt:
- shard-skl: [PASS][50] -> [INCOMPLETE][51] ([i915#123] / [i915#667])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl1/igt@kms_frontbuffer_tracking@psr-1p-primscrn-indfb-plflip-blt.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl8/igt@kms_frontbuffer_tracking@psr-1p-primscrn-indfb-plflip-blt.html
* igt@kms_psr@psr2_sprite_mmap_gtt:
- shard-iclb: [PASS][52] -> [SKIP][53] ([fdo#109441]) +3 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb1/igt@kms_psr@psr2_sprite_mmap_gtt.html
* igt@kms_psr@psr2_suspend:
- shard-tglb: [PASS][54] -> [DMESG-WARN][55] ([i915#402])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb4/igt@kms_psr@psr2_suspend.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb8/igt@kms_psr@psr2_suspend.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-apl: [PASS][56] -> [DMESG-WARN][57] ([i915#180]) +1 similar issue
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-apl3/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-apl4/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
* igt@perf_pmu@busy-no-semaphores-vcs1:
- shard-iclb: [PASS][58] -> [SKIP][59] ([fdo#112080]) +2 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb2/igt@perf_pmu@busy-no-semaphores-vcs1.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb6/igt@perf_pmu@busy-no-semaphores-vcs1.html
#### Possible fixes ####
* igt@gem_busy@close-race:
- shard-tglb: [INCOMPLETE][60] ([i915#435]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb6/igt@gem_busy@close-race.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb1/igt@gem_busy@close-race.html
* igt@gem_ctx_persistence@vcs0-mixed-process:
- shard-skl: [FAIL][62] ([i915#679]) -> [PASS][63]
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl10/igt@gem_ctx_persistence@vcs0-mixed-process.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl2/igt@gem_ctx_persistence@vcs0-mixed-process.html
* igt@gem_ctx_persistence@vcs1-persistence:
- shard-iclb: [SKIP][64] ([fdo#109276] / [fdo#112080]) -> [PASS][65]
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb8/igt@gem_ctx_persistence@vcs1-persistence.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb1/igt@gem_ctx_persistence@vcs1-persistence.html
* igt@gem_eio@kms:
- shard-snb: [INCOMPLETE][66] ([i915#82]) -> [PASS][67]
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-snb7/igt@gem_eio@kms.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-snb1/igt@gem_eio@kms.html
* igt@gem_exec_parallel@vcs0:
- shard-tglb: [INCOMPLETE][68] ([fdo#111593]) -> [PASS][69]
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb5/igt@gem_exec_parallel@vcs0.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb1/igt@gem_exec_parallel@vcs0.html
* igt@gem_exec_schedule@preempt-hang-bsd:
- shard-iclb: [SKIP][70] ([fdo#112146]) -> [PASS][71]
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb2/igt@gem_exec_schedule@preempt-hang-bsd.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb6/igt@gem_exec_schedule@preempt-hang-bsd.html
* igt@gem_sync@basic-store-each:
- shard-tglb: [INCOMPLETE][72] ([i915#435] / [i915#472]) -> [PASS][73]
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb8/igt@gem_sync@basic-store-each.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb1/igt@gem_sync@basic-store-each.html
* igt@gem_userptr_blits@sync-unmap-after-close:
- shard-snb: [DMESG-WARN][74] ([fdo#111870]) -> [PASS][75] +1 similar issue
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-snb4/igt@gem_userptr_blits@sync-unmap-after-close.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-snb2/igt@gem_userptr_blits@sync-unmap-after-close.html
* igt@gem_workarounds@suspend-resume-fd:
- shard-kbl: [DMESG-WARN][76] ([i915#180]) -> [PASS][77] +4 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-kbl4/igt@gem_workarounds@suspend-resume-fd.html
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-kbl6/igt@gem_workarounds@suspend-resume-fd.html
* igt@i915_selftest@mock_sanitycheck:
- shard-skl: [DMESG-WARN][78] ([i915#747]) -> [PASS][79]
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl2/igt@i915_selftest@mock_sanitycheck.html
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl5/igt@i915_selftest@mock_sanitycheck.html
- shard-snb: [DMESG-WARN][80] ([i915#747]) -> [PASS][81]
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-snb1/igt@i915_selftest@mock_sanitycheck.html
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-snb5/igt@i915_selftest@mock_sanitycheck.html
* igt@i915_suspend@fence-restore-untiled:
- shard-apl: [DMESG-WARN][82] ([i915#180]) -> [PASS][83] +1 similar issue
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-apl4/igt@i915_suspend@fence-restore-untiled.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-apl8/igt@i915_suspend@fence-restore-untiled.html
* igt@kms_cursor_crc@pipe-a-cursor-64x64-sliding:
- shard-skl: [FAIL][84] ([i915#54]) -> [PASS][85]
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl3/igt@kms_cursor_crc@pipe-a-cursor-64x64-sliding.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl3/igt@kms_cursor_crc@pipe-a-cursor-64x64-sliding.html
* igt@kms_draw_crc@draw-method-xrgb2101010-blt-ytiled:
- shard-skl: [INCOMPLETE][86] ([i915#667]) -> [PASS][87]
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl6/igt@kms_draw_crc@draw-method-xrgb2101010-blt-ytiled.html
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl4/igt@kms_draw_crc@draw-method-xrgb2101010-blt-ytiled.html
* igt@kms_draw_crc@draw-method-xrgb2101010-render-untiled:
- shard-skl: [INCOMPLETE][88] ([i915#646] / [i915#667]) -> [PASS][89]
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl3/igt@kms_draw_crc@draw-method-xrgb2101010-render-untiled.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl1/igt@kms_draw_crc@draw-method-xrgb2101010-render-untiled.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-skl: [FAIL][90] ([i915#79]) -> [PASS][91]
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl9/igt@kms_flip@flip-vs-expired-vblank.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl3/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-glk: [FAIL][92] ([i915#79]) -> [PASS][93]
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render:
- shard-tglb: [INCOMPLETE][94] ([i915#474] / [i915#667]) -> [PASS][95]
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render.html
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
- shard-tglb: [FAIL][96] ([i915#49]) -> [PASS][97]
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
* igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min:
- shard-skl: [FAIL][98] ([fdo#108145]) -> [PASS][99] +1 similar issue
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl5/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl1/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
* igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
- shard-skl: [FAIL][100] ([fdo#108145] / [i915#265]) -> [PASS][101]
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl3/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl1/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
* igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend:
- shard-skl: [INCOMPLETE][102] ([i915#69]) -> [PASS][103]
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-skl10/igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend.html
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-skl8/igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend.html
* igt@prime_vgem@fence-wait-bsd2:
- shard-iclb: [SKIP][104] ([fdo#109276]) -> [PASS][105] +1 similar issue
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-iclb8/igt@prime_vgem@fence-wait-bsd2.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-iclb1/igt@prime_vgem@fence-wait-bsd2.html
#### Warnings ####
* igt@gem_ctx_isolation@vcs2-nonpriv:
- shard-tglb: [SKIP][106] ([fdo#112080]) -> [SKIP][107] ([fdo#111912] / [fdo#112080])
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7566/shard-tglb9/igt@gem_ctx_isolation@vcs2-nonpriv.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/shard-tglb4/igt@gem_ctx_isolation@vcs2-nonpriv.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
[fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
[fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
[fdo#111736]: https://bugs.freedesktop.org/show_bug.cgi?id=111736
[fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870
[fdo#111912]: https://bugs.freedesktop.org/show_bug.cgi?id=111912
[fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
[fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
[i915#123]: https://gitlab.freedesktop.org/drm/intel/issues/123
[i915#177]: https://gitlab.freedesktop.org/drm/intel/issues/177
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
[i915#435]: https://gitlab.freedesktop.org/drm/intel/issues/435
[i915#456]: https://gitlab.freedesktop.org/drm/intel/issues/456
[i915#460]: https://gitlab.freedesktop.org/drm/intel/issues/460
[i915#463]: https://gitlab.freedesktop.org/drm/intel/issues/463
[i915#472]: https://gitlab.freedesktop.org/drm/intel/issues/472
[i915#474]: https://gitlab.freedesktop.org/drm/intel/issues/474
[i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
[i915#516]: https://gitlab.freedesktop.org/drm/intel/issues/516
[i915#52]: https://gitlab.freedesktop.org/drm/intel/issues/52
[i915#530]: https://gitlab.freedesktop.org/drm/intel/issues/530
[i915#534]: https://gitlab.freedesktop.org/drm/intel/issues/534
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#58]: https://gitlab.freedesktop.org/drm/intel/issues/58
[i915#646]: https://gitlab.freedesktop.org/drm/intel/issues/646
[i915#667]: https://gitlab.freedesktop.org/drm/intel/issues/667
[i915#679]: https://gitlab.freedesktop.org/drm/intel/issues/679
[i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69
[i915#747]: https://gitlab.freedesktop.org/drm/intel/issues/747
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7566 -> Patchwork_15754
CI-20190529: 20190529
CI_DRM_7566: 87c99602f2beb1b0ee7bdb3310bf12133f4d3f7f @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5349: 048f58513d8b8ec6bb307a939f0ac959bc0f0e10 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15754: 1eb2087450a1e802554fcd950feff6b55a145fcd @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15754/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[]
2019-12-13 21:03 ` Tom Anderson
@ 2019-12-16 15:23 ` Ville Syrjälä
0 siblings, 0 replies; 11+ messages in thread
From: Ville Syrjälä @ 2019-12-16 15:23 UTC (permalink / raw)
To: Tom Anderson; +Cc: Hans Verkuil, intel-gfx, dri-devel
On Fri, Dec 13, 2019 at 01:03:30PM -0800, Tom Anderson wrote:
> Latest patch looks good to me, thanks for the changes!
>
> Reviewed-by: Thomas Anderson <thomasanderson@google.com>
Thanks for the review. Series pushed to drm-misc-next.
>
> On Fri, Dec 13, 2019 at 07:43:45PM +0200, Ville Syrjala wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > We're going to need two cea mode tables (one for VICs < 128,
> > another one for VICs >= 193). To that end replace the direct
> > edid_cea_modes[] lookups with a function call. And we'll rename
> > the array to edid_cea_modes_0[] to indicate how it's to be
> > indexed.
> >
> > v2: Fix typos (Tom)
> > Drop the pointless NULL checks in the loops (Tom)
> > Assign when declaring (Tom)
> > Improve the comment for cea_modes_*[] to indicate
> > that one should always use cea_mode_for_vic() (Tom)
> >
> > Cc: Tom Anderson <thomasanderson@google.com>
> > Cc: Hans Verkuil <hansverk@cisco.com>
> > Cc: Manasi Navare <manasi.d.navare@intel.com>
> > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > ---
> > drivers/gpu/drm/drm_edid.c | 67 +++++++++++++++++++++++++-------------
> > 1 file changed, 45 insertions(+), 22 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> > index 5b33b7cfd645..00a543b9daab 100644
> > --- a/drivers/gpu/drm/drm_edid.c
> > +++ b/drivers/gpu/drm/drm_edid.c
> > @@ -710,12 +710,11 @@ static const struct minimode extra_modes[] = {
> > };
> >
> > /*
> > - * Probably taken from CEA-861 spec.
> > - * This table is converted from xorg's hw/xfree86/modes/xf86EdidModes.c.
> > + * From CEA/CTA-861 spec.
> > *
> > - * Index using the VIC.
> > + * Do not access directly, instead always use cea_mode_for_vic().
> > */
> > -static const struct drm_display_mode edid_cea_modes[] = {
> > +static const struct drm_display_mode edid_cea_modes_0[] = {
> > /* 0 - dummy, VICs start at 1 */
> > { },
> > /* 1 - 640x480@60Hz 4:3 */
> > @@ -3071,6 +3070,25 @@ static u8 *drm_find_cea_extension(const struct edid *edid)
> > return cea;
> > }
> >
> > +static const struct drm_display_mode *cea_mode_for_vic(u8 vic)
> > +{
> > + if (!vic)
> > + return NULL;
> > + if (vic < ARRAY_SIZE(edid_cea_modes_0))
> > + return &edid_cea_modes_0[vic];
> > + return NULL;
> > +}
> > +
> > +static u8 cea_num_vics(void)
> > +{
> > + return ARRAY_SIZE(edid_cea_modes_0);
> > +}
> > +
> > +static u8 cea_next_vic(u8 vic)
> > +{
> > + return vic + 1;
> > +}
> > +
> > /*
> > * Calculate the alternate clock for the CEA mode
> > * (60Hz vs. 59.94Hz etc.)
> > @@ -3108,14 +3126,14 @@ cea_mode_alternate_timings(u8 vic, struct drm_display_mode *mode)
> > * get the other variants by simply increasing the
> > * vertical front porch length.
> > */
> > - BUILD_BUG_ON(edid_cea_modes[8].vtotal != 262 ||
> > - edid_cea_modes[9].vtotal != 262 ||
> > - edid_cea_modes[12].vtotal != 262 ||
> > - edid_cea_modes[13].vtotal != 262 ||
> > - edid_cea_modes[23].vtotal != 312 ||
> > - edid_cea_modes[24].vtotal != 312 ||
> > - edid_cea_modes[27].vtotal != 312 ||
> > - edid_cea_modes[28].vtotal != 312);
> > + BUILD_BUG_ON(cea_mode_for_vic(8)->vtotal != 262 ||
> > + cea_mode_for_vic(9)->vtotal != 262 ||
> > + cea_mode_for_vic(12)->vtotal != 262 ||
> > + cea_mode_for_vic(13)->vtotal != 262 ||
> > + cea_mode_for_vic(23)->vtotal != 312 ||
> > + cea_mode_for_vic(24)->vtotal != 312 ||
> > + cea_mode_for_vic(27)->vtotal != 312 ||
> > + cea_mode_for_vic(28)->vtotal != 312);
> >
> > if (((vic == 8 || vic == 9 ||
> > vic == 12 || vic == 13) && mode->vtotal < 263) ||
> > @@ -3143,8 +3161,8 @@ static u8 drm_match_cea_mode_clock_tolerance(const struct drm_display_mode *to_m
> > if (to_match->picture_aspect_ratio)
> > match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
> >
> > - for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
> > - struct drm_display_mode cea_mode = edid_cea_modes[vic];
> > + for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
> > + struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
> > unsigned int clock1, clock2;
> >
> > /* Check both 60Hz and 59.94Hz */
> > @@ -3182,8 +3200,8 @@ u8 drm_match_cea_mode(const struct drm_display_mode *to_match)
> > if (to_match->picture_aspect_ratio)
> > match_flags |= DRM_MODE_MATCH_ASPECT_RATIO;
> >
> > - for (vic = 1; vic < ARRAY_SIZE(edid_cea_modes); vic++) {
> > - struct drm_display_mode cea_mode = edid_cea_modes[vic];
> > + for (vic = 1; vic < cea_num_vics(); vic = cea_next_vic(vic)) {
> > + struct drm_display_mode cea_mode = *cea_mode_for_vic(vic);
> > unsigned int clock1, clock2;
> >
> > /* Check both 60Hz and 59.94Hz */
> > @@ -3206,12 +3224,17 @@ EXPORT_SYMBOL(drm_match_cea_mode);
> >
> > static bool drm_valid_cea_vic(u8 vic)
> > {
> > - return vic > 0 && vic < ARRAY_SIZE(edid_cea_modes);
> > + return cea_mode_for_vic(vic) != NULL;
> > }
> >
> > static enum hdmi_picture_aspect drm_get_cea_aspect_ratio(const u8 video_code)
> > {
> > - return edid_cea_modes[video_code].picture_aspect_ratio;
> > + const struct drm_display_mode *mode = cea_mode_for_vic(video_code);
> > +
> > + if (mode)
> > + return mode->picture_aspect_ratio;
> > +
> > + return HDMI_PICTURE_ASPECT_NONE;
> > }
> >
> > static enum hdmi_picture_aspect drm_get_hdmi_aspect_ratio(const u8 video_code)
> > @@ -3323,7 +3346,7 @@ add_alternate_cea_modes(struct drm_connector *connector, struct edid *edid)
> > unsigned int clock1, clock2;
> >
> > if (drm_valid_cea_vic(vic)) {
> > - cea_mode = &edid_cea_modes[vic];
> > + cea_mode = cea_mode_for_vic(vic);
> > clock2 = cea_mode_alternate_clock(cea_mode);
> > } else {
> > vic = drm_match_hdmi_mode(mode);
> > @@ -3398,7 +3421,7 @@ drm_display_mode_from_vic_index(struct drm_connector *connector,
> > if (!drm_valid_cea_vic(vic))
> > return NULL;
> >
> > - newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
> > + newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
> > if (!newmode)
> > return NULL;
> >
> > @@ -3432,7 +3455,7 @@ static int do_y420vdb_modes(struct drm_connector *connector,
> > if (!drm_valid_cea_vic(vic))
> > continue;
> >
> > - newmode = drm_mode_duplicate(dev, &edid_cea_modes[vic]);
> > + newmode = drm_mode_duplicate(dev, cea_mode_for_vic(vic));
> > if (!newmode)
> > break;
> > bitmap_set(hdmi->y420_vdb_modes, vic, 1);
> > @@ -4001,7 +4024,7 @@ static void fixup_detailed_cea_mode_clock(struct drm_display_mode *mode)
> > vic = drm_match_cea_mode_clock_tolerance(mode, 5);
> > if (drm_valid_cea_vic(vic)) {
> > type = "CEA";
> > - cea_mode = &edid_cea_modes[vic];
> > + cea_mode = cea_mode_for_vic(vic);
> > clock1 = cea_mode->clock;
> > clock2 = cea_mode_alternate_clock(cea_mode);
> > } else {
> > --
> > 2.23.0
> >
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-12-16 15:24 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-12-13 17:43 [Intel-gfx] [PATCH v4 0/4] drm/edid: Add new modes from CTA-861-G Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 1/4] drm/edid: Abstract away cea_edid_modes[] Ville Syrjala
2019-12-13 21:03 ` Tom Anderson
2019-12-16 15:23 ` Ville Syrjälä
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 2/4] drm/edid: Add CTA-861-G modes with VIC >= 193 Ville Syrjala
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 3/4] drm/edid: Throw away the dummy VIC 0 cea mode Ville Syrjala
2019-12-13 21:05 ` Tom Anderson
2019-12-13 17:43 ` [Intel-gfx] [PATCH v4 4/4] drm/edid: Make sure the CEA mode arrays have the correct amount of modes Ville Syrjala
2019-12-14 0:29 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/edid: Add new modes from CTA-861-G (rev3) Patchwork
2019-12-14 1:04 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2019-12-15 0:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox