tests/modetest: silence compiler warning about uninitialised variables
[platform/upstream/libdrm.git] / tests / modetest / modetest.c
index 237eac1..f0ed56b 100644 (file)
@@ -40,6 +40,7 @@
 #include "config.h"
 
 #include <assert.h>
+#include <ctype.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -634,8 +635,49 @@ error:
        return NULL;
 }
 
+static int get_crtc_index(struct device *dev, uint32_t id)
+{
+       int i;
+
+       for (i = 0; i < dev->resources->res->count_crtcs; ++i) {
+               drmModeCrtc *crtc = dev->resources->crtcs[i].crtc;
+               if (crtc && crtc->crtc_id == id)
+                       return i;
+       }
+
+       return -1;
+}
+
+static drmModeConnector *get_connector_by_id(struct device *dev, uint32_t id)
+{
+       drmModeConnector *connector;
+       int i;
+
+       for (i = 0; i < dev->resources->res->count_connectors; i++) {
+               connector = dev->resources->connectors[i].connector;
+               if (connector && connector->connector_id == id)
+                       return connector;
+       }
+
+       return NULL;
+}
+
+static drmModeEncoder *get_encoder_by_id(struct device *dev, uint32_t id)
+{
+       drmModeEncoder *encoder;
+       int i;
+
+       for (i = 0; i < dev->resources->res->count_encoders; i++) {
+               encoder = dev->resources->encoders[i].encoder;
+               if (encoder && encoder->encoder_id == id)
+                       return encoder;
+       }
+
+       return NULL;
+}
+
 /* -----------------------------------------------------------------------------
- * Connectors and planes
+ * Pipes and planes
  */
 
 /*
@@ -645,8 +687,9 @@ error:
  * Then you need to find the encoder attached to that connector so you
  * can bind it with a free crtc.
  */
