[PATCH] drm/i915/tgl: disable DDIC

[PATCH] drm/i915/tgl: disable DDIC

[Lucas De Marchi]

The current SKUs added for Tiger Lake don't have DDIC hooked up, even though it
is supported by the SoC. The current state for these SKUs is problematic
since while enabling the combo phy, PORT_COMP_DW* return 0xFFFFFFFF,
which is invalid per register definition.

During initialization we check what phys are not yet enabled by reading
PHY_MISC_C and try to enable it by toggling the "DE to IO Comp Pwr Down"
bit.  But after that any read to the PORT_COMP_DW* returns invalid
results. This removes the following warning

[56997.634353] Missing case (val == 4294967295)
[56997.639241] WARNING: CPU: 5 PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54 cnl_get_procmon_ref_values+0xc9/0xf0 [i915]
[56997.639808] Modules linked in: i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm irqbypass crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e [last unloaded: prime_numbers]
[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-demarchi+ #65
[56997.639808] Hardware name: Intel Corporation Tiger Lake Client Platform/TigerLake U DDR4 SODIMM RVP, BIOS TGLSFWI1.R00.2252.A03.1906270154 06/27/2019
[56997.639808] RIP: 0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915]
[56997.639808] Code: 2c a0 85 c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7 c6 3c 3a 31 a0 48 c7 c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48 c7 c0 24 a4 2
c a0 eb a8 e8 be
[56997.639808] RSP: 0018:ffffc9000068f8a8 EFLAGS: 00010286
[56997.639808] RAX: 0000000000000000 RBX: ffff88848fa90000 RCX: 0000000000000000
[56997.639808] RDX: ffff8884a08b5ef8 RSI: ffff8884a08a6658 RDI: 00000000ffffffff
[56997.639808] RBP: 0000000000000002 R08: 0000000000000000 R09: 0000000000000000
[56997.639808] R10: 0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000
[56997.639808] R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
[56997.639808] FS:  00007f61ca3d12c0(0000) GS:ffff8884a0880000(0000) knlGS:0000000000000000
[56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[56997.639808] CR2: 00007f71be6a92c0 CR3: 0000000494750006 CR4: 0000000000760ee0
[56997.639808] PKRU: 55555554
[56997.639808] Call Trace:
[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915]
[56997.639808]  ? rcu_read_lock_sched_held+0x6f/0x80
[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915]
[56997.639808]  icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915]
[56997.639808]  intel_combo_phy_init+0x17e/0x3e0 [i915]
[56997.639808]  ? icl_display_core_init+0x2c/0x1a0 [i915]
[56997.639808]  ? _raw_spin_unlock_irqrestore+0x4c/0x60
[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915]
[56997.639808]  intel_power_domains_init_hw+0x200/0x570 [i915]
[56997.639808]  i915_driver_probe+0x103b/0x17e0 [i915]
[56997.639808]  ? printk+0x53/0x6a
[56997.639808]  i915_pci_probe+0x3b/0x190 [i915]

We may or may not need to change the implementation to account for DDIC
being available on other SKUs. For now I think the best thing to do is
to just disable the port.

Cc: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 5b733e38eae3..6c6a5a5f41bb 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct drm_i915_private *dev_priv, enum phy phy)
 	if (phy == PHY_NONE)
 		return false;
 
-	if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
+	if (IS_ELKHARTLAKE(dev_priv))
 		return phy <= PHY_C;
 
 	if (INTEL_GEN(dev_priv) >= 11)
@@ -15317,7 +15317,6 @@ static void intel_setup_outputs(struct drm_i915_private *dev_priv)
 		/* TODO: initialize TC ports as well */
 		intel_ddi_init(dev_priv, PORT_A);
 		intel_ddi_init(dev_priv, PORT_B);
