Possible fix for mplane support
This commit is contained in:
parent
07adf898e7
commit
868291e0c6
42
src/camera.c
42
src/camera.c
|
@ -264,7 +264,6 @@ mp_camera_new(int video_fd, int subdev_fd)
|
||||||
bool use_mplane;
|
bool use_mplane;
|
||||||
if (cap.capabilities & V4L2_CAP_VIDEO_CAPTURE_MPLANE) {
|
if (cap.capabilities & V4L2_CAP_VIDEO_CAPTURE_MPLANE) {
|
||||||
use_mplane = true;
|
use_mplane = true;
|
||||||
printf("!!\n");
|
|
||||||
} else if (cap.capabilities & V4L2_CAP_VIDEO_CAPTURE) {
|
} else if (cap.capabilities & V4L2_CAP_VIDEO_CAPTURE) {
|
||||||
use_mplane = false;
|
use_mplane = false;
|
||||||
} else {
|
} else {
|
||||||
|
@ -394,10 +393,19 @@ mp_camera_get_subdev_fd(MPCamera *camera)
|
||||||
return camera->subdev_fd;
|
return camera->subdev_fd;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static enum v4l2_buf_type
|
||||||
|
get_buf_type(MPCamera *camera)
|
||||||
|
{
|
||||||
|
if (camera->use_mplane) {
|
||||||
|
return V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
|
||||||
|
}
|
||||||
|
return V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
||||||
|
}
|
||||||
|
|
||||||
static bool
|
static bool
|
||||||
camera_mode_impl(MPCamera *camera, int request, MPCameraMode *mode)
|
camera_mode_impl(MPCamera *camera, int request, MPCameraMode *mode)
|
||||||
{
|
{
|
||||||
uint32_t pixfmt = mp_pixel_format_from_v4l_pixel_format(mode->pixel_format);
|
uint32_t pixfmt = mp_pixel_format_to_v4l_pixel_format(mode->pixel_format);
|
||||||
struct v4l2_format fmt = {};
|
struct v4l2_format fmt = {};
|
||||||
if (camera->use_mplane) {
|
if (camera->use_mplane) {
|
||||||
fmt.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
|
fmt.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
|
||||||
|
@ -460,7 +468,6 @@ mp_camera_set_mode(MPCamera *camera, MPCameraMode *mode)
|
||||||
VIDIOC_SUBDEV_S_FRAME_INTERVAL,
|
VIDIOC_SUBDEV_S_FRAME_INTERVAL,
|
||||||
&interval) == -1) {
|
&interval) == -1) {
|
||||||
errno_printerr("VIDIOC_SUBDEV_S_FRAME_INTERVAL");
|
errno_printerr("VIDIOC_SUBDEV_S_FRAME_INTERVAL");
|
||||||
return false;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
bool did_set_frame_rate = interval.interval.numerator ==
|
bool did_set_frame_rate = interval.interval.numerator ==
|
||||||
|
@ -522,10 +529,7 @@ mp_camera_start_capture(MPCamera *camera)
|
||||||
g_return_val_if_fail(camera->has_set_mode, false);
|
g_return_val_if_fail(camera->has_set_mode, false);
|
||||||
g_return_val_if_fail(camera->num_buffers == 0, false);
|
g_return_val_if_fail(camera->num_buffers == 0, false);
|
||||||
|
|
||||||
enum v4l2_buf_type buftype = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
const enum v4l2_buf_type buftype = get_buf_type(camera);
|
||||||
if (camera->use_mplane) {
|
|
||||||
buftype = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
|
|
||||||
}
|
|
||||||
|
|
||||||
// Start by requesting buffers
|
// Start by requesting buffers
|
||||||
struct v4l2_requestbuffers req = {};
|
struct v4l2_requestbuffers req = {};
|
||||||
|
@ -588,7 +592,7 @@ mp_camera_start_capture(MPCamera *camera)
|
||||||
}
|
}
|
||||||
|
|
||||||
struct v4l2_exportbuffer expbuf = {
|
struct v4l2_exportbuffer expbuf = {
|
||||||
.type = V4L2_BUF_TYPE_VIDEO_CAPTURE,
|
.type = buftype,
|
||||||
.index = i,
|
.index = i,
|
||||||
};
|
};
|
||||||
if (xioctl(camera->video_fd, VIDIOC_EXPBUF, &expbuf) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_EXPBUF, &expbuf) == -1) {
|
||||||
|
@ -627,7 +631,7 @@ mp_camera_start_capture(MPCamera *camera)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Start capture
|
// Start capture
|
||||||
enum v4l2_buf_type type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
enum v4l2_buf_type type = buftype;
|
||||||
if (xioctl(camera->video_fd, VIDIOC_STREAMON, &type) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_STREAMON, &type) == -1) {
|
||||||
errno_printerr("VIDIOC_STREAMON");
|
errno_printerr("VIDIOC_STREAMON");
|
||||||
goto error;
|
goto error;
|
||||||
|
@ -653,7 +657,7 @@ error:
|
||||||
{
|
{
|
||||||
struct v4l2_requestbuffers req = {};
|
struct v4l2_requestbuffers req = {};
|
||||||
req.count = 0;
|
req.count = 0;
|
||||||
req.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
req.type = buftype;
|
||||||
req.memory = V4L2_MEMORY_MMAP;
|
req.memory = V4L2_MEMORY_MMAP;
|
||||||
|
|
||||||
if (xioctl(camera->video_fd, VIDIOC_REQBUFS, &req) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_REQBUFS, &req) == -1) {
|
||||||
|
@ -669,7 +673,9 @@ mp_camera_stop_capture(MPCamera *camera)
|
||||||
{
|
{
|
||||||
g_return_val_if_fail(camera->num_buffers > 0, false);
|
g_return_val_if_fail(camera->num_buffers > 0, false);
|
||||||
|
|
||||||
enum v4l2_buf_type type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
const enum v4l2_buf_type buftype = get_buf_type(camera);
|
||||||
|
|
||||||
|
enum v4l2_buf_type type = buftype;
|
||||||
if (xioctl(camera->video_fd, VIDIOC_STREAMOFF, &type) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_STREAMOFF, &type) == -1) {
|
||||||
errno_printerr("VIDIOC_STREAMOFF");
|
errno_printerr("VIDIOC_STREAMOFF");
|
||||||
}
|
}
|
||||||
|
@ -690,7 +696,7 @@ mp_camera_stop_capture(MPCamera *camera)
|
||||||
|
|
||||||
struct v4l2_requestbuffers req = {};
|
struct v4l2_requestbuffers req = {};
|
||||||
req.count = 0;
|
req.count = 0;
|
||||||
req.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
req.type = buftype;
|
||||||
req.memory = V4L2_MEMORY_MMAP;
|
req.memory = V4L2_MEMORY_MMAP;
|
||||||
if (xioctl(camera->video_fd, VIDIOC_REQBUFS, &req) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_REQBUFS, &req) == -1) {
|
||||||
errno_printerr("VIDIOC_REQBUFS");
|
errno_printerr("VIDIOC_REQBUFS");
|
||||||
|
@ -708,8 +714,10 @@ mp_camera_is_capturing(MPCamera *camera)
|
||||||
bool
|
bool
|
||||||
mp_camera_capture_buffer(MPCamera *camera, MPBuffer *buffer)
|
mp_camera_capture_buffer(MPCamera *camera, MPBuffer *buffer)
|
||||||
{
|
{
|
||||||
|
const enum v4l2_buf_type buftype = get_buf_type(camera);
|
||||||
|
|
||||||
struct v4l2_buffer buf = {};
|
struct v4l2_buffer buf = {};
|
||||||
buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
buf.type = buftype;
|
||||||
buf.memory = V4L2_MEMORY_MMAP;
|
buf.memory = V4L2_MEMORY_MMAP;
|
||||||
|
|
||||||
struct v4l2_plane planes[1];
|
struct v4l2_plane planes[1];
|
||||||
|
@ -757,8 +765,10 @@ mp_camera_capture_buffer(MPCamera *camera, MPBuffer *buffer)
|
||||||
bool
|
bool
|
||||||
mp_camera_release_buffer(MPCamera *camera, uint32_t buffer_index)
|
mp_camera_release_buffer(MPCamera *camera, uint32_t buffer_index)
|
||||||
{
|
{
|
||||||
|
const enum v4l2_buf_type buftype = get_buf_type(camera);
|
||||||
|
|
||||||
struct v4l2_buffer buf = {};
|
struct v4l2_buffer buf = {};
|
||||||
buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
buf.type = buftype;
|
||||||
buf.memory = V4L2_MEMORY_MMAP;
|
buf.memory = V4L2_MEMORY_MMAP;
|
||||||
buf.index = buffer_index;
|
buf.index = buffer_index;
|
||||||
if (xioctl(camera->video_fd, VIDIOC_QBUF, &buf) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_QBUF, &buf) == -1) {
|
||||||
|
@ -863,12 +873,14 @@ get_subdev_modes(MPCamera *camera, bool (*check)(MPCamera *, MPCameraMode *))
|
||||||
static MPCameraModeList *
|
static MPCameraModeList *
|
||||||
get_video_modes(MPCamera *camera, bool (*check)(MPCamera *, MPCameraMode *))
|
get_video_modes(MPCamera *camera, bool (*check)(MPCamera *, MPCameraMode *))
|
||||||
{
|
{
|
||||||
|
const enum v4l2_buf_type buftype = get_buf_type(camera);
|
||||||
|
|
||||||
MPCameraModeList *item = NULL;
|
MPCameraModeList *item = NULL;
|
||||||
|
|
||||||
for (uint32_t fmt_index = 0;; ++fmt_index) {
|
for (uint32_t fmt_index = 0;; ++fmt_index) {
|
||||||
struct v4l2_fmtdesc fmt = {};
|
struct v4l2_fmtdesc fmt = {};
|
||||||
fmt.index = fmt_index;
|
fmt.index = fmt_index;
|
||||||
fmt.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
|
fmt.type = buftype;
|
||||||
if (xioctl(camera->video_fd, VIDIOC_ENUM_FMT, &fmt) == -1) {
|
if (xioctl(camera->video_fd, VIDIOC_ENUM_FMT, &fmt) == -1) {
|
||||||
if (errno != EINVAL) {
|
if (errno != EINVAL) {
|
||||||
errno_printerr("VIDIOC_ENUM_FMT");
|
errno_printerr("VIDIOC_ENUM_FMT");
|
||||||
|
|
Loading…
Reference in New Issue