Add missing stdlib.h and ctype.h includes
[media-ctl.git] / src / v4l2subdev.c
index 38dcdda..1370962 100644 (file)
@@ -1,37 +1,42 @@
 /*
  * V4L2 subdev interface library
  *
- * Copyright (C) 2010 Ideas on board SPRL <laurent.pinchart@ideasonboard.com>
+ * Copyright (C) 2010-2011 Ideas on board SPRL
+ *
+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
  *
  * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * it under the terms of the GNU Lesser General Public License as published
+ * by the Free Software Foundation; either version 2.1 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
+ * GNU Lesser General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 #include <sys/ioctl.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 
+#include <ctype.h>
 #include <errno.h>
 #include <fcntl.h>
+#include <stdbool.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 
 #include <linux/v4l2-subdev.h>
 
 #include "mediactl.h"
-#include "v4l2subdev.h"
 #include "tools.h"
+#include "v4l2subdev.h"
 
 int v4l2_subdev_open(struct media_entity *entity)
 {
@@ -40,8 +45,9 @@ int v4l2_subdev_open(struct media_entity *entity)
 
        entity->fd = open(entity->devname, O_RDWR);
        if (entity->fd == -1) {
-               printf("%s: Failed to open subdev device node %s\n", __func__,
-                       entity->devname);
+               media_dbg(entity->media,
+                         "%s: Failed to open subdev device node %s\n", __func__,
+                         entity->devname);
                return -errno;
        }
 
@@ -101,48 +107,83 @@ int v4l2_subdev_set_format(struct media_entity *entity,
        return 0;
 }
 
-int v4l2_subdev_get_crop(struct media_entity *entity, struct v4l2_rect *rect,
-                        unsigned int pad, enum v4l2_subdev_format_whence which)
+int v4l2_subdev_get_selection(struct media_entity *entity,
+       struct v4l2_rect *rect, unsigned int pad, unsigned int target,
+       enum v4l2_subdev_format_whence which)
 {
-       struct v4l2_subdev_crop crop;
+       union {
+               struct v4l2_subdev_selection sel;
+               struct v4l2_subdev_crop crop;
+       } u;
        int ret;
 
        ret = v4l2_subdev_open(entity);
        if (ret < 0)
                return ret;
 
-       memset(&crop, 0, sizeof(crop));
-       crop.pad = pad;
-       crop.which = which;
+       memset(&u.sel, 0, sizeof(u.sel));
+       u.sel.pad = pad;
+       u.sel.target = target;
+       u.sel.which = which;
 
-       ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_CROP, &crop);
+       ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_SELECTION, &u.sel);
+       if (ret >= 0) {
+               *rect = u.sel.r;
+               return 0;
+       }
+       if (errno != ENOTTY || target != V4L2_SEL_TGT_CROP)
+               return -errno;
+
+       memset(&u.crop, 0, sizeof(u.crop));
+       u.crop.pad = pad;
+       u.crop.which = which;
+
+       ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_CROP, &u.crop);
        if (ret < 0)
                return -errno;
 
-       *rect = crop.rect;
+       *rect = u.crop.rect;
        return 0;
 }
 
-int v4l2_subdev_set_crop(struct media_entity *entity, struct v4l2_rect *rect,
-                        unsigned int pad, enum v4l2_subdev_format_whence which)
+int v4l2_subdev_set_selection(struct media_entity *entity,
+       struct v4l2_rect *rect, unsigned int pad, unsigned int target,
+       enum v4l2_subdev_format_whence which)
 {
-       struct v4l2_subdev_crop crop;
+       union {
+               struct v4l2_subdev_selection sel;
+               struct v4l2_subdev_crop crop;
+       } u;
        int ret;
 
        ret = v4l2_subdev_open(entity);
        if (ret < 0)
                return ret;
 
-       memset(&crop, 0, sizeof(crop));
-       crop.pad = pad;
-       crop.which = which;
-       crop.rect = *rect;
+       memset(&u.sel, 0, sizeof(u.sel));
+       u.sel.pad = pad;
+       u.sel.target = target;
+       u.sel.which = which;
+       u.sel.r = *rect;
+
+       ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_SELECTION, &u.sel);
+       if (ret >= 0) {
+               *rect = u.sel.r;
+               return 0;
+       }
+       if (errno != ENOTTY || target != V4L2_SEL_TGT_CROP)
+               return -errno;
 
-       ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_CROP, &crop);
+       memset(&u.crop, 0, sizeof(u.crop));
+       u.crop.pad = pad;
+       u.crop.which = which;
+       u.crop.rect = *rect;
+
+       ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_CROP, &u.crop);
        if (ret < 0)
                return -errno;
 
-       *rect = crop.rect;
+       *rect = u.crop.rect;
        return 0;
 }
 
@@ -194,14 +235,19 @@ static int v4l2_subdev_parse_format(struct v4l2_mbus_framefmt *format,
        unsigned int width, height;
        char *end;
 
+       /*
+        * Compatibility with the old syntax: consider space as valid
+        * separator between the media bus pixel code and the size.
+        */
        for (; isspace(*p); ++p);
