lib/igt.cocci: Add stanza for for_each_pipe
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 11 Jul 2014 15:41:27 +0000 (17:41 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 11 Jul 2014 15:43:22 +0000 (17:43 +0200)
Damien dodged this ...

Also run the script while at it.

v2: Don't just capture identifiers for pipe, but also expressions.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
lib/igt.cocci
tests/kms_cursor_crc.c
tests/kms_fbc_crc.c
tests/kms_fence_pin_leak.c

index 3246924..d399496 100644 (file)
@@ -74,3 +74,13 @@ expression list[n] Ep;
 @@
 -abort();
 +igt_fail(1);
+
+@@
+iterator name for_each_pipe;
+igt_display_t *display;
+expression pipe;
+@@
+- for (pipe = 0; pipe < igt_display_get_n_pipes(display); pipe++) {
++ for_each_pipe (display, pipe) {
+...
+}
index 82807b7..bbbf053 100644 (file)
@@ -303,7 +303,7 @@ static void run_test(data_t *data, void (*testfunc)(data_t *), int cursor_w, int
 
        for_each_connected_output(display, output) {
                data->output = output;
-               for (p = 0; p < igt_display_get_n_pipes(display); p++) {
+               for_each_pipe(display, p) {
                        data->pipe = p;
 
                        if (!prepare_crtc(data, output, cursor_w, cursor_h))
index a99bf36..b0108b1 100644 (file)
@@ -454,7 +454,7 @@ static void run_test(data_t *data, enum test_mode mode)
        reset_display(data);
 
        for_each_connected_output(display, data->output) {
-               for (data->pipe = 0; data->pipe < igt_display_get_n_pipes(display); data->pipe++) {
+               for_each_pipe(display, data->pipe) {
                        if (!prepare_crtc(data))
                                continue;
 
index ef2e687..6a05d97 100644 (file)
@@ -196,7 +196,7 @@ static void run_test(data_t *data)
        enum pipe p;
 
        for_each_connected_output(display, output) {
-               for (p = 0; p < igt_display_get_n_pipes(display); p++) {
+               for_each_pipe(display, p) {
                        if (run_single_test(data, p, output))
                                return; /* one time ought to be enough */
                }