[PATCH IGT] tests/chamelium: Require HPD storm control for basic hotplug test

[PATCH IGT] tests/chamelium: Require HPD storm control for basic hotplug test

[Paul Kocialkowski]

The basic hotplug test (test_basic_hotplug) makes calls to
igt_hpd_storm_set_threshold, which requires HPD storm control.

This adds a check for that control, since it was missing.

Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
---
 tests/chamelium.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/chamelium.c b/tests/chamelium.c
index dc631161..acd17886 100644
--- a/tests/chamelium.c
+++ b/tests/chamelium.c
@@ -153,6 +153,8 @@ test_basic_hotplug(data_t *data, struct chamelium_port *port)
 	struct udev_monitor *mon = igt_watch_hotplug();
 	int i;
 
+	igt_require_hpd_storm_ctl(data->drm_fd);
+
 	reset_state(data, port);
 	igt_hpd_storm_set_threshold(data->drm_fd, 0);
 
-- 
2.13.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH IGT] tests/chamelium: Require HPD storm control for basic hotplug test

[Paul Kocialkowski]

On Mon, 2017-06-12 at 16:25 +0300, Paul Kocialkowski wrote:
> The basic hotplug test (test_basic_hotplug) makes calls to
> igt_hpd_storm_set_threshold, which requires HPD storm control.
> 
> This adds a check for that control, since it was missing.

+Lyude for Chamelium patches

> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
> ---
>  tests/chamelium.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/chamelium.c b/tests/chamelium.c
> index dc631161..acd17886 100644
> --- a/tests/chamelium.c
> +++ b/tests/chamelium.c
> @@ -153,6 +153,8 @@ test_basic_hotplug(data_t *data, struct chamelium_port
> *port)
>  	struct udev_monitor *mon = igt_watch_hotplug();
>  	int i;
>  
> +	igt_require_hpd_storm_ctl(data->drm_fd);
> +
>  	reset_state(data, port);
>  	igt_hpd_storm_set_threshold(data->drm_fd, 0);
>  
-- 
Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH IGT] tests/chamelium: Require HPD storm control for basic hotplug test

[Paul Kocialkowski]

On Tue, 2017-06-13 at 10:37 +0300, Paul Kocialkowski wrote:
> On Mon, 2017-06-12 at 16:25 +0300, Paul Kocialkowski wrote:
> > The basic hotplug test (test_basic_hotplug) makes calls to
> > igt_hpd_storm_set_threshold, which requires HPD storm control.
> > 
> > This adds a check for that control, since it was missing.
> 
> +Lyude for Chamelium patches

... and adding back others. Sorry for the noise.

> > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
> > ---
> >  tests/chamelium.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/tests/chamelium.c b/tests/chamelium.c
> > index dc631161..acd17886 100644
> > --- a/tests/chamelium.c
> > +++ b/tests/chamelium.c
> > @@ -153,6 +153,8 @@ test_basic_hotplug(data_t *data, struct chamelium_port
> > *port)
> >  	struct udev_monitor *mon = igt_watch_hotplug();
> >  	int i;
> >  
> > +	igt_require_hpd_storm_ctl(data->drm_fd);
> > +
> >  	reset_state(data, port);
> >  	igt_hpd_storm_set_threshold(data->drm_fd, 0);
> >  
-- 
Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx