mbox series

[0/2] Cleanup a few functions in C10/C20 handling

Message ID 20231018222831.4132968-1-lucas.demarchi@intel.com (mailing list archive)
Headers show
Series Cleanup a few functions in C10/C20 handling | expand

Message

Lucas De Marchi Oct. 18, 2023, 10:28 p.m. UTC
I started a cleanup on the c10/c20 while adding LNL, but had to stop due
to other priorities, so this is not a complete cleanup.  More details
and ask for feedback at https://patchwork.freedesktop.org/series/125322/

Maybe it's worth getting these in regardless.

Lucas De Marchi (2):
  drm/i915/display: Abstract C10/C20 pll hw readout
  drm/i915/display: Abstract C10/C20 pll calculation

 drivers/gpu/drm/i915/display/intel_cx0_phy.c | 40 ++++++++++++++++----
 drivers/gpu/drm/i915/display/intel_cx0_phy.h | 14 +++----
 drivers/gpu/drm/i915/display/intel_ddi.c     |  9 +----
 drivers/gpu/drm/i915/display/intel_dpll.c    |  7 +---
 4 files changed, 42 insertions(+), 28 deletions(-)

Comments

Lucas De Marchi Oct. 29, 2023, 11:53 p.m. UTC | #1
On Sat, Oct 28, 2023 at 07:55:24PM +0000, Patchwork wrote:
>== Series Details ==
>
>Series: Cleanup a few functions in C10/C20 handling (rev3)
>URL   : https://patchwork.freedesktop.org/series/125323/
>State : failure
>
>== Summary ==
>
>CI Bug Log - changes from CI_DRM_13799_full -> Patchwork_125323v3_full
>====================================================
>
>Summary
>-------
>
>  **FAILURE**
>
>  Serious unknown changes coming with Patchwork_125323v3_full absolutely need to be
>  verified manually.
>
>  If you think the reported changes have nothing to do with the changes
>  introduced in Patchwork_125323v3_full, please notify your bug team (lgci.bug.filing@intel.com) to allow them
>  to document this new failure mode, which will reduce false positives in CI.
>
>
>
>Participating hosts (12 -> 11)
>------------------------------
>
>  Missing    (1): shard-mtlp0
>
>Possible new issues
>-------------------
>
>  Here are the unknown changes that may have been introduced in Patchwork_125323v3_full:
>
>### IGT changes ###
>
>#### Possible regressions ####
>
>  * igt@gem_exec_balancer@fairslice:
>    - shard-rkl:          [PASS][1] -> [SKIP][2]
>   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13799/shard-rkl-4/igt@gem_exec_balancer@fairslice.html
>   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125323v3/shard-rkl-5/igt@gem_exec_balancer@fairslice.html
>
>  * igt@i915_module_load@reload-with-fault-injection:
>    - shard-tglu:         [PASS][3] -> [INCOMPLETE][4]
>   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13799/shard-tglu-8/igt@i915_module_load@reload-with-fault-injection.html
>   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125323v3/shard-tglu-7/igt@i915_module_load@reload-with-fault-injection.html
>
>  * igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted-submitted-signaled:
>    - shard-glk:          NOTRUN -> [FAIL][5]
>   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125323v3/shard-glk5/igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted-submitted-signaled.html
>
>  * igt@syncobj_timeline@invalid-multi-wait-available-unsubmitted-submitted-signaled:
>    - shard-rkl:          NOTRUN -> [FAIL][6]
>   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125323v3/shard-rkl-1/igt@syncobj_timeline@invalid-multi-wait-available-unsubmitted-submitted-signaled.html


not related to the display patches in this series. Patches applied to
drm-intel-next. Thanks for the reviews.

Lucas De Marchi