: 1x1 only
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
.../devicetree/bindings/media/st,st-hva.txt| 24 ++
management utilities (hva-mem.c)
This patch doesn't include the support of specific codec (e.g. H.264)
video encoding: this support is part of subsequent patches.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
Streaming ioctls:
test read/write: OK (Not Supported)
test MMAP: OK
test USERPTR: OK (Not Supported)
test DMABUF: Cannot test, specify --expbuf-device
Total: 46, Succeeded: 46, Failed: 0, Warnings: 8
Jean-Christophe Troti
On 07/21/2016 11:49 AM, Hans Verkuil wrote:
>
>
> On 07/21/2016 09:30 AM, Jean Christophe TROTIN wrote:
>>
>> On 07/18/2016 01:45 PM, Hans Verkuil wrote:
>>> Hi Jean-Christophe,
>>>
>>> See my review comments below. Nothing really major, but I
ll the other comments.
All these modifications will be reflected in the version 3.
Best regards,
Jean-Christophe.
> On 07/11/2016 05:14 PM, Jean-Christophe Trotin wrote:
>> This patch adds V4L2 HVA (Hardware Video Accelerator) video encoder
>> driver for STMicroelectronics SoC
On 07/18/2016 01:55 PM, Hans Verkuil wrote:
> On 07/11/2016 05:14 PM, Jean-Christophe Trotin wrote:
>> This patch adds the H.264 video encoding capability in the V4L2 HVA
>> video encoder driver for STMicroelectronics SoC (hva-h264.c).
>>
>> The main supported
: 1x1 only
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/
management utilities (hva-mem.c)
This patch doesn't include the support of specific codec (e.g. H.264)
video encoding: this support is part of subsequent patches.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
.../devicetree/bindings/media/st,sti-hva.txt | 24 ++
BUFS/QUERYBUF: OK
test VIDIOC_EXPBUF: OK
Test input 0:
Streaming ioctls:
test read/write: OK (Not Supported)
test MMAP: OK
test USERPTR: OK (Not Supported)
test DMABUF: Cannot test, specify --expbuf-device
Total
Hi Hans,
Thank you for your comments.
I've started to take them into account.
I've got a question about V4L2_FIELD_ANY in buf_prepare (please see below).
[snip]
>> +static int hva_buf_prepare(struct vb2_buffer *vb)
>> +{
>> + struct hva_ctx *ctx = vb2_get_drv_priv(vb->vb2_queue);
>> +
On 07/11/2016 08:57 PM, Javier Martinez Canillas wrote:
> On Mon, Jul 11, 2016 at 1:48 PM, Nicolas Dufresne
> <nicolas.dufre...@gmail.com> wrote:
>> Le lundi 11 juillet 2016 à 17:14 +0200, Jean-Christophe Trotin a
>> écrit :
>
> [snip]
>
>>&g
On 07/11/2016 08:00 PM, Nicolas Dufresne wrote:
> Le lundi 11 juillet 2016 à 17:14 +0200, Jean-Christophe Trotin a écrit :
[snip]
>> +static int hva_g_fmt_stream(struct file *file, void *fh, struct v4l2_format
>> *f)
>> +{
>> +struct hva_ctx *ctx = f
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
.../devicetree/bindings/media/st,sti-hva.txt | 24 ++
DMABUF: Cannot test, specify --expbuf-device
Total: 45, Succeeded: 45, Failed: 0, Warnings: 12
Jean-Christophe Trotin (3):
Documentation: DT: add bindings for STI HVA
[media] hva: multi-format video encoder V4L2 driver
[media] hva: add H.264 video encoding support
.../devicetree/bindings
management utilities (hva-mem.c)
This patch doesn't include the support of specific codec (e.g. H.264)
video encoding: this support is part of subsequent patches.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
: 1x1 only
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/
he last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/Kconfig | 11 +
drivers/media/platform/sti/hva/Makefile
, encoding...) errors
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/hva-h264.c | 6
drivers/media/platform/sti/hva/hva-hw.c | 5
drivers/media/platform/sti/hva/hva
including the debug
information) is saved to feed, on demand, the last closed instance debugfs
entry.
These debug capabilities are mainly implemented in the hva-debugfs.c file.
Jean-Christophe Trotin (2):
st-hva: encoding summary at instance release
st-hva: add debug file system
drivers/media/platfo
On 01/30/2017 06:28 PM, Mauro Carvalho Chehab wrote:
> Em Mon, 28 Nov 2016 11:30:52 +0100
> Jean-Christophe Trotin <jean-christophe.tro...@st.com> escreveu:
>
>> This patch prints unconditionnaly a short summary
>
> Why? Is this driver so broken that everyone would nee
On 09/05/2016 10:24 AM, Hans Verkuil wrote:
> On 08/29/2016 03:21 PM, Jean-Christophe Trotin wrote:
>> This patch adds V4L2 HVA (Hardware Video Accelerator) video encoder
>> driver for STMicroelectronics SoC. It uses the V4L2 mem2mem framework.
>>
>> This patch o
: 1x1 only
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
Acked-by: Peter Griffin <peter.grif...@linaro.org>
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva
in the next version.
Regards,
JC.
Jean-Christophe TROTIN | TINA: 1667397 | Tel: +33 244027397 | Mobile: +33
624726135
STMicroelectronics
9-11 rue Pierre-Félix Delarue | 72100 Le Mans | France
ST online: www.st.com
-Original Message-
From: Peter Griffin [mailto:peter.grif...@linaro.org]
...) errors
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile| 2 +-
drivers/media/platform/sti/hva/hva-debug.c | 125 +
drivers/media/plat
on demand, the last closed instance debugfs
entry.
These debug capabilities are mainly implemented in the hva-debug.c file.
Jean-Christophe Trotin (2):
st-hva: encoding summary at instance release
st-hva: add debug file system
drivers/media/platform/sti/hva/Makefile| 2 +-
Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/hva-debug.c | 362 +
drivers/media/platform/sti/hva/hva-hw.c| 39
drivers/media/platform/sti/hva/hva-hw.
ory
makes sense. It might be part of a different patch series dealing with the "sti
coherency" (we discussed about that on IRC with Benjamin Gaignard few weeks
ago).
Regards,
Jean-Christophe.
>
> On 09/12/2016 06:01 PM, Jean-Christophe Trotin wrote:
>> This patch creates 4 static debug
he last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/hva-debug.c | 417 +
drivers/media/platform/sti/hva/hva-hw.c
instance debugfs
entry.
These debug capabilities are mainly implemented in the hva-debug.c file.
Jean-Christophe Trotin (2):
[media] st-hva: encoding summary at instance release
[media] st-hva: add debug file system
drivers/media/platform/sti/hva/Makefile| 2 +-
drivers/media/platform/sti/hva/hv
, encoding...) errors
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile| 2 +-
drivers/media/platform/sti/hva/hva-debug.c | 71 ++
drivers/me
t; test VIDIOC_G/S_EDID: OK (Not Supported)
>>
>> Control ioctls:
>> test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
>> test VIDIOC_QUERYCTRL: OK
>> test VIDIOC_G/S_CTRL: OK
>> test VIDIOC_G/S/TRY_EXT_CTRLS:
: 1x1 only
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1050 +
drivers/media/
management utilities (hva-mem.c)
This patch doesn't include the support of specific codec (e.g. H.264)
video encoding: this support is part of subsequent patches.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
.../devicetree/bindings/media/st,st-hva.txt| 24 ++
test DMABUF: Cannot test, specify --expbuf-device
Total: 46, Succeeded: 46, Failed: 0, Warnings: 8
Jean-Christophe Trotin (3):
Documentation: DT: add bindings for ST HVA
st-hva: multi-format video encoder V4L2 driver
st-hva: add H.264 video encoding support
.../devi
Thanks (and sorry for the delay of my answer),
Acked-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
On 09/19/2016 08:19 AM, Colin King wrote:
> From: Colin Ian King <colin.k...@canonical.com>
>
> The second check for an error on hva->lmi_err_reg appears
> t
Thanks,
Acked-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
On 10/14/2016 09:32 AM, Dan Carpenter wrote:
> The devm_ioremap_resource() returns error pointers, never NULL. The
> platform_get_resource() returns NULL on error, never error pointers.
> The error code
encoding (HW processing
duration, average bitrate, average framerate...)
Each time a running instance is closed, its context (including the debug
information) is saved to feed, on demand, the last closed instance debugfs
entry.
These debug capabilities are mainly implemented in the hva-debugfs.c f
he last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/Kconfig | 11 +
drivers/media/platform/sti/hva/Makefile
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/multi_v7_defconfig
b/arch/arm/configs/multi_v7_defconfig
index eb14ab6..7a15107 100644
--- a/arch/arm/c
, encoding...) errors
Signed-off-by: Yannick Fertre <yannick.fer...@st.com>
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/platform/sti/hva/hva-h264.c | 6
drivers/media/platform/sti/hva/hva-hw.c | 5
drivers/media/platform/sti/hva/hva
e you sure you want to enable it in the defconfig? I think in general
> DEBUGFS config options aren't enabled by default.
>
> Regards,
>
> Hans
>
> On 11/28/2016 11:30 AM, Jean-Christophe Trotin wrote:
>> Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com&
Correct the calculation of the rounding to nearest aligned value in
the clamp_align() function. For example, clamp_align(1277, 1, 9600, 2)
returns 1276, while it should return 1280.
Signed-off-by: Jean-Christophe Trotin <jean-christophe.tro...@st.com>
---
drivers/media/v4l2-core/v4l2-co
.
Regards,
Jean-Christophe.
On 12/16/2016 02:56 PM, Sakari Ailus wrote:
> Hi Jean-Christophe,
>
> On Fri, Dec 16, 2016 at 02:32:15PM +0100, Jean-Christophe Trotin wrote:
>> Correct the calculation of the rounding to nearest aligned value in
>> the clamp_align() function. For example
46 matches
Mail list logo