-		intel_ddi_init(dev_priv, PORT_C);
 		icl_dsi_init(dev_priv);
 	} else if (IS_ELKHARTLAKE(dev_priv)) {
 		intel_ddi_init(dev_priv, PORT_A);
-- 
2.21.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✗ Fi.CI.CHECKPATCH: warning for drm/i915/tgl: disable DDIC

[Patchwork]

== Series Details ==

Series: drm/i915/tgl: disable DDIC
URL   : https://patchwork.freedesktop.org/series/65217/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
335a81e24ca8 drm/i915/tgl: disable DDIC
-:9: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#9: 
The current SKUs added for Tiger Lake don't have DDIC hooked up, even though it

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

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.BAT: success for drm/i915/tgl: disable DDIC

[Patchwork]

== Series Details ==

Series: drm/i915/tgl: disable DDIC
URL   : https://patchwork.freedesktop.org/series/65217/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_6710 -> Patchwork_14020
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/

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

  Here are the changes found in Patchwork_14020 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_create@basic:
    - fi-icl-u3:          [PASS][1] -> [DMESG-WARN][2] ([fdo#107724])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/fi-icl-u3/igt@gem_ctx_create@basic.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/fi-icl-u3/igt@gem_ctx_create@basic.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-hsw-peppy:       [PASS][3] -> [DMESG-WARN][4] ([fdo#102614])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html

  
#### Possible fixes ####

  * igt@i915_selftest@live_mman:
    - fi-bsw-n3050:       [DMESG-WARN][5] ([fdo#111373]) -> [PASS][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/fi-bsw-n3050/igt@i915_selftest@live_mman.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/fi-bsw-n3050/igt@i915_selftest@live_mman.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#109635 ]: https://bugs.freedesktop.org/show_bug.cgi?id=109635 
  [fdo#110387]: https://bugs.freedesktop.org/show_bug.cgi?id=110387
  [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
  [fdo#111046 ]: https://bugs.freedesktop.org/show_bug.cgi?id=111046 
  [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
  [fdo#111373]: https://bugs.freedesktop.org/show_bug.cgi?id=111373


Participating hosts (53 -> 44)
------------------------------

  Additional (1): fi-kbl-8809g 
  Missing    (10): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-icl-u2 fi-bsw-cyan fi-icl-y fi-icl-guc fi-byt-clapper fi-bdw-samus 


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

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_6710 -> Patchwork_14020

  CI-20190529: 20190529
  CI_DRM_6710: 131c6ccdf21739498689f22c973b1b77660ae7b9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5134: 81df2f22385bc275975cf199d962eed9bc10f916 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_14020: 335a81e24ca83f0c0270bb22889a2805b6e20f91 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

335a81e24ca8 drm/i915/tgl: disable DDIC

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.IGT: success for drm/i915/tgl: disable DDIC

[Patchwork]

== Series Details ==

Series: drm/i915/tgl: disable DDIC
URL   : https://patchwork.freedesktop.org/series/65217/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_6710_full -> Patchwork_14020_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

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

  Here are the changes found in Patchwork_14020_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_shared@exec-single-timeline-bsd:
    - shard-iclb:         [PASS][1] -> [SKIP][2] ([fdo#110841])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb3/igt@gem_ctx_shared@exec-single-timeline-bsd.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb1/igt@gem_ctx_shared@exec-single-timeline-bsd.html

  * igt@gem_exec_balancer@smoke:
    - shard-iclb:         [PASS][3] -> [SKIP][4] ([fdo#110854])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb2/igt@gem_exec_balancer@smoke.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb5/igt@gem_exec_balancer@smoke.html

  * igt@gem_exec_schedule@preempt-other-chain-bsd:
    - shard-iclb:         [PASS][5] -> [SKIP][6] ([fdo#111325]) +5 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb7/igt@gem_exec_schedule@preempt-other-chain-bsd.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb2/igt@gem_exec_schedule@preempt-other-chain-bsd.html

  * igt@i915_suspend@fence-restore-tiled2untiled:
    - shard-apl:          [PASS][7] -> [DMESG-WARN][8] ([fdo#108566]) +3 similar issues
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-apl7/igt@i915_suspend@fence-restore-tiled2untiled.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-apl5/igt@i915_suspend@fence-restore-tiled2untiled.html

  * igt@kms_fbcon_fbt@fbc-suspend:
    - shard-skl:          [PASS][9] -> [INCOMPLETE][10] ([fdo#104108])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-skl4/igt@kms_fbcon_fbt@fbc-suspend.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-skl1/igt@kms_fbcon_fbt@fbc-suspend.html

  * igt@kms_flip@2x-flip-vs-expired-vblank:
    - shard-hsw:          [PASS][11] -> [FAIL][12] ([fdo#102887])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-hsw5/igt@kms_flip@2x-flip-vs-expired-vblank.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-hsw2/igt@kms_flip@2x-flip-vs-expired-vblank.html

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
    - shard-iclb:         [PASS][13] -> [FAIL][14] ([fdo#103167]) +4 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html

  * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
    - shard-skl:          [PASS][15] -> [FAIL][16] ([fdo#108145] / [fdo#110403])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-skl2/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html

  * igt@kms_psr@psr2_cursor_render:
    - shard-iclb:         [PASS][17] -> [SKIP][18] ([fdo#109441]) +2 similar issues
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb2/igt@kms_psr@psr2_cursor_render.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb4/igt@kms_psr@psr2_cursor_render.html

  * igt@prime_busy@hang-bsd2:
    - shard-iclb:         [PASS][19] -> [SKIP][20] ([fdo#109276]) +15 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb4/igt@prime_busy@hang-bsd2.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb3/igt@prime_busy@hang-bsd2.html

  
#### Possible fixes ####

  * igt@gem_eio@reset-stress:
    - shard-snb:          [FAIL][21] ([fdo#109661]) -> [PASS][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-snb2/igt@gem_eio@reset-stress.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-snb2/igt@gem_eio@reset-stress.html

  * igt@gem_exec_async@concurrent-writes-bsd:
    - shard-iclb:         [SKIP][23] ([fdo#111325]) -> [PASS][24] +4 similar issues
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb2/igt@gem_exec_async@concurrent-writes-bsd.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb6/igt@gem_exec_async@concurrent-writes-bsd.html

  * igt@gem_exec_schedule@preempt-contexts-bsd2:
    - shard-iclb:         [SKIP][25] ([fdo#109276]) -> [PASS][26] +16 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb8/igt@gem_exec_schedule@preempt-contexts-bsd2.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb4/igt@gem_exec_schedule@preempt-contexts-bsd2.html

  * igt@gem_softpin@noreloc-s3:
    - shard-skl:          [INCOMPLETE][27] ([fdo#104108]) -> [PASS][28] +1 similar issue
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-skl3/igt@gem_softpin@noreloc-s3.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-skl1/igt@gem_softpin@noreloc-s3.html

  * igt@kms_flip@plain-flip-ts-check:
    - shard-skl:          [FAIL][29] ([fdo#100368]) -> [PASS][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-skl8/igt@kms_flip@plain-flip-ts-check.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-skl8/igt@kms_flip@plain-flip-ts-check.html

  * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite:
    - shard-iclb:         [FAIL][31] ([fdo#103167]) -> [PASS][32] +5 similar issues
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite.html

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-pwrite:
    - shard-snb:          [SKIP][33] ([fdo#109271]) -> [PASS][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-snb7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-pwrite.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-snb4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-pwrite.html

  * igt@kms_plane@pixel-format-pipe-b-planes-source-clamping:
    - shard-iclb:         [INCOMPLETE][35] ([fdo#107713] / [fdo#110036 ]) -> [PASS][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb7/igt@kms_plane@pixel-format-pipe-b-planes-source-clamping.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb2/igt@kms_plane@pixel-format-pipe-b-planes-source-clamping.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
    - shard-iclb:         [INCOMPLETE][37] ([fdo#107713] / [fdo#110042]) -> [PASS][38]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb8/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html

  * igt@kms_psr@psr2_dpms:
    - shard-iclb:         [SKIP][39] ([fdo#109441]) -> [PASS][40] +1 similar issue
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb4/igt@kms_psr@psr2_dpms.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb2/igt@kms_psr@psr2_dpms.html

  * igt@kms_vblank@pipe-b-ts-continuation-suspend:
    - shard-apl:          [DMESG-WARN][41] ([fdo#108566]) -> [PASS][42] +2 similar issues
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-apl7/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-apl1/igt@kms_vblank@pipe-b-ts-continuation-suspend.html

  * igt@perf@polling:
    - shard-skl:          [FAIL][43] ([fdo#110728]) -> [PASS][44]
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-skl1/igt@perf@polling.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-skl4/igt@perf@polling.html

  
#### Warnings ####

  * igt@gem_ctx_isolation@vcs1-nonpriv:
    - shard-iclb:         [FAIL][45] ([fdo#111329]) -> [SKIP][46] ([fdo#109276])
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb2/igt@gem_ctx_isolation@vcs1-nonpriv.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb5/igt@gem_ctx_isolation@vcs1-nonpriv.html

  * igt@gem_mocs_settings@mocs-isolation-bsd2:
    - shard-iclb:         [FAIL][47] ([fdo#111330]) -> [SKIP][48] ([fdo#109276])
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb4/igt@gem_mocs_settings@mocs-isolation-bsd2.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb3/igt@gem_mocs_settings@mocs-isolation-bsd2.html

  * igt@kms_dp_dsc@basic-dsc-enable-edp:
    - shard-iclb:         [SKIP][49] ([fdo#109349]) -> [DMESG-WARN][50] ([fdo#107724])
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6710/shard-iclb4/igt@kms_dp_dsc@basic-dsc-enable-edp.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14020/shard-iclb2/igt@kms_dp_dsc@basic-dsc-enable-edp.html

  
  [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
  [fdo#102887]: https://bugs.freedesktop.org/show_bug.cgi?id=102887
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109349]: https://bugs.freedesktop.org/show_bug.cgi?id=109349
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661
  [fdo#110036 ]: https://bugs.freedesktop.org/show_bug.cgi?id=110036 
  [fdo#110042]: https://bugs.freedesktop.org/show_bug.cgi?id=110042
  [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403
  [fdo#110728]: https://bugs.freedesktop.org/show_bug.cgi?id=110728
  [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841
  [fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
  [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325
  [fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329
  [fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330


Participating hosts (10 -> 10)
------------------------------

  No changes in participating hosts


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

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_6710 -> Patchwork_14020

  CI-20190529: 20190529
  CI_DRM_6710: 131c6ccdf21739498689f22c973b1b77660ae7b9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5134: 81df2f22385bc275975cf199d962eed9bc10f916 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_14020: 335a81e24ca83f0c0270bb22889a2805b6e20f91 @ 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_14020/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/tgl: disable DDIC

[Shankar, Uma]

>-----Original Message-----
>From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of Lucas
>De Marchi
>Sent: Thursday, August 15, 2019 5:25 AM
>To: intel-gfx@lists.freedesktop.org
>Subject: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>
>The current SKUs added for Tiger Lake don't have DDIC hooked up, even though it is
>supported by the SoC. The current state for these SKUs is problematic since while
>enabling the combo phy, PORT_COMP_DW* return 0xFFFFFFFF, which is invalid per
>register definition.
>
>During initialization we check what phys are not yet enabled by reading PHY_MISC_C
>and try to enable it by toggling the "DE to IO Comp Pwr Down"
>bit.  But after that any read to the PORT_COMP_DW* returns invalid results. This
>removes the following warning
>
>[56997.634353] Missing case (val == 4294967295) [56997.639241] WARNING: CPU: 5
>PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54
>cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Modules linked in:
>i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm irqbypass
>crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e [last unloaded:
>prime_numbers]
>[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-
>demarchi+ #65
>[56997.639808] Hardware name: Intel Corporation Tiger Lake Client
>Platform/TigerLake U DDR4 SODIMM RVP, BIOS
>TGLSFWI1.R00.2252.A03.1906270154 06/27/2019 [56997.639808] RIP:
>0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Code: 2c a0 85
>c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7 c6 3c 3a 31 a0 48 c7
>c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48 c7 c0 24 a4 2 c a0
>eb a8 e8 be [56997.639808] RSP: 0018:ffffc9000068f8a8 EFLAGS: 00010286
>[56997.639808] RAX: 0000000000000000 RBX: ffff88848fa90000 RCX:
>0000000000000000 [56997.639808] RDX: ffff8884a08b5ef8 RSI: ffff8884a08a6658
>RDI: 00000000ffffffff [56997.639808] RBP: 0000000000000002 R08:
>0000000000000000 R09: 0000000000000000 [56997.639808] R10:
>0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000 [56997.639808]
>R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
>[56997.639808] FS:  00007f61ca3d12c0(0000) GS:ffff8884a0880000(0000)
>knlGS:0000000000000000 [56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0:
>0000000080050033 [56997.639808] CR2: 00007f71be6a92c0 CR3:
>0000000494750006 CR4: 0000000000760ee0 [56997.639808] PKRU: 55555554
>[56997.639808] Call Trace:
>[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915] [56997.639808]  ?
>rcu_read_lock_sched_held+0x6f/0x80
>[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915] [56997.639808]
>icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915] [56997.639808]
>intel_combo_phy_init+0x17e/0x3e0 [i915] [56997.639808]  ?
>icl_display_core_init+0x2c/0x1a0 [i915] [56997.639808]  ?
>_raw_spin_unlock_irqrestore+0x4c/0x60
>[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915] [56997.639808]
>intel_power_domains_init_hw+0x200/0x570 [i915] [56997.639808]
>i915_driver_probe+0x103b/0x17e0 [i915] [56997.639808]  ? printk+0x53/0x6a
>[56997.639808]  i915_pci_probe+0x3b/0x190 [i915]
>
>We may or may not need to change the implementation to account for DDIC being
>available on other SKUs. For now I think the best thing to do is to just disable the port.

This information ideally should be coming from VBT and based on that driver can take a call
whether to enable the port or not. So is this an interim solution and later would be dropped,
since there will/may be SKU's with PORT C enabled. 

I feel revocation of this port in VBT should be the right approach, instead of an interim solution.

Regards,
Uma Shankar

>Cc: José Roberto de Souza <jose.souza@intel.com>
>Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>---
> drivers/gpu/drm/i915/display/intel_display.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>b/drivers/gpu/drm/i915/display/intel_display.c
>index 5b733e38eae3..6c6a5a5f41bb 100644
>--- a/drivers/gpu/drm/i915/display/intel_display.c
>+++ b/drivers/gpu/drm/i915/display/intel_display.c
>@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct drm_i915_private
>*dev_priv, enum phy phy)
> 	if (phy == PHY_NONE)
> 		return false;
>
>-	if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
>+	if (IS_ELKHARTLAKE(dev_priv))
> 		return phy <= PHY_C;
>
> 	if (INTEL_GEN(dev_priv) >= 11)
>@@ -15317,7 +15317,6 @@ static void intel_setup_outputs(struct drm_i915_private
>*dev_priv)
> 		/* TODO: initialize TC ports as well */
> 		intel_ddi_init(dev_priv, PORT_A);
> 		intel_ddi_init(dev_priv, PORT_B);
>-		intel_ddi_init(dev_priv, PORT_C);
> 		icl_dsi_init(dev_priv);
> 	} else if (IS_ELKHARTLAKE(dev_priv)) {
> 		intel_ddi_init(dev_priv, PORT_A);
>--
>2.21.0
>
>_______________________________________________
>Intel-gfx mailing list
>Intel-gfx@lists.freedesktop.org
>https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/tgl: disable DDIC

[Lucas De Marchi]

On Fri, Aug 16, 2019 at 3:14 AM Shankar, Uma <uma.shankar@intel.com> wrote:
>
>
>
> >-----Original Message-----
> >From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of Lucas
> >De Marchi
> >Sent: Thursday, August 15, 2019 5:25 AM
> >To: intel-gfx@lists.freedesktop.org
> >Subject: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
> >
> >The current SKUs added for Tiger Lake don't have DDIC hooked up, even though it is
> >supported by the SoC. The current state for these SKUs is problematic since while
> >enabling the combo phy, PORT_COMP_DW* return 0xFFFFFFFF, which is invalid per
> >register definition.
> >
> >During initialization we check what phys are not yet enabled by reading PHY_MISC_C
> >and try to enable it by toggling the "DE to IO Comp Pwr Down"
> >bit.  But after that any read to the PORT_COMP_DW* returns invalid results. This
> >removes the following warning
> >
> >[56997.634353] Missing case (val == 4294967295) [56997.639241] WARNING: CPU: 5
> >PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54
> >cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Modules linked in:
> >i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm irqbypass
> >crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e [last unloaded:
> >prime_numbers]
> >[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-
> >demarchi+ #65
> >[56997.639808] Hardware name: Intel Corporation Tiger Lake Client
> >Platform/TigerLake U DDR4 SODIMM RVP, BIOS
> >TGLSFWI1.R00.2252.A03.1906270154 06/27/2019 [56997.639808] RIP:
> >0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Code: 2c a0 85
> >c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7 c6 3c 3a 31 a0 48 c7
> >c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48 c7 c0 24 a4 2 c a0
> >eb a8 e8 be [56997.639808] RSP: 0018:ffffc9000068f8a8 EFLAGS: 00010286
> >[56997.639808] RAX: 0000000000000000 RBX: ffff88848fa90000 RCX:
> >0000000000000000 [56997.639808] RDX: ffff8884a08b5ef8 RSI: ffff8884a08a6658
> >RDI: 00000000ffffffff [56997.639808] RBP: 0000000000000002 R08:
> >0000000000000000 R09: 0000000000000000 [56997.639808] R10:
> >0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000 [56997.639808]
> >R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
> >[56997.639808] FS:  00007f61ca3d12c0(0000) GS:ffff8884a0880000(0000)
> >knlGS:0000000000000000 [56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0:
> >0000000080050033 [56997.639808] CR2: 00007f71be6a92c0 CR3:
> >0000000494750006 CR4: 0000000000760ee0 [56997.639808] PKRU: 55555554
> >[56997.639808] Call Trace:
> >[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915] [56997.639808]  ?
> >rcu_read_lock_sched_held+0x6f/0x80
> >[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915] [56997.639808]
> >icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915] [56997.639808]
> >intel_combo_phy_init+0x17e/0x3e0 [i915] [56997.639808]  ?
> >icl_display_core_init+0x2c/0x1a0 [i915] [56997.639808]  ?
> >_raw_spin_unlock_irqrestore+0x4c/0x60
> >[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915] [56997.639808]
> >intel_power_domains_init_hw+0x200/0x570 [i915] [56997.639808]
> >i915_driver_probe+0x103b/0x17e0 [i915] [56997.639808]  ? printk+0x53/0x6a
> >[56997.639808]  i915_pci_probe+0x3b/0x190 [i915]
> >
> >We may or may not need to change the implementation to account for DDIC being
> >available on other SKUs. For now I think the best thing to do is to just disable the port.
>
> This information ideally should be coming from VBT and based on that driver can take a call
> whether to enable the port or not. So is this an interim solution and later would be dropped,
> since there will/may be SKU's with PORT C enabled.
>
> I feel revocation of this port in VBT should be the right approach, instead of an interim solution.

Ideally yes, but it's better something that works than the ideal
solution that doesn't. I wanted to go the VBT
route, but it wouldn't work with any machine that I have available.
Hence this patch.

When/if there's one sku with ddic, hopefully VBT will already be fixed
or we may go the route of differentiating
by PCI id.

Lucas De Marchi

>
> Regards,
> Uma Shankar
>
> >Cc: José Roberto de Souza <jose.souza@intel.com>
> >Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> >---
> > drivers/gpu/drm/i915/display/intel_display.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> >b/drivers/gpu/drm/i915/display/intel_display.c
> >index 5b733e38eae3..6c6a5a5f41bb 100644
> >--- a/drivers/gpu/drm/i915/display/intel_display.c
> >+++ b/drivers/gpu/drm/i915/display/intel_display.c
> >@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct drm_i915_private
> >*dev_priv, enum phy phy)
> >       if (phy == PHY_NONE)
> >               return false;
> >
> >-      if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
> >+      if (IS_ELKHARTLAKE(dev_priv))
> >               return phy <= PHY_C;
> >
> >       if (INTEL_GEN(dev_priv) >= 11)
> >@@ -15317,7 +15317,6 @@ static void intel_setup_outputs(struct drm_i915_private
> >*dev_priv)
> >               /* TODO: initialize TC ports as well */
> >               intel_ddi_init(dev_priv, PORT_A);
> >               intel_ddi_init(dev_priv, PORT_B);
> >-              intel_ddi_init(dev_priv, PORT_C);
> >               icl_dsi_init(dev_priv);
> >       } else if (IS_ELKHARTLAKE(dev_priv)) {
> >               intel_ddi_init(dev_priv, PORT_A);
> >--
> >2.21.0
> >
> >_______________________________________________
> >Intel-gfx mailing list
> >Intel-gfx@lists.freedesktop.org
> >https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Lucas De Marchi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/tgl: disable DDIC

[Shankar, Uma]

>-----Original Message-----
>From: Lucas De Marchi [mailto:lucas.de.marchi@gmail.com]
>Sent: Friday, August 16, 2019 8:44 PM
>To: Shankar, Uma <uma.shankar@intel.com>
>Cc: De Marchi, Lucas <lucas.demarchi@intel.com>; intel-gfx@lists.freedesktop.org
>Subject: Re: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>
>On Fri, Aug 16, 2019 at 3:14 AM Shankar, Uma <uma.shankar@intel.com> wrote:
>>
>>
>>
>> >-----Original Message-----
>> >From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On
>> >Behalf Of Lucas De Marchi
>> >Sent: Thursday, August 15, 2019 5:25 AM
>> >To: intel-gfx@lists.freedesktop.org
>> >Subject: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>> >
>> >The current SKUs added for Tiger Lake don't have DDIC hooked up, even
>> >though it is supported by the SoC. The current state for these SKUs
>> >is problematic since while enabling the combo phy, PORT_COMP_DW*
>> >return 0xFFFFFFFF, which is invalid per register definition.
>> >
>> >During initialization we check what phys are not yet enabled by
>> >reading PHY_MISC_C and try to enable it by toggling the "DE to IO Comp Pwr
>Down"
>> >bit.  But after that any read to the PORT_COMP_DW* returns invalid
>> >results. This removes the following warning
>> >
>> >[56997.634353] Missing case (val == 4294967295) [56997.639241]
>> >WARNING: CPU: 5
>> >PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54
>> >cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Modules linked in:
>> >i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm
>> >irqbypass crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e [last
>unloaded:
>> >prime_numbers]
>> >[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-
>> >demarchi+ #65
>> >[56997.639808] Hardware name: Intel Corporation Tiger Lake Client
>> >Platform/TigerLake U DDR4 SODIMM RVP, BIOS
>> >TGLSFWI1.R00.2252.A03.1906270154 06/27/2019 [56997.639808] RIP:
>> >0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Code:
>> >2c a0 85
>> >c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7 c6
>> >3c 3a 31 a0 48 c7
>> >c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48
>> >c7 c0 24 a4 2 c a0 eb a8 e8 be [56997.639808] RSP:
>> >0018:ffffc9000068f8a8 EFLAGS: 00010286 [56997.639808] RAX:
>0000000000000000 RBX: ffff88848fa90000 RCX:
>> >0000000000000000 [56997.639808] RDX: ffff8884a08b5ef8 RSI:
>> >ffff8884a08a6658
>> >RDI: 00000000ffffffff [56997.639808] RBP: 0000000000000002 R08:
>> >0000000000000000 R09: 0000000000000000 [56997.639808] R10:
>> >0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000
>> >[56997.639808]
>> >R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
>> >[56997.639808] FS:  00007f61ca3d12c0(0000) GS:ffff8884a0880000(0000)
>> >knlGS:0000000000000000 [56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0:
>> >0000000080050033 [56997.639808] CR2: 00007f71be6a92c0 CR3:
>> >0000000494750006 CR4: 0000000000760ee0 [56997.639808] PKRU: 55555554
>> >[56997.639808] Call Trace:
>> >[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915] [56997.639808]
>?
>> >rcu_read_lock_sched_held+0x6f/0x80
>> >[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915]
>> >[56997.639808]
>> >icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915] [56997.639808]
>> >intel_combo_phy_init+0x17e/0x3e0 [i915] [56997.639808]  ?
>> >icl_display_core_init+0x2c/0x1a0 [i915] [56997.639808]  ?
>> >_raw_spin_unlock_irqrestore+0x4c/0x60
>> >[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915]
>> >[56997.639808]
>> >intel_power_domains_init_hw+0x200/0x570 [i915] [56997.639808]
>> >i915_driver_probe+0x103b/0x17e0 [i915] [56997.639808]  ?
>> >printk+0x53/0x6a [56997.639808]  i915_pci_probe+0x3b/0x190 [i915]
>> >
>> >We may or may not need to change the implementation to account for
>> >DDIC being available on other SKUs. For now I think the best thing to do is to just
>disable the port.
>>
>> This information ideally should be coming from VBT and based on that
>> driver can take a call whether to enable the port or not. So is this
>> an interim solution and later would be dropped, since there will/may be SKU's with
>PORT C enabled.
>>
>> I feel revocation of this port in VBT should be the right approach, instead of an
>interim solution.
>
>Ideally yes, but it's better something that works than the ideal solution that doesn't. I
>wanted to go the VBT route, but it wouldn't work with any machine that I have
>available.
>Hence this patch.
>
>When/if there's one sku with ddic, hopefully VBT will already be fixed or we may go
>the route of differentiating by PCI id.

I feel this is a mistake in VBT which exposes an non-existent port and should be fixed. I don't think
adding PCIID based checks for a platform is a good idea. We should call init for all ports spec claims
to be supporting and based on VBT configuration just return if port is not available for use on that
particular design. Now with current approach, if on a new SKU this is fixed  (PORT C gets working) and
VBT exposes it also correctly, we will not even attempt to initialize it until we fix this patch and re-send
a somewhat similar to a revert version of this patch.

So my take would be that we should push VBT guys to fix the problem instead of adding a temporary WA in our
driver. I will leave to maintainers to suggest what is the optimum approach.

Regards,
Uma Shankar

>Lucas De Marchi
>
>>
>> Regards,
>> Uma Shankar
>>
>> >Cc: José Roberto de Souza <jose.souza@intel.com>
>> >Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> >---
>> > drivers/gpu/drm/i915/display/intel_display.c | 3 +--
>> > 1 file changed, 1 insertion(+), 2 deletions(-)
>> >
>> >diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>> >b/drivers/gpu/drm/i915/display/intel_display.c
>> >index 5b733e38eae3..6c6a5a5f41bb 100644
>> >--- a/drivers/gpu/drm/i915/display/intel_display.c
>> >+++ b/drivers/gpu/drm/i915/display/intel_display.c
>> >@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct drm_i915_private
>> >*dev_priv, enum phy phy)
>> >       if (phy == PHY_NONE)
>> >               return false;
>> >
>> >-      if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
>> >+      if (IS_ELKHARTLAKE(dev_priv))
>> >               return phy <= PHY_C;
>> >
>> >       if (INTEL_GEN(dev_priv) >= 11) @@ -15317,7 +15317,6 @@ static
>> >void intel_setup_outputs(struct drm_i915_private
>> >*dev_priv)
>> >               /* TODO: initialize TC ports as well */
>> >               intel_ddi_init(dev_priv, PORT_A);
>> >               intel_ddi_init(dev_priv, PORT_B);
>> >-              intel_ddi_init(dev_priv, PORT_C);
>> >               icl_dsi_init(dev_priv);
>> >       } else if (IS_ELKHARTLAKE(dev_priv)) {
>> >               intel_ddi_init(dev_priv, PORT_A);
>> >--
>> >2.21.0
>> >
>> >_______________________________________________
>> >Intel-gfx mailing list
>> >Intel-gfx@lists.freedesktop.org
>> >https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
>--
>Lucas De Marchi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/tgl: disable DDIC

[Lucas De Marchi]

On Mon, Aug 19, 2019 at 06:26:08AM -0700, Shankar, Uma wrote:
>
>
>>-----Original Message-----
>>From: Lucas De Marchi [mailto:lucas.de.marchi@gmail.com]
>>Sent: Friday, August 16, 2019 8:44 PM
>>To: Shankar, Uma <uma.shankar@intel.com>
>>Cc: De Marchi, Lucas <lucas.demarchi@intel.com>; intel-gfx@lists.freedesktop.org
>>Subject: Re: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>>
>>On Fri, Aug 16, 2019 at 3:14 AM Shankar, Uma <uma.shankar@intel.com> wrote:
>>>
>>>
>>>
>>> >-----Original Message-----
>>> >From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On
>>> >Behalf Of Lucas De Marchi
>>> >Sent: Thursday, August 15, 2019 5:25 AM
>>> >To: intel-gfx@lists.freedesktop.org
>>> >Subject: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>>> >
>>> >The current SKUs added for Tiger Lake don't have DDIC hooked up, even
>>> >though it is supported by the SoC. The current state for these SKUs
>>> >is problematic since while enabling the combo phy, PORT_COMP_DW*
>>> >return 0xFFFFFFFF, which is invalid per register definition.
>>> >
>>> >During initialization we check what phys are not yet enabled by
>>> >reading PHY_MISC_C and try to enable it by toggling the "DE to IO Comp Pwr
>>Down"
>>> >bit.  But after that any read to the PORT_COMP_DW* returns invalid
>>> >results. This removes the following warning
>>> >
>>> >[56997.634353] Missing case (val == 4294967295) [56997.639241]
>>> >WARNING: CPU: 5
>>> >PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54
>>> >cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Modules linked in:
>>> >i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm
>>> >irqbypass crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e [last
>>unloaded:
>>> >prime_numbers]
>>> >[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-
>>> >demarchi+ #65
>>> >[56997.639808] Hardware name: Intel Corporation Tiger Lake Client
>>> >Platform/TigerLake U DDR4 SODIMM RVP, BIOS
>>> >TGLSFWI1.R00.2252.A03.1906270154 06/27/2019 [56997.639808] RIP:
>>> >0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Code:
>>> >2c a0 85
>>> >c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7 c6
>>> >3c 3a 31 a0 48 c7
>>> >c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48
>>> >c7 c0 24 a4 2 c a0 eb a8 e8 be [56997.639808] RSP:
>>> >0018:ffffc9000068f8a8 EFLAGS: 00010286 [56997.639808] RAX:
>>0000000000000000 RBX: ffff88848fa90000 RCX:
>>> >0000000000000000 [56997.639808] RDX: ffff8884a08b5ef8 RSI:
>>> >ffff8884a08a6658
>>> >RDI: 00000000ffffffff [56997.639808] RBP: 0000000000000002 R08:
>>> >0000000000000000 R09: 0000000000000000 [56997.639808] R10:
>>> >0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000
>>> >[56997.639808]
>>> >R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
>>> >[56997.639808] FS:  00007f61ca3d12c0(0000) GS:ffff8884a0880000(0000)
>>> >knlGS:0000000000000000 [56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0:
>>> >0000000080050033 [56997.639808] CR2: 00007f71be6a92c0 CR3:
>>> >0000000494750006 CR4: 0000000000760ee0 [56997.639808] PKRU: 55555554
>>> >[56997.639808] Call Trace:
>>> >[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915] [56997.639808]
>>?
>>> >rcu_read_lock_sched_held+0x6f/0x80
>>> >[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915]
>>> >[56997.639808]
>>> >icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915] [56997.639808]
>>> >intel_combo_phy_init+0x17e/0x3e0 [i915] [56997.639808]  ?
>>> >icl_display_core_init+0x2c/0x1a0 [i915] [56997.639808]  ?
>>> >_raw_spin_unlock_irqrestore+0x4c/0x60
>>> >[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915]
>>> >[56997.639808]
>>> >intel_power_domains_init_hw+0x200/0x570 [i915] [56997.639808]
>>> >i915_driver_probe+0x103b/0x17e0 [i915] [56997.639808]  ?
>>> >printk+0x53/0x6a [56997.639808]  i915_pci_probe+0x3b/0x190 [i915]
>>> >
>>> >We may or may not need to change the implementation to account for
>>> >DDIC being available on other SKUs. For now I think the best thing to do is to just
>>disable the port.
>>>
>>> This information ideally should be coming from VBT and based on that
>>> driver can take a call whether to enable the port or not. So is this
>>> an interim solution and later would be dropped, since there will/may be SKU's with
>>PORT C enabled.
>>>
>>> I feel revocation of this port in VBT should be the right approach, instead of an
>>interim solution.
>>
>>Ideally yes, but it's better something that works than the ideal solution that doesn't. I
>>wanted to go the VBT route, but it wouldn't work with any machine that I have
>>available.
>>Hence this patch.
>>
>>When/if there's one sku with ddic, hopefully VBT will already be fixed or we may go
>>the route of differentiating by PCI id.
>
>I feel this is a mistake in VBT which exposes an non-existent port and should be fixed. I don't think
>adding PCIID based checks for a platform is a good idea. We should call init for all ports spec claims
>to be supporting and based on VBT configuration just return if port is not available for use on that
>particular design. Now with current approach, if on a new SKU this is fixed  (PORT C gets working) and

I agree with all of that. But it would not fix the problem that we have
*today*. There's no forseable fix for VBT.

>VBT exposes it also correctly, we will not even attempt to initialize it until we fix this patch and re-send
>a somewhat similar to a revert version of this patch.

the "fix"  would be "make DDIC work and *start* exporting it". More like
an additional feature than a fix in itself. It was my mistake to send
the initial patch with DDIC there when it was actually not working. See
that we still don't expose the TC ports and initially we didn't expose
the DSI port neither.

In other words, expose the things that are *currently* working. DDIC is
not and fixing it by hooking up VBT if we should initialize it (my
initial attempt) would not fix things.

>
>So my take would be that we should push VBT guys to fix the problem instead of adding a temporary WA in our

this has already been reported to people responsible for VBT. There's
currently not ETA for if/when this will be done. When/if it's done we
can conditionally add DDIC.

Lucas De Marchi

>driver. I will leave to maintainers to suggest what is the optimum approach.
>
>Regards,
>Uma Shankar
>
>>Lucas De Marchi
>>
>>>
>>> Regards,
>>> Uma Shankar
>>>
>>> >Cc: José Roberto de Souza <jose.souza@intel.com>
>>> >Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>>> >---
>>> > drivers/gpu/drm/i915/display/intel_display.c | 3 +--
>>> > 1 file changed, 1 insertion(+), 2 deletions(-)
>>> >
>>> >diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>>> >b/drivers/gpu/drm/i915/display/intel_display.c
>>> >index 5b733e38eae3..6c6a5a5f41bb 100644
>>> >--- a/drivers/gpu/drm/i915/display/intel_display.c
>>> >+++ b/drivers/gpu/drm/i915/display/intel_display.c
>>> >@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct drm_i915_private
>>> >*dev_priv, enum phy phy)
>>> >       if (phy == PHY_NONE)
>>> >               return false;
>>> >
>>> >-      if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
>>> >+      if (IS_ELKHARTLAKE(dev_priv))
>>> >               return phy <= PHY_C;
>>> >
>>> >       if (INTEL_GEN(dev_priv) >= 11) @@ -15317,7 +15317,6 @@ static
>>> >void intel_setup_outputs(struct drm_i915_private
>>> >*dev_priv)
>>> >               /* TODO: initialize TC ports as well */
>>> >               intel_ddi_init(dev_priv, PORT_A);
>>> >               intel_ddi_init(dev_priv, PORT_B);
>>> >-              intel_ddi_init(dev_priv, PORT_C);
>>> >               icl_dsi_init(dev_priv);
>>> >       } else if (IS_ELKHARTLAKE(dev_priv)) {
>>> >               intel_ddi_init(dev_priv, PORT_A);
>>> >--
>>> >2.21.0
>>> >
>>> >_______________________________________________
>>> >Intel-gfx mailing list
>>> >Intel-gfx@lists.freedesktop.org
>>> >https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>
>>
>>
>>--
>>Lucas De Marchi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915/tgl: disable DDIC

[Shankar, Uma]

>>>> >-----Original Message-----
>>>> >From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On
>>>> >Behalf Of Lucas De Marchi
>>>> >Sent: Thursday, August 15, 2019 5:25 AM
>>>> >To: intel-gfx@lists.freedesktop.org
>>>> >Subject: [Intel-gfx] [PATCH] drm/i915/tgl: disable DDIC
>>>> >
>>>> >The current SKUs added for Tiger Lake don't have DDIC hooked up,
>>>> >even though it is supported by the SoC. The current state for these
>>>> >SKUs is problematic since while enabling the combo phy,
>>>> >PORT_COMP_DW* return 0xFFFFFFFF, which is invalid per register definition.
>>>> >
>>>> >During initialization we check what phys are not yet enabled by
>>>> >reading PHY_MISC_C and try to enable it by toggling the "DE to IO
>>>> >Comp Pwr
>>>Down"
>>>> >bit.  But after that any read to the PORT_COMP_DW* returns invalid
>>>> >results. This removes the following warning
>>>> >
>>>> >[56997.634353] Missing case (val == 4294967295) [56997.639241]
>>>> >WARNING: CPU: 5
>>>> >PID: 768 at drivers/gpu/drm/i915/display/intel_combo_phy.c:54
>>>> >cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Modules linked
>in:
>>>> >i915(+) prime_numbers x86_pkg_temp_thermal coretemp kvm_intel kvm
>>>> >irqbypass crct10dif_pclmul crc32_pclmul ghash_clmulni_intel e1000e
>>>> >[last
>>>unloaded:
>>>> >prime_numbers]
>>>> >[56997.639808] CPU: 5 PID: 768 Comm: insmod Tainted: G     U  W         5.2.0-
>>>> >demarchi+ #65
>>>> >[56997.639808] Hardware name: Intel Corporation Tiger Lake Client
>>>> >Platform/TigerLake U DDR4 SODIMM RVP, BIOS
>>>> >TGLSFWI1.R00.2252.A03.1906270154 06/27/2019 [56997.639808] RIP:
>>>> >0010:cnl_get_procmon_ref_values+0xc9/0xf0 [i915] [56997.639808] Code:
>>>> >2c a0 85
>>>> >c9 74 e0 81 f9 00 00 00 01 75 09 48 c7 c0 0c a4 2c a0 eb cf 48 c7
>>>> >c6 3c 3a 31 a0 48 c7
>>>> >c7 40 3a 31 a0 e8 6b 4d ea e0 <0f> 0b 48 c7 c0 00 a4 2c a0 eb b1 48
>>>> >c7 c0 24 a4 2 c a0 eb a8 e8 be [56997.639808] RSP:
>>>> >0018:ffffc9000068f8a8 EFLAGS: 00010286 [56997.639808] RAX:
>>>0000000000000000 RBX: ffff88848fa90000 RCX:
>>>> >0000000000000000 [56997.639808] RDX: ffff8884a08b5ef8 RSI:
>>>> >ffff8884a08a6658
>>>> >RDI: 00000000ffffffff [56997.639808] RBP: 0000000000000002 R08:
>>>> >0000000000000000 R09: 0000000000000000 [56997.639808] R10:
>>>> >0000000000000000 R11: 0000000000000000 R12: ffff88848fa90000
>>>> >[56997.639808]
>>>> >R13: 0000000000000000 R14: 0000000000000002 R15: 0006c00000162000
>>>> >[56997.639808] FS:  00007f61ca3d12c0(0000)
>>>> >GS:ffff8884a0880000(0000)
>>>> >knlGS:0000000000000000 [56997.639808] CS:  0010 DS: 0000 ES: 0000 CR0:
>>>> >0000000080050033 [56997.639808] CR2: 00007f71be6a92c0 CR3:
>>>> >0000000494750006 CR4: 0000000000760ee0 [56997.639808] PKRU:
>>>> >55555554 [56997.639808] Call Trace:
>>>> >[56997.639808]  cnl_verify_procmon_ref_values+0x36/0xf0 [i915]
>>>> >[56997.639808]
>>>?
>>>> >rcu_read_lock_sched_held+0x6f/0x80
>>>> >[56997.639808]  ? gen11_fwtable_read32+0x257/0x290 [i915]
>>>> >[56997.639808]
>>>> >icl_combo_phy_verify_state.part.0+0x22/0xa0 [i915] [56997.639808]
>>>> >intel_combo_phy_init+0x17e/0x3e0 [i915] [56997.639808]  ?
>>>> >icl_display_core_init+0x2c/0x1a0 [i915] [56997.639808]  ?
>>>> >_raw_spin_unlock_irqrestore+0x4c/0x60
>>>> >[56997.639808]  icl_display_core_init+0x34/0x1a0 [i915]
>>>> >[56997.639808]
>>>> >intel_power_domains_init_hw+0x200/0x570 [i915] [56997.639808]
>>>> >i915_driver_probe+0x103b/0x17e0 [i915] [56997.639808]  ?
>>>> >printk+0x53/0x6a [56997.639808]  i915_pci_probe+0x3b/0x190 [i915]
>>>> >
>>>> >We may or may not need to change the implementation to account for
>>>> >DDIC being available on other SKUs. For now I think the best thing
>>>> >to do is to just
>>>disable the port.
>>>>
>>>> This information ideally should be coming from VBT and based on that
>>>> driver can take a call whether to enable the port or not. So is this
>>>> an interim solution and later would be dropped, since there will/may
>>>> be SKU's with
>>>PORT C enabled.
>>>>
>>>> I feel revocation of this port in VBT should be the right approach,
>>>> instead of an
>>>interim solution.
>>>
>>>Ideally yes, but it's better something that works than the ideal
>>>solution that doesn't. I wanted to go the VBT route, but it wouldn't
>>>work with any machine that I have available.
>>>Hence this patch.
>>>
>>>When/if there's one sku with ddic, hopefully VBT will already be fixed
>>>or we may go the route of differentiating by PCI id.
>>
>>I feel this is a mistake in VBT which exposes an non-existent port and
>>should be fixed. I don't think adding PCIID based checks for a platform
>>is a good idea. We should call init for all ports spec claims to be
>>supporting and based on VBT configuration just return if port is not
>>available for use on that particular design. Now with current approach,
>>if on a new SKU this is fixed  (PORT C gets working) and
>
>I agree with all of that. But it would not fix the problem that we have *today*.
>There's no forseable fix for VBT.
>
>>VBT exposes it also correctly, we will not even attempt to initialize
>>it until we fix this patch and re-send a somewhat similar to a revert version of this
>patch.
>
>the "fix"  would be "make DDIC work and *start* exporting it". More like an additional
>feature than a fix in itself. It was my mistake to send the initial patch with DDIC there
>when it was actually not working. See that we still don't expose the TC ports and
>initially we didn't expose the DSI port neither.
>
>In other words, expose the things that are *currently* working. DDIC is not and fixing
>it by hooking up VBT if we should initialize it (my initial attempt) would not fix things.
>
>>
>>So my take would be that we should push VBT guys to fix the problem
>>instead of adding a temporary WA in our
>
>this has already been reported to people responsible for VBT. There's currently not
>ETA for if/when this will be done. When/if it's done we can conditionally add DDIC.

Ok fair point, if VBT fix is a long shot we can remove existence of PORT C and introduce it back
later when/if it becomes alive again. 

With the above justifications, we can go ahead with this patch, but still keep pushing VBT guys to update
their stuff.

Reviewed-by: Uma Shankar <uma.shankar@intel.com>

>Lucas De Marchi
>
>>driver. I will leave to maintainers to suggest what is the optimum approach.
>>
>>Regards,
>>Uma Shankar
>>
>>>Lucas De Marchi
>>>
>>>>
>>>> Regards,
>>>> Uma Shankar
>>>>
>>>> >Cc: José Roberto de Souza <jose.souza@intel.com>
>>>> >Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>>>> >---
>>>> > drivers/gpu/drm/i915/display/intel_display.c | 3 +--
>>>> > 1 file changed, 1 insertion(+), 2 deletions(-)
>>>> >
>>>> >diff --git a/drivers/gpu/drm/i915/display/intel_display.c
>>>> >b/drivers/gpu/drm/i915/display/intel_display.c
>>>> >index 5b733e38eae3..6c6a5a5f41bb 100644
>>>> >--- a/drivers/gpu/drm/i915/display/intel_display.c
>>>> >+++ b/drivers/gpu/drm/i915/display/intel_display.c
>>>> >@@ -6683,7 +6683,7 @@ bool intel_phy_is_combo(struct
>>>> >drm_i915_private *dev_priv, enum phy phy)
>>>> >       if (phy == PHY_NONE)
>>>> >               return false;
>>>> >
>>>> >-      if (IS_ELKHARTLAKE(dev_priv) || INTEL_GEN(dev_priv) >= 12)
>>>> >+      if (IS_ELKHARTLAKE(dev_priv))
>>>> >               return phy <= PHY_C;
>>>> >
>>>> >       if (INTEL_GEN(dev_priv) >= 11) @@ -15317,7 +15317,6 @@
>>>> >static void intel_setup_outputs(struct drm_i915_private
>>>> >*dev_priv)
>>>> >               /* TODO: initialize TC ports as well */
>>>> >               intel_ddi_init(dev_priv, PORT_A);
>>>> >               intel_ddi_init(dev_priv, PORT_B);
>>>> >-              intel_ddi_init(dev_priv, PORT_C);
>>>> >               icl_dsi_init(dev_priv);
>>>> >       } else if (IS_ELKHARTLAKE(dev_priv)) {
>>>> >               intel_ddi_init(dev_priv, PORT_A);
>>>> >--
>>>> >2.21.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx