Re: [PATCH i-g-t 0/7] tests: lots of s/crtc->pipe/crtc->crtc_index/

Jani Nikula
2026-03-24T17:54:14Z

On Tue, 24 Mar 2026, Alex Hung <alex.hung@amd.com> wrote:
> This series is Reviewed-by: Alex Hung <alex.hung@amd.com>

Thanks a lot, pushed to master.

As a tip, putting the Reviewed-by on a line of its own helps 'b4 shazam'
pick up the trailers from the mailing list while applying.

BR,
Jani.

>
> On 3/20/26 05:24, Jani Nikula wrote:
>> Do a bunch of s/crtc->pipe/crtc->crtc_index/ where the crtc->pipe isn't
>> actually needed or relevant, e.g. basically all of amdgpu tests.
>>
>> Cc: Aurabindo Pillai <aurabindo.pillai@amd.com>
>> Cc: Harry Wentland <harry.wentland@amd.com>
>> Cc: Vitaly Prosyak <vitaly.prosyak@amd.com>
>> Cc: Alex Hung <alex.hung@amd.com>
>>
>> Jani Nikula (7):
>> tests/amdgpu/amd_plane: s/crtc->pipe/crtc->crtc_index/
>> tests/amdgpu/amd_dp_dsc: s/crtc->pipe/crtc->crtc_index/
>> tests/amdgpu/amd_subvp: s/crtc->pipe/crtc->crtc_index/
>> tests/amdgpu/amd_hotplug: s/crtc->pipe/crtc->crtc_index/
>> tests/amdgpu/amd_multidisplay_modeset: s/crtc->pipe/crtc->crtc_index/
>> tests/amdgpu/amd_vrr_range: s/crtc->pipe/crtc->crtc_index/
>> tests/kms_atomic_transition: s/crtc->pipe/crtc->crtc_index/
>>
>> tests/amdgpu/amd_dp_dsc.c | 142 ++++++++++++------------
>> tests/amdgpu/amd_hotplug.c | 52 +++++----
>> tests/amdgpu/amd_multidisplay_modeset.c | 30 ++---
>> tests/amdgpu/amd_plane.c | 27 ++---
>> tests/amdgpu/amd_subvp.c | 16 +--
>> tests/amdgpu/amd_vrr_range.c | 4 +-
>> tests/kms_atomic_transition.c | 18 +--
>> 7 files changed, 144 insertions(+), 145 deletions(-)
>>
>

--
Jani Nikula, Intel