Re: [PATCH] libv4l1: Move VIDIOCGFBUF into libv4l1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Thanks, I've applied your patch with one small fix,
The else block at the end of was wrongly indented
(one indent level too much) It is the else for the first if, not the second.
Note the first if has a { at the end of the line, and the second does not,
and the else starts with a }.

Regards,

Hans


On 05/31/2010 10:03 AM, huzaifas@xxxxxxxxxx wrote:
From: Huzaifa Sidhpurwala<huzaifas@fedora-12.(none)>

Move VIDIOCGFBUF into libv4l1

Signed-off-by: Huzaifa Sidhpurwala<huzaifas@xxxxxxxxxx>
---
  lib/libv4l1/libv4l1.c |   45 +++++++++++++++++++++++++++++++++++++++++++++
  1 files changed, 45 insertions(+), 0 deletions(-)

diff --git a/lib/libv4l1/libv4l1.c b/lib/libv4l1/libv4l1.c
index e13feba..5b2dc29 100644
--- a/lib/libv4l1/libv4l1.c
+++ b/lib/libv4l1/libv4l1.c
@@ -804,6 +804,51 @@ int v4l1_ioctl(int fd, unsigned long int request, ...)
  		break;
  	}

+	case VIDIOCGFBUF: {
+		struct video_buffer *buffer = arg;
+		struct v4l2_framebuffer fbuf = { 0, };
+
+		result = v4l2_ioctl(fd, VIDIOC_G_FBUF, buffer);
+		if (result<  0)
+			break;
+
+		buffer->base = fbuf.base;
+		buffer->height = fbuf.fmt.height;
+		buffer->width = fbuf.fmt.width;
+
+		switch (fbuf.fmt.pixelformat) {
+		case V4L2_PIX_FMT_RGB332:
+			buffer->depth = 8;
+			break;
+		case V4L2_PIX_FMT_RGB555:
+			buffer->depth = 15;
+			break;
+		case V4L2_PIX_FMT_RGB565:
+			buffer->depth = 16;
+			break;
+		case V4L2_PIX_FMT_BGR24:
+			buffer->depth = 24;
+			break;
+		case V4L2_PIX_FMT_BGR32:
+			buffer->depth = 32;
+			break;
+		default:
+			buffer->depth = 0;
+		}
+
+		if (fbuf.fmt.bytesperline) {
+			buffer->bytesperline = fbuf.fmt.bytesperline;
+			if (!buffer->depth&&  buffer->width)
+				buffer->depth = ((fbuf.fmt.bytesperline<<3)
+						+ (buffer->width-1))
+						/ buffer->width;
+			} else {
+				buffer->bytesperline =
+					(buffer->width * buffer->depth + 7)&  7;
+				buffer->bytesperline>>= 3;
+			}
+	}
+
  	default:
  		/* Pass through libv4l2 for applications which are using v4l2 through
  		   libv4l1 (this can happen with the v4l1compat.so wrapper preloaded */
--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux