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

Alex Hung
2026-03-24T18:02:29Z

On 3/24/26 11:54, Jani Nikula wrote:
> 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.

Got it. Thanks for the tip.

>
> 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(-)
>>>
>>
>