Support data_offset for multi plane buffers. Also add an option to write the data in the buffer before data offset (--buffer-prefix). Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> --- yavta.c | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/yavta.c b/yavta.c index 77e5a41..cf8239b 100644 --- a/yavta.c +++ b/yavta.c @@ -80,6 +80,8 @@ struct device void *pattern[VIDEO_MAX_PLANES]; unsigned int patternsize[VIDEO_MAX_PLANES]; + + bool write_buffer_prefix; }; static bool video_is_mplane(struct device *dev) @@ -1545,14 +1547,21 @@ static void video_save_image(struct device *dev, struct v4l2_buffer *buf, return; for (i = 0; i < dev->num_planes; i++) { + void *data = dev->buffers[buf->index].mem[i]; unsigned int length; - if (video_is_mplane(dev)) + if (video_is_mplane(dev)) { length = buf->m.planes[i].bytesused; - else + + if (!dev->write_buffer_prefix) { + data += buf->m.planes[i].data_offset; + length -= buf->m.planes[i].data_offset; + } + } else { length = buf->bytesused; + } - ret = write(fd, dev->buffers[buf->index].mem[i], length); + ret = write(fd, data, length); if (ret < 0) { printf("write error: %s (%d)\n", strerror(errno), errno); break; @@ -1717,6 +1726,7 @@ static void usage(const char *argv0) printf("-t, --time-per-frame num/denom Set the time per frame (eg. 1/25 = 25 fps)\n"); printf("-u, --userptr Use the user pointers streaming method\n"); printf("-w, --set-control 'ctrl value' Set control 'ctrl' to 'value'\n"); + printf(" --buffer-prefix Write portions of buffer before data_offset\n"); printf(" --buffer-size Buffer size in bytes\n"); printf(" --enum-formats Enumerate formats\n"); printf(" --enum-inputs Enumerate inputs\n"); @@ -1749,10 +1759,12 @@ static void usage(const char *argv0) #define OPT_BUFFER_SIZE 268 #define OPT_PREMULTIPLIED 269 #define OPT_QUEUE_LATE 270 +#define OPT_BUFFER_PREFIX 271 static struct option opts[] = { {"buffer-size", 1, 0, OPT_BUFFER_SIZE}, {"buffer-type", 1, 0, 'B'}, + {"buffer-prefix", 1, 0, OPT_BUFFER_PREFIX}, {"capture", 2, 0, 'c'}, {"check-overrun", 0, 0, 'C'}, {"delay", 1, 0, 'd'}, @@ -2016,6 +2028,8 @@ int main(int argc, char *argv[]) case OPT_USERPTR_OFFSET: userptr_offset = atoi(optarg); break; + case OPT_BUFFER_PREFIX: + dev.write_buffer_prefix = true; default: printf("Invalid option -%c\n", c); printf("Run %s -h for help.\n", argv[0]); -- 2.1.0.231.g7484e3b -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html