[PATCH i-g-t] tests/chamelium/kms_chamelium_color: poll connector status instead of HPD uevents
Sharma, Swati2
swati2.sharma at intel.com
Tue Aug 5 11:53:52 UTC 2025
Hi Kunal,
On 28-07-2025 01:45 pm, Kunal Joshi wrote:
> Stop using udev monitors and chamelium_wait_for_hotplug().
> Instead, reprobe until the connector becomes DRM_MODE_CONNECTED.
> Temoves HPD/udev dependency.
Nitpick: s/T/R
Patch LGTM
Reviewed-by: Swati Sharma <swati2.sharma at intel.com>
>
> Signed-off-by: Kunal Joshi <kunal1.joshi at intel.com>
> ---
> tests/chamelium/kms_chamelium_color.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/tests/chamelium/kms_chamelium_color.c b/tests/chamelium/kms_chamelium_color.c
> index cee613233..1d4f16da4 100644
> --- a/tests/chamelium/kms_chamelium_color.c
> +++ b/tests/chamelium/kms_chamelium_color.c
> @@ -748,13 +748,12 @@ igt_main
> * prevents any ports from being unintentionally skipped in test_setup.
> */
> for (i = 0; i < data.port_count; i++) {
> - struct udev_monitor *mon;
> - int timeout = CHAMELIUM_HOTPLUG_TIMEOUT;
> chamelium_unplug(data.chamelium, data.ports[i]);
> - mon = igt_watch_uevents();
> chamelium_plug(data.chamelium, data.ports[i]);
> - igt_assert(chamelium_wait_for_hotplug(mon, &timeout));
> - igt_cleanup_uevents(mon);
> + chamelium_wait_for_conn_status_change(&data.display,
> + data.chamelium,
> + data.ports[i],
> + DRM_MODE_CONNECTED);
> igt_assert_f(chamelium_reprobe_connector(&data.display,
> data.chamelium,
> data.ports[i]) == DRM_MODE_CONNECTED,
More information about the igt-dev
mailing list