[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