-struct connector_arg {
-       uint32_t id;
+struct pipe_arg {
+       uint32_t *con_ids;
+       unsigned int num_cons;
        uint32_t crtc_id;
        char mode_str[64];
        char format_str[5];
@@ -669,69 +712,116 @@ struct plane_arg {
        unsigned int fourcc;
 };
 
-static void connector_find_mode(struct device *dev, struct connector_arg *c)
+static drmModeModeInfo *
+connector_find_mode(struct device *dev, uint32_t con_id, const char *mode_str)
 {
        drmModeConnector *connector;
-       drmModeEncoder *encoder;
-       int i, j;
+       drmModeModeInfo *mode;
+       int i;
 
-       /* First, find the connector & mode */
-       c->mode = NULL;
-       for (i = 0; i < dev->resources->res->count_connectors; i++) {
-               connector = dev->resources->connectors[i].connector;
+       connector = get_connector_by_id(dev, con_id);
+       if (!connector || !connector->count_modes)
+               return NULL;
+
+       for (i = 0; i < connector->count_modes; i++) {
+               mode = &connector->modes[i];
+               if (!strcmp(mode->name, mode_str))
+                       return mode;
+       }
+
+       return NULL;
+}
+
+static struct crtc *pipe_find_crtc(struct device *dev, struct pipe_arg *pipe)
+{
+       uint32_t possible_crtcs = ~0;
+       uint32_t active_crtcs = 0;
+       unsigned int crtc_idx;
+       unsigned int i;
+       int j;
+
+       for (i = 0; i < pipe->num_cons; ++i) {
+               uint32_t crtcs_for_connector = 0;
+               drmModeConnector *connector;
+               drmModeEncoder *encoder;
+               int idx;
+
+               connector = get_connector_by_id(dev, pipe->con_ids[i]);
                if (!connector)
-                       continue;
+                       return NULL;
 
-               if (!connector->count_modes)
-                       continue;
+               for (j = 0; j < connector->count_encoders; ++j) {
+                       encoder = get_encoder_by_id(dev, connector->encoders[j]);
+                       if (!encoder)
+                               continue;
 
-               if (connector->connector_id != c->id)
-                       continue;
+                       crtcs_for_connector |= encoder->possible_crtcs;
 
-               for (j = 0; j < connector->count_modes; j++) {
-                       c->mode = &connector->modes[j];
-                       if (!strcmp(c->mode->name, c->mode_str))
-                               break;
+                       idx = get_crtc_index(dev, encoder->crtc_id);
+                       if (idx >= 0)
+                               active_crtcs |= 1 << idx;
                }
 
-               /* Found it, break out */
-               if (c->mode)
-                       break;
+               possible_crtcs &= crtcs_for_connector;
        }
 
-       if (!c->mode) {
-               fprintf(stderr, "failed to find mode \"%s\"\n", c->mode_str);
-               return;
+       if (!possible_crtcs)
+               return NULL;
+
+       /* Return the first possible and active CRTC if one exists, or the first
+        * possible CRTC otherwise.
+        */
+       if (possible_crtcs & active_crtcs)
+               crtc_idx = ffs(possible_crtcs & active_crtcs);
+       else
+               crtc_idx = ffs(possible_crtcs);
+
+       return &dev->resources->crtcs[crtc_idx - 1];
+}
+
+static int pipe_find_crtc_and_mode(struct device *dev, struct pipe_arg *pipe)
+{
+       drmModeModeInfo *mode = NULL;
+       int i;
+
+       pipe->mode = NULL;
+
+       for (i = 0; i < (int)pipe->num_cons; i++) {
+               mode = connector_find_mode(dev, pipe->con_ids[i],
+                                          pipe->mode_str);
+               if (mode == NULL) {
+                       fprintf(stderr,
+                               "failed to find mode \"%s\" for connector %u\n",
+                               pipe->mode_str, pipe->con_ids[i]);
+                       return -EINVAL;
+               }
        }
 
        /* If the CRTC ID was specified, get the corresponding CRTC. Otherwise
-        * locate a CRTC that can be attached to the connector.
+        * locate a CRTC that can be attached to all the connectors.
         */
-       if (c->crtc_id == (uint32_t)-1) {
-               for (i = 0; i < dev->resources->res->count_encoders; i++) {
-                       encoder = dev->resources->encoders[i].encoder;
-                       if (!encoder)
-                               continue;
+       if (pipe->crtc_id != (uint32_t)-1) {
+               for (i = 0; i < dev->resources->res->count_crtcs; i++) {
+                       struct crtc *crtc = &dev->resources->crtcs[i];
 
-                       if (encoder->encoder_id  == connector->encoder_id) {
-                               c->crtc_id = encoder->crtc_id;
+                       if (pipe->crtc_id == crtc->crtc->crtc_id) {
+                               pipe->crtc = crtc;
                                break;
                        }
                }
+       } else {
+               pipe->crtc = pipe_find_crtc(dev, pipe);
        }
 
-       if (c->crtc_id == (uint32_t)-1)
-               return;
+       if (!pipe->crtc) {
+               fprintf(stderr, "failed to find CRTC for pipe\n");
+               return -EINVAL;
+       }
 
-       for (i = 0; i < dev->resources->res->count_crtcs; i++) {
-               struct crtc *crtc = &dev->resources->crtcs[i];
+       pipe->mode = mode;
+       pipe->crtc->mode = mode;
 
-               if (c->crtc_id == crtc->crtc->crtc_id) {
-                       crtc->mode = c->mode;
-                       c->crtc = crtc;
-                       break;
-               }
-       }
+       return 0;
 }
 
 /* -----------------------------------------------------------------------------
@@ -748,8 +838,8 @@ struct property_arg {
 
 static void set_property(struct device *dev, struct property_arg *p)
 {
-       drmModeObjectProperties *props;
-       drmModePropertyRes **props_info;
+       drmModeObjectProperties *props = NULL;
+       drmModePropertyRes **props_info = NULL;
        const char *obj_type;
        int ret;
        int i;
@@ -815,28 +905,28 @@ static void
 page_flip_handler(int fd, unsigned int frame,
                  unsigned int sec, unsigned int usec, void *data)
 {
-       struct connector_arg *c;
+       struct pipe_arg *pipe;
        unsigned int new_fb_id;
        struct timeval end;
        double t;
 
-       c = data;
-       if (c->current_fb_id == c->fb_id[0])
-               new_fb_id = c->fb_id[1];
+       pipe = data;
+       if (pipe->current_fb_id == pipe->fb_id[0])
+               new_fb_id = pipe->fb_id[1];
        else
-               new_fb_id = c->fb_id[0];
+               new_fb_id = pipe->fb_id[0];
 
-       drmModePageFlip(fd, c->crtc_id, new_fb_id,
-                       DRM_MODE_PAGE_FLIP_EVENT, c);
-       c->current_fb_id = new_fb_id;
-       c->swap_count++;
-       if (c->swap_count == 60) {
+       drmModePageFlip(fd, pipe->crtc->crtc->crtc_id, new_fb_id,
+                       DRM_MODE_PAGE_FLIP_EVENT, pipe);
+       pipe->current_fb_id = new_fb_id;
+       pipe->swap_count++;
+       if (pipe->swap_count == 60) {
                gettimeofday(&end, NULL);
                t = end.tv_sec + end.tv_usec * 1e-6 -
-                       (c->start.tv_sec + c->start.tv_usec * 1e-6);
-               fprintf(stderr, "freq: %.02fHz\n", c->swap_count / t);
-               c->swap_count = 0;
-               c->start = end;
+                       (pipe->start.tv_sec + pipe->start.tv_usec * 1e-6);
+               fprintf(stderr, "freq: %.02fHz\n", pipe->swap_count / t);
+               pipe->swap_count = 0;
+               pipe->start = end;
        }
 }
 
@@ -923,34 +1013,38 @@ static int set_plane(struct device *dev, struct plane_arg *p)
        return 0;
 }
 
-static void set_mode(struct device *dev, struct connector_arg *c, unsigned int count)
+static void set_mode(struct device *dev, struct pipe_arg *pipes, unsigned int count)
 {
        uint32_t handles[4], pitches[4], offsets[4] = {0}; /* we only use [0] */
        unsigned int fb_id;
        struct kms_bo *bo;
        unsigned int i;
+       unsigned int j;
        int ret, x;
 
        dev->mode.width = 0;
        dev->mode.height = 0;
 
        for (i = 0; i < count; i++) {
-               connector_find_mode(dev, &c[i]);
-               if (c[i].mode == NULL)
+               struct pipe_arg *pipe = &pipes[i];
+
+               ret = pipe_find_crtc_and_mode(dev, pipe);
+               if (ret < 0)
                        continue;
-               dev->mode.width += c[i].mode->hdisplay;
-               if (dev->mode.height < c[i].mode->vdisplay)
-                       dev->mode.height = c[i].mode->vdisplay;
+
+               dev->mode.width += pipe->mode->hdisplay;
+               if (dev->mode.height < pipe->mode->vdisplay)
+                       dev->mode.height = pipe->mode->vdisplay;
        }
 
-       bo = create_test_buffer(dev->kms, c->fourcc,
+       bo = create_test_buffer(dev->kms, pipes[0].fourcc,
                                dev->mode.width, dev->mode.height,
                                handles, pitches, offsets, PATTERN_SMPTE);
        if (bo == NULL)
                return;
 
-       ret = drmModeAddFB2(dev->fd, dev->mode.width, dev->mode.height, c->fourcc,
-                           handles, pitches, offsets, &fb_id, 0);
+       ret = drmModeAddFB2(dev->fd, dev->mode.width, dev->mode.height,
+                           pipes[0].fourcc, handles, pitches, offsets, &fb_id, 0);
        if (ret) {
                fprintf(stderr, "failed to add fb (%ux%u): %s\n",
                        dev->mode.width, dev->mode.height, strerror(errno));
@@ -959,19 +1053,25 @@ static void set_mode(struct device *dev, struct connector_arg *c, unsigned int c
 
        x = 0;
        for (i = 0; i < count; i++) {
-               if (c[i].mode == NULL)
+               struct pipe_arg *pipe = &pipes[i];
+
+               if (pipe->mode == NULL)
                        continue;
 
-               printf("setting mode %s@%s on connector %d, crtc %d\n",
-                      c[i].mode_str, c[i].format_str, c[i].id, c[i].crtc_id);
+               printf("setting mode %s@%s on connectors ",
+                      pipe->mode_str, pipe->format_str);
+               for (j = 0; j < pipe->num_cons; ++j)
+                       printf("%u, ", pipe->con_ids[j]);
+               printf("crtc %d\n", pipe->crtc->crtc->crtc_id);
 
-               ret = drmModeSetCrtc(dev->fd, c[i].crtc_id, fb_id, x, 0,
-                                    &c[i].id, 1, c[i].mode);
+               ret = drmModeSetCrtc(dev->fd, pipe->crtc->crtc->crtc_id, fb_id,
+                                    x, 0, pipe->con_ids, pipe->num_cons,
+                                    pipe->mode);
 
                /* XXX: Actually check if this is needed */
                drmModeDirtyFB(dev->fd, fb_id, NULL, 0);
 
-               x += c[i].mode->hdisplay;
+               x += pipe->mode->hdisplay;
 
                if (ret) {
                        fprintf(stderr, "failed to set mode: %s\n", strerror(errno));
@@ -993,7 +1093,7 @@ static void set_planes(struct device *dev, struct plane_arg *p, unsigned int cou
                        return;
 }
 
-static void test_page_flip(struct device *dev, struct connector_arg *c, unsigned int count)
+static void test_page_flip(struct device *dev, struct pipe_arg *pipes, unsigned int count)
 {
        uint32_t handles[4], pitches[4], offsets[4] = {0}; /* we only use [0] */
        unsigned int other_fb_id;
@@ -1002,34 +1102,38 @@ static void test_page_flip(struct device *dev, struct connector_arg *c, unsigned
        unsigned int i;
        int ret;
 
-       other_bo = create_test_buffer(dev->kms, c->fourcc,
+       other_bo = create_test_buffer(dev->kms, pipes[0].fourcc,
                                      dev->mode.width, dev->mode.height,
                                      handles, pitches, offsets, PATTERN_PLAIN);
        if (other_bo == NULL)
                return;
 
-       ret = drmModeAddFB2(dev->fd, dev->mode.width, dev->mode.height, c->fourcc,
-                           handles, pitches, offsets, &other_fb_id, 0);
+       ret = drmModeAddFB2(dev->fd, dev->mode.width, dev->mode.height,
+                           pipes[0].fourcc, handles, pitches, offsets,
+                           &other_fb_id, 0);
        if (ret) {
                fprintf(stderr, "failed to add fb: %s\n", strerror(errno));
                return;
        }
 
        for (i = 0; i < count; i++) {
-               if (c[i].mode == NULL)
+               struct pipe_arg *pipe = &pipes[i];
+
+               if (pipe->mode == NULL)
                        continue;
 
-               ret = drmModePageFlip(dev->fd, c[i].crtc_id, other_fb_id,
-                                     DRM_MODE_PAGE_FLIP_EVENT, &c[i]);
+               ret = drmModePageFlip(dev->fd, pipe->crtc->crtc->crtc_id,
+                                     other_fb_id, DRM_MODE_PAGE_FLIP_EVENT,
+                                     pipe);
                if (ret) {
                        fprintf(stderr, "failed to page flip: %s\n", strerror(errno));
                        return;
                }
-               gettimeofday(&c[i].start, NULL);
-               c[i].swap_count = 0;
-               c[i].fb_id[0] = dev->mode.fb_id;
-               c[i].fb_id[1] = other_fb_id;
-               c[i].current_fb_id = other_fb_id;
+               gettimeofday(&pipe->start, NULL);
+               pipe->swap_count = 0;
+               pipe->fb_id[0] = dev->mode.fb_id;
+               pipe->fb_id[1] = other_fb_id;
+               pipe->current_fb_id = other_fb_id;
        }
 
        memset(&evctx, 0, sizeof evctx);
@@ -1080,19 +1184,41 @@ static void test_page_flip(struct device *dev, struct connector_arg *c, unsigned
 
 #define min(a, b)      ((a) < (b) ? (a) : (b))
 
-static int parse_connector(struct connector_arg *c, const char *arg)
+static int parse_connector(struct pipe_arg *pipe, const char *arg)
 {
        unsigned int len;
+       unsigned int i;
        const char *p;
        char *endp;
 
-       c->crtc_id = (uint32_t)-1;
-       strcpy(c->format_str, "XR24");
+       pipe->crtc_id = (uint32_t)-1;
+       strcpy(pipe->format_str, "XR24");
+
+       /* Count the number of connectors and allocate them. */
+       pipe->num_cons = 1;
+       for (p = arg; isdigit(*p) || *p == ','; ++p) {
+               if (*p == ',')
+                       pipe->num_cons++;
+       }
+
+       pipe->con_ids = malloc(pipe->num_cons * sizeof *pipe->con_ids);
+       if (pipe->con_ids == NULL)
+               return -1;
+
+       /* Parse the connectors. */
+       for (i = 0, p = arg; i < pipe->num_cons; ++i, p = endp + 1) {
+               pipe->con_ids[i] = strtoul(p, &endp, 10);
+               if (*endp != ',')
+                       break;
+       }
+
+       if (i != pipe->num_cons - 1)
+               return -1;
 
-       c->id = strtoul(arg, &endp, 10);
+       /* Parse the remaining parameters. */
        if (*endp == '@') {
                arg = endp + 1;
-               c->crtc_id = strtoul(arg, &endp, 10);
+               pipe->crtc_id = strtoul(arg, &endp, 10);
        }
        if (*endp != ':')
                return -1;
@@ -1100,18 +1226,18 @@ static int parse_connector(struct connector_arg *c, const char *arg)
        arg = endp + 1;
 
        p = strchrnul(arg, '@');
-       len = min(sizeof c->mode_str - 1, (unsigned int)(p - arg));
-       strncpy(c->mode_str, arg, len);
-       c->mode_str[len] = '\0';
+       len = min(sizeof pipe->mode_str - 1, (unsigned int)(p - arg));
+       strncpy(pipe->mode_str, arg, len);
+       pipe->mode_str[len] = '\0';
 
        if (*p == '@') {
-               strncpy(c->format_str, p + 1, 4);
-               c->format_str[4] = '\0';
+               strncpy(pipe->format_str, p + 1, 4);
+               pipe->format_str[4] = '\0';
        }
 
-       c->fourcc = format_fourcc(c->format_str);
-       if (c->fourcc == 0)  {
-               fprintf(stderr, "unknown format %s\n", c->format_str);
+       pipe->fourcc = format_fourcc(pipe->format_str);
+       if (pipe->fourcc == 0)  {
+               fprintf(stderr, "unknown format %s\n", pipe->format_str);
                return -1;
        }
 
@@ -1187,7 +1313,7 @@ static void usage(char *name)
 
        fprintf(stderr, "\n Test options:\n\n");
        fprintf(stderr, "\t-P <crtc_id>:<w>x<h>[+<x>+<y>][@<format>]\tset a plane\n");
-       fprintf(stderr, "\t-s <connector_id>[@<crtc_id>]:<mode>[@<format>]\tset a mode\n");
+       fprintf(stderr, "\t-s <connector_id>[,<connector_id>][@<crtc_id>]:<mode>[@<format>]\tset a mode\n");
        fprintf(stderr, "\t-v\ttest vsynced page flipping\n");
        fprintf(stderr, "\t-w <obj_id>:<prop_name>:<value>\tset property\n");
 
@@ -1230,12 +1356,12 @@ int main(int argc, char **argv)
        int encoders = 0, connectors = 0, crtcs = 0, planes = 0, framebuffers = 0;
        int drop_master = 0;
        int test_vsync = 0;
-       const char *modules[] = { "i915", "radeon", "nouveau", "vmwgfx", "omapdrm", "exynos", "tilcdc" };
+       const char *modules[] = { "i915", "radeon", "nouveau", "vmwgfx", "omapdrm", "exynos", "tilcdc", "msm" };
        char *module = NULL;
        unsigned int i;
        int count = 0, plane_count = 0;
        unsigned int prop_count = 0;
-       struct connector_arg *con_args = NULL;
+       struct pipe_arg *pipe_args = NULL;
        struct plane_arg *plane_args = NULL;
        struct property_arg *prop_args = NULL;
        unsigned int args = 0;
@@ -1283,14 +1409,14 @@ int main(int argc, char **argv)
                        planes = 1;
                        break;
                case 's':
-                       con_args = realloc(con_args,
-                                          (count + 1) * sizeof *con_args);
-                       if (con_args == NULL) {
+                       pipe_args = realloc(pipe_args,
+                                           (count + 1) * sizeof *pipe_args);
+                       if (pipe_args == NULL) {
                                fprintf(stderr, "memory allocation failed\n");
                                return 1;
                        }
 
-                       if (parse_connector(&con_args[count], optarg) < 0)
+                       if (parse_connector(&pipe_args[count], optarg) < 0)
                                usage(argv[0]);
 
                        count++;                                      
@@ -1380,13 +1506,13 @@ int main(int argc, char **argv)
                }
 
                if (count)
-                       set_mode(&dev, con_args, count);
+                       set_mode(&dev, pipe_args, count);
 
                if (plane_count)
                        set_planes(&dev, plane_args, plane_count);
 
                if (test_vsync)
-                       test_page_flip(&dev, con_args, count);
+                       test_page_flip(&dev, pipe_args, count);
 
                if (drop_master)
                        drmDropMaster(dev.fd);