* [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2019-12-24 8:42 ` Kai-Heng Feng 0 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2019-12-24 8:42 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: ville.syrjala, swati2.sharma, intel-gfx, dri-devel, linux-kernel, Kai-Heng Feng On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port becomes useless and never responds to cable hotplugging: [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D Seems like the lspcon chip on the system in question only gets powered after the cable is plugged. So let's call lspcon_init() dynamically to properly initialize the lspcon chip and make HDMI port work. Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> --- v3: - Make sure it's handled under long HPD case. v2: - Move lspcon_init() inside of intel_dp_hpd_pulse(). drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index fe31bbfd6c62..a72c9c041c60 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -6573,6 +6573,7 @@ enum irqreturn intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) { struct intel_dp *intel_dp = &intel_dig_port->dp; + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { /* @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) long_hpd ? "long" : "short"); if (long_hpd) { - intel_dp->reset_link_params = true; + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) + lspcon_init(intel_dig_port); + else + intel_dp->reset_link_params = true; + return IRQ_NONE; } -- 2.17.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [Intel-gfx] [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2019-12-24 8:42 ` Kai-Heng Feng 0 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2019-12-24 8:42 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: intel-gfx, linux-kernel, dri-devel, Kai-Heng Feng On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port becomes useless and never responds to cable hotplugging: [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D Seems like the lspcon chip on the system in question only gets powered after the cable is plugged. So let's call lspcon_init() dynamically to properly initialize the lspcon chip and make HDMI port work. Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> --- v3: - Make sure it's handled under long HPD case. v2: - Move lspcon_init() inside of intel_dp_hpd_pulse(). drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index fe31bbfd6c62..a72c9c041c60 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -6573,6 +6573,7 @@ enum irqreturn intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) { struct intel_dp *intel_dp = &intel_dig_port->dp; + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { /* @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) long_hpd ? "long" : "short"); if (long_hpd) { - intel_dp->reset_link_params = true; + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) + lspcon_init(intel_dig_port); + else + intel_dp->reset_link_params = true; + return IRQ_NONE; } -- 2.17.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2019-12-24 8:42 ` Kai-Heng Feng 0 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2019-12-24 8:42 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: intel-gfx, linux-kernel, dri-devel, Kai-Heng Feng, swati2.sharma On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port becomes useless and never responds to cable hotplugging: [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D Seems like the lspcon chip on the system in question only gets powered after the cable is plugged. So let's call lspcon_init() dynamically to properly initialize the lspcon chip and make HDMI port work. Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> --- v3: - Make sure it's handled under long HPD case. v2: - Move lspcon_init() inside of intel_dp_hpd_pulse(). drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index fe31bbfd6c62..a72c9c041c60 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -6573,6 +6573,7 @@ enum irqreturn intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) { struct intel_dp *intel_dp = &intel_dig_port->dp; + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { /* @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) long_hpd ? "long" : "short"); if (long_hpd) { - intel_dp->reset_link_params = true; + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) + lspcon_init(intel_dig_port); + else + intel_dp->reset_link_params = true; + return IRQ_NONE; } -- 2.17.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev3) 2019-12-24 8:42 ` Kai-Heng Feng (?) (?) @ 2019-12-24 10:26 ` Patchwork -1 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2019-12-24 10:26 UTC (permalink / raw) To: Kai-Heng Feng; +Cc: intel-gfx == Series Details == Series: drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev3) URL : https://patchwork.freedesktop.org/series/71314/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7631 -> Patchwork_15914 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_15914 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_15914, 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_15914/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_15914: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live_objects: - fi-bwr-2160: [PASS][1] -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-bwr-2160/igt@i915_selftest@live_objects.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-bwr-2160/igt@i915_selftest@live_objects.html * igt@kms_chamelium@dp-crc-fast: - fi-kbl-7500u: [PASS][3] -> [DMESG-WARN][4] +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: [PASS][5] -> [DMESG-WARN][6] +4 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html - fi-kbl-7500u: [PASS][7] -> [DMESG-FAIL][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html Known issues ------------ Here are the changes found in Patchwork_15914 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_close_race@basic-threads: - fi-byt-n2820: [PASS][9] -> [TIMEOUT][10] ([i915#816]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-byt-n2820/igt@gem_close_race@basic-threads.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-byt-n2820/igt@gem_close_race@basic-threads.html * igt@gem_exec_suspend@basic-s3: - fi-cml-s: [PASS][11] -> [DMESG-WARN][12] ([fdo#111764]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-cml-s/igt@gem_exec_suspend@basic-s3.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-cml-s/igt@gem_exec_suspend@basic-s3.html * igt@gem_tiled_blits@basic: - fi-byt-j1900: [PASS][13] -> [FAIL][14] ([i915#832]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-byt-j1900/igt@gem_tiled_blits@basic.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-byt-j1900/igt@gem_tiled_blits@basic.html * igt@i915_pm_rpm@module-reload: - fi-skl-6770hq: [PASS][15] -> [FAIL][16] ([i915#178]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html * igt@i915_selftest@live_blt: - fi-hsw-4770: [PASS][17] -> [DMESG-FAIL][18] ([i915#725]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-hsw-4770/igt@i915_selftest@live_blt.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-hsw-4770/igt@i915_selftest@live_blt.html * igt@i915_selftest@live_gem_contexts: - fi-cfl-8700k: [PASS][19] -> [INCOMPLETE][20] ([i915#424]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html * igt@i915_selftest@live_hugepages: - fi-byt-j1900: [PASS][21] -> [DMESG-FAIL][22] ([i915#845]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-byt-j1900/igt@i915_selftest@live_hugepages.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-byt-j1900/igt@i915_selftest@live_hugepages.html * igt@kms_chamelium@dp-crc-fast: - fi-icl-u2: [PASS][23] -> [DMESG-FAIL][24] ([fdo#109635] / [i915#262]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-icl-u2/igt@kms_chamelium@dp-crc-fast.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-icl-u2/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@dp-edid-read: - fi-kbl-7500u: [PASS][25] -> [DMESG-FAIL][26] ([fdo#109635]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-7500u/igt@kms_chamelium@dp-edid-read.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-7500u/igt@kms_chamelium@dp-edid-read.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s0: - fi-cml-s: [FAIL][27] ([fdo#103375]) -> [PASS][28] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-cml-s/igt@gem_exec_suspend@basic-s0.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-cml-s/igt@gem_exec_suspend@basic-s0.html * igt@i915_module_load@reload-with-fault-injection: - fi-cfl-guc: [INCOMPLETE][29] ([i915#505] / [i915#671]) -> [PASS][30] [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html - fi-skl-lmem: [INCOMPLETE][31] ([i915#671]) -> [PASS][32] [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html #### Warnings #### * igt@i915_module_load@reload-with-fault-injection: - fi-kbl-x1275: [INCOMPLETE][33] ([i915#879]) -> [DMESG-WARN][34] ([i915#62] / [i915#92] / [i915#95]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html * igt@i915_selftest@live_blt: - fi-hsw-4770r: [DMESG-FAIL][35] ([i915#553] / [i915#725]) -> [DMESG-FAIL][36] ([i915#725]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-hsw-4770r/igt@i915_selftest@live_blt.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-hsw-4770r/igt@i915_selftest@live_blt.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-icl-u2: [FAIL][37] ([i915#217]) -> [DMESG-WARN][38] ([IGT#4] / [i915#263]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_cursor_legacy@basic-flip-after-cursor-legacy: - fi-kbl-x1275: [DMESG-WARN][39] ([i915#62] / [i915#92]) -> [DMESG-WARN][40] ([i915#62] / [i915#92] / [i915#95]) +3 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html * igt@kms_pipe_crc_basic@read-crc-pipe-a: - fi-kbl-x1275: [DMESG-WARN][41] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][42] ([i915#62] / [i915#92]) +3 similar issues [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7631/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html [IGT#4]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/4 [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375 [fdo#109635]: https://bugs.freedesktop.org/show_bug.cgi?id=109635 [fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764 [i915#178]: https://gitlab.freedesktop.org/drm/intel/issues/178 [i915#217]: https://gitlab.freedesktop.org/drm/intel/issues/217 [i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262 [i915#263]: https://gitlab.freedesktop.org/drm/intel/issues/263 [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424 [i915#505]: https://gitlab.freedesktop.org/drm/intel/issues/505 [i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553 [i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62 [i915#671]: https://gitlab.freedesktop.org/drm/intel/issues/671 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 [i915#832]: https://gitlab.freedesktop.org/drm/intel/issues/832 [i915#845]: https://gitlab.freedesktop.org/drm/intel/issues/845 [i915#879]: https://gitlab.freedesktop.org/drm/intel/issues/879 [i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92 [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95 Participating hosts (48 -> 41) ------------------------------ Additional (4): fi-bsw-kefka fi-kbl-8809g fi-ivb-3770 fi-bsw-nick Missing (11): fi-ilk-m540 fi-bsw-n3050 fi-hsw-peppy fi-byt-squawks fi-bsw-cyan fi-gdg-551 fi-kbl-7560u fi-tgl-y fi-byt-clapper fi-bdw-samus fi-snb-2600 Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7631 -> Patchwork_15914 CI-20190529: 20190529 CI_DRM_7631: ae1ee9002c3140488a3250b1ff3eab4ccbaf0097 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5352: 0586d205f651674e575351c2d5a7d0760716c9f1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_15914: 794c310f5807a1ed2f11774114c08372482f0174 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 794c310f5807 drm/i915: Re-init lspcon after HPD if lspcon probe failed == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15914/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev4) 2019-12-24 8:42 ` Kai-Heng Feng ` (2 preceding siblings ...) (?) @ 2020-01-02 8:56 ` Patchwork -1 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2020-01-02 8:56 UTC (permalink / raw) To: Kai-Heng Feng; +Cc: intel-gfx == Series Details == Series: drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev4) URL : https://patchwork.freedesktop.org/series/71314/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7664 -> Patchwork_15973 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_15973 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_15973, 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_15973/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_15973: ### IGT changes ### #### Possible regressions #### * igt@kms_chamelium@dp-crc-fast: - fi-kbl-7500u: NOTRUN -> [DMESG-WARN][1] +5 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: [PASS][2] -> [DMESG-WARN][3] +3 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_flip@basic-flip-vs-dpms: - fi-skl-guc: [PASS][4] -> [DMESG-WARN][5] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-skl-guc/igt@kms_flip@basic-flip-vs-dpms.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-skl-guc/igt@kms_flip@basic-flip-vs-dpms.html Known issues ------------ Here are the changes found in Patchwork_15973 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_module_load@reload-with-fault-injection: - fi-bxt-dsi: [PASS][6] -> [INCOMPLETE][7] ([fdo#103927]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html * igt@i915_selftest@live_blt: - fi-hsw-4770: [PASS][8] -> [DMESG-FAIL][9] ([i915#563]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-hsw-4770/igt@i915_selftest@live_blt.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-hsw-4770/igt@i915_selftest@live_blt.html * igt@i915_selftest@live_gt_engines: - fi-cfl-8700k: [PASS][10] -> [DMESG-FAIL][11] ([i915#889]) +7 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-cfl-8700k/igt@i915_selftest@live_gt_engines.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-cfl-8700k/igt@i915_selftest@live_gt_engines.html * igt@i915_selftest@live_gt_pm: - fi-cfl-8700k: [PASS][12] -> [DMESG-WARN][13] ([i915#889]) +23 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-cfl-8700k/igt@i915_selftest@live_gt_pm.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-cfl-8700k/igt@i915_selftest@live_gt_pm.html * igt@kms_chamelium@dp-crc-fast: - fi-icl-u2: [PASS][14] -> [DMESG-FAIL][15] ([fdo#109635] / [i915#262]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-icl-u2/igt@kms_chamelium@dp-crc-fast.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-icl-u2/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@dp-hpd-fast: - fi-icl-u2: [PASS][16] -> [DMESG-FAIL][17] ([fdo#109635]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-icl-u2/igt@kms_chamelium@dp-hpd-fast.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-icl-u2/igt@kms_chamelium@dp-hpd-fast.html #### Possible fixes #### * igt@gem_close_race@basic-threads: - fi-byt-j1900: [TIMEOUT][18] ([i915#816]) -> [PASS][19] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-byt-j1900/igt@gem_close_race@basic-threads.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-byt-j1900/igt@gem_close_race@basic-threads.html * igt@i915_selftest@live_gt_pm: - fi-bwr-2160: [FAIL][20] ([i915#878]) -> [PASS][21] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-bwr-2160/igt@i915_selftest@live_gt_pm.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-bwr-2160/igt@i915_selftest@live_gt_pm.html #### Warnings #### * igt@i915_module_load@reload-with-fault-injection: - fi-kbl-x1275: [INCOMPLETE][22] ([i915#879]) -> [DMESG-WARN][23] ([i915#62] / [i915#92]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html * igt@i915_selftest@live_blt: - fi-hsw-4770r: [DMESG-FAIL][24] ([i915#725]) -> [DMESG-FAIL][25] ([i915#553] / [i915#725]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-hsw-4770r/igt@i915_selftest@live_blt.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-hsw-4770r/igt@i915_selftest@live_blt.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-icl-u2: [DMESG-WARN][26] ([IGT#4] / [i915#263]) -> [FAIL][27] ([i915#217]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy: - fi-kbl-x1275: [DMESG-WARN][28] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][29] ([i915#62] / [i915#92]) +5 similar issues [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html * igt@kms_cursor_legacy@basic-flip-after-cursor-legacy: - fi-kbl-x1275: [DMESG-WARN][30] ([i915#62] / [i915#92]) -> [DMESG-WARN][31] ([i915#62] / [i915#92] / [i915#95]) +7 similar issues [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7664/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html [IGT#4]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/4 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#109635]: https://bugs.freedesktop.org/show_bug.cgi?id=109635 [i915#217]: https://gitlab.freedesktop.org/drm/intel/issues/217 [i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262 [i915#263]: https://gitlab.freedesktop.org/drm/intel/issues/263 [i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553 [i915#563]: https://gitlab.freedesktop.org/drm/intel/issues/563 [i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 [i915#878]: https://gitlab.freedesktop.org/drm/intel/issues/878 [i915#879]: https://gitlab.freedesktop.org/drm/intel/issues/879 [i915#889]: https://gitlab.freedesktop.org/drm/intel/issues/889 [i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92 [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95 Participating hosts (46 -> 40) ------------------------------ Additional (6): fi-bdw-5557u fi-skl-6770hq fi-kbl-7500u fi-ivb-3770 fi-skl-6700k2 fi-snb-2600 Missing (12): fi-ilk-m540 fi-bsw-n3050 fi-hsw-4200u fi-hsw-peppy fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-skl-lmem fi-bdw-samus fi-byt-clapper fi-skl-6600u Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7664 -> Patchwork_15973 CI-20190529: 20190529 CI_DRM_7664: d207bb67a1192ff32488fdc403e6708def18d80f @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5357: a555a4b98f90dab655d24bb3d07e9291a8b8dac8 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_15973: 2e56996acbe5ea3ce0975453b82f1b82c182cbea @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 2e56996acbe5 drm/i915: Re-init lspcon after HPD if lspcon probe failed == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15973/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed 2019-12-24 8:42 ` Kai-Heng Feng (?) @ 2020-01-06 8:35 ` Kai-Heng Feng -1 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2020-01-06 8:35 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: ville.syrjala, swati2.sharma, intel-gfx, dri-devel, linux-kernel Hi Jani, > On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > becomes useless and never responds to cable hotplugging: > [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > > Seems like the lspcon chip on the system in question only gets powered > after the cable is plugged. > > So let's call lspcon_init() dynamically to properly initialize the > lspcon chip and make HDMI port work. Do you have any further suggestion for this patch? Kai-Heng > > Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > --- > v3: > - Make sure it's handled under long HPD case. > > v2: > - Move lspcon_init() inside of intel_dp_hpd_pulse(). > > drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index fe31bbfd6c62..a72c9c041c60 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -6573,6 +6573,7 @@ enum irqreturn > intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > { > struct intel_dp *intel_dp = &intel_dig_port->dp; > + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > > if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > /* > @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > long_hpd ? "long" : "short"); > > if (long_hpd) { > - intel_dp->reset_link_params = true; > + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > + lspcon_init(intel_dig_port); > + else > + intel_dp->reset_link_params = true; > + > return IRQ_NONE; > } > > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-06 8:35 ` Kai-Heng Feng 0 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2020-01-06 8:35 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: dri-devel, intel-gfx, linux-kernel Hi Jani, > On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > becomes useless and never responds to cable hotplugging: > [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > > Seems like the lspcon chip on the system in question only gets powered > after the cable is plugged. > > So let's call lspcon_init() dynamically to properly initialize the > lspcon chip and make HDMI port work. Do you have any further suggestion for this patch? Kai-Heng > > Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > --- > v3: > - Make sure it's handled under long HPD case. > > v2: > - Move lspcon_init() inside of intel_dp_hpd_pulse(). > > drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index fe31bbfd6c62..a72c9c041c60 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -6573,6 +6573,7 @@ enum irqreturn > intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > { > struct intel_dp *intel_dp = &intel_dig_port->dp; > + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > > if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > /* > @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > long_hpd ? "long" : "short"); > > if (long_hpd) { > - intel_dp->reset_link_params = true; > + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > + lspcon_init(intel_dig_port); > + else > + intel_dp->reset_link_params = true; > + > return IRQ_NONE; > } > > -- > 2.17.1 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-06 8:35 ` Kai-Heng Feng 0 siblings, 0 replies; 16+ messages in thread From: Kai-Heng Feng @ 2020-01-06 8:35 UTC (permalink / raw) To: jani.nikula, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: dri-devel, intel-gfx, swati2.sharma, linux-kernel Hi Jani, > On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > becomes useless and never responds to cable hotplugging: > [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > > Seems like the lspcon chip on the system in question only gets powered > after the cable is plugged. > > So let's call lspcon_init() dynamically to properly initialize the > lspcon chip and make HDMI port work. Do you have any further suggestion for this patch? Kai-Heng > > Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > --- > v3: > - Make sure it's handled under long HPD case. > > v2: > - Move lspcon_init() inside of intel_dp_hpd_pulse(). > > drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index fe31bbfd6c62..a72c9c041c60 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -6573,6 +6573,7 @@ enum irqreturn > intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > { > struct intel_dp *intel_dp = &intel_dig_port->dp; > + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > > if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > /* > @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > long_hpd ? "long" : "short"); > > if (long_hpd) { > - intel_dp->reset_link_params = true; > + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > + lspcon_init(intel_dig_port); > + else > + intel_dp->reset_link_params = true; > + > return IRQ_NONE; > } > > -- > 2.17.1 > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed 2020-01-06 8:35 ` Kai-Heng Feng (?) @ 2020-01-14 14:11 ` Jani Nikula -1 siblings, 0 replies; 16+ messages in thread From: Jani Nikula @ 2020-01-14 14:11 UTC (permalink / raw) To: Kai-Heng Feng, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: ville.syrjala, swati2.sharma, intel-gfx, dri-devel, linux-kernel On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > Hi Jani, > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: >> >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port >> becomes useless and never responds to cable hotplugging: >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D >> >> Seems like the lspcon chip on the system in question only gets powered >> after the cable is plugged. >> >> So let's call lspcon_init() dynamically to properly initialize the >> lspcon chip and make HDMI port work. > > Do you have any further suggestion for this patch? Sorry for taking so long. Comments inline. > > Kai-Heng > >> >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> >> --- >> v3: >> - Make sure it's handled under long HPD case. >> >> v2: >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). >> >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c >> index fe31bbfd6c62..a72c9c041c60 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c >> @@ -6573,6 +6573,7 @@ enum irqreturn >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> { >> struct intel_dp *intel_dp = &intel_dig_port->dp; >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); >> >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { >> /* >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> long_hpd ? "long" : "short"); >> >> if (long_hpd) { >> - intel_dp->reset_link_params = true; >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) >> + lspcon_init(intel_dig_port); >> + else >> + intel_dp->reset_link_params = true; >> + Hmm, I think this will try to init lspcon on ports that do not have lspcon. Also, why wouldn't we reset the params? I think this boils down to just adding the following lines: if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && !intel_dig_port->lspcon.active) lspcon_init(intel_dig_port); Ville? BR, Jani. >> return IRQ_NONE; >> } >> >> -- >> 2.17.1 >> > -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-14 14:11 ` Jani Nikula 0 siblings, 0 replies; 16+ messages in thread From: Jani Nikula @ 2020-01-14 14:11 UTC (permalink / raw) To: Kai-Heng Feng, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: dri-devel, intel-gfx, linux-kernel On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > Hi Jani, > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: >> >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port >> becomes useless and never responds to cable hotplugging: >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D >> >> Seems like the lspcon chip on the system in question only gets powered >> after the cable is plugged. >> >> So let's call lspcon_init() dynamically to properly initialize the >> lspcon chip and make HDMI port work. > > Do you have any further suggestion for this patch? Sorry for taking so long. Comments inline. > > Kai-Heng > >> >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> >> --- >> v3: >> - Make sure it's handled under long HPD case. >> >> v2: >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). >> >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c >> index fe31bbfd6c62..a72c9c041c60 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c >> @@ -6573,6 +6573,7 @@ enum irqreturn >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> { >> struct intel_dp *intel_dp = &intel_dig_port->dp; >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); >> >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { >> /* >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> long_hpd ? "long" : "short"); >> >> if (long_hpd) { >> - intel_dp->reset_link_params = true; >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) >> + lspcon_init(intel_dig_port); >> + else >> + intel_dp->reset_link_params = true; >> + Hmm, I think this will try to init lspcon on ports that do not have lspcon. Also, why wouldn't we reset the params? I think this boils down to just adding the following lines: if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && !intel_dig_port->lspcon.active) lspcon_init(intel_dig_port); Ville? BR, Jani. >> return IRQ_NONE; >> } >> >> -- >> 2.17.1 >> > -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-14 14:11 ` Jani Nikula 0 siblings, 0 replies; 16+ messages in thread From: Jani Nikula @ 2020-01-14 14:11 UTC (permalink / raw) To: Kai-Heng Feng, joonas.lahtinen, rodrigo.vivi, airlied, daniel Cc: dri-devel, intel-gfx, swati2.sharma, linux-kernel On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > Hi Jani, > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: >> >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port >> becomes useless and never responds to cable hotplugging: >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D >> >> Seems like the lspcon chip on the system in question only gets powered >> after the cable is plugged. >> >> So let's call lspcon_init() dynamically to properly initialize the >> lspcon chip and make HDMI port work. > > Do you have any further suggestion for this patch? Sorry for taking so long. Comments inline. > > Kai-Heng > >> >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> >> --- >> v3: >> - Make sure it's handled under long HPD case. >> >> v2: >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). >> >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c >> index fe31bbfd6c62..a72c9c041c60 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c >> @@ -6573,6 +6573,7 @@ enum irqreturn >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> { >> struct intel_dp *intel_dp = &intel_dig_port->dp; >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); >> >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { >> /* >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) >> long_hpd ? "long" : "short"); >> >> if (long_hpd) { >> - intel_dp->reset_link_params = true; >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) >> + lspcon_init(intel_dig_port); >> + else >> + intel_dp->reset_link_params = true; >> + Hmm, I think this will try to init lspcon on ports that do not have lspcon. Also, why wouldn't we reset the params? I think this boils down to just adding the following lines: if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && !intel_dig_port->lspcon.active) lspcon_init(intel_dig_port); Ville? BR, Jani. >> return IRQ_NONE; >> } >> >> -- >> 2.17.1 >> > -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed 2020-01-14 14:11 ` Jani Nikula (?) @ 2020-01-14 14:55 ` Ville Syrjälä -1 siblings, 0 replies; 16+ messages in thread From: Ville Syrjälä @ 2020-01-14 14:55 UTC (permalink / raw) To: Jani Nikula Cc: Kai-Heng Feng, joonas.lahtinen, rodrigo.vivi, airlied, daniel, swati2.sharma, intel-gfx, dri-devel, linux-kernel On Tue, Jan 14, 2020 at 04:11:40PM +0200, Jani Nikula wrote: > On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > Hi Jani, > > > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > >> > >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > >> becomes useless and never responds to cable hotplugging: > >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > >> > >> Seems like the lspcon chip on the system in question only gets powered > >> after the cable is plugged. > >> > >> So let's call lspcon_init() dynamically to properly initialize the > >> lspcon chip and make HDMI port work. > > > > Do you have any further suggestion for this patch? > > Sorry for taking so long. Comments inline. > > > > > Kai-Heng > > > >> > >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > >> --- > >> v3: > >> - Make sure it's handled under long HPD case. > >> > >> v2: > >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). > >> > >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > >> 1 file changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > >> index fe31bbfd6c62..a72c9c041c60 100644 > >> --- a/drivers/gpu/drm/i915/display/intel_dp.c > >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c > >> @@ -6573,6 +6573,7 @@ enum irqreturn > >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> { > >> struct intel_dp *intel_dp = &intel_dig_port->dp; > >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > >> > >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > >> /* > >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> long_hpd ? "long" : "short"); > >> > >> if (long_hpd) { > >> - intel_dp->reset_link_params = true; > >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > >> + lspcon_init(intel_dig_port); > >> + else > >> + intel_dp->reset_link_params = true; > >> + > > Hmm, I think this will try to init lspcon on ports that do not have > lspcon. Also, why wouldn't we reset the params? > > I think this boils down to just adding the following lines: > > if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && > !intel_dig_port->lspcon.active) > lspcon_init(intel_dig_port); > > > Ville? This won't work right. Eg. intel_infoframe_init() assumes that lspcon init happens during driver load. We should probably change that to just trust the VBT and simply move the lspcon probe (if we even need one) into dp_detect() instead of sprinkling it around in several places. -- Ville Syrjälä Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-14 14:55 ` Ville Syrjälä 0 siblings, 0 replies; 16+ messages in thread From: Ville Syrjälä @ 2020-01-14 14:55 UTC (permalink / raw) To: Jani Nikula; +Cc: airlied, intel-gfx, linux-kernel, Kai-Heng Feng, dri-devel On Tue, Jan 14, 2020 at 04:11:40PM +0200, Jani Nikula wrote: > On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > Hi Jani, > > > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > >> > >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > >> becomes useless and never responds to cable hotplugging: > >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > >> > >> Seems like the lspcon chip on the system in question only gets powered > >> after the cable is plugged. > >> > >> So let's call lspcon_init() dynamically to properly initialize the > >> lspcon chip and make HDMI port work. > > > > Do you have any further suggestion for this patch? > > Sorry for taking so long. Comments inline. > > > > > Kai-Heng > > > >> > >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > >> --- > >> v3: > >> - Make sure it's handled under long HPD case. > >> > >> v2: > >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). > >> > >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > >> 1 file changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > >> index fe31bbfd6c62..a72c9c041c60 100644 > >> --- a/drivers/gpu/drm/i915/display/intel_dp.c > >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c > >> @@ -6573,6 +6573,7 @@ enum irqreturn > >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> { > >> struct intel_dp *intel_dp = &intel_dig_port->dp; > >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > >> > >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > >> /* > >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> long_hpd ? "long" : "short"); > >> > >> if (long_hpd) { > >> - intel_dp->reset_link_params = true; > >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > >> + lspcon_init(intel_dig_port); > >> + else > >> + intel_dp->reset_link_params = true; > >> + > > Hmm, I think this will try to init lspcon on ports that do not have > lspcon. Also, why wouldn't we reset the params? > > I think this boils down to just adding the following lines: > > if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && > !intel_dig_port->lspcon.active) > lspcon_init(intel_dig_port); > > > Ville? This won't work right. Eg. intel_infoframe_init() assumes that lspcon init happens during driver load. We should probably change that to just trust the VBT and simply move the lspcon probe (if we even need one) into dp_detect() instead of sprinkling it around in several places. -- 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] 16+ messages in thread
* Re: [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed @ 2020-01-14 14:55 ` Ville Syrjälä 0 siblings, 0 replies; 16+ messages in thread From: Ville Syrjälä @ 2020-01-14 14:55 UTC (permalink / raw) To: Jani Nikula Cc: airlied, intel-gfx, linux-kernel, swati2.sharma, Kai-Heng Feng, dri-devel, rodrigo.vivi On Tue, Jan 14, 2020 at 04:11:40PM +0200, Jani Nikula wrote: > On Mon, 06 Jan 2020, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > > Hi Jani, > > > >> On Dec 24, 2019, at 16:42, Kai-Heng Feng <kai.heng.feng@canonical.com> wrote: > >> > >> On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > >> becomes useless and never responds to cable hotplugging: > >> [ 3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > >> [ 3.031945] [drm:intel_ddi_init [i915]] *ERROR* LSPCON init failed on port D > >> > >> Seems like the lspcon chip on the system in question only gets powered > >> after the cable is plugged. > >> > >> So let's call lspcon_init() dynamically to properly initialize the > >> lspcon chip and make HDMI port work. > > > > Do you have any further suggestion for this patch? > > Sorry for taking so long. Comments inline. > > > > > Kai-Heng > > > >> > >> Closes: https://gitlab.freedesktop.org/drm/intel/issues/203 > >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com> > >> --- > >> v3: > >> - Make sure it's handled under long HPD case. > >> > >> v2: > >> - Move lspcon_init() inside of intel_dp_hpd_pulse(). > >> > >> drivers/gpu/drm/i915/display/intel_dp.c | 8 +++++++- > >> 1 file changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > >> index fe31bbfd6c62..a72c9c041c60 100644 > >> --- a/drivers/gpu/drm/i915/display/intel_dp.c > >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c > >> @@ -6573,6 +6573,7 @@ enum irqreturn > >> intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> { > >> struct intel_dp *intel_dp = &intel_dig_port->dp; > >> + struct drm_i915_private *dev_priv = dp_to_i915(intel_dp); > >> > >> if (long_hpd && intel_dig_port->base.type == INTEL_OUTPUT_EDP) { > >> /* > >> @@ -6593,7 +6594,12 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd) > >> long_hpd ? "long" : "short"); > >> > >> if (long_hpd) { > >> - intel_dp->reset_link_params = true; > >> + if (intel_dig_port->base.type == INTEL_OUTPUT_DDI && > >> + HAS_LSPCON(dev_priv) && !intel_dig_port->lspcon.active) > >> + lspcon_init(intel_dig_port); > >> + else > >> + intel_dp->reset_link_params = true; > >> + > > Hmm, I think this will try to init lspcon on ports that do not have > lspcon. Also, why wouldn't we reset the params? > > I think this boils down to just adding the following lines: > > if (intel_bios_is_lspcon_present(dev_priv, intel_dig_port->base.port) && > !intel_dig_port->lspcon.active) > lspcon_init(intel_dig_port); > > > Ville? This won't work right. Eg. intel_infoframe_init() assumes that lspcon init happens during driver load. We should probably change that to just trust the VBT and simply move the lspcon probe (if we even need one) into dp_detect() instead of sprinkling it around in several places. -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev5) 2019-12-24 8:42 ` Kai-Heng Feng ` (4 preceding siblings ...) (?) @ 2020-01-14 16:14 ` Patchwork -1 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2020-01-14 16:14 UTC (permalink / raw) To: Kai-Heng Feng; +Cc: intel-gfx == Series Details == Series: drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev5) URL : https://patchwork.freedesktop.org/series/71314/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7739 -> Patchwork_16094 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_16094 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_16094, 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_16094/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_16094: ### IGT changes ### #### Possible regressions #### * igt@kms_chamelium@dp-crc-fast: - fi-kbl-7500u: [PASS][1] -> [DMESG-WARN][2] +5 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: [PASS][3] -> [DMESG-WARN][4] +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_flip@basic-flip-vs-dpms: - fi-skl-guc: [PASS][5] -> [DMESG-WARN][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-skl-guc/igt@kms_flip@basic-flip-vs-dpms.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-skl-guc/igt@kms_flip@basic-flip-vs-dpms.html Known issues ------------ Here are the changes found in Patchwork_16094 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_module_load@reload-with-fault-injection: - fi-bxt-dsi: [PASS][7] -> [DMESG-WARN][8] ([i915#889]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html * igt@i915_selftest@live_blt: - fi-ivb-3770: [PASS][9] -> [DMESG-FAIL][10] ([i915#725]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-ivb-3770/igt@i915_selftest@live_blt.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-ivb-3770/igt@i915_selftest@live_blt.html * igt@i915_selftest@live_execlists: - fi-glk-dsi: [PASS][11] -> [INCOMPLETE][12] ([i915#529] / [i915#58] / [k.org#198133]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-glk-dsi/igt@i915_selftest@live_execlists.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-glk-dsi/igt@i915_selftest@live_execlists.html * igt@kms_chamelium@dp-edid-read: - fi-icl-u2: [PASS][13] -> [DMESG-FAIL][14] ([fdo#109635]) +1 similar issue [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html * igt@vgem_basic@unload: - fi-tgl-y: [PASS][15] -> [DMESG-WARN][16] ([i915#402]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-tgl-y/igt@vgem_basic@unload.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-tgl-y/igt@vgem_basic@unload.html #### Possible fixes #### * igt@gem_close_race@basic-threads: - fi-byt-j1900: [TIMEOUT][17] ([fdo#112271] / [i915#816]) -> [PASS][18] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-byt-j1900/igt@gem_close_race@basic-threads.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-byt-j1900/igt@gem_close_race@basic-threads.html * igt@gem_exec_fence@basic-wait-default: - {fi-ehl-1}: [INCOMPLETE][19] ([i915#937]) -> [PASS][20] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-ehl-1/igt@gem_exec_fence@basic-wait-default.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-ehl-1/igt@gem_exec_fence@basic-wait-default.html * igt@i915_pm_rpm@module-reload: - fi-skl-6770hq: [FAIL][21] ([i915#178]) -> [PASS][22] [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html * igt@i915_selftest@live_blt: - fi-hsw-4770: [DMESG-FAIL][23] ([i915#725]) -> [PASS][24] [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-hsw-4770/igt@i915_selftest@live_blt.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-hsw-4770/igt@i915_selftest@live_blt.html * igt@vgem_basic@dmabuf-export: - fi-tgl-y: [DMESG-WARN][25] ([i915#402]) -> [PASS][26] +1 similar issue [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-tgl-y/igt@vgem_basic@dmabuf-export.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-tgl-y/igt@vgem_basic@dmabuf-export.html #### Warnings #### * igt@i915_module_load@reload-with-fault-injection: - fi-skl-lmem: [INCOMPLETE][27] ([i915#671]) -> [DMESG-WARN][28] ([i915#889]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7739/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109635]: https://bugs.freedesktop.org/show_bug.cgi?id=109635 [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#178]: https://gitlab.freedesktop.org/drm/intel/issues/178 [i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402 [i915#529]: https://gitlab.freedesktop.org/drm/intel/issues/529 [i915#58]: https://gitlab.freedesktop.org/drm/intel/issues/58 [i915#671]: https://gitlab.freedesktop.org/drm/intel/issues/671 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 [i915#889]: https://gitlab.freedesktop.org/drm/intel/issues/889 [i915#937]: https://gitlab.freedesktop.org/drm/intel/issues/937 [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133 Participating hosts (53 -> 46) ------------------------------ Missing (7): fi-hsw-4770r fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7739 -> Patchwork_16094 CI-20190529: 20190529 CI_DRM_7739: 757c25a357ea6e34d5eba9b6efee6f45e7961334 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5365: e9ec0ed63b25c86861ffac3c8601cc4d1b910b65 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16094: 7f8839452eb202f2796622c5dd5cbab1cf6742a8 @ git://anongit.freedesktop.org/gfx-ci/linux == Kernel 32bit build == Warning: Kernel 32bit buildtest failed: https://intel-gfx-ci.01.org/Patchwork_16094/build_32bit.log CALL scripts/checksyscalls.sh CALL scripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x86/boot/bzImage is ready (#1) Building modules, stage 2. MODPOST 122 modules ERROR: "__udivdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! scripts/Makefile.modpost:93: recipe for target '__modpost' failed make[1]: *** [__modpost] Error 1 Makefile:1282: recipe for target 'modules' failed make: *** [modules] Error 2 == Linux commits == 7f8839452eb2 drm/i915: Re-init lspcon after HPD if lspcon probe failed == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev5) 2019-12-24 8:42 ` Kai-Heng Feng ` (5 preceding siblings ...) (?) @ 2020-01-14 16:14 ` Patchwork -1 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2020-01-14 16:14 UTC (permalink / raw) To: Kai-Heng Feng; +Cc: intel-gfx == Series Details == Series: drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev5) URL : https://patchwork.freedesktop.org/series/71314/ State : warning == Summary == CALL scripts/checksyscalls.sh CALL scripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x86/boot/bzImage is ready (#1) Building modules, stage 2. MODPOST 122 modules ERROR: "__udivdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! scripts/Makefile.modpost:93: recipe for target '__modpost' failed make[1]: *** [__modpost] Error 1 Makefile:1282: recipe for target 'modules' failed make: *** [modules] Error 2 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16094/build_32bit.log _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2020-01-14 16:14 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-12-24 8:42 [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed Kai-Heng Feng 2019-12-24 8:42 ` [Intel-gfx] " Kai-Heng Feng 2019-12-24 8:42 ` Kai-Heng Feng 2019-12-24 10:26 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev3) Patchwork 2020-01-02 8:56 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev4) Patchwork 2020-01-06 8:35 ` [PATCH v3] drm/i915: Re-init lspcon after HPD if lspcon probe failed Kai-Heng Feng 2020-01-06 8:35 ` [Intel-gfx] " Kai-Heng Feng 2020-01-06 8:35 ` Kai-Heng Feng 2020-01-14 14:11 ` Jani Nikula 2020-01-14 14:11 ` [Intel-gfx] " Jani Nikula 2020-01-14 14:11 ` Jani Nikula 2020-01-14 14:55 ` Ville Syrjälä 2020-01-14 14:55 ` [Intel-gfx] " Ville Syrjälä 2020-01-14 14:55 ` Ville Syrjälä 2020-01-14 16:14 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Re-init lspcon after HPD if lspcon probe failed (rev5) Patchwork 2020-01-14 16:14 ` [Intel-gfx] ✗ Fi.CI.BUILD: warning " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.