* [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling
@ 2020-12-07 20:35 Ville Syrjala
2020-12-07 20:35 ` [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables Ville Syrjala
` (5 more replies)
0 siblings, 6 replies; 8+ messages in thread
From: Ville Syrjala @ 2020-12-07 20:35 UTC (permalink / raw)
To: intel-gfx
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
The MH PHY vswing table does have all the entries these days. Get
rid of the old hacks in the code which claim otherwise.
This hack was totally bogus anyway. The correct way to handle the
lack of those two entries would have been to declare our max
vswing and pre-emph to both be level 2.
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Clinton Taylor <clinton.a.taylor@intel.com>
Fixes: 9f7ffa297978 ("drm/i915/tc/icl: Update TC vswing tables")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/display/intel_ddi.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 5193473c838c..c3a15ce66478 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -2760,12 +2760,11 @@ static void icl_mg_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
u32 val;
ddi_translations = icl_get_mg_buf_trans(encoder, crtc_state, &n_entries);
- /* The table does not have values for level 3 and level 9. */
- if (level >= n_entries || level == 3 || level == 9) {
+ if (level >= n_entries) {
drm_dbg_kms(&dev_priv->drm,
"DDI translation not found for level %d. Using %d instead.",
- level, n_entries - 2);
- level = n_entries - 2;
+ level, n_entries - 1);
+ level = n_entries - 1;
}
/* Set MG_TX_LINK_PARAMS cri_use_fs32 to 0. */
--
2.26.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
@ 2020-12-07 20:35 ` Ville Syrjala
2021-01-20 19:22 ` Souza, Jose
2020-12-07 21:26 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling Patchwork
` (4 subsequent siblings)
5 siblings, 1 reply; 8+ messages in thread
From: Ville Syrjala @ 2020-12-07 20:35 UTC (permalink / raw)
To: intel-gfx
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Get rid of the "I like my random new style best" approach and unify
the handling for the DDI buf trans table sanity checks once again.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/display/intel_ddi.c | 23 ++++++++++-------------
1 file changed, 10 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index c3a15ce66478..68693d4538e2 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -2631,15 +2631,11 @@ static void icl_ddi_combo_vswing_program(struct intel_encoder *encoder,
ddi_translations = ehl_get_combo_buf_trans(encoder, crtc_state, &n_entries);
else
ddi_translations = icl_get_combo_buf_trans(encoder, crtc_state, &n_entries);
- if (!ddi_translations)
- return;
- if (level >= n_entries) {
- drm_dbg_kms(&dev_priv->drm,
- "DDI translation not found for level %d. Using %d instead.",
- level, n_entries - 1);
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
+ return;
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
level = n_entries - 1;
- }
if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP)) {
struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
@@ -2760,12 +2756,11 @@ static void icl_mg_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
u32 val;
ddi_translations = icl_get_mg_buf_trans(encoder, crtc_state, &n_entries);
- if (level >= n_entries) {
- drm_dbg_kms(&dev_priv->drm,
- "DDI translation not found for level %d. Using %d instead.",
- level, n_entries - 1);
+
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
+ return;
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
level = n_entries - 1;
- }
/* Set MG_TX_LINK_PARAMS cri_use_fs32 to 0. */
for (ln = 0; ln < 2; ln++) {
@@ -2897,7 +2892,9 @@ tgl_dkl_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
ddi_translations = tgl_get_dkl_buf_trans(encoder, crtc_state, &n_entries);
- if (level >= n_entries)
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
+ return;
+ if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
level = n_entries - 1;
dpcnt_mask = (DKL_TX_PRESHOOT_COEFF_MASK |
--
2.26.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
2020-12-07 20:35 ` [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables Ville Syrjala
@ 2020-12-07 21:26 ` Patchwork
2020-12-14 16:27 ` [Intel-gfx] ✗ Fi.CI.DOCS: warning for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2) Patchwork
` (3 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2020-12-07 21:26 UTC (permalink / raw)
To: Ville Syrjala; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 3401 bytes --]
== Series Details ==
Series: series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling
URL : https://patchwork.freedesktop.org/series/84651/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9456 -> Patchwork_19077
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_19077 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_19077, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19077/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_19077:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_suspend@basic-s3:
- fi-snb-2600: [PASS][1] -> [DMESG-WARN][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9456/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19077/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
New tests
---------
New tests have been introduced between CI_DRM_9456 and Patchwork_19077:
### New CI tests (1) ###
* boot:
- Statuses : 1 fail(s) 38 pass(s)
- Exec time: [0.0] s
Known issues
------------
Here are the changes found in Patchwork_19077 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@gt_heartbeat:
- fi-bsw-kefka: [PASS][3] -> [DMESG-FAIL][4] ([i915#2675] / [i915#541])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9456/fi-bsw-kefka/igt@i915_selftest@live@gt_heartbeat.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19077/fi-bsw-kefka/igt@i915_selftest@live@gt_heartbeat.html
#### Warnings ####
* igt@i915_selftest@live@gt_pm:
- fi-kbl-soraka: [DMESG-FAIL][5] ([i915#2291]) -> [DMESG-FAIL][6] ([i915#1886] / [i915#2291])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9456/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19077/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291
[i915#2675]: https://gitlab.freedesktop.org/drm/intel/issues/2675
[i915#541]: https://gitlab.freedesktop.org/drm/intel/issues/541
Participating hosts (44 -> 39)
------------------------------
Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-tgl-y fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_9456 -> Patchwork_19077
CI-20190529: 20190529
CI_DRM_9456: 4c841475fa2fe28db44c090a65ec58c632f705dd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5884: b1015a3267bbccb985b2fa7e3accb778c7bff0ed @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19077: c4a7651820892f181e8aa4e4389f0243d4ef79a4 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
c4a765182089 drm/i915: Unify the sanity checks for the buf trans tables
6c4f73fa8880 drm/i915: Fix ICL MG PHY vswing handling
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19077/index.html
[-- Attachment #1.2: Type: text/html, Size: 4156 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.DOCS: warning for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
2020-12-07 20:35 ` [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables Ville Syrjala
2020-12-07 21:26 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling Patchwork
@ 2020-12-14 16:27 ` Patchwork
2020-12-14 16:53 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
` (2 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2020-12-14 16:27 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
URL : https://patchwork.freedesktop.org/series/84651/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
Error: Cannot open file ./drivers/gpu/drm/i915/gt/intel_lrc.c
WARNING: kernel-doc './scripts/kernel-doc -rst -enable-lineno -sphinx-version 1.7.9 -function Logical Rings, Logical Ring Contexts and Execlists ./drivers/gpu/drm/i915/gt/intel_lrc.c' failed with return code 1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
` (2 preceding siblings ...)
2020-12-14 16:27 ` [Intel-gfx] ✗ Fi.CI.DOCS: warning for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2) Patchwork
@ 2020-12-14 16:53 ` Patchwork
2020-12-14 19:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-01-20 19:19 ` [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Souza, Jose
5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2020-12-14 16:53 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 3544 bytes --]
== Series Details ==
Series: series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
URL : https://patchwork.freedesktop.org/series/84651/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9481 -> Patchwork_19136
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/index.html
Known issues
------------
Here are the changes found in Patchwork_19136 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@debugfs_test@read_all_entries:
- fi-tgl-y: [PASS][1] -> [DMESG-WARN][2] ([i915#402]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/fi-tgl-y/igt@debugfs_test@read_all_entries.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-tgl-y/igt@debugfs_test@read_all_entries.html
* igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
- fi-snb-2600: NOTRUN -> [SKIP][3] ([fdo#109271]) +30 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-snb-2600/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html
* igt@kms_chamelium@hdmi-crc-fast:
- fi-snb-2600: NOTRUN -> [SKIP][4] ([fdo#109271] / [fdo#111827]) +8 similar issues
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-snb-2600/igt@kms_chamelium@hdmi-crc-fast.html
* igt@runner@aborted:
- fi-bdw-5557u: NOTRUN -> [FAIL][5] ([i915#2029] / [i915#2722])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-bdw-5557u/igt@runner@aborted.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3:
- fi-snb-2600: [DMESG-WARN][6] -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
* igt@prime_self_import@basic-with_one_bo_two_files:
- fi-tgl-y: [DMESG-WARN][8] ([i915#402]) -> [PASS][9] +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/fi-tgl-y/igt@prime_self_import@basic-with_one_bo_two_files.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/fi-tgl-y/igt@prime_self_import@basic-with_one_bo_two_files.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029
[i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
Participating hosts (44 -> 39)
------------------------------
Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_9481 -> Patchwork_19136
CI-20190529: 20190529
CI_DRM_9481: 968d6b8835487575d68732a7c0a1588795900de4 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5893: ec4073d30b428aaf199c0f4de6c0a5ebdc1c1c65 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19136: 7c1db3bd75a26fb9c181a1c0c6e0450d74da51be @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
7c1db3bd75a2 drm/i915: Unify the sanity checks for the buf trans tables
18cf8bf189b3 drm/i915: Fix ICL MG PHY vswing handling
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/index.html
[-- Attachment #1.2: Type: text/html, Size: 4429 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
` (3 preceding siblings ...)
2020-12-14 16:53 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2020-12-14 19:46 ` Patchwork
2021-01-20 19:19 ` [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Souza, Jose
5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2020-12-14 19:46 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 16762 bytes --]
== Series Details ==
Series: series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2)
URL : https://patchwork.freedesktop.org/series/84651/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9481_full -> Patchwork_19136_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_19136_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_19136_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_19136_full:
### IGT changes ###
#### Possible regressions ####
* igt@kms_cursor_legacy@pipe-a-single-bo:
- shard-glk: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-glk5/igt@kms_cursor_legacy@pipe-a-single-bo.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-glk7/igt@kms_cursor_legacy@pipe-a-single-bo.html
Known issues
------------
Here are the changes found in Patchwork_19136_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gen9_exec_parse@allowed-all:
- shard-skl: [PASS][3] -> [DMESG-WARN][4] ([i915#1436] / [i915#716])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl3/igt@gen9_exec_parse@allowed-all.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl7/igt@gen9_exec_parse@allowed-all.html
* igt@kms_chamelium@hdmi-aspect-ratio:
- shard-skl: NOTRUN -> [SKIP][5] ([fdo#109271] / [fdo#111827]) +7 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_chamelium@hdmi-aspect-ratio.html
* igt@kms_color@pipe-a-ctm-0-75:
- shard-skl: [PASS][6] -> [DMESG-WARN][7] ([i915#1982]) +1 similar issue
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl1/igt@kms_color@pipe-a-ctm-0-75.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_color@pipe-a-ctm-0-75.html
* igt@kms_color_chamelium@pipe-c-ctm-blue-to-red:
- shard-hsw: NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827]) +2 similar issues
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-hsw7/igt@kms_color_chamelium@pipe-c-ctm-blue-to-red.html
* igt@kms_cursor_crc@pipe-b-cursor-64x21-random:
- shard-skl: NOTRUN -> [FAIL][9] ([i915#54])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_cursor_crc@pipe-b-cursor-64x21-random.html
* igt@kms_cursor_crc@pipe-b-cursor-suspend:
- shard-kbl: [PASS][10] -> [DMESG-WARN][11] ([i915#180])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-kbl7/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-kbl7/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
* igt@kms_cursor_crc@pipe-c-cursor-64x21-random:
- shard-skl: [PASS][12] -> [FAIL][13] ([i915#54]) +2 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl2/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl9/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
* igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge:
- shard-skl: NOTRUN -> [SKIP][14] ([fdo#109271]) +80 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl10/igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge.html
* igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
- shard-tglb: [PASS][15] -> [FAIL][16] ([i915#2346])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-tglb6/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-tglb3/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
* igt@kms_flip@2x-flip-vs-dpms:
- shard-iclb: NOTRUN -> [SKIP][17] ([fdo#109274])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-iclb3/igt@kms_flip@2x-flip-vs-dpms.html
* igt@kms_flip@flip-vs-expired-vblank@a-edp1:
- shard-tglb: [PASS][18] -> [FAIL][19] ([i915#2598])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-tglb1/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-tglb7/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
* igt@kms_flip@flip-vs-suspend@c-edp1:
- shard-skl: [PASS][20] -> [INCOMPLETE][21] ([i915#198])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl6/igt@kms_flip@flip-vs-suspend@c-edp1.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl6/igt@kms_flip@flip-vs-suspend@c-edp1.html
* igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1:
- shard-skl: [PASS][22] -> [FAIL][23] ([i915#2122]) +1 similar issue
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl7/igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl1/igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs:
- shard-skl: NOTRUN -> [SKIP][24] ([fdo#109271] / [i915#2672])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-draw-pwrite:
- shard-hsw: NOTRUN -> [SKIP][25] ([fdo#109271]) +22 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-hsw7/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-draw-pwrite.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-d:
- shard-skl: NOTRUN -> [SKIP][26] ([fdo#109271] / [i915#533])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html
* igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min:
- shard-skl: [PASS][27] -> [FAIL][28] ([fdo#108145] / [i915#265])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl10/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl8/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: NOTRUN -> [FAIL][29] ([fdo#108145] / [i915#265]) +1 similar issue
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
* igt@kms_psr@psr2_sprite_mmap_cpu:
- shard-iclb: [PASS][30] -> [SKIP][31] ([fdo#109441]) +1 similar issue
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_cpu.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-iclb5/igt@kms_psr@psr2_sprite_mmap_cpu.html
* igt@prime_vgem@sync@bcs0:
- shard-tglb: [PASS][32] -> [INCOMPLETE][33] ([i915#409])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-tglb8/igt@prime_vgem@sync@bcs0.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-tglb1/igt@prime_vgem@sync@bcs0.html
#### Possible fixes ####
* {igt@gem_create@busy-create}:
- shard-hsw: [INCOMPLETE][34] -> [PASS][35]
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-hsw6/igt@gem_create@busy-create.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-hsw7/igt@gem_create@busy-create.html
* igt@gem_exec_reloc@basic-many-active@rcs0:
- shard-hsw: [FAIL][36] ([i915#2389]) -> [PASS][37]
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-hsw1/igt@gem_exec_reloc@basic-many-active@rcs0.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-hsw6/igt@gem_exec_reloc@basic-many-active@rcs0.html
* igt@gem_exec_whisper@basic-queues-forked:
- shard-glk: [DMESG-WARN][38] ([i915#118] / [i915#95]) -> [PASS][39]
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-glk6/igt@gem_exec_whisper@basic-queues-forked.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-glk2/igt@gem_exec_whisper@basic-queues-forked.html
* igt@i915_selftest@live@gt_heartbeat:
- shard-glk: [DMESG-FAIL][40] ([i915#2291]) -> [PASS][41]
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-glk3/igt@i915_selftest@live@gt_heartbeat.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-glk1/igt@i915_selftest@live@gt_heartbeat.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-skl: [INCOMPLETE][42] ([i915#2295] / [i915#300]) -> [PASS][43]
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl3/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl10/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_crc@pipe-c-cursor-64x64-offscreen:
- shard-skl: [FAIL][44] ([i915#54]) -> [PASS][45] +1 similar issue
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl9/igt@kms_cursor_crc@pipe-c-cursor-64x64-offscreen.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl4/igt@kms_cursor_crc@pipe-c-cursor-64x64-offscreen.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1:
- shard-glk: [FAIL][46] ([i915#79]) -> [PASS][47] +1 similar issue
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-glk6/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-glk2/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1.html
* igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1:
- shard-skl: [FAIL][48] ([i915#2122]) -> [PASS][49]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl1/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl5/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html
* igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
- shard-skl: [FAIL][50] ([fdo#108145] / [i915#265]) -> [PASS][51] +1 similar issue
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl9/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl4/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
* igt@kms_psr2_su@frontbuffer:
- shard-iclb: [SKIP][52] ([fdo#109642] / [fdo#111068]) -> [PASS][53]
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-iclb4/igt@kms_psr2_su@frontbuffer.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-iclb2/igt@kms_psr2_su@frontbuffer.html
* igt@kms_psr@psr2_sprite_plane_onoff:
- shard-iclb: [SKIP][54] ([fdo#109441]) -> [PASS][55]
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-iclb1/igt@kms_psr@psr2_sprite_plane_onoff.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html
#### Warnings ####
* igt@i915_pm_rc6_residency@rc6-fence:
- shard-iclb: [WARN][56] ([i915#2681] / [i915#2684]) -> [WARN][57] ([i915#1804] / [i915#2684]) +1 similar issue
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-iclb1/igt@i915_pm_rc6_residency@rc6-fence.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-iclb6/igt@i915_pm_rc6_residency@rc6-fence.html
* igt@runner@aborted:
- shard-kbl: [FAIL][58] ([i915#2295] / [i915#2722] / [i915#483]) -> ([FAIL][59], [FAIL][60]) ([i915#1814] / [i915#2295] / [i915#2722] / [i915#483])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-kbl3/igt@runner@aborted.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-kbl4/igt@runner@aborted.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-kbl7/igt@runner@aborted.html
- shard-tglb: [FAIL][61] ([i915#2295] / [i915#2722]) -> ([FAIL][62], [FAIL][63]) ([i915#2295] / [i915#2426] / [i915#2722] / [i915#409])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-tglb7/igt@runner@aborted.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-tglb1/igt@runner@aborted.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-tglb1/igt@runner@aborted.html
- shard-skl: [FAIL][64] ([i915#2295] / [i915#2722] / [i915#483]) -> ([FAIL][65], [FAIL][66]) ([i915#1436] / [i915#2295] / [i915#2722] / [i915#483])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9481/shard-skl9/igt@runner@aborted.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl7/igt@runner@aborted.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19136/shard-skl4/igt@runner@aborted.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#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
[i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291
[i915#2295]: https://gitlab.freedesktop.org/drm/intel/issues/2295
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2389]: https://gitlab.freedesktop.org/drm/intel/issues/2389
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2598]: https://gitlab.freedesktop.org/drm/intel/issues/2598
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681
[i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
[i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722
[i915#300]: https://gitlab.freedesktop.org/drm/intel/issues/300
[i915#409]: https://gitlab.freedesktop.org/drm/intel/issues/409
[i915#483]: https://gitlab.freedesktop.org/drm/intel/issues/483
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_9481 -> Patchwork_19136
CI-20190529: 20190529
CI_DRM_9481: 968d6b8835487575d68732a7c0a1588795900de4 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5893: ec4073d30b428aaf199c0f4de6c0a5ebdc1c1c65 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19136: 7c1db3bd75a26fb9c181a1c0c6e0450d74da51be @ 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_19136/index.html
[-- Attachment #1.2: Type: text/html, Size: 20265 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
` (4 preceding siblings ...)
2020-12-14 19:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
@ 2021-01-20 19:19 ` Souza, Jose
5 siblings, 0 replies; 8+ messages in thread
From: Souza, Jose @ 2021-01-20 19:19 UTC (permalink / raw)
To: ville.syrjala@linux.intel.com, intel-gfx@lists.freedesktop.org
On Mon, 2020-12-07 at 22:35 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The MH PHY vswing table does have all the entries these days. Get
> rid of the old hacks in the code which claim otherwise.
>
> This hack was totally bogus anyway. The correct way to handle the
> lack of those two entries would have been to declare our max
> vswing and pre-emph to both be level 2.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Cc: Clinton Taylor <clinton.a.taylor@intel.com>
> Fixes: 9f7ffa297978 ("drm/i915/tc/icl: Update TC vswing tables")
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> index 5193473c838c..c3a15ce66478 100644
> --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> @@ -2760,12 +2760,11 @@ static void icl_mg_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
> u32 val;
>
>
>
>
> ddi_translations = icl_get_mg_buf_trans(encoder, crtc_state, &n_entries);
> - /* The table does not have values for level 3 and level 9. */
> - if (level >= n_entries || level == 3 || level == 9) {
> + if (level >= n_entries) {
> drm_dbg_kms(&dev_priv->drm,
> "DDI translation not found for level %d. Using %d instead.",
> - level, n_entries - 2);
> - level = n_entries - 2;
> + level, n_entries - 1);
> + level = n_entries - 1;
> }
>
>
>
>
> /* Set MG_TX_LINK_PARAMS cri_use_fs32 to 0. */
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables
2020-12-07 20:35 ` [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables Ville Syrjala
@ 2021-01-20 19:22 ` Souza, Jose
0 siblings, 0 replies; 8+ messages in thread
From: Souza, Jose @ 2021-01-20 19:22 UTC (permalink / raw)
To: ville.syrjala@linux.intel.com, intel-gfx@lists.freedesktop.org
On Mon, 2020-12-07 at 22:35 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Get rid of the "I like my random new style best" approach and unify
> the handling for the DDI buf trans table sanity checks once again.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 23 ++++++++++-------------
> 1 file changed, 10 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> index c3a15ce66478..68693d4538e2 100644
> --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> @@ -2631,15 +2631,11 @@ static void icl_ddi_combo_vswing_program(struct intel_encoder *encoder,
> ddi_translations = ehl_get_combo_buf_trans(encoder, crtc_state, &n_entries);
> else
> ddi_translations = icl_get_combo_buf_trans(encoder, crtc_state, &n_entries);
> - if (!ddi_translations)
> - return;
>
>
>
>
> - if (level >= n_entries) {
> - drm_dbg_kms(&dev_priv->drm,
> - "DDI translation not found for level %d. Using %d instead.",
> - level, n_entries - 1);
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
> + return;
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
> level = n_entries - 1;
> - }
>
>
>
>
> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP)) {
> struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
> @@ -2760,12 +2756,11 @@ static void icl_mg_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
> u32 val;
>
>
>
>
> ddi_translations = icl_get_mg_buf_trans(encoder, crtc_state, &n_entries);
> - if (level >= n_entries) {
> - drm_dbg_kms(&dev_priv->drm,
> - "DDI translation not found for level %d. Using %d instead.",
> - level, n_entries - 1);
> +
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
> + return;
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
> level = n_entries - 1;
> - }
>
>
>
>
> /* Set MG_TX_LINK_PARAMS cri_use_fs32 to 0. */
> for (ln = 0; ln < 2; ln++) {
> @@ -2897,7 +2892,9 @@ tgl_dkl_phy_ddi_vswing_sequence(struct intel_encoder *encoder,
>
>
>
>
> ddi_translations = tgl_get_dkl_buf_trans(encoder, crtc_state, &n_entries);
>
>
>
>
> - if (level >= n_entries)
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, !ddi_translations))
> + return;
> + if (drm_WARN_ON_ONCE(&dev_priv->drm, level >= n_entries))
> level = n_entries - 1;
>
>
>
>
> dpcnt_mask = (DKL_TX_PRESHOOT_COEFF_MASK |
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-01-20 19:22 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-07 20:35 [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Ville Syrjala
2020-12-07 20:35 ` [Intel-gfx] [PATCH 2/2] drm/i915: Unify the sanity checks for the buf trans tables Ville Syrjala
2021-01-20 19:22 ` Souza, Jose
2020-12-07 21:26 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling Patchwork
2020-12-14 16:27 ` [Intel-gfx] ✗ Fi.CI.DOCS: warning for series starting with [1/2] drm/i915: Fix ICL MG PHY vswing handling (rev2) Patchwork
2020-12-14 16:53 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2020-12-14 19:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-01-20 19:19 ` [Intel-gfx] [PATCH 1/2] drm/i915: Fix ICL MG PHY vswing handling Souza, Jose
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox