diff mbox series

drm/xe/display: fix type of intel_uncore_read*() functions

Message ID 20240314065221.1181158-1-luciano.coelho@intel.com (mailing list archive)
State New, archived
Headers show
Series drm/xe/display: fix type of intel_uncore_read*() functions | expand

Commit Message

Luca Coelho March 14, 2024, 6:52 a.m. UTC 
Some of the backported intel_uncore_read*() functions used the wrong
types.  Change the function declarations accordingly.

Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
---
 drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Luca Coelho March 19, 2024, 10:46 a.m. UTC | #1 
Hi,

FTR, this failure is not related to my patch.  It seems to be happening
sporadically on this machine.

--
Cheers,
Luca.

On Thu, 2024-03-14 at 23:16 +0000, Patchwork wrote:
> Patch Details
> Series:drm/xe/display: fix type of intel_uncore_read*() functions
> URL:https://patchwork.freedesktop.org/series/131122/
> State:failure
> Details:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131122v1/index.html
> CI Bug Log - changes from CI_DRM_14429_full -> Patchwork_131122v1_fullSummaryFAILURE
> Serious unknown changes coming with Patchwork_131122v1_full
> absolutely need to be
> verified manually.
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_131122v1_full, please notify your bug team
> (I915-ci-infra@lists.freedesktop.org) to allow them
> to document this new failure mode, which will reduce false positives
> in CI.
> Participating hosts (9 -> 8)Missing (1): shard-snb-0 
> Possible new issuesHere are the unknown changes that may have been introduced in
> Patchwork_131122v1_full:
> IGT changesPossible regressions * igt@kms_flip@2x-flip-vs-blocking-wf-vblank@ab-vga1-hdmi-a1:shard-
>    snb: PASS -> ABORT
> Known issuesHere are the changes found in Patchwork_131122v1_full that come from
> known issues:
> IGT changesIssues hit * igt@api_intel_bb@blit-reloc-purge-cache:shard-dg2: NOTRUN -> SKIP
>    (i915#8411)
>  * igt@debugfs_test@basic-hwmon:shard-mtlp: NOTRUN -> SKIP (i915#9318)
>  * igt@drm_fdinfo@busy-hang@rcs0:shard-mtlp: NOTRUN -> SKIP (i915#8414)
>    +5 other tests skip
>  * igt@drm_fdinfo@most-busy-idle-check-all@vecs1:shard-dg2: NOTRUN ->
>    SKIP (i915#8414) +19 other tests skip
>  * igt@drm_fdinfo@virtual-idle:shard-rkl: PASS -> FAIL (i915#7742) +2
>    other tests fail
>  * igt@gem_basic@multigpu-create-close:shard-tglu: NOTRUN -> SKIP
>    (i915#7697) +1 other test skip
>  * igt@gem_busy@semaphore:shard-mtlp: NOTRUN -> SKIP (i915#3936)
>  * igt@gem_close_race@multigpu-basic-threads:shard-dg2: NOTRUN -> SKIP
>    (i915#7697)
>  * igt@gem_create@create-ext-set-pat:shard-dg1: NOTRUN -> SKIP
>    (i915#8562)
>  * igt@gem_ctx_exec@basic-nohangcheck:shard-tglu: PASS -> FAIL
>    (i915#6268)
>  * igt@gem_ctx_persistence@file:shard-snb: NOTRUN -> SKIP (i915#1099)
>    +1 other test skip
>  * igt@gem_ctx_persistence@heartbeat-hang:shard-dg2: NOTRUN -> SKIP
>    (i915#8555)
>  * igt@gem_ctx_sseu@mmap-args:shard-dg1: NOTRUN -> SKIP (i915#280)
>  * igt@gem_eio@unwedge-stress:shard-snb: NOTRUN -> FAIL (i915#8898)
>  * igt@gem_exec_balancer@bonded-dual:shard-mtlp: NOTRUN -> SKIP
>    (i915#4771) +1 other test skip
>  * igt@gem_exec_capture@many-4k-zero:shard-dg2: NOTRUN -> FAIL
>    (i915#9606)
>  * igt@gem_exec_fair@basic-none-share:shard-dg2: NOTRUN -> SKIP
>    (i915#3539 / i915#4852) +2 other tests skip
>  * igt@gem_exec_fair@basic-none-solo@rcs0:shard-rkl: PASS -> FAIL
>    (i915#2842)
>  * igt@gem_exec_fair@basic-pace:shard-mtlp: NOTRUN -> SKIP (i915#4473 /
>    i915#4771)
>  * igt@gem_exec_fair@basic-pace-solo@rcs0:shard-glk: NOTRUN -> FAIL
>    (i915#2842)
>  * igt@gem_exec_fence@submit3:shard-dg2: NOTRUN -> SKIP (i915#4812) +1
>    other test skip
>  * igt@gem_exec_flush@basic-uc-ro-default:shard-dg1: NOTRUN -> SKIP
>    (i915#3539 / i915#4852) +1 other test skip
>  * igt@gem_exec_gttfill@multigpu-basic:shard-mtlp: NOTRUN -> SKIP
>    (i915#7697)shard-dg1: NOTRUN -> SKIP (i915#7697)
>  * igt@gem_exec_reloc@basic-gtt-cpu-active:shard-dg2: NOTRUN -> SKIP
>    (i915#3281) +4 other tests skip
>  * igt@gem_exec_reloc@basic-gtt-noreloc:shard-mtlp: NOTRUN -> SKIP
>    (i915#3281) +4 other tests skip
>  * igt@gem_exec_reloc@basic-scanout:shard-dg1: NOTRUN -> SKIP
>    (i915#3281) +3 other tests skip
>  * igt@gem_exec_schedule@preempt-queue:shard-dg2: NOTRUN -> SKIP
>    (i915#4537 / i915#4812)
>  * igt@gem_exec_schedule@preempt-queue-contexts-chain:shard-mtlp:
>    NOTRUN -> SKIP (i915#4537 / i915#4812) +1 other test skip
>  * igt@gem_exec_schedule@semaphore-power:shard-dg1: NOTRUN -> SKIP
>    (i915#4812) +1 other test skip
>  * igt@gem_fence_thrash@bo-write-verify-threaded-none:shard-dg1: NOTRUN
>    -> SKIP (i915#4860) +1 other test skip
>  * igt@gem_lmem_swapping@heavy-multi:shard-tglu: NOTRUN -> SKIP
>    (i915#4613) +1 other test skip
>  * igt@gem_lmem_swapping@heavy-random@lmem0:shard-dg1: NOTRUN -> FAIL
>    (i915#10378)
>  * igt@gem_lmem_swapping@heavy-verify-multi-ccs:shard-mtlp: NOTRUN ->
>    SKIP (i915#4613) +3 other tests skip
>  * igt@gem_lmem_swapping@heavy-verify-multi@lmem0:shard-dg2: PASS ->
>    FAIL (i915#10378)
>  * igt@gem_lmem_swapping@heavy-verify-random-ccs@lmem0:shard-dg1:
>    NOTRUN -> SKIP (i915#4565)
>  * igt@gem_lmem_swapping@verify-random-ccs:shard-glk: NOTRUN -> SKIP
>    (i915#4613)
>  * igt@gem_media_vme:shard-dg2: NOTRUN -> SKIP (i915#284)
>  * igt@gem_mmap@basic-small-bo:shard-mtlp: NOTRUN -> SKIP (i915#4083)
>    +3 other tests skip
>  * igt@gem_mmap_gtt@cpuset-medium-copy-xy:shard-dg2: NOTRUN -> SKIP
>    (i915#4077) +6 other tests skip
>  * igt@gem_mmap_gtt@fault-concurrent-x:shard-dg1: NOTRUN -> SKIP
>    (i915#4077) +1 other test skip
>  * igt@gem_mmap_wc@bad-object:shard-dg2: NOTRUN -> SKIP (i915#4083) +3
>    other tests skip
>  * igt@gem_partial_pwrite_pread@reads:shard-dg2: NOTRUN -> SKIP
>    (i915#3282) +3 other tests skip
>  * igt@gem_pwrite@basic-exhaustion:shard-glk: NOTRUN -> WARN
>    (i915#2658)
>  * igt@gem_pxp@create-regular-context-1:shard-mtlp: NOTRUN -> SKIP
>    (i915#4270)
>  * igt@gem_pxp@display-protected-crc:shard-dg1: NOTRUN -> SKIP
>    (i915#4270)
>  * igt@gem_pxp@reject-modify-context-protection-off-3:shard-snb: NOTRUN
>    -> SKIP +136 other tests skip
>  * igt@gem_pxp@verify-pxp-execution-after-suspend-resume:shard-tglu:
>    NOTRUN -> SKIP (i915#4270)
>  * igt@gem_pxp@verify-pxp-key-change-after-suspend-resume:shard-dg2:
>    NOTRUN -> SKIP (i915#4270)
>  * igt@gem_readwrite@read-write:shard-mtlp: NOTRUN -> SKIP (i915#3282)
>    +4 other tests skip
>  * igt@gem_render_copy@y-tiled-ccs-to-y-tiled:shard-dg2: NOTRUN -> SKIP
>    (i915#5190) +4 other tests skip
>  * igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs:shard-mtlp: NOTRUN
>    -> SKIP (i915#8428) +5 other tests skip
>  * igt@gem_set_tiling_vs_gtt:shard-dg1: NOTRUN -> SKIP (i915#4079)
>  * igt@gem_tiled_pread_pwrite:shard-dg2: NOTRUN -> SKIP (i915#4079)
>  * igt@gem_tiled_wb:shard-mtlp: NOTRUN -> SKIP (i915#4077) +5 other
>    tests skip
>  * igt@gem_userptr_blits@access-control:shard-dg2: NOTRUN -> SKIP
>    (i915#3297)
>  * igt@gem_userptr_blits@readonly-unsync:shard-dg1: NOTRUN -> SKIP
>    (i915#3297)
>  * igt@gem_userptr_blits@sd-probe:shard-dg2: NOTRUN -> SKIP (i915#3297
>    / i915#4958)
>  * igt@gem_userptr_blits@unsync-overlap:shard-mtlp: NOTRUN -> SKIP
>    (i915#3297) +3 other tests skip
>  * igt@gen3_render_linear_blits:shard-mtlp: NOTRUN -> SKIP +17 other
>    tests skip
>  * igt@gen9_exec_parse@basic-rejected:shard-dg2: NOTRUN -> SKIP
>    (i915#2856) +1 other test skip
>  * igt@gen9_exec_parse@bb-start-cmd:shard-mtlp: NOTRUN -> SKIP
>    (i915#2856) +2 other tests skip
>  * igt@i915_module_load@load:shard-glk: NOTRUN -> SKIP
>    (i915#6227)shard-dg2: NOTRUN -> SKIP (i915#6227)
>  * igt@i915_module_load@reload-with-fault-injection:shard-dg2: NOTRUN -
>    > ABORT (i915#9820)
>  * igt@i915_pm_freq_api@freq-reset:shard-tglu: NOTRUN -> SKIP
>    (i915#8399)
>  * igt@i915_pm_rps@reset:shard-mtlp: NOTRUN -> FAIL (i915#8346)
>  * igt@i915_pm_rps@thresholds-idle@gt0:shard-dg1: NOTRUN -> SKIP
>    (i915#8925)
>  * igt@i915_pm_sseu@full-enable:shard-mtlp: NOTRUN -> SKIP (i915#8437)
>  * igt@i915_selftest@live@workarounds:shard-dg2: PASS -> DMESG-FAIL
>    (i915#9500)
>  * igt@i915_suspend@basic-s3-without-i915:shard-tglu: NOTRUN ->
>    INCOMPLETE (i915#7443)
>  * igt@intel_hwmon@hwmon-write:shard-mtlp: NOTRUN -> SKIP (i915#7707)
>  * igt@kms_addfb_basic@bo-too-small-due-to-tiling:shard-mtlp: NOTRUN ->
>    SKIP (i915#4212) +2 other tests skip
>  * igt@kms_addfb_basic@clobberred-modifier:shard-dg1: NOTRUN -> SKIP
>    (i915#4212) +1 other test skip
>  * igt@kms_addfb_basic@framebuffer-vs-set-tiling:shard-dg2: NOTRUN ->
>    SKIP (i915#4212) +1 other test skip
>  * igt@kms_async_flips@async-flip-with-page-flip-events@pipe-b-hdmi-a-
>    2-y-rc-ccs-cc:shard-rkl: NOTRUN -> SKIP (i915#8709) +3 other tests
>    skip
>  * igt@kms_async_flips@async-flip-with-page-flip-events@pipe-d-hdmi-a-
>    3-4-mc-ccs:shard-dg2: NOTRUN -> SKIP (i915#8709) +11 other tests
>    skip
>  * igt@kms_atomic_transition@plane-all-modeset-transition:shard-mtlp:
>    NOTRUN -> SKIP (i915#1769 / i915#3555)
>  * igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-0:shard-dg1:
>    NOTRUN -> SKIP (i915#4538 / i915#5286)
>  * igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-0-async-
>    flip:shard-tglu: NOTRUN -> SKIP (i915#5286)
>  * igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0:shard-mtlp: PASS
>    -> FAIL (i915#5138)
>  * igt@kms_big_fb@x-tiled-8bpp-rotate-270:shard-dg1: NOTRUN -> SKIP
>    (i915#3638)
>  * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-
>    flip:shard-dg2: NOTRUN -> SKIP (i915#4538 / i915#5190) +6 other
>    tests skip
>  * igt@kms_big_fb@yf-tiled-addfb:shard-mtlp: NOTRUN -> SKIP (i915#6187)
>  * igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-
>    flip:shard-dg1: NOTRUN -> SKIP (i915#4538)
>  * igt@kms_ccs@bad-aux-stride-4-tiled-mtl-mc-ccs@pipe-a-hdmi-a-4:shard-
>    dg1: NOTRUN -> SKIP (i915#6095) +51 other tests skip
>  * igt@kms_ccs@bad-aux-stride-y-tiled-gen12-rc-ccs-cc@pipe-d-hdmi-a-
>    1:shard-dg2: NOTRUN -> SKIP (i915#10307 / i915#10434) +15 other
>    tests skip
>  * igt@kms_ccs@bad-rotation-90-4-tiled-mtl-rc-ccs@pipe-b-hdmi-a-
>    2:shard-rkl: NOTRUN -> SKIP (i915#6095) +49 other tests skip
>  * igt@kms_ccs@bad-rotation-90-y-tiled-gen12-rc-ccs-cc@pipe-b-edp-
>    1:shard-mtlp: NOTRUN -> SKIP (i915#6095) +35 other tests skip
>  * igt@kms_ccs@bad-rotation-90-yf-tiled-ccs@pipe-a-hdmi-a-1:shard-tglu:
>    NOTRUN -> SKIP (i915#6095) +15 other tests skip
>  * igt@kms_ccs@ccs-on-another-bo-yf-tiled-ccs@pipe-a-hdmi-a-3:shard-
>    dg2: NOTRUN -> SKIP (i915#10307) +127 other tests skip
>  * igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3:shard-dg2: NOTRUN ->
>    SKIP (i915#7213) +3 other tests skip
>  * igt@kms_cdclk@plane-scaling@pipe-d-dp-4:shard-dg2: NOTRUN -> SKIP
>    (i915#4087) +3 other tests skip
>  * igt@kms_chamelium_edid@hdmi-edid-read:shard-dg1: NOTRUN -> SKIP
>    (i915#7828) +1 other test skip
>  * igt@kms_chamelium_frames@dp-crc-fast:shard-dg2: NOTRUN -> SKIP
>    (i915#7828) +6 other tests skip
>  * igt@kms_chamelium_hpd@hdmi-hpd-enable-disable-mode:shard-mtlp:
>    NOTRUN -> SKIP (i915#7828) +4 other tests skip
>  * igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe:shard-tglu: NOTRUN ->
>    SKIP (i915#7828) +2 other tests skip
>  * igt@kms_content_protection@dp-mst-type-0:shard-dg2: NOTRUN -> SKIP
>    (i915#3299)
>  * igt@kms_content_protection@dp-mst-type-1:shard-mtlp: NOTRUN -> SKIP
>    (i915#3299)
>  * igt@kms_content_protection@type1:shard-mtlp: NOTRUN -> SKIP
>    (i915#3555 / i915#6944 / i915#9424)
>  * igt@kms_cursor_crc@cursor-offscreen-32x10:shard-dg2: NOTRUN -> SKIP
>    (i915#3555) +2 other tests skip
>  * igt@kms_cursor_crc@cursor-offscreen-32x32:shard-dg1: NOTRUN -> SKIP
>    (i915#3555)
>  * igt@kms_cursor_crc@cursor-onscreen-32x10:shard-mtlp: NOTRUN -> SKIP
>    (i915#3555 / i915#8814)
>  * igt@kms_cursor_crc@cursor-onscreen-512x170:shard-mtlp: NOTRUN ->
>    SKIP (i915#3359)
>  * igt@kms_cursor_crc@cursor-rapid-movement-128x42:shard-mtlp: NOTRUN -
>    > SKIP (i915#8814)
>  * igt@kms_cursor_legacy@2x-cursor-vs-flip-legacy:shard-mtlp: NOTRUN ->
>    SKIP (i915#9809) +3 other tests skip
>  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:shard-
>    mtlp: NOTRUN -> SKIP (i915#4213) +1 other test skip
>  * igt@kms_cursor_legacy@torture-move@pipe-a:shard-dg1: PASS -> DMESG-
>    WARN (i915#10166)
>  * igt@kms_dirtyfb@fbc-dirtyfb-ioctl@a-hdmi-a-4:shard-dg1: NOTRUN ->
>    SKIP (i915#9723)
>  * igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1:shard-rkl:
>    NOTRUN -> SKIP (i915#3804)
>  * igt@kms_draw_crc@draw-method-mmap-gtt:shard-dg2: NOTRUN -> SKIP
>    (i915#8812)
>  * igt@kms_dsc@dsc-fractional-bpp-with-bpc:shard-dg2: NOTRUN -> SKIP
>    (i915#3840)
>  * igt@kms_dsc@dsc-with-output-formats:shard-mtlp: NOTRUN -> SKIP
>    (i915#3555 / i915#3840)
>  * igt@kms_feature_discovery@psr1:shard-dg2: NOTRUN -> SKIP (i915#658)
>  * igt@kms_feature_discovery@psr2:shard-dg1: NOTRUN -> SKIP (i915#658)
>  * igt@kms_flip@2x-flip-vs-expired-vblank:shard-tglu: NOTRUN -> SKIP
>    (i915#3637)
>  * igt@kms_flip@2x-modeset-vs-vblank-race:shard-dg1: NOTRUN -> SKIP
>    (i915#9934) +3 other tests skip
>  * igt@kms_flip@2x-modeset-vs-vblank-race-interruptible:shard-mtlp:
>    NOTRUN -> SKIP (i915#3637) +6 other tests skip
>  * igt@kms_flip@flip-vs-absolute-wf_vblank@b-vga1:shard-snb: PASS ->
>    FAIL (i915#2122) +1 other test fail
>  * igt@kms_flip@flip-vs-fences:shard-dg2: NOTRUN -> SKIP (i915#8381)
>  * igt@kms_flip_scaled_crc@flip-32bpp-4tile-to-32bpp-4tiledg2rcccs-
>    downscaling@pipe-a-default-mode:shard-mtlp: NOTRUN -> SKIP
>    (i915#2672) +3 other tests skip
>  * igt@kms_flip_scaled_crc@flip-32bpp-ytileccs-to-64bpp-ytile-
>    downscaling@pipe-a-default-mode:shard-mtlp: NOTRUN -> SKIP
>    (i915#2672 / i915#3555)
>  * igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-
>    downscaling@pipe-a-valid-mode:shard-dg2: NOTRUN -> SKIP (i915#2672)
>    +2 other tests skip
>  * igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-
>    downscaling@pipe-a-valid-mode:shard-dg1: NOTRUN -> SKIP (i915#2587 /
>    i915#2672)
>  * igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-
>    upscaling@pipe-a-valid-mode:shard-tglu: NOTRUN -> SKIP (i915#2587 /
>    i915#2672)
>  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-
>    blt:shard-mtlp: NOTRUN -> SKIP (i915#1825) +27 other tests skip
>  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-plflip-blt:shard-
>    dg2: NOTRUN -> SKIP (i915#5354) +26 other tests skip
>  * igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt:shard-dg2:
>    NOTRUN -> SKIP (i915#8708) +6 other tests skip
>  * igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-
>    pwrite:shard-dg2: NOTRUN -> SKIP (i915#3458) +8 other tests skip
>  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-mmap-
>    gtt:shard-glk: NOTRUN -> SKIP +176 other tests skip
>  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-shrfb-draw-mmap-
>    gtt:shard-dg1: NOTRUN -> SKIP (i915#8708) +4 other tests skip
>  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-
>    fullscreen:shard-tglu: NOTRUN -> SKIP +14 other tests skip
>  * igt@kms_frontbuffer_tracking@pipe-fbc-rte:shard-dg2: NOTRUN -> SKIP
>    (i915#9766)
>  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-
>    cpu:shard-dg2: NOTRUN -> SKIP (i915#10433 / i915#3458) +2 other
>    tests skip
>  * igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-indfb-draw-mmap-
>    gtt:shard-mtlp: NOTRUN -> SKIP (i915#8708) +5 other tests skip
>  * igt@kms_frontbuffer_tracking@psr-suspend:shard-dg1: NOTRUN -> SKIP
>    (i915#3458) +5 other tests skip
>  * igt@kms_hdr@static-swap:shard-mtlp: NOTRUN -> SKIP (i915#3555 /
>    i915#8228) +1 other test skip
>  * igt@kms_plane_multiple@tiling-yf:shard-dg2: NOTRUN -> SKIP
>    (i915#3555 / i915#8806)
>  * igt@kms_plane_scaling@plane-downscale-factor-0-25-with-pixel-
>    format@pipe-a-dp-4:shard-dg2: NOTRUN -> SKIP (i915#9423) +11 other
>    tests skip
>  * igt@kms_plane_scaling@plane-scaler-unity-scaling-with-rotation@pipe-
>    b-hdmi-a-1:shard-rkl: NOTRUN -> SKIP (i915#9423) +3 other tests skip
>  * igt@kms_plane_scaling@plane-scaler-unity-scaling-with-rotation@pipe-
>    d-hdmi-a-4:shard-dg1: NOTRUN -> SKIP (i915#9423) +11 other tests
>    skip
>  * igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-
>    rotation@pipe-c-hdmi-a-4:shard-dg1: NOTRUN -> SKIP (i915#5176 /
>    i915#9423) +3 other tests skip
>  * igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-
>    25@pipe-b-hdmi-a-1:shard-dg2: NOTRUN -> SKIP (i915#5235 / i915#9423)
>    +11 other tests skip
>  * igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-
>    25@pipe-b-hdmi-a-3:shard-dg1: NOTRUN -> SKIP (i915#5235) +3 other
>    tests skip
>  * igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-
>    25@pipe-c-edp-1:shard-mtlp: NOTRUN -> SKIP (i915#5235) +8 other
>    tests skip
>  * igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-
>    25@pipe-b-hdmi-a-2:shard-rkl: NOTRUN -> SKIP (i915#5235) +3 other
>    tests skip
>  * igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-
>    5@pipe-d-edp-1:shard-mtlp: NOTRUN -> SKIP (i915#3555 / i915#5235) +2
>    other tests skip
>  * igt@kms_pm_dc@dc6-psr:shard-mtlp: NOTRUN -> SKIP (i915#10139)
>  * igt@kms_pm_rpm@dpms-mode-unset-non-lpsp:shard-tglu: NOTRUN -> SKIP
>    (i915#9519)
>  * igt@kms_pm_rpm@dpms-non-lpsp:shard-mtlp: NOTRUN -> SKIP (i915#9519)
>    +1 other test skip
>  * igt@kms_pm_rpm@modeset-lpsp-stress-no-wait:shard-dg2: PASS -> SKIP
>    (i915#9519)
>  * igt@kms_pm_rpm@modeset-non-lpsp:shard-dg2: NOTRUN -> SKIP
>    (i915#9519)
>  * igt@kms_pm_rpm@modeset-non-lpsp-stress:shard-rkl: PASS -> SKIP
>    (i915#9519) +1 other test skip
>  * igt@kms_prime@d3hot:shard-mtlp: NOTRUN -> SKIP (i915#6524)
>  * igt@kms_psr2_sf@fbc-overlay-plane-move-continuous-exceed-fully-
>    sf:shard-dg2: NOTRUN -> SKIP +13 other tests skip
>  * igt@kms_psr2_sf@fbc-primary-plane-update-sf-dmg-area@psr2-pipe-a-
>    edp-1:shard-mtlp: NOTRUN -> SKIP (i915#9808) +1 other test skip
>  * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area:shard-dg1: NOTRUN
>    -> SKIP +16 other tests skip
>  * igt@kms_psr2_su@page_flip-nv12:shard-dg2: NOTRUN -> SKIP (i915#9683)
>  * igt@kms_psr@fbc-pr-sprite-blt:shard-dg2: NOTRUN -> SKIP (i915#9732)
>    +14 other tests skip
>  * igt@kms_psr@fbc-psr2-cursor-plane-move:shard-tglu: NOTRUN -> SKIP
>    (i915#9732) +1 other test skip
>  * igt@kms_psr@pr-no-drrs:shard-mtlp: NOTRUN -> SKIP (i915#9688) +9
>    other tests skip
>  * igt@kms_psr@psr-primary-mmap-gtt@edp-1:shard-mtlp: NOTRUN -> SKIP
>    (i915#4077 / i915#9688)
>  * igt@kms_psr@psr-sprite-mmap-cpu:shard-dg1: NOTRUN -> SKIP
>    (i915#9732) +6 other tests skip
>  * igt@kms_rotation_crc@exhaust-fences:shard-dg2: NOTRUN -> SKIP
>    (i915#4235)
>  * igt@kms_rotation_crc@primary-4-tiled-reflect-x-180:shard-tglu:
>    NOTRUN -> SKIP (i915#5289)
>  * igt@kms_rotation_crc@primary-rotation-90:shard-mtlp: NOTRUN -> SKIP
>    (i915#4235) +2 other tests skip
>  * igt@kms_universal_plane@cursor-fb-leak@pipe-a-hdmi-a-1:shard-rkl:
>    PASS -> FAIL (i915#9196)
>  * igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-4:shard-dg1:
>    PASS -> FAIL (i915#9196)
>  * igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1:shard-tglu:
>    PASS -> FAIL (i915#9196) +1 other test fail
>  * igt@kms_vrr@flip-basic-fastset:shard-mtlp: NOTRUN -> SKIP (i915#8808
>    / i915#9906) +1 other test skip
>  * igt@kms_vrr@flip-suspend:shard-mtlp: NOTRUN -> SKIP (i915#3555 /
>    i915#8808)
>  * igt@kms_writeback@writeback-invalid-parameters:shard-dg1: NOTRUN ->
>    SKIP (i915#2437)
>  * igt@perf@global-sseu-config:shard-dg2: NOTRUN -> SKIP (i915#7387)
>  * igt@perf_pmu@cpu-hotplug:shard-mtlp: NOTRUN -> SKIP (i915#8850)
>  * igt@perf_pmu@event-wait@rcs0:shard-mtlp: NOTRUN -> SKIP (i915#3555 /
>    i915#8807)
>  * igt@perf_pmu@module-unload:shard-dg2: NOTRUN -> FAIL (i915#5793)
>  * igt@prime_vgem@basic-fence-flip:shard-dg2: NOTRUN -> SKIP
>    (i915#3708)
>  * igt@prime_vgem@basic-read:shard-dg2: NOTRUN -> SKIP (i915#3291 /
>    i915#3708)
>  * igt@prime_vgem@coherency-gtt:shard-dg2: NOTRUN -> SKIP (i915#3708 /
>    i915#4077)
>  * igt@prime_vgem@fence-flip-hang:shard-dg1: NOTRUN -> SKIP (i915#3708)
>    +1 other test skipshard-mtlp: NOTRUN -> SKIP (i915#3708)
>  * igt@sriov_basic@enable-vfs-bind-unbind-each:shard-dg1: NOTRUN ->
>    SKIP (i915#9917)
>  * igt@syncobj_timeline@invalid-wait-zero-handles:shard-mtlp: NOTRUN ->
>    FAIL (i915#9781)
>  * igt@syncobj_wait@invalid-wait-zero-handles:shard-snb: NOTRUN -> FAIL
>    (i915#9779)shard-glk: NOTRUN -> FAIL (i915#9779)
>  * igt@tools_test@sysfs_l3_parity:shard-mtlp: NOTRUN -> SKIP
>    (i915#4818)
>  * igt@v3d/v3d_mmap@mmap-bo:shard-mtlp: NOTRUN -> SKIP (i915#2575) +7
>    other tests skip
>  * igt@v3d/v3d_perfmon@create-single-perfmon:shard-dg2: NOTRUN -> SKIP
>    (i915#2575) +7 other tests skip
>  * igt@v3d/v3d_submit_cl@job-perfmon:shard-dg1: NOTRUN -> SKIP
>    (i915#2575) +1 other test skip
>  * igt@v3d/v3d_submit_csd@job-perfmon:shard-tglu: NOTRUN -> SKIP
>    (i915#2575) +3 other tests skip
>  * igt@vc4/vc4_purgeable_bo@access-purged-bo-mem:shard-mtlp: NOTRUN ->
>    SKIP (i915#7711) +7 other tests skip
>  * igt@vc4/vc4_purgeable_bo@mark-purgeable:shard-dg1: NOTRUN -> SKIP
>    (i915#7711) +3 other tests skip
>  * igt@vc4/vc4_tiling@set-get:shard-dg2: NOTRUN -> SKIP (i915#7711) +4
>    other tests skip
> Possible fixes * igt@gem_ctx_freq@sysfs@gt0:shard-dg2: FAIL (i915#9561) -> PASS
>  * igt@gem_exec_fair@basic-pace-solo@rcs0:shard-tglu: FAIL (i915#2842)
>    -> PASS
>  * igt@gem_exec_fair@basic-pace@rcs0:shard-rkl: FAIL (i915#2842) ->
>    PASS +2 other tests pass
>  * igt@gem_lmem_swapping@heavy-verify-multi-ccs@lmem0:shard-dg2: FAIL
>    (i915#10378) -> PASS
>  * igt@gem_lmem_swapping@smem-oom@lmem0:shard-dg1: TIMEOUT (i915#5493)
>    -> PASS
>  * igt@i915_module_load@reload-with-fault-injection:shard-snb:
>    INCOMPLETE (i915#9849) -> PASS +1 other test passshard-dg1:
>    INCOMPLETE (i915#9849) -> PASSshard-tglu: INCOMPLETE (i915#9820) ->
>    PASSshard-mtlp: ABORT (i915#10131 / i915#9697) -> PASS
>  * igt@i915_pm_rc6_residency@rc6-idle@gt0-vcs0:shard-dg1: FAIL
>    (i915#3591) -> PASS
>  * igt@kms_async_flips@alternate-sync-async-flip@pipe-c-hdmi-a-1:shard-
>    glk: FAIL (i915#2521) -> PASS
>  * igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-180-hflip:shard-
>    mtlp: FAIL (i915#5138) -> PASS +1 other test pass
>  * igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-
>    flip:shard-tglu: FAIL (i915#3743) -> PASS
>  * igt@kms_cursor_legacy@torture-bo@pipe-a:shard-tglu: DMESG-WARN
>    (i915#10166) -> PASS
>  * igt@kms_pm_rpm@dpms-lpsp:shard-rkl: SKIP (i915#9519) -> PASS +1
>    other test pass
>  * igt@kms_pm_rpm@i2c:shard-dg2: INCOMPLETE -> PASS
> Warnings * igt@kms_multipipe_modeset@basic-max-pipe-crc-check:shard-rkl: SKIP
>    (i915#4070 / i915#4816) -> SKIP (i915#4816)
>  * igt@kms_psr@psr-cursor-mmap-cpu:shard-dg2: SKIP (i915#9732) -> SKIP
>    (i915#9673 / i915#9732) +8 other tests skip
>  * igt@perf@non-zero-reason@0-rcs0:shard-dg2: FAIL (i915#7484) -> FAIL
>    (i915#9100)
> Build changes * Linux: CI_DRM_14429 -> Patchwork_131122v1
> CI-20190529: 20190529
> CI_DRM_14429: 96e47e3f3a5952e104d56352872becdc0048d26e @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_7760: 7760
> Patchwork_131122v1: 96e47e3f3a5952e104d56352872becdc0048d26e @
> git://anongit.freedesktop.org/gfx-ci/linux
> piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @
> git://anongit.freedesktop.org/piglit
Lucas De Marchi March 19, 2024, 2:52 p.m. UTC | #2 
On Thu, Mar 14, 2024 at 08:52:21AM +0200, Luca Coelho wrote:
>Some of the backported intel_uncore_read*() functions used the wrong
>types.  Change the function declarations accordingly.
>
>Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
>Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

if a patch affects xe, please always Cc the xe mailing list so it
triggers CI on those machines

Lucas De Marchi
Jani Nikula March 19, 2024, 3:06 p.m. UTC | #3 
On Tue, 19 Mar 2024, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
> On Thu, Mar 14, 2024 at 08:52:21AM +0200, Luca Coelho wrote:
>>Some of the backported intel_uncore_read*() functions used the wrong
>>types.  Change the function declarations accordingly.
>>
>>Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
>>Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
>
> if a patch affects xe, please always Cc the xe mailing list so it
> triggers CI on those machines

Indeed. I just bounced the patch to intel-xe to trigger CI.

BR,
Jani.

>
> Lucas De Marchi
Luca Coelho March 19, 2024, 3:14 p.m. UTC | #4 
On Tue, 2024-03-19 at 17:06 +0200, Jani Nikula wrote:
> On Tue, 19 Mar 2024, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
> > On Thu, Mar 14, 2024 at 08:52:21AM +0200, Luca Coelho wrote:
> > > Some of the backported intel_uncore_read*() functions used the wrong
> > > types.  Change the function declarations accordingly.
> > > 
> > > Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
> > > Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
> > 
> > if a patch affects xe, please always Cc the xe mailing list so it
> > triggers CI on those machines
> 
> Indeed. I just bounced the patch to intel-xe to trigger CI.

Thanks, guys! I'll do it always from now on.

--
Cheers,
Luca.
Jani Nikula March 20, 2024, 10:11 a.m. UTC | #5 
On Tue, 19 Mar 2024, Luca Coelho <luca@coelho.fi> wrote:
> On Tue, 2024-03-19 at 17:06 +0200, Jani Nikula wrote:
>> On Tue, 19 Mar 2024, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
>> > On Thu, Mar 14, 2024 at 08:52:21AM +0200, Luca Coelho wrote:
>> > > Some of the backported intel_uncore_read*() functions used the wrong
>> > > types.  Change the function declarations accordingly.
>> > > 
>> > > Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> > > Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
>> > 
>> > if a patch affects xe, please always Cc the xe mailing list so it
>> > triggers CI on those machines
>> 
>> Indeed. I just bounced the patch to intel-xe to trigger CI.
>
> Thanks, guys! I'll do it always from now on.

Thanks for the patch and review, pushed to drm-xe-next.

BR,
Jani.

>
> --
> Cheers,
> Luca.
diff mbox series

Patch

diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
index cd26ddc0f69e..ef79793caa72 100644
--- a/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
+++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
@@ -25,15 +25,15 @@  static inline u32 intel_uncore_read(struct intel_uncore *uncore,
 	return xe_mmio_read32(__compat_uncore_to_gt(uncore), reg);
 }
 
-static inline u32 intel_uncore_read8(struct intel_uncore *uncore,
-				     i915_reg_t i915_reg)
+static inline u8 intel_uncore_read8(struct intel_uncore *uncore,
+				    i915_reg_t i915_reg)
 {
 	struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
 
 	return xe_mmio_read8(__compat_uncore_to_gt(uncore), reg);
 }
 
-static inline u32 intel_uncore_read16(struct intel_uncore *uncore,
+static inline u16 intel_uncore_read16(struct intel_uncore *uncore,
 				      i915_reg_t i915_reg)
 {
 	struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));