[RFC/PATCH 1/1] v4l: Introduce sensor operation for getting interface configuration

2011-02-22 Thread Stanimir Varbanov
Introduce g_interface_parms sensor operation for getting sensor interface parameters. These parameters are needed from the host side to determine it's own configuration. Signed-off-by: Stanimir Varbanov svarba...@mm-sol.com --- include/media/v4l2-subdev.h | 42

Re: [PATCH v2 03/11] v4l: Introduce sensor operation for getting interface configuration

2011-12-02 Thread Stanimir Varbanov
Hi, Sergio This change in interface is not used from the omap4iss driver. You could drop it from the patch set, if so. On 12/01/2011 02:14 AM, Sergio Aguirre wrote: From: Stanimir Varbanov svarba...@mm-sol.com Introduce g_interface_parms sensor operation for getting sensor interface

gstreamer: v4l2videodec plugin

2016-04-11 Thread Stanimir Varbanov
Hi, I'm working on QCOM v4l2 video decoder/encoder driver and in order to test its functionalities I'm using gstreamer v4l2videodec plugin. I am able to use the v4l2videodec plugin with MMAP, now I want to try the dmabuf export from v4l2 and import dmabuf buffers to glimagesink. I upgraded gst to

Re: gstreamer: v4l2videodec plugin

2016-04-11 Thread Stanimir Varbanov
adding gstreamer-devel On 04/11/2016 03:03 PM, Stanimir Varbanov wrote: > Hi, > > I'm working on QCOM v4l2 video decoder/encoder driver and in order to > test its functionalities I'm using gstreamer v4l2videodec plugin. I am > able to use the v4l2videodec plugin with MMAP, no

Re: gstreamer: v4l2videodec plugin

2016-04-12 Thread Stanimir Varbanov
Hi Victor, On 04/11/2016 03:55 PM, Víctor M. Jáquez L. wrote: > On 04/11/16 at 03:11pm, Stanimir Varbanov wrote: >> adding gstreamer-devel >> >> On 04/11/2016 03:03 PM, Stanimir Varbanov wrote: >>> Hi, >>> >>> I'm working on QCOM v4l2 video dec

Re: gstreamer: v4l2videodec plugin

2016-04-12 Thread Stanimir Varbanov
Hi Nicolas, On 04/11/2016 07:25 PM, Nicolas Dufresne wrote: > Le lundi 11 avril 2016 à 15:11 +0300, Stanimir Varbanov a écrit : >> adding gstreamer-devel >> >> On 04/11/2016 03:03 PM, Stanimir Varbanov wrote: >>> >>> Hi, >>> >>> I'm worki

Re: gstreamer: v4l2videodec plugin

2016-04-12 Thread Stanimir Varbanov
I'm using the following pipeline: GST_GL_PLATFORM=egl GST_GL_API=gles2 gst-launch-1.0 $GSTDEBUG $GSTFILESRC ! qtdemux name=m m.video_0 ! h264parse ! v4l2video32dec capture-io-mode=dmabuf ! glimagesink I stalled on this error: eglimagememory

Re: gstreamer: v4l2videodec plugin

2016-04-28 Thread Stanimir Varbanov
On 04/15/2016 07:09 PM, Nicolas Dufresne wrote: > Le vendredi 15 avril 2016 à 11:58 -0400, Rob Clark a écrit : >> The issue is probably the YUV format, which we cannot really deal >> with >> properly in gallium.. it's a similar issue to multi-planer even if >> it >> is in a single buffer. >> >>

Re: gstreamer: v4l2videodec plugin

2016-04-29 Thread Stanimir Varbanov
cc: mesa-dev ML On 04/28/2016 02:33 PM, Stanimir Varbanov wrote: > On 04/15/2016 07:09 PM, Nicolas Dufresne wrote: >> Le vendredi 15 avril 2016 à 11:58 -0400, Rob Clark a écrit : >>> The issue is probably the YUV format, which we cannot really deal >>> with >

Re: [PATCH v2 2/2] media: Add a driver for the ov5645 camera sensor.

