diff mbox

[i-g-t,1/3] tests/chamelium: Introduce fast basic hpd tests, with limited toggles

Message ID 20170823152114.16559-1-paul.kocialkowski@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Paul Kocialkowki Aug. 23, 2017, 3:21 p.m. UTC
This introduces a fast fashion of testing basic hotplug detect, with
only 3 toggles. It drastically reduces the run time of the test, which
is necessary for the fast-feedback test list.

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

Comments

Lyude Paul Aug. 23, 2017, 7:27 p.m. UTC | #1
R-B'd and pushed, thanks!

On Wed, 2017-08-23 at 18:21 +0300, Paul Kocialkowski wrote:
> This introduces a fast fashion of testing basic hotplug detect, with
> only 3 toggles. It drastically reduces the run time of the test,
> which
> is necessary for the fast-feedback test list.
> 
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com>
> ---
>  tests/chamelium.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/tests/chamelium.c b/tests/chamelium.c
> index 881b7fa9..0339c635 100644
> --- a/tests/chamelium.c
> +++ b/tests/chamelium.c
> @@ -48,6 +48,7 @@ typedef struct {
>  
>  #define HPD_TOGGLE_COUNT_VGA 5
>  #define HPD_TOGGLE_COUNT_DP_HDMI 15
> +#define HPD_TOGGLE_COUNT_FAST 3
>  
>  static void
>  get_connectors_link_status_failed(data_t *data, bool
> *link_status_failed)
> @@ -775,6 +776,10 @@ igt_main
>  			test_basic_hotplug(&data, port,
>  					   HPD_TOGGLE_COUNT_DP_HDMI)
> ;
>  
> +		connector_subtest("dp-hpd-fast", DisplayPort)
> +			test_basic_hotplug(&data, port,
> +					   HPD_TOGGLE_COUNT_FAST);
> +
>  		connector_subtest("dp-edid-read", DisplayPort) {
>  			test_edid_read(&data, port, edid_id,
>  				       igt_kms_get_base_edid());
> @@ -832,6 +837,10 @@ igt_main
>  			test_basic_hotplug(&data, port,
>  					   HPD_TOGGLE_COUNT_DP_HDMI)
> ;
>  
> +		connector_subtest("hdmi-hpd-fast", HDMIA)
> +			test_basic_hotplug(&data, port,
> +					   HPD_TOGGLE_COUNT_FAST);
> +
>  		connector_subtest("hdmi-edid-read", HDMIA) {
>  			test_edid_read(&data, port, edid_id,
>  				       igt_kms_get_base_edid());
> @@ -888,6 +897,9 @@ igt_main
>  		connector_subtest("vga-hpd", VGA)
>  			test_basic_hotplug(&data, port,
> HPD_TOGGLE_COUNT_VGA);
>  
> +		connector_subtest("vga-hpd-fast", VGA)
> +			test_basic_hotplug(&data, port,
> HPD_TOGGLE_COUNT_FAST);
> +
>  		connector_subtest("vga-edid-read", VGA) {
>  			test_edid_read(&data, port, edid_id,
>  				       igt_kms_get_base_edid());
diff mbox

Patch

diff --git a/tests/chamelium.c b/tests/chamelium.c
index 881b7fa9..0339c635 100644
--- a/tests/chamelium.c
+++ b/tests/chamelium.c
@@ -48,6 +48,7 @@  typedef struct {
 
 #define HPD_TOGGLE_COUNT_VGA 5
 #define HPD_TOGGLE_COUNT_DP_HDMI 15
+#define HPD_TOGGLE_COUNT_FAST 3
 
 static void
 get_connectors_link_status_failed(data_t *data, bool *link_status_failed)
@@ -775,6 +776,10 @@  igt_main
 			test_basic_hotplug(&data, port,
 					   HPD_TOGGLE_COUNT_DP_HDMI);
 
+		connector_subtest("dp-hpd-fast", DisplayPort)
+			test_basic_hotplug(&data, port,
+					   HPD_TOGGLE_COUNT_FAST);
+
 		connector_subtest("dp-edid-read", DisplayPort) {
 			test_edid_read(&data, port, edid_id,
 				       igt_kms_get_base_edid());
@@ -832,6 +837,10 @@  igt_main
 			test_basic_hotplug(&data, port,
 					   HPD_TOGGLE_COUNT_DP_HDMI);
 
+		connector_subtest("hdmi-hpd-fast", HDMIA)
+			test_basic_hotplug(&data, port,
+					   HPD_TOGGLE_COUNT_FAST);
+
 		connector_subtest("hdmi-edid-read", HDMIA) {
 			test_edid_read(&data, port, edid_id,
 				       igt_kms_get_base_edid());
@@ -888,6 +897,9 @@  igt_main
 		connector_subtest("vga-hpd", VGA)
 			test_basic_hotplug(&data, port, HPD_TOGGLE_COUNT_VGA);
 
+		connector_subtest("vga-hpd-fast", VGA)
+			test_basic_hotplug(&data, port, HPD_TOGGLE_COUNT_FAST);
+
 		connector_subtest("vga-edid-read", VGA) {
 			test_edid_read(&data, port, edid_id,
 				       igt_kms_get_base_edid());