Print more detailed parse error messages
authorSascha Hauer <s.hauer@pengutronix.de>
Wed, 8 May 2013 13:27:53 +0000 (15:27 +0200)
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Mon, 10 Jun 2013 13:30:23 +0000 (15:30 +0200)
The following errors usually resulted in the same 'Unable to parse link'
message:

- one of the given entities does not exist
- one of the pads of a given entity does not exist
- No link exists between given pads
- syntax error in link description

Add more detailed error messages to give the user a clue what is going wrong.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
src/mediactl.c

index 4783a58..c65de50 100644 (file)
@@ -537,31 +537,42 @@ struct media_pad *media_parse_pad(struct media_device *media,
 
        if (*p == '"') {
                for (end = (char *)p + 1; *end && *end != '"'; ++end);
 
        if (*p == '"') {
                for (end = (char *)p + 1; *end && *end != '"'; ++end);
-               if (*end != '"')
+               if (*end != '"') {
+                       media_dbg(media, "missing matching '\"'\n");
                        return NULL;
                        return NULL;
+               }
 
                entity = media_get_entity_by_name(media, p + 1, end - p - 1);
 
                entity = media_get_entity_by_name(media, p + 1, end - p - 1);
-               if (entity == NULL)
+               if (entity == NULL) {
+                       media_dbg(media, "no such entity \"%.*s\"\n", end - p - 1, p + 1);
                        return NULL;
                        return NULL;
+               }
 
                ++end;
        } else {
                entity_id = strtoul(p, &end, 10);
                entity = media_get_entity_by_id(media, entity_id);
 
                ++end;
        } else {
                entity_id = strtoul(p, &end, 10);
                entity = media_get_entity_by_id(media, entity_id);
-               if (entity == NULL)
+               if (entity == NULL) {
+                       media_dbg(media, "no such entity %d\n", entity_id);
                        return NULL;
                        return NULL;
+               }
        }
        for (; isspace(*end); ++end);
 
        }
        for (; isspace(*end); ++end);
 
-       if (*end != ':')
+       if (*end != ':') {
+               media_dbg(media, "Expected ':'\n", *end);
                return NULL;
                return NULL;
+       }
+
        for (p = end + 1; isspace(*p); ++p);
 
        pad = strtoul(p, &end, 10);
        for (p = end + 1; isspace(*p); ++p);
 
        pad = strtoul(p, &end, 10);
-       for (p = end; isspace(*p); ++p);
 
 
-       if (pad >= entity->info.pads)
+       if (pad >= entity->info.pads) {
+               media_dbg(media, "No pad '%d' on entity \"%s\". Maximum pad number is %d\n",
+                               pad, entity->info.name, entity->info.pads - 1);
                return NULL;
                return NULL;
+       }
 
        for (p = end; isspace(*p); ++p);
        if (endp)
 
        for (p = end; isspace(*p); ++p);
        if (endp)
@@ -583,8 +594,11 @@ struct media_link *media_parse_link(struct media_device *media,
        if (source == NULL)
                return NULL;
 
        if (source == NULL)
                return NULL;
 
-       if (end[0] != '-' || end[1] != '>')
+       if (end[0] != '-' || end[1] != '>') {
+               media_dbg(media, "Expected '->'\n");
                return NULL;
                return NULL;
+       }
+
        p = end + 2;
 
        sink = media_parse_pad(media, p, &end);
        p = end + 2;
 
        sink = media_parse_pad(media, p, &end);
@@ -600,6 +614,9 @@ struct media_link *media_parse_link(struct media_device *media,
                        return link;
        }
 
                        return link;
        }
 
+       media_dbg(media, "No link between \"%s\":%d and \"%s\":%d\n",
+                       source->entity->info.name, source->index,
+                       sink->entity->info.name, sink->index);
        return NULL;
 }
 
        return NULL;
 }
 
@@ -619,14 +636,14 @@ int media_parse_setup_link(struct media_device *media,
 
        p = end;
        if (*p++ != '[') {
 
        p = end;
        if (*p++ != '[') {
-               media_dbg(media, "Unable to parse link flags\n");
+               media_dbg(media, "Unable to parse link flags: expected '['.\n");
                return -EINVAL;
        }
 
        flags = strtoul(p, &end, 10);
        for (p = end; isspace(*p); p++);
        if (*p++ != ']') {
                return -EINVAL;
        }
 
        flags = strtoul(p, &end, 10);
        for (p = end; isspace(*p); p++);
        if (*p++ != ']') {
-               media_dbg(media, "Unable to parse link flags\n");
+               media_dbg(media, "Unable to parse link flags: expected ']'.\n");
                return -EINVAL;
        }
 
                return -EINVAL;
        }