2016-05-20 Thread Stanimir Varbanov
Hi Todor, Thanks for the patch, few comments below. On 05/18/2016 02:50 PM, Todor Tomov wrote: > The ov5645 sensor from Omnivision supports up to 2592x1944 > and CSI2 interface. > > The driver adds support for the following modes: > - 1280x960 > - 1920x1080 > - 2592x1944 > > Output format is

Re: gstreamer: v4l2videodec plugin

2016-05-09 Thread Stanimir Varbanov
On 05/02/2016 12:48 AM, Nicolas Dufresne wrote: > Le jeudi 28 avril 2016 à 14:33 +0300, Stanimir Varbanov a écrit : >> So I'm wondering is that intensional? >> >> Depending on the answer I should make the same in the Gallium dri2 in >> dri2_from_dma_bufs(). > > I

Re: gstreamer: v4l2videodec plugin

2016-05-13 Thread Stanimir Varbanov
cc: ayaka On 04/12/2016 06:57 PM, Nicolas Dufresne wrote: > Le mardi 12 avril 2016 à 11:57 +0300, Stanimir Varbanov a écrit : >>> I'm very happy to see this report. So far, we only had report that >> this >>> element works on Freescale IMX.6 (CODA) and Exynos 4/5. >

Re: [PATCH] media: Add a driver for the ov5645 camera sensor.

2016-05-15 Thread Stanimir Varbanov
Hi Todor, On 05/12/2016 05:59 PM, Todor Tomov wrote: > The ov5645 sensor from Omnivision supports up to 2592x1944 > and CSI2 interface. > > The driver adds support for the following modes: > - 1280x960 > - 1920x1080 > - 2592x1944 > > Output format is packed 8bit UYVY. > > Signed-off-by: Todor

Re: gstreamer: v4l2videodec plugin

2016-05-14 Thread Stanimir Varbanov
On 05/14/2016 01:59 PM, Nicolas Dufresne wrote: > Le vendredi 13 mai 2016 à 11:45 +0300, Stanimir Varbanov a écrit : >> yes, of course :) >> >> Just FYI, I applied the WIP patches on my side and I'm very happy to >> report that they just works. So If you need so

[PATCH] media: v4l2-ctrls: append missing h264 profile string

2016-08-16 Thread Stanimir Varbanov
return+0x0/0x4 Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/v4l2-core/v4l2-ctrls.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index f7abfad9ad23..adc2147fcff7 100644 ---

[PATCH v6 0/9] Qualcomm video decoder/encoder driver

2017-02-07 Thread Stanimir Varbanov
previous remoteproc API). * fixed various issues. Build dependencies: - qcom_scm_set_remote_state is in linux-next - qcom mdt_loader will be soon in linux-next regards, Stan Stanimir Varbanov (9): media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management doc: DT

[PATCH v6 2/9] doc: DT: venus: binding document for Qualcomm video driver

