| Message ID | BAY176-W19A194B095C32CE30B0B8DA90D0@phx.gbl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 07/07/2014 09:27 AM, Divneil Wadhawan wrote: > Hi Hans, > > >> include/media/davinci/vpfe_capture.h > > It uses videobuf-dma-contig.h, so, I left it out. Ah, you are right. I thought that one was converted to vb2, but it isn't. > > >> drivers/media/platform/vivi-core.c > > Cannot find this one. Checked with find, in case it changed location, but couldn't. Sorry, my mistake. I'm working on the vivi driver and I'm actually the one who introduced VIDEO_MAX_FRAME there. But since that isn't committed yet you can't find it either :-) > > >> drivers/media/pci/saa7134/* > > Updated. > > > Please find below the patch. I am hoping it's good to go. I'll take a look at it. Regards, Hans > > > Regards, > > Divneil > > > From 1792d75dc0f893a181d991a0b238bbd0ead945c1 Mon Sep 17 00:00:00 2001 > From: Divneil Wadhawan <divneil.wadhawan@st.com> > Date: Mon, 7 Jul 2014 12:38:06 +0530 > Subject: [PATCH] v4l2: vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME > > - vb2 drivers to rely on VB2_MAX_FRAME. > > - VB2_MAX_FRAME bumps the value to 64 from current 32 > > Change-Id: I3d7998898df43553486166c44b54524aac449deb > Signed-off-by: Divneil Wadhawan <divneil.wadhawan@st.com> > --- > drivers/media/pci/saa7134/saa7134-ts.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-vbi.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-video.c | 2 +- > drivers/media/platform/mem2mem_testdev.c | 2 +- > drivers/media/platform/ti-vpe/vpe.c | 2 +- > drivers/media/v4l2-core/videobuf2-core.c | 8 ++++---- > include/media/videobuf2-core.h | 4 +++- > 7 files changed, 14 insertions(+), 12 deletions(-) > > diff --git a/drivers/media/pci/saa7134/saa7134-ts.c b/drivers/media/pci/saa7134/saa7134-ts.c > index bd25323..0d04995 100644 > --- a/drivers/media/pci/saa7134/saa7134-ts.c > +++ b/drivers/media/pci/saa7134/saa7134-ts.c > @@ -227,8 +227,8 @@ int saa7134_ts_init1(struct saa7134_dev *dev) > /* sanitycheck insmod options */ > if (tsbufs < 2) > tsbufs = 2; > - if (tsbufs> VIDEO_MAX_FRAME) > - tsbufs = VIDEO_MAX_FRAME; > + if (tsbufs> VB2_MAX_FRAME) > + tsbufs = VB2_MAX_FRAME; > if (ts_nr_packets < 4) > ts_nr_packets = 4; > if (ts_nr_packets> 312) > diff --git a/drivers/media/pci/saa7134/saa7134-vbi.c b/drivers/media/pci/saa7134/saa7134-vbi.c > index c06dbe1..15b5860 100644 > --- a/drivers/media/pci/saa7134/saa7134-vbi.c > +++ b/drivers/media/pci/saa7134/saa7134-vbi.c > @@ -203,8 +203,8 @@ int saa7134_vbi_init1(struct saa7134_dev *dev) > > if (vbibufs < 2) > vbibufs = 2; > - if (vbibufs> VIDEO_MAX_FRAME) > - vbibufs = VIDEO_MAX_FRAME; > + if (vbibufs> VB2_MAX_FRAME) > + vbibufs = VB2_MAX_FRAME; > return 0; > } > > diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa7134-video.c > index d375999..47dda6c 100644 > --- a/drivers/media/pci/saa7134/saa7134-video.c > +++ b/drivers/media/pci/saa7134/saa7134-video.c > @@ -2032,7 +2032,7 @@ int saa7134_video_init1(struct saa7134_dev *dev) > int ret; > > /* sanitycheck insmod options */ > - if (gbuffers < 2 || gbuffers> VIDEO_MAX_FRAME) > + if (gbuffers < 2 || gbuffers> VB2_MAX_FRAME) > gbuffers = 2; > if (gbufsize> gbufsize_max) > gbufsize = gbufsize_max; > diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/platform/mem2mem_testdev.c > index 0714070..fe3235d 100644 > --- a/drivers/media/platform/mem2mem_testdev.c > +++ b/drivers/media/platform/mem2mem_testdev.c > @@ -55,7 +55,7 @@ MODULE_PARM_DESC(debug, "activates debug info"); > #define MEM2MEM_NAME "m2m-testdev" > > /* Per queue */ > -#define MEM2MEM_DEF_NUM_BUFS VIDEO_MAX_FRAME > +#define MEM2MEM_DEF_NUM_BUFS VB2_MAX_FRAME > /* In bytes, per queue */ > #define MEM2MEM_VID_MEM_LIMIT (16 * 1024 * 1024) > > diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c > index 972f43f..6b370ed 100644 > --- a/drivers/media/platform/ti-vpe/vpe.c > +++ b/drivers/media/platform/ti-vpe/vpe.c > @@ -1970,7 +1970,7 @@ static const struct v4l2_ctrl_config vpe_bufs_per_job = { > .type = V4L2_CTRL_TYPE_INTEGER, > .def = VPE_DEF_BUFS_PER_JOB, > .min = 1, > - .max = VIDEO_MAX_FRAME, > + .max = VB2_MAX_FRAME, > .step = 1, > }; > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 7c4489c..09bc9bb 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -904,7 +904,7 @@ static int __reqbufs(struct vb2_queue *q, struct v4l2_requestbuffers *req) > /* > * Make sure the requested values and current defaults are sane. > */ > - num_buffers = min_t(unsigned int, req->count, VIDEO_MAX_FRAME); > + num_buffers = min_t(unsigned int, req->count, VB2_MAX_FRAME); > num_buffers = max_t(unsigned int, num_buffers, q->min_buffers_needed); > memset(q->plane_sizes, 0, sizeof(q->plane_sizes)); > memset(q->alloc_ctx, 0, sizeof(q->alloc_ctx)); > @@ -1005,7 +1005,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > unsigned int num_planes = 0, num_buffers, allocated_buffers; > int ret; > > - if (q->num_buffers == VIDEO_MAX_FRAME) { > + if (q->num_buffers == VB2_MAX_FRAME) { > dprintk(1, "maximum number of buffers already allocated\n"); > return -ENOBUFS; > } > @@ -1016,7 +1016,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > q->memory = create->memory; > } > > - num_buffers = min(create->count, VIDEO_MAX_FRAME - q->num_buffers); > + num_buffers = min(create->count, VB2_MAX_FRAME - q->num_buffers); > > /* > * Ask the driver, whether the requested number of buffers, planes per > @@ -2686,7 +2686,7 @@ struct vb2_fileio_data { > struct v4l2_requestbuffers req; > struct v4l2_plane p; > struct v4l2_buffer b; > - struct vb2_fileio_buf bufs[VIDEO_MAX_FRAME]; > + struct vb2_fileio_buf bufs[VB2_MAX_FRAME]; > unsigned int cur_index; > unsigned int initial_index; > unsigned int q_count; > diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h > index 8fab6fa..3702a33 100644 > --- a/include/media/videobuf2-core.h > +++ b/include/media/videobuf2-core.h > @@ -18,6 +18,8 @@ > #include <linux/videodev2.h> > #include <linux/dma-buf.h> > > +#define VB2_MAX_FRAME 64 > + > struct vb2_alloc_ctx; > struct vb2_fileio_data; > struct vb2_threadio_data; > @@ -395,7 +397,7 @@ struct vb2_queue { > > /* private: internal use only */ > enum v4l2_memory memory; > - struct vb2_buffer *bufs[VIDEO_MAX_FRAME]; > + struct vb2_buffer *bufs[VB2_MAX_FRAME]; > unsigned int num_buffers; > > struct list_head queued_list; > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 07/07/2014 09:27 AM, Divneil Wadhawan wrote: > Hi Hans, > > >> include/media/davinci/vpfe_capture.h > > It uses videobuf-dma-contig.h, so, I left it out. > > >> drivers/media/platform/vivi-core.c > > Cannot find this one. Checked with find, in case it changed location, but couldn't. > > >> drivers/media/pci/saa7134/* > > Updated. > > > Please find below the patch. I am hoping it's good to go. Acked-by: Hans Verkuil <hans.verkuil@cisco.com> Pawel, Kamil, can you take a look as well? To my knowledge there drivers/media/platform/s5p-mfc is the only one that uses the buffer index as a bit index (1 << v4l2_buf.index), but that driver has its own maximum buffer limit of 32 so it should not be affected by this patch. Regards, Hans > > > Regards, > > Divneil > > > From 1792d75dc0f893a181d991a0b238bbd0ead945c1 Mon Sep 17 00:00:00 2001 > From: Divneil Wadhawan <divneil.wadhawan@st.com> > Date: Mon, 7 Jul 2014 12:38:06 +0530 > Subject: [PATCH] v4l2: vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME > > - vb2 drivers to rely on VB2_MAX_FRAME. > > - VB2_MAX_FRAME bumps the value to 64 from current 32 > > Change-Id: I3d7998898df43553486166c44b54524aac449deb > Signed-off-by: Divneil Wadhawan <divneil.wadhawan@st.com> > --- > drivers/media/pci/saa7134/saa7134-ts.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-vbi.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-video.c | 2 +- > drivers/media/platform/mem2mem_testdev.c | 2 +- > drivers/media/platform/ti-vpe/vpe.c | 2 +- > drivers/media/v4l2-core/videobuf2-core.c | 8 ++++---- > include/media/videobuf2-core.h | 4 +++- > 7 files changed, 14 insertions(+), 12 deletions(-) > > diff --git a/drivers/media/pci/saa7134/saa7134-ts.c b/drivers/media/pci/saa7134/saa7134-ts.c > index bd25323..0d04995 100644 > --- a/drivers/media/pci/saa7134/saa7134-ts.c > +++ b/drivers/media/pci/saa7134/saa7134-ts.c > @@ -227,8 +227,8 @@ int saa7134_ts_init1(struct saa7134_dev *dev) > /* sanitycheck insmod options */ > if (tsbufs < 2) > tsbufs = 2; > - if (tsbufs> VIDEO_MAX_FRAME) > - tsbufs = VIDEO_MAX_FRAME; > + if (tsbufs> VB2_MAX_FRAME) > + tsbufs = VB2_MAX_FRAME; > if (ts_nr_packets < 4) > ts_nr_packets = 4; > if (ts_nr_packets> 312) > diff --git a/drivers/media/pci/saa7134/saa7134-vbi.c b/drivers/media/pci/saa7134/saa7134-vbi.c > index c06dbe1..15b5860 100644 > --- a/drivers/media/pci/saa7134/saa7134-vbi.c > +++ b/drivers/media/pci/saa7134/saa7134-vbi.c > @@ -203,8 +203,8 @@ int saa7134_vbi_init1(struct saa7134_dev *dev) > > if (vbibufs < 2) > vbibufs = 2; > - if (vbibufs> VIDEO_MAX_FRAME) > - vbibufs = VIDEO_MAX_FRAME; > + if (vbibufs> VB2_MAX_FRAME) > + vbibufs = VB2_MAX_FRAME; > return 0; > } > > diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa7134-video.c > index d375999..47dda6c 100644 > --- a/drivers/media/pci/saa7134/saa7134-video.c > +++ b/drivers/media/pci/saa7134/saa7134-video.c > @@ -2032,7 +2032,7 @@ int saa7134_video_init1(struct saa7134_dev *dev) > int ret; > > /* sanitycheck insmod options */ > - if (gbuffers < 2 || gbuffers> VIDEO_MAX_FRAME) > + if (gbuffers < 2 || gbuffers> VB2_MAX_FRAME) > gbuffers = 2; > if (gbufsize> gbufsize_max) > gbufsize = gbufsize_max; > diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/platform/mem2mem_testdev.c > index 0714070..fe3235d 100644 > --- a/drivers/media/platform/mem2mem_testdev.c > +++ b/drivers/media/platform/mem2mem_testdev.c > @@ -55,7 +55,7 @@ MODULE_PARM_DESC(debug, "activates debug info"); > #define MEM2MEM_NAME "m2m-testdev" > > /* Per queue */ > -#define MEM2MEM_DEF_NUM_BUFS VIDEO_MAX_FRAME > +#define MEM2MEM_DEF_NUM_BUFS VB2_MAX_FRAME > /* In bytes, per queue */ > #define MEM2MEM_VID_MEM_LIMIT (16 * 1024 * 1024) > > diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c > index 972f43f..6b370ed 100644 > --- a/drivers/media/platform/ti-vpe/vpe.c > +++ b/drivers/media/platform/ti-vpe/vpe.c > @@ -1970,7 +1970,7 @@ static const struct v4l2_ctrl_config vpe_bufs_per_job = { > .type = V4L2_CTRL_TYPE_INTEGER, > .def = VPE_DEF_BUFS_PER_JOB, > .min = 1, > - .max = VIDEO_MAX_FRAME, > + .max = VB2_MAX_FRAME, > .step = 1, > }; > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 7c4489c..09bc9bb 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -904,7 +904,7 @@ static int __reqbufs(struct vb2_queue *q, struct v4l2_requestbuffers *req) > /* > * Make sure the requested values and current defaults are sane. > */ > - num_buffers = min_t(unsigned int, req->count, VIDEO_MAX_FRAME); > + num_buffers = min_t(unsigned int, req->count, VB2_MAX_FRAME); > num_buffers = max_t(unsigned int, num_buffers, q->min_buffers_needed); > memset(q->plane_sizes, 0, sizeof(q->plane_sizes)); > memset(q->alloc_ctx, 0, sizeof(q->alloc_ctx)); > @@ -1005,7 +1005,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > unsigned int num_planes = 0, num_buffers, allocated_buffers; > int ret; > > - if (q->num_buffers == VIDEO_MAX_FRAME) { > + if (q->num_buffers == VB2_MAX_FRAME) { > dprintk(1, "maximum number of buffers already allocated\n"); > return -ENOBUFS; > } > @@ -1016,7 +1016,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > q->memory = create->memory; > } > > - num_buffers = min(create->count, VIDEO_MAX_FRAME - q->num_buffers); > + num_buffers = min(create->count, VB2_MAX_FRAME - q->num_buffers); > > /* > * Ask the driver, whether the requested number of buffers, planes per > @@ -2686,7 +2686,7 @@ struct vb2_fileio_data { > struct v4l2_requestbuffers req; > struct v4l2_plane p; > struct v4l2_buffer b; > - struct vb2_fileio_buf bufs[VIDEO_MAX_FRAME]; > + struct vb2_fileio_buf bufs[VB2_MAX_FRAME]; > unsigned int cur_index; > unsigned int initial_index; > unsigned int q_count; > diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h > index 8fab6fa..3702a33 100644 > --- a/include/media/videobuf2-core.h > +++ b/include/media/videobuf2-core.h > @@ -18,6 +18,8 @@ > #include <linux/videodev2.h> > #include <linux/dma-buf.h> > > +#define VB2_MAX_FRAME 64 > + > struct vb2_alloc_ctx; > struct vb2_fileio_data; > struct vb2_threadio_data; > @@ -395,7 +397,7 @@ struct vb2_queue { > > /* private: internal use only */ > enum v4l2_memory memory; > - struct vb2_buffer *bufs[VIDEO_MAX_FRAME]; > + struct vb2_buffer *bufs[VB2_MAX_FRAME]; > unsigned int num_buffers; > > struct list_head queued_list; > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Hans, > From: Hans Verkuil [mailto:hverkuil@xs4all.nl] > Sent: Tuesday, July 08, 2014 9:52 AM > To: Divneil Wadhawan; linux-media@vger.kernel.org; Kamil Debski; > pawel@osciak.com > Subject: Re: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME > > On 07/07/2014 09:27 AM, Divneil Wadhawan wrote: > > Hi Hans, > > > > > >> include/media/davinci/vpfe_capture.h > > > > It uses videobuf-dma-contig.h, so, I left it out. > > > > > >> drivers/media/platform/vivi-core.c > > > > Cannot find this one. Checked with find, in case it changed location, > but couldn't. > > > > > >> drivers/media/pci/saa7134/* > > > > Updated. > > > > > > Please find below the patch. I am hoping it's good to go. > > Acked-by: Hans Verkuil <hans.verkuil@cisco.com> > > Pawel, Kamil, can you take a look as well? > > To my knowledge there drivers/media/platform/s5p-mfc is the only one > that uses > the buffer index as a bit index (1 << v4l2_buf.index), but that driver > has > its own maximum buffer limit of 32 so it should not be affected by this > patch. Yes, MFC has a hardware limit of 32 buffers. It will not be affected by this patch as it uses internal #define MFC_MAX_BUFFERS 32. Best wishes,
Hi Divneil, This patch is all messed up and doesn't apply. Check your mailer settings: it clearly replaced hard tabs by a space. Can you repost? Regards, Hans On 07/07/2014 09:27 AM, Divneil Wadhawan wrote: > Hi Hans, > > >> include/media/davinci/vpfe_capture.h > > It uses videobuf-dma-contig.h, so, I left it out. > > >> drivers/media/platform/vivi-core.c > > Cannot find this one. Checked with find, in case it changed location, but couldn't. > > >> drivers/media/pci/saa7134/* > > Updated. > > > Please find below the patch. I am hoping it's good to go. > > > Regards, > > Divneil > > > From 1792d75dc0f893a181d991a0b238bbd0ead945c1 Mon Sep 17 00:00:00 2001 > From: Divneil Wadhawan <divneil.wadhawan@st.com> > Date: Mon, 7 Jul 2014 12:38:06 +0530 > Subject: [PATCH] v4l2: vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME > > - vb2 drivers to rely on VB2_MAX_FRAME. > > - VB2_MAX_FRAME bumps the value to 64 from current 32 > > Change-Id: I3d7998898df43553486166c44b54524aac449deb > Signed-off-by: Divneil Wadhawan <divneil.wadhawan@st.com> > --- > drivers/media/pci/saa7134/saa7134-ts.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-vbi.c | 4 ++-- > drivers/media/pci/saa7134/saa7134-video.c | 2 +- > drivers/media/platform/mem2mem_testdev.c | 2 +- > drivers/media/platform/ti-vpe/vpe.c | 2 +- > drivers/media/v4l2-core/videobuf2-core.c | 8 ++++---- > include/media/videobuf2-core.h | 4 +++- > 7 files changed, 14 insertions(+), 12 deletions(-) > > diff --git a/drivers/media/pci/saa7134/saa7134-ts.c b/drivers/media/pci/saa7134/saa7134-ts.c > index bd25323..0d04995 100644 > --- a/drivers/media/pci/saa7134/saa7134-ts.c > +++ b/drivers/media/pci/saa7134/saa7134-ts.c > @@ -227,8 +227,8 @@ int saa7134_ts_init1(struct saa7134_dev *dev) > /* sanitycheck insmod options */ > if (tsbufs < 2) > tsbufs = 2; > - if (tsbufs> VIDEO_MAX_FRAME) > - tsbufs = VIDEO_MAX_FRAME; > + if (tsbufs> VB2_MAX_FRAME) > + tsbufs = VB2_MAX_FRAME; > if (ts_nr_packets < 4) > ts_nr_packets = 4; > if (ts_nr_packets> 312) > diff --git a/drivers/media/pci/saa7134/saa7134-vbi.c b/drivers/media/pci/saa7134/saa7134-vbi.c > index c06dbe1..15b5860 100644 > --- a/drivers/media/pci/saa7134/saa7134-vbi.c > +++ b/drivers/media/pci/saa7134/saa7134-vbi.c > @@ -203,8 +203,8 @@ int saa7134_vbi_init1(struct saa7134_dev *dev) > > if (vbibufs < 2) > vbibufs = 2; > - if (vbibufs> VIDEO_MAX_FRAME) > - vbibufs = VIDEO_MAX_FRAME; > + if (vbibufs> VB2_MAX_FRAME) > + vbibufs = VB2_MAX_FRAME; > return 0; > } > > diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa7134-video.c > index d375999..47dda6c 100644 > --- a/drivers/media/pci/saa7134/saa7134-video.c > +++ b/drivers/media/pci/saa7134/saa7134-video.c > @@ -2032,7 +2032,7 @@ int saa7134_video_init1(struct saa7134_dev *dev) > int ret; > > /* sanitycheck insmod options */ > - if (gbuffers < 2 || gbuffers> VIDEO_MAX_FRAME) > + if (gbuffers < 2 || gbuffers> VB2_MAX_FRAME) > gbuffers = 2; > if (gbufsize> gbufsize_max) > gbufsize = gbufsize_max; > diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/platform/mem2mem_testdev.c > index 0714070..fe3235d 100644 > --- a/drivers/media/platform/mem2mem_testdev.c > +++ b/drivers/media/platform/mem2mem_testdev.c > @@ -55,7 +55,7 @@ MODULE_PARM_DESC(debug, "activates debug info"); > #define MEM2MEM_NAME "m2m-testdev" > > /* Per queue */ > -#define MEM2MEM_DEF_NUM_BUFS VIDEO_MAX_FRAME > +#define MEM2MEM_DEF_NUM_BUFS VB2_MAX_FRAME > /* In bytes, per queue */ > #define MEM2MEM_VID_MEM_LIMIT (16 * 1024 * 1024) > > diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c > index 972f43f..6b370ed 100644 > --- a/drivers/media/platform/ti-vpe/vpe.c > +++ b/drivers/media/platform/ti-vpe/vpe.c > @@ -1970,7 +1970,7 @@ static const struct v4l2_ctrl_config vpe_bufs_per_job = { > .type = V4L2_CTRL_TYPE_INTEGER, > .def = VPE_DEF_BUFS_PER_JOB, > .min = 1, > - .max = VIDEO_MAX_FRAME, > + .max = VB2_MAX_FRAME, > .step = 1, > }; > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 7c4489c..09bc9bb 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -904,7 +904,7 @@ static int __reqbufs(struct vb2_queue *q, struct v4l2_requestbuffers *req) > /* > * Make sure the requested values and current defaults are sane. > */ > - num_buffers = min_t(unsigned int, req->count, VIDEO_MAX_FRAME); > + num_buffers = min_t(unsigned int, req->count, VB2_MAX_FRAME); > num_buffers = max_t(unsigned int, num_buffers, q->min_buffers_needed); > memset(q->plane_sizes, 0, sizeof(q->plane_sizes)); > memset(q->alloc_ctx, 0, sizeof(q->alloc_ctx)); > @@ -1005,7 +1005,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > unsigned int num_planes = 0, num_buffers, allocated_buffers; > int ret; > > - if (q->num_buffers == VIDEO_MAX_FRAME) { > + if (q->num_buffers == VB2_MAX_FRAME) { > dprintk(1, "maximum number of buffers already allocated\n"); > return -ENOBUFS; > } > @@ -1016,7 +1016,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create > q->memory = create->memory; > } > > - num_buffers = min(create->count, VIDEO_MAX_FRAME - q->num_buffers); > + num_buffers = min(create->count, VB2_MAX_FRAME - q->num_buffers); > > /* > * Ask the driver, whether the requested number of buffers, planes per > @@ -2686,7 +2686,7 @@ struct vb2_fileio_data { > struct v4l2_requestbuffers req; > struct v4l2_plane p; > struct v4l2_buffer b; > - struct vb2_fileio_buf bufs[VIDEO_MAX_FRAME]; > + struct vb2_fileio_buf bufs[VB2_MAX_FRAME]; > unsigned int cur_index; > unsigned int initial_index; > unsigned int q_count; > diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h > index 8fab6fa..3702a33 100644 > --- a/include/media/videobuf2-core.h > +++ b/include/media/videobuf2-core.h > @@ -18,6 +18,8 @@ > #include <linux/videodev2.h> > #include <linux/dma-buf.h> > > +#define VB2_MAX_FRAME 64 > + > struct vb2_alloc_ctx; > struct vb2_fileio_data; > struct vb2_threadio_data; > @@ -395,7 +397,7 @@ struct vb2_queue { > > /* private: internal use only */ > enum v4l2_memory memory; > - struct vb2_buffer *bufs[VIDEO_MAX_FRAME]; > + struct vb2_buffer *bufs[VB2_MAX_FRAME]; > unsigned int num_buffers; > > struct list_head queued_list; > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Hans, > This patch is all messed up and doesn't apply. > > Check your mailer settings: it clearly replaced hard tabs by a space. > > Can you repost? I tried to find out if I can change the mailer settings. Seems, that is the problem. I tried using mutt, but, seems the configuration is missing. If you have a simple method, on using mutt, I will send it from there, as, mutt is respecting the TAB spaces. I didn't spend much time with it for time being. Regards, Divneil -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Why dont you use git send-email ? On Mon, Jul 21, 2014 at 8:03 AM, Divneil Wadhawan <divneil@outlook.com> wrote: > > Hi Hans, > >> This patch is all messed up and doesn't apply. >> >> Check your mailer settings: it clearly replaced hard tabs by a space. >> >> Can you repost? > > I tried to find out if I can change the mailer settings. Seems, that is the problem. > > > I tried using mutt, but, seems the configuration is missing. > > If you have a simple method, on using mutt, I will send it from there, as, mutt is respecting the TAB spaces. > > I didn't spend much time with it for time being. > > > Regards, > > Divneil -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
On 07/21/2014 08:03 AM, Divneil Wadhawan wrote: > > Hi Hans, > >> This patch is all messed up and doesn't apply. >> >> Check your mailer settings: it clearly replaced hard tabs by a space. >> >> Can you repost? > > I tried to find out if I can change the mailer settings. Seems, that is the problem. > > > I tried using mutt, but, seems the configuration is missing. > > If you have a simple method, on using mutt, I will send it from there, as, mutt is respecting the TAB spaces. > > I didn't spend much time with it for time being. Ping? If you really can't get your mailer to behave, then mail it as an attachment. Regards, Hans > > > Regards, > > Divneil -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/media/pci/saa7134/saa7134-ts.c b/drivers/media/pci/saa7134/saa7134-ts.c index bd25323..0d04995 100644 --- a/drivers/media/pci/saa7134/saa7134-ts.c +++ b/drivers/media/pci/saa7134/saa7134-ts.c @@ -227,8 +227,8 @@ int saa7134_ts_init1(struct saa7134_dev *dev) /* sanitycheck insmod options */ if (tsbufs < 2) tsbufs = 2; - if (tsbufs> VIDEO_MAX_FRAME) - tsbufs = VIDEO_MAX_FRAME; + if (tsbufs> VB2_MAX_FRAME) + tsbufs = VB2_MAX_FRAME; if (ts_nr_packets < 4) ts_nr_packets = 4; if (ts_nr_packets> 312) diff --git a/drivers/media/pci/saa7134/saa7134-vbi.c b/drivers/media/pci/saa7134/saa7134-vbi.c index c06dbe1..15b5860 100644 --- a/drivers/media/pci/saa7134/saa7134-vbi.c +++ b/drivers/media/pci/saa7134/saa7134-vbi.c @@ -203,8 +203,8 @@ int saa7134_vbi_init1(struct saa7134_dev *dev) if (vbibufs < 2) vbibufs = 2; - if (vbibufs> VIDEO_MAX_FRAME) - vbibufs = VIDEO_MAX_FRAME; + if (vbibufs> VB2_MAX_FRAME) + vbibufs = VB2_MAX_FRAME; return 0; } diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa7134-video.c index d375999..47dda6c 100644 --- a/drivers/media/pci/saa7134/saa7134-video.c +++ b/drivers/media/pci/saa7134/saa7134-video.c @@ -2032,7 +2032,7 @@ int saa7134_video_init1(struct saa7134_dev *dev) int ret; /* sanitycheck insmod options */ - if (gbuffers < 2 || gbuffers> VIDEO_MAX_FRAME) + if (gbuffers < 2 || gbuffers> VB2_MAX_FRAME) gbuffers = 2; if (gbufsize> gbufsize_max) gbufsize = gbufsize_max; diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/platform/mem2mem_testdev.c index 0714070..fe3235d 100644 --- a/drivers/media/platform/mem2mem_testdev.c +++ b/drivers/media/platform/mem2mem_testdev.c @@ -55,7 +55,7 @@ MODULE_PARM_DESC(debug, "activates debug info"); #define MEM2MEM_NAME "m2m-testdev" /* Per queue */ -#define MEM2MEM_DEF_NUM_BUFS VIDEO_MAX_FRAME +#define MEM2MEM_DEF_NUM_BUFS VB2_MAX_FRAME /* In bytes, per queue */ #define MEM2MEM_VID_MEM_LIMIT (16 * 1024 * 1024) diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c index 972f43f..6b370ed 100644 --- a/drivers/media/platform/ti-vpe/vpe.c +++ b/drivers/media/platform/ti-vpe/vpe.c @@ -1970,7 +1970,7 @@ static const struct v4l2_ctrl_config vpe_bufs_per_job = { .type = V4L2_CTRL_TYPE_INTEGER, .def = VPE_DEF_BUFS_PER_JOB, .min = 1, - .max = VIDEO_MAX_FRAME, + .max = VB2_MAX_FRAME, .step = 1, }; diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 7c4489c..09bc9bb 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c @@ -904,7 +904,7 @@ static int __reqbufs(struct vb2_queue *q, struct v4l2_requestbuffers *req) /* * Make sure the requested values and current defaults are sane. */ - num_buffers = min_t(unsigned int, req->count, VIDEO_MAX_FRAME); + num_buffers = min_t(unsigned int, req->count, VB2_MAX_FRAME); num_buffers = max_t(unsigned int, num_buffers, q->min_buffers_needed); memset(q->plane_sizes, 0, sizeof(q->plane_sizes)); memset(q->alloc_ctx, 0, sizeof(q->alloc_ctx)); @@ -1005,7 +1005,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create unsigned int num_planes = 0, num_buffers, allocated_buffers; int ret; - if (q->num_buffers == VIDEO_MAX_FRAME) { + if (q->num_buffers == VB2_MAX_FRAME) { dprintk(1, "maximum number of buffers already allocated\n"); return -ENOBUFS; } @@ -1016,7 +1016,7 @@ static int __create_bufs(struct vb2_queue *q, struct v4l2_create_buffers *create q->memory = create->memory; } - num_buffers = min(create->count, VIDEO_MAX_FRAME - q->num_buffers); + num_buffers = min(create->count, VB2_MAX_FRAME - q->num_buffers); /* * Ask the driver, whether the requested number of buffers, planes per @@ -2686,7 +2686,7 @@ struct vb2_fileio_data { struct v4l2_requestbuffers req; struct v4l2_plane p; struct v4l2_buffer b; - struct vb2_fileio_buf bufs[VIDEO_MAX_FRAME]; + struct vb2_fileio_buf bufs[VB2_MAX_FRAME]; unsigned int cur_index; unsigned int initial_index; unsigned int q_count; diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h index 8fab6fa..3702a33 100644 --- a/include/media/videobuf2-core.h +++ b/include/media/videobuf2-core.h @@ -18,6 +18,8 @@ #include <linux/videodev2.h> #include <linux/dma-buf.h> +#define VB2_MAX_FRAME 64 + struct vb2_alloc_ctx; struct vb2_fileio_data; struct vb2_threadio_data; @@ -395,7 +397,7 @@ struct vb2_queue { /* private: internal use only */ enum v4l2_memory memory; - struct vb2_buffer *bufs[VIDEO_MAX_FRAME]; + struct vb2_buffer *bufs[VB2_MAX_FRAME]; unsigned int num_buffers; struct list_head queued_list;