-       for (end = (char *)p; !isspace(*end) && *end != '\0'; ++end);
+       for (end = (char *)p;
+            *end != '/' && *end != ' ' && *end != '\0'; ++end);
 
        code = v4l2_subdev_string_to_pixelcode(p, end - p);
        if (code == (enum v4l2_mbus_pixelcode)-1)
                return -EINVAL;
 
-       for (p = end; isspace(*p); ++p);
+       p = end + 1;
        width = strtoul(p, &end, 10);
        if (*end != 'x')
                return -EINVAL;
@@ -218,32 +264,32 @@ static int v4l2_subdev_parse_format(struct v4l2_mbus_framefmt *format,
        return 0;
 }
 
-static int v4l2_subdev_parse_crop(
-       struct v4l2_rect *crop, const char *p, char **endp)
+static int v4l2_subdev_parse_rectangle(
+       struct v4l2_rect *r, const char *p, char **endp)
 {
        char *end;
 
        if (*p++ != '(')
                return -EINVAL;
 
-       crop->left = strtoul(p, &end, 10);
+       r->left = strtoul(p, &end, 10);
        if (*end != ',')
                return -EINVAL;
 
        p = end + 1;
-       crop->top = strtoul(p, &end, 10);
+       r->top = strtoul(p, &end, 10);
        if (*end++ != ')')
                return -EINVAL;
        if (*end != '/')
                return -EINVAL;
 
        p = end + 1;
-       crop->width = strtoul(p, &end, 10);
+       r->width = strtoul(p, &end, 10);
        if (*end != 'x')
                return -EINVAL;
 
        p = end + 1;
-       crop->height = strtoul(p, &end, 10);
+       r->height = strtoul(p, &end, 10);
        *endp = end;
 
        return 0;
@@ -269,12 +315,29 @@ static int v4l2_subdev_parse_frame_interval(struct v4l2_fract *interval,
        return 0;
 }
 
+/*
+ * The debate over whether this function should be named icanhasstr() instead
+ * has been strong and heated. If you feel like this would be an important
+ * change, patches are welcome (or not).
+ */
+static bool strhazit(const char *str, const char **p)
+{
+       int len = strlen(str);
+
+       if (strncmp(str, *p, len))
+               return false;
+
+       for (*p += len; isspace(**p); ++*p);
+       return true;
+}
+
 static struct media_pad *v4l2_subdev_parse_pad_format(
        struct media_device *media, struct v4l2_mbus_framefmt *format,
-       struct v4l2_rect *crop, struct v4l2_fract *interval, const char *p,
-       char **endp)
+       struct v4l2_rect *crop, struct v4l2_rect *compose,
+       struct v4l2_fract *interval, const char *p, char **endp)
 {
        struct media_pad *pad;
+       bool first;
        char *end;
        int ret;
 
@@ -288,30 +351,54 @@ static struct media_pad *v4l2_subdev_parse_pad_format(
        if (*p++ != '[')
                return NULL;
 
-       for (; isspace(*p); ++p);
+       for (first = true; ; first = false) {
+               for (; isspace(*p); p++);
 
-       if (isalnum(*p)) {
-               ret = v4l2_subdev_parse_format(format, p, &end);
-               if (ret < 0)
-                       return NULL;
+               /*
+                * Backward compatibility: if the first property starts with an
+                * uppercase later, process it as a format description.
+                */
+               if (strhazit("fmt:", &p) || (first && isupper(*p))) {
+                       ret = v4l2_subdev_parse_format(format, p, &end);
+                       if (ret < 0)
+                               return NULL;
 
-               for (p = end; isspace(*p); p++);
-       }
+                       p = end;
+                       continue;
+               }
 
-       if (*p == '(') {
-               ret = v4l2_subdev_parse_crop(crop, p, &end);
-               if (ret < 0)
-                       return NULL;
+               /*
+                * Backward compatibility: crop rectangles can be specified
+                * implicitly without the 'crop:' property name.
+                */
+               if (strhazit("crop:", &p) || *p == '(') {
+                       ret = v4l2_subdev_parse_rectangle(crop, p, &end);
+                       if (ret < 0)
+                               return NULL;
+
+                       p = end;
+                       continue;
+               }
 
-               for (p = end; isspace(*p); p++);
-       }
+               if (strhazit("compose:", &p)) {
+                       ret = v4l2_subdev_parse_rectangle(compose, p, &end);
+                       if (ret < 0)
+                               return NULL;
 
-       if (*p == '@') {
-               ret = v4l2_subdev_parse_frame_interval(interval, ++p, &end);
-               if (ret < 0)
-                       return NULL;
+                       for (p = end; isspace(*p); p++);
+                       continue;
+               }
+
+               if (*p == '@') {
+                       ret = v4l2_subdev_parse_frame_interval(interval, ++p, &end);
+                       if (ret < 0)
+                               return NULL;
+
+                       p = end;
+                       continue;
+               }
 
-               for (p = end; isspace(*p); p++);
+               break;
        }
 
        if (*p != ']')
@@ -329,45 +416,54 @@ static int set_format(struct media_pad *pad,
        if (format->width == 0 || format->height == 0)
                return 0;
 
-       printf("Setting up format %s %ux%u on pad %s/%u\n",
-              v4l2_subdev_pixelcode_to_string(format->code),
-              format->width, format->height,
-              pad->entity->info.name, pad->index);
+       media_dbg(pad->entity->media,
+                 "Setting up format %s %ux%u on pad %s/%u\n",
+                 v4l2_subdev_pixelcode_to_string(format->code),
+                 format->width, format->height,
+                 pad->entity->info.name, pad->index);
 
        ret = v4l2_subdev_set_format(pad->entity, format, pad->index,
                                     V4L2_SUBDEV_FORMAT_ACTIVE);
        if (ret < 0) {
-               printf("Unable to set format: %s (%d)\n", strerror(-ret), ret);
+               media_dbg(pad->entity->media,
+                         "Unable to set format: %s (%d)\n",
+                         strerror(-ret), ret);
                return ret;
        }
 
-       printf("Format set: %s %ux%u\n",
-              v4l2_subdev_pixelcode_to_string(format->code),
-              format->width, format->height);
+       media_dbg(pad->entity->media,
+                 "Format set: %s %ux%u\n",
+                 v4l2_subdev_pixelcode_to_string(format->code),
+                 format->width, format->height);
 
        return 0;
 }
 
-static int set_crop(struct media_pad *pad, struct v4l2_rect *crop)
+static int set_selection(struct media_pad *pad, unsigned int target,
+                        struct v4l2_rect *rect)
 {
        int ret;
 
-       if (crop->left == -1 || crop->top == -1)
+       if (rect->left == -1 || rect->top == -1)
                return 0;
 
-       printf("Setting up crop rectangle (%u,%u)/%ux%u on pad %s/%u\n",
-               crop->left, crop->top, crop->width, crop->height,
-               pad->entity->info.name, pad->index);
+       media_dbg(pad->entity->media,
+                 "Setting up selection target %u rectangle (%u,%u)/%ux%u on pad %s/%u\n",
+                 target, rect->left, rect->top, rect->width, rect->height,
+                 pad->entity->info.name, pad->index);
 
-       ret = v4l2_subdev_set_crop(pad->entity, crop, pad->index,
-                                  V4L2_SUBDEV_FORMAT_ACTIVE);
+       ret = v4l2_subdev_set_selection(pad->entity, rect, pad->index,
+                                       target, V4L2_SUBDEV_FORMAT_ACTIVE);
        if (ret < 0) {
-               printf("Unable to set crop rectangle: %s (%d)\n", strerror(-ret), ret);
+               media_dbg(pad->entity->media,
+                         "Unable to set selection rectangle: %s (%d)\n",
+                         strerror(-ret), ret);
                return ret;
        }
 
-       printf("Crop rectangle set: (%u,%u)/%ux%u\n",
-               crop->left, crop->top, crop->width, crop->height);
+       media_dbg(pad->entity->media,
+                 "Selection rectangle set: (%u,%u)/%ux%u\n",
+                 rect->left, rect->top, rect->width, rect->height);
 
        return 0;
 }
@@ -380,17 +476,21 @@ static int set_frame_interval(struct media_entity *entity,
        if (interval->numerator == 0)
                return 0;
 
-       printf("Setting up frame interval %u/%u on entity %s\n",
-               interval->numerator, interval->denominator, entity->info.name);
+       media_dbg(entity->media,
+                 "Setting up frame interval %u/%u on entity %s\n",
+                 interval->numerator, interval->denominator,
+                 entity->info.name);
 
        ret = v4l2_subdev_set_frame_interval(entity, interval);
        if (ret < 0) {
-               printf("Unable to set frame interval: %s (%d)", strerror(-ret), ret);
+               media_dbg(entity->media,
+                         "Unable to set frame interval: %s (%d)",
+                         strerror(-ret), ret);
                return ret;
        }
 
-       printf("Frame interval set: %u/%u\n",
-               interval->numerator, interval->denominator);
+       media_dbg(entity->media, "Frame interval set: %u/%u\n",
+                 interval->numerator, interval->denominator);
 
        return 0;
 }
@@ -402,30 +502,35 @@ static int v4l2_subdev_parse_setup_format(struct media_device *media,
        struct v4l2_mbus_framefmt format = { 0, 0, 0 };
        struct media_pad *pad;
        struct v4l2_rect crop = { -1, -1, -1, -1 };
+       struct v4l2_rect compose = crop;
        struct v4l2_fract interval = { 0, 0 };
        unsigned int i;
        char *end;
        int ret;
 
-       pad = v4l2_subdev_parse_pad_format(media, &format, &crop, &interval,
-                                          p, &end);
+       pad = v4l2_subdev_parse_pad_format(media, &format, &crop, &compose,
+                                          &interval, p, &end);
        if (pad == NULL) {
-               printf("Unable to parse format\n");
+               media_dbg(media, "Unable to parse format\n");
                return -EINVAL;
        }
 
-       if (pad->flags & MEDIA_PAD_FL_SOURCE) {
-               ret = set_crop(pad, &crop);
+       if (pad->flags & MEDIA_PAD_FL_SINK) {
+               ret = set_format(pad, &format);
                if (ret < 0)
                        return ret;
        }
 
-       ret = set_format(pad, &format);
+       ret = set_selection(pad, V4L2_SEL_TGT_CROP, &crop);
        if (ret < 0)
                return ret;
 
-       if (pad->flags & MEDIA_PAD_FL_SINK) {
-               ret = set_crop(pad, &crop);
+       ret = set_selection(pad, V4L2_SEL_TGT_COMPOSE, &compose);
+       if (ret < 0)
+               return ret;
+
+       if (pad->flags & MEDIA_PAD_FL_SOURCE) {
+               ret = set_format(pad, &format);
                if (ret < 0)
                        return ret;
        }
@@ -482,7 +587,11 @@ static struct {
        { "Y10", V4L2_MBUS_FMT_Y10_1X10 },
        { "Y12", V4L2_MBUS_FMT_Y12_1X12 },
        { "YUYV", V4L2_MBUS_FMT_YUYV8_1X16 },
+       { "YUYV1_5X8", V4L2_MBUS_FMT_YUYV8_1_5X8 },
+       { "YUYV2X8", V4L2_MBUS_FMT_YUYV8_2X8 },
        { "UYVY", V4L2_MBUS_FMT_UYVY8_1X16 },
+       { "UYVY1_5X8", V4L2_MBUS_FMT_UYVY8_1_5X8 },
+       { "UYVY2X8", V4L2_MBUS_FMT_UYVY8_2X8 },
        { "SBGGR8", V4L2_MBUS_FMT_SBGGR8_1X8 },
        { "SGBRG8", V4L2_MBUS_FMT_SGBRG8_1X8 },
        { "SGRBG8", V4L2_MBUS_FMT_SGRBG8_1X8 },