2017-02-07 Thread Stanimir Varbanov
Add binding document for Venus video encoder/decoder driver Cc: Rob Herring <robh...@kernel.org> Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- Changes since previous v5: * dropped rproc phandle (remoteproc is not used anymo

[PATCH v6 3/9] MAINTAINERS: Add Qualcomm Venus video accelerator driver

2017-02-07 Thread Stanimir Varbanov
Add an entry for Venus video encoder/decoder accelerator driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 421adffbe376..fb2c79b567e1 100644 --- a/MAINTAINERS

[PATCH v6 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2017-02-07 Thread Stanimir Varbanov
this add functions for: - remove buffers from src/dst queue by index - remove exact buffer from src/dst queue also extends m2m API to iterate over a list of src/dst buffers in safely and non-safely manner. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers

[PATCH v6 7/9] media: venus: hfi: add Host Firmware Interface (HFI)

2017-02-07 Thread Stanimir Varbanov
for session and core initialisation. - hfi_cmds has packetization operations which preparing packets to be send from host to firmware. - hfi_msgs takes care of messages sent from firmware to the host. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platfor

[PATCH v6 4/9] media: venus: adding core part and helper functions

2017-02-07 Thread Stanimir Varbanov
loader Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/core.c | 368 +++ drivers/media/platform/qcom/venus/core.h | 303 + drivers/media/platform/qcom/venus/firmware.c | 151 +++ drivers/media/platform/qcom

[PATCH v6 5/9] media: venus: vdec: add video decoder files

2017-02-07 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/vdec.c | 1083 drivers/media/platform/qcom/venus/vdec.h | 23 + drivers/media/pl

[PATCH v6 6/9] media: venus: venc: add video encoder files

2017-02-07 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/venc.c | 1231 drivers/media/platform/qcom/venus/venc.h | 23 + drivers/media/platfor

[PATCH v6 9/9] media: venus: enable building of Venus video driver

2017-02-07 Thread Stanimir Varbanov
This adds Venus driver Makefile and changes v4l2 platform Makefile/Kconfig in order to enable building of the driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/Kconfig | 14 ++ drivers/media/platform/Ma

[PATCH v6 8/9] media: venus: hfi: add Venus HFI files

2017-02-07 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

Re: [PATCH v6 4/9] media: venus: adding core part and helper functions

2017-02-08 Thread Stanimir Varbanov
Bjorn, thanks for the comments! On 02/08/2017 01:32 AM, Bjorn Andersson wrote: > On Tue 07 Feb 05:10 PST 2017, Stanimir Varbanov wrote: > >> * firmware loader >> > > I like the way this turns out, just some style comments below. > > [..] >> diff --git

Re: metadata node

2017-01-30 Thread Stanimir Varbanov
Hi Guennadi, On 01/11/2017 11:42 AM, Guennadi Liakhovetski wrote: > Hi Laurent, > > As you know, I'm working on a project, that involves streaming metadata, > obtained from UVC payload headers to the userspace. Luckily, you have > created "metadata node" patces a while ago. The core patch has

Re: [PATCH v6 2/9] doc: DT: venus: binding document for Qualcomm video driver

2017-02-22 Thread Stanimir Varbanov
Hi Rob, On 02/22/2017 02:09 AM, Rob Herring wrote: > On Tue, Feb 07, 2017 at 03:10:17PM +0200, Stanimir Varbanov wrote: >> Add binding document for Venus video encoder/decoder driver >> >> Cc: Rob Herring <robh...@kernel.org> >> Cc: devicet...@vger.kernel.org &g

Re: [PATCH v6 2/9] doc: DT: venus: binding document for Qualcomm video driver

2017-02-23 Thread Stanimir Varbanov
On 02/22/2017 04:17 PM, Rob Herring wrote: > On Wed, Feb 22, 2017 at 3:25 AM, Stanimir Varbanov > <stanimir.varba...@linaro.org> wrote: >> Hi Rob, >> >> On 02/22/2017 02:09 AM, Rob Herring wrote: >>> On Tue, Feb 07, 2017 at 03:10:17PM +0200, Stanimir Va

Re: metadata node

2017-02-10 Thread Stanimir Varbanov
Hi Guennadi, On 02/02/2017 08:35 PM, Guennadi Liakhovetski wrote: > Hi Stanimir, > > On Mon, 30 Jan 2017, Stanimir Varbanov wrote: > >> Hi Guennadi, >> >> On 01/11/2017 11:42 AM, Guennadi Liakhovetski wrote: > > [snip] > >>> In any case, _if_

Re: [PATCH v6 9/9] media: venus: enable building of Venus video driver

2017-02-09 Thread Stanimir Varbanov
On 02/08/2017 12:38 AM, kbuild test robot wrote: > Hi Stanimir, > > [auto build test ERROR on linuxtv-media/master] > [also build test ERROR on v4.10-rc7 next-20170207] The error is not reproducible on v4.10-rc7 next-20170209! -- regards, Stan -- To unsubscribe from this list: send the line

Re: [PATCH 2/8] media: vidc: adding core part and helper functions

2016-08-22 Thread Stanimir Varbanov
Hi Hans, Thanks for the express comments! >> + >> +struct vidc_core { >> +struct list_head list; >> +void __iomem *base; >> +int irq; >> +struct clk *clks[VIDC_CLKS_NUM_MAX]; >> +struct mutex lock; >> +struct hfi_core hfi; >> +struct video_device vdev_dec; >> +

[PATCH 4/8] media: vidc: encoder: add video encoder files

2016-08-22 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/venc.c | 1261 + drivers/media/platform/qcom/vidc/venc.h | 27 + drivers/media/platform/qco

[PATCH 5/8] media: vidc: add Host Firmware Interface (HFI)

2016-08-22 Thread Stanimir Varbanov
for session and core initialisation. - hfi_cmds has packetization operations which preparing packets to be send from host to firmware. - hfi_msgs takes care of messages sent from firmware to the host. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qco

[PATCH 7/8] media: vidc: add Makefiles and Kconfig files

2016-08-22 Thread Stanimir Varbanov
Makefile and Kconfig files to build the video codec driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/Kconfig | 8 drivers/media/platform/qcom/Makefile | 6 ++ drivers/media/platform/qcom/vidc/Makefil

[PATCH 1/8] doc: DT: vidc: binding document for Qualcomm video driver

2016-08-22 Thread Stanimir Varbanov
Adds binding document for vidc video encoder/decoder driver Cc: Rob Herring <robh...@kernel.org> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- .../devicetree/bindings/media/qcom,vi

[PATCH 8/8] media: vidc: enable building of the video codec driver

2016-08-22 Thread Stanimir Varbanov
This adds changes in v4l2 platform directory to include the vidc driver and show it in kernel config. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/Kconfig | 1 + drivers/media/platform/Makefile | 1 + 2 files changed, 2 insertions(+) diff

[PATCH 0/8] Qualcomm video decoder/encoder driver

2016-08-22 Thread Stanimir Varbanov
6/8/19/570 [2] https://www.spinics.net/lists/arm-kernel/msg522505.html Stanimir Varbanov (8): doc: DT: vidc: binding document for Qualcomm video driver media: vidc: adding core part and helper functions media: vidc: decoder: add video decoder files media: vidc: encoder: add video encoder fi

[PATCH 2/8] media: vidc: adding core part and helper functions

2016-08-22 Thread Stanimir Varbanov
Interface) interface commands. - resources.c exports a structure describing the details specific to platform and SoC. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/core.c | 548 +++ drivers/media/platform/qco

[PATCH 3/8] media: vidc: decoder: add video decoder files

2016-08-22 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/vdec.c | 1100 + drivers/media/platform/qcom/vidc/vdec.h | 27 + drivers/media/pl

[PATCH 6/8] media: vidc: add Venus HFI files

2016-08-22 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

Re: [PATCH 2/8] media: vidc: adding core part and helper functions

2016-08-25 Thread Stanimir Varbanov
Hi Bjorn, Thanks for the review and comments! On 08/23/2016 05:50 AM, Bjorn Andersson wrote: > On Mon 22 Aug 06:13 PDT 2016, Stanimir Varbanov wrote: > > Hi Stan, > >> This adds core part of the vidc driver common helper functions >> used by encoder and decoder specif

[PATCH v2 0/8] Qualcomm video decoder/encoder driver

2016-09-07 Thread Stanimir Varbanov
ted) Buffer ioctls: test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK test VIDIOC_EXPBUF: OK Test input 0: Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 Stanimir Varbanov (8): doc: DT: vidc: binding document for Qualcomm video driver media: vidc: adding core p

[PATCH v2 2/8] media: vidc: adding core part and helper functions

2016-09-07 Thread Stanimir Varbanov
on the count of instances and resolutions it selects the best clock rate for the video core. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/core.c| 559 ++ drivers/media/platform/qcom/vidc/core.h

[PATCH v2 8/8] media: vidc: enable building of the video codec driver

2016-09-07 Thread Stanimir Varbanov
This adds changes in v4l2 platform directory to include the vidc driver and show it in kernel config. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/Kconfig | 1 + drivers/media/platform/Makefile | 1 + 2 files changed, 2 insertions(+) diff

Re: [PATCH 0/8] Qualcomm video decoder/encoder driver

2016-09-07 Thread Stanimir Varbanov
Hi Hans, On 09/05/2016 05:47 PM, Hans Verkuil wrote: > On 08/22/2016 03:13 PM, Stanimir Varbanov wrote: >> This patchset introduces a basic support for Qualcomm video >> acceleration hardware used for video stream decoding/encoding. >> The video IP can found on various q

[PATCH v2 4/8] media: vidc: encoder: add video encoder files

2016-09-07 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/venc.c | 1252 + drivers/media/platform/qcom/vidc/venc.h | 29 + drivers/media/platform/qco

[PATCH v2 7/8] media: vidc: add Makefiles and Kconfig files

2016-09-07 Thread Stanimir Varbanov
Makefile and Kconfig files to build the video codec driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/Kconfig | 8 drivers/media/platform/qcom/Makefile | 6 ++ drivers/media/platform/qcom/vidc/Makefil

[PATCH v2 5/8] media: vidc: add Host Firmware Interface (HFI)

2016-09-07 Thread Stanimir Varbanov
for session and core initialisation. - hfi_cmds has packetization operations which preparing packets to be send from host to firmware. - hfi_msgs takes care of messages sent from firmware to the host. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qco

[PATCH v2 3/8] media: vidc: decoder: add video decoder files

2016-09-07 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/vidc/vdec.c | 1091 + drivers/media/platform/qcom/vidc/vdec.h | 29 + drivers/media/pl

[PATCH v2 1/8] doc: DT: vidc: binding document for Qualcomm video driver

2016-09-07 Thread Stanimir Varbanov
Adds binding document for vidc video encoder/decoder driver Cc: Rob Herring <robh...@kernel.org> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- .../devicetree/bindings/media/qcom,vi

[PATCH v2 6/8] media: vidc: add Venus HFI files

2016-09-07 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

Re: [PATCH v2 1/8] doc: DT: vidc: binding document for Qualcomm video driver

2016-09-26 Thread Stanimir Varbanov
Hi Rob, Thanks for the review! On 09/16/2016 05:19 PM, Rob Herring wrote: > On Wed, Sep 07, 2016 at 02:37:02PM +0300, Stanimir Varbanov wrote: >> Adds binding document for vidc video encoder/decoder driver >> >> Cc: Rob Herring <robh...@kernel.org> >> Cc: M

Re: [PATCH 5/8] media: vidc: add Host Firmware Interface (HFI)

2016-08-26 Thread Stanimir Varbanov
Hi Bjorn, Thanks for the comments! On 08/23/2016 06:25 AM, Bjorn Andersson wrote: > On Mon 22 Aug 06:13 PDT 2016, Stanimir Varbanov wrote: > >> This is the implementation of HFI. It is loaded with the >> responsibility to comunicate with the firmware through an >

Re: [PATCH 3/8] media: vidc: decoder: add video decoder files

2016-08-23 Thread Stanimir Varbanov
Hi Hans, Thanks for the valuable comments! >> + >> +static int vdec_g_fmt(struct file *file, void *fh, struct v4l2_format *f) >> +{ >> +struct vidc_inst *inst = to_inst(file); >> +const struct vidc_format *fmt = NULL; >> +struct v4l2_pix_format_mplane *pixmp = >fmt.pix_mp; >> + >>

Re: [PATCH 3/8] media: vidc: decoder: add video decoder files

2016-08-29 Thread Stanimir Varbanov
Hi Hans, +static int vdec_start_streaming(struct vb2_queue *q, unsigned int count) +{ + struct vidc_inst *inst = vb2_get_drv_priv(q); + struct hfi_core *hfi = >core->hfi; + struct device *dev = inst->core->dev; + struct hfi_buffer_requirements bufreq; +

Re: [PATCH v2 3/8] media: vidc: decoder: add video decoder files

2016-09-28 Thread Stanimir Varbanov
Hi, On 09/19/2016 01:12 PM, Hans Verkuil wrote: > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This consists of video decoder implementation plus decoder >> controls. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> >> --- >

Re: [PATCH v2 3/8] media: vidc: decoder: add video decoder files

2016-09-28 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 09/19/2016 01:04 PM, Hans Verkuil wrote: > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This consists of video decoder implementation plus decoder >> controls. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varba...@

Re: [PATCH v2 7/8] media: vidc: add Makefiles and Kconfig files

2016-09-28 Thread Stanimir Varbanov
Hi Hans, On 09/19/2016 01:35 PM, Hans Verkuil wrote: > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> Makefile and Kconfig files to build the video codec driver. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> >> --- >> drivers

Re: [PATCH v2 4/8] media: vidc: encoder: add video encoder files

2016-09-28 Thread Stanimir Varbanov
Hi Hans, On 09/19/2016 01:15 PM, Hans Verkuil wrote: > Many of my review comments for the decoder apply to the encoder as well, > so I won't repeat those. Sure, will address them too. > > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This adds encoder part of the d

[PATCH v3 5/9] media: venus: venc: add video encoder files

2016-11-07 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/venc.c | 1212 drivers/media/platform/qcom/venus/venc.h | 32 + drivers/media/platfor

[PATCH v3 2/9] MAINTAINERS: Add Qualcomm Venus video accelerator driver

2016-11-07 Thread Stanimir Varbanov
Add an entry for Venus video encoder/decoder accelerator driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 93e9f4227c53..5c2e70e83ff5 100644 --- a/MAINTAINERS

[PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-07 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/vdec.c | 1108 drivers/media/platform/qcom/venus/vdec.h | 32 + drivers/media/pl

[PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-07 Thread Stanimir Varbanov
-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/core.c| 557 + drivers/media/platform/qcom/venus/core.h| 261 drivers/media/platform/qcom/venus/helpers.c | 612 drivers/media/pl

[PATCH v3 0/9] Qualcomm video decoder/encoder driver

2016-11-07 Thread Stanimir Varbanov
fer ioctls: test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK test VIDIOC_EXPBUF: OK Test input 0: Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 regards, Stan Stanimir Varbanov (9): doc: DT: vidc: binding document for Qualcomm video driver MAINTAINERS: Add Qualc

[PATCH v3 7/9] media: venus: hfi: add Venus HFI files

2016-11-07 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

[PATCH v3 9/9] media: venus: enable building of Venus video codec driver

2016-11-07 Thread Stanimir Varbanov
This adds changes in v4l2 platform directory to include the vidc driver and show it in kernel config. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/Kconfig | 1 + drivers/media/platform/Makefile | 1 + 2 files changed, 2 insertions(+) diff

[PATCH v3 1/9] doc: DT: vidc: binding document for Qualcomm video driver

2016-11-07 Thread Stanimir Varbanov
Add binding document for Venus video encoder/decoder driver Cc: Rob Herring <robh...@kernel.org> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- .../devicetree/bindings/media/qcom,ve

[PATCH v3 8/9] media: venus: add Makefiles and Kconfig files

2016-11-07 Thread Stanimir Varbanov
Makefile and Kconfig files to build the Venus video codec driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/Kconfig| 7 +++ drivers/media/platform/qcom/Makefile | 1 + drivers/media/platform/qcom/venus/Makefil

Re: [PATCH v3 8/9] media: venus: add Makefiles and Kconfig files

2016-11-11 Thread Stanimir Varbanov
Hi Vivek, On 11/11/2016 08:12 AM, Vivek Gautam wrote: > On Mon, Nov 7, 2016 at 11:04 PM, Stanimir Varbanov > <stanimir.varba...@linaro.org> wrote: >> Makefile and Kconfig files to build the Venus video codec driver. >> >> Signed-off-by: Stanimir Varbanov

Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver

2016-11-14 Thread Stanimir Varbanov
Hi, On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: > Hello Hans, > > On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuil wrote: >> Hi Stanimir, >> >> Overall it looks good. As you saw, I do have some comments but nothing major. >> >> One question: you use qcom as the

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Stanimir Varbanov
Hi Hans, >>> +void vidc_vb2_stop_streaming(struct vb2_queue *q) +{ + struct venus_inst *inst = vb2_get_drv_priv(q); + struct venus_core *core = inst->core; + struct device *dev = core->dev; + struct vb2_queue *other_queue; + struct vidc_buffer *buf, *n;

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:39 PM, Hans Verkuil wrote: > I made some comments about start_streaming in my review of patch 3/9, so > I am not going to repeat that here. Sure. > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> This consist

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:32 PM, Hans Verkuil wrote: > Hi Stanimir, > > Some comments: > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> * core.c has implemented the platform dirver methods, file >> operations and v4l2 registrat

Re: [PATCH v3 5/9] media: venus: venc: add video encoder files

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:43 PM, Hans Verkuil wrote: > The comments I made before about start_streaming and the use of struct > venus_ctrl > apply here as well and I won't repeat them. > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> T

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-24 Thread Stanimir Varbanov
Hi Nicolas, On 11/23/2016 10:24 PM, Nicolas Dufresne wrote: > Le lundi 21 novembre 2016 à 18:09 +0200, Stanimir Varbanov a écrit : >>>> Meanwhile I have found bigger obstacle - I cannot run multiple >> instances >>>> simultaneously. By m2m design it can exe

Re: [PATCH v3 1/9] doc: DT: vidc: binding document for Qualcomm video driver

2016-11-15 Thread Stanimir Varbanov
Hi Rob, Thanks for the comments! On 11/14/2016 07:04 PM, Rob Herring wrote: > On Mon, Nov 07, 2016 at 07:33:55PM +0200, Stanimir Varbanov wrote: >> Add binding document for Venus video encoder/decoder driver >> >> Cc: Rob Herring <robh...@kernel.org> >> Cc: M

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-17 Thread Stanimir Varbanov
Hi Hans, On 11/14/2016 12:25 PM, Hans Verkuil wrote: > On 11/14/2016 11:11 AM, Stanimir Varbanov wrote: >> Hi Hans, >> >> >> >>>>> >>>>>> +void vidc_vb2_stop_streaming(struct vb2_queue *q) >>>>>> +{ >>>>&g

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-18 Thread Stanimir Varbanov
Hi Hans, >>> + >>> +static int >>> +vdec_reqbufs(struct file *file, void *fh, struct v4l2_requestbuffers *b) >>> +{ >>> + struct vb2_queue *queue = to_vb2q(file, b->type); >>> + >>> + if (!queue) >>> + return -EINVAL; >>> + >>> + return vb2_reqbufs(queue, b); >>> +} >> >> Is there

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-11 Thread Stanimir Varbanov
Hi Stephen, Thanks for the comments! On 11/10/2016 11:43 PM, Stephen Boyd wrote: > On 11/07, Stanimir Varbanov wrote: >> diff --git a/drivers/media/platform/qcom/venus/core.c >> b/drivers/media/platform/qcom/venus/core.c >> new file mode 100644 >> index 000

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-21 Thread Stanimir Varbanov
Hi Hans, On 11/21/2016 05:04 PM, Hans Verkuil wrote: > On 18/11/16 10:11, Stanimir Varbanov wrote: >> Hi Hans, >> >>>>> + >>>>> +static int >>>>> +vdec_reqbufs(struct file *file, void *fh, struct >>>>> v4l2_requestbuff

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-21 Thread Stanimir Varbanov
On 11/21/2016 05:33 PM, Hans Verkuil wrote: > On 21/11/16 16:29, Stanimir Varbanov wrote: >> Hi Hans, >> >> On 11/21/2016 05:04 PM, Hans Verkuil wrote: >>> On 18/11/16 10:11, Stanimir Varbanov wrote: >>>> Hi Hans, >>>> >>>>>&

Re: [PATCH v2 3/8] media: vidc: decoder: add video decoder files

2016-11-03 Thread Stanimir Varbanov
Hi Hans, On 09/19/2016 01:04 PM, Hans Verkuil wrote: > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> This consists of video decoder implementation plus decoder >> controls. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> >> -

Re: [PATCH v2 3/8] media: vidc: decoder: add video decoder files

2016-11-03 Thread Stanimir Varbanov
Hi, On 11/03/2016 12:55 PM, Hans Verkuil wrote: > On 03/11/16 11:45, Stanimir Varbanov wrote: >> Hi Hans, >> >> On 09/19/2016 01:04 PM, Hans Verkuil wrote: >>> On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >>>> This consists of video decoder

[PATCH v5 9/9] media: venus: enable building of Venus video driver

2016-12-15 Thread Stanimir Varbanov
This adds Venus driver Makefile and changes v4l2 platform Makefile/Kconfig in order to enable building of the driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile

[PATCH v5 8/9] media: venus: hfi: add Venus HFI files

2016-12-15 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

[PATCH v5 7/9] media: venus: hfi: add Host Firmware Interface (HFI)

2016-12-15 Thread Stanimir Varbanov
for session and core initialisation. - hfi_cmds has packetization operations which preparing packets to be send from host to firmware. - hfi_msgs takes care of messages sent from firmware to the host. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platfor

[PATCH v5 4/9] media: venus: adding core part and helper functions

2016-12-15 Thread Stanimir Varbanov
-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/core.c| 468 + drivers/media/platform/qcom/venus/core.h| 296 + drivers/media/platform/qcom/venus/helpers.c | 621 drivers/media/platfor

[PATCH v5 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2016-12-15 Thread Stanimir Varbanov
this add functions for: - remove buffers from src/dst queue by index - remove exact buffer from src/dst queue also extends m2m API to iterate over a list of src/dst buffers in safely and non-safely manner. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers

[PATCH v5 2/9] doc: DT: venus: binding document for Qualcomm video driver

2016-12-15 Thread Stanimir Varbanov
Add binding document for Venus video encoder/decoder driver Cc: Rob Herring <robh...@kernel.org> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- .../devicetree/bindings/media/qcom,ve

[PATCH v5 0/9] Qualcomm video decoder/encoder driver

2016-12-15 Thread Stanimir Varbanov
(TRY_)DECODER_CMD: OK (Not Supported) Buffer ioctls: test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK test VIDIOC_EXPBUF: OK Test input 0: Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 regards, Stan Stanimir Varbanov (9): media: v4l2-mem2mem: extend m2m

[PATCH v5 5/9] media: venus: vdec: add video decoder files

2016-12-15 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/vdec.c | 952 + drivers/media/platform/qcom/venus/vdec.h | 32 + drivers/media/pl

[PATCH v5 6/9] media: venus: venc: add video encoder files

2016-12-15 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platform/qcom/venus/venc.c | 1100 drivers/media/platform/qcom/venus/venc.h | 32 + drivers/media/platfor

[PATCH v5 3/9] MAINTAINERS: Add Qualcomm Venus video accelerator driver

2016-12-15 Thread Stanimir Varbanov
Add an entry for Venus video encoder/decoder accelerator driver. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 52cc0775a799..9528488f538e 100644 --- a/MAINTAINERS

Re: [PATCH v4 8/9] media: venus: hfi: add Venus HFI files

2016-12-05 Thread Stanimir Varbanov
Hi Hans, On 12/05/2016 02:05 PM, Hans Verkuil wrote: > On 12/01/2016 10:03 AM, Stanimir Varbanov wrote: >> Here is the implementation of Venus video accelerator low-level >> functionality. It contanins code which setup the registers and >> startup uthe processor, all

Re: [PATCH v4 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2016-12-05 Thread Stanimir Varbanov
Hi Hans, On 12/05/2016 01:25 PM, Hans Verkuil wrote: > On 12/01/2016 10:03 AM, Stanimir Varbanov wrote: >> this add functions for: >> - remove buffers from src/dst queue by index >> - remove exact buffer from src/dst queue >> >> also extends m2m API to itera

Re: [PATCH v4 5/9] media: venus: vdec: add video decoder files

2016-12-05 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 12/05/2016 01:32 PM, Hans Verkuil wrote: > I have two comments (and the same two comments apply to the video encoder > patch > as well): > > On 12/01/2016 10:03 AM, Stanimir Varbanov wrote: >> This consists of video decoder implem

[PATCH v4 0/9] Qualcomm video decoder/encoder driver

2016-12-01 Thread Stanimir Varbanov
memory (vmem) DT properties from binding document, this will need more work. The previous version can be found at https://lkml.org/lkml/2016/11/7/554 regards, Stan Stanimir Varbanov (9): media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management doc: DT: venus: binding

[PATCH v4 7/9] media: venus: hfi: add Host Firmware Interface (HFI)

2016-12-01 Thread Stanimir Varbanov
for session and core initialisation. - hfi_cmds has packetization operations which preparing packets to be send from host to firmware. - hfi_msgs takes care of messages sent from firmware to the host. Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> --- drivers/media/platfor

[PATCH v4 8/9] media: venus: hfi: add Venus HFI files

2016-12-01 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov <stanimir.va

  1   2   3   4   >