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 file.
Jean-Christophe Trotin (2):
st-hva: encoding summary at instance release
st-hva: add debug file system
drive
, encoding...) errors
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/hva-h264.c | 6
drivers/media/platform/sti/hva/hva-hw.c | 5
drivers/media/platform/sti/hva/hva-mem.c | 5 +++-
drivers/media/platform/sti/hva/hva-v4l2.c | 49
last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig | 11 +
drivers/media/platform/sti/hva/Makefile | 1 +
drivers/media/platform/sti/hva/hva-debugfs.c | 422 +++
On 01/30/2017 06:28 PM, Mauro Carvalho Chehab wrote:
> Em Mon, 28 Nov 2016 11:30:52 +0100
> Jean-Christophe Trotin escreveu:
>
>> This patch prints unconditionnaly a short summary
>
> Why? Is this driver so broken that everyone would need an
> unconditional "short
disruption.
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. F
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
---
drivers/media/v4l2-core/v4l2-common.c | 2 +-
1 file changed, 1
ut are 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
>>
Signed-off-by: Jean-Christophe Trotin
---
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/configs/multi_v7_defconfig
+++ b/arch/arm
, encoding...) errors
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/hva-h264.c | 6
drivers/media/platform/sti/hva/hva-hw.c | 5
drivers/media/platform/sti/hva/hva-mem.c | 5 +++-
drivers/media/platform/sti/hva/hva-v4l2.c | 49
tion about the 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
last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig | 11 +
drivers/media/platform/sti/hva/Makefile | 1 +
drivers/media/platform/sti/hva/hva-debugfs.c | 422 +++
Thanks,
Acked-by: Jean-Christophe Trotin
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 needs to be set, as well. The cur
Thanks (and sorry for the delay of my answer),
Acked-by: Jean-Christophe Trotin
On 09/19/2016 08:19 AM, Colin King wrote:
> From: Colin Ian King
>
> The second check for an error on hva->lmi_err_reg appears
> to be a copy-and-paste error, it should be hva->emi_err_reg
> in
, encoding...) errors
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile| 2 +-
drivers/media/platform/sti/hva/hva-debug.c | 71 ++
drivers/media/platform/sti/hva/hva-h264.c | 6 +++
drivers/media/platform/sti
the sti directory
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 cr
last closed
instance debugfs entry.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/hva-debug.c | 417 +
drivers/media/platform/sti/hva/hva-hw.c| 39 +++
drivers/media/platform/sti/hva/hva-hw.h| 1 +
he last closed 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/plat
ing...) errors
- the performance information about the 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.
Signed-off-
nd, 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 +-
...) errors
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile| 2 +-
drivers/media/platform/sti/hva/hva-debug.c | 125 +
drivers/media/platform/sti/hva/hva-h264.c | 6 ++
drivers/media/platform/sti/hva
ng I need :-)
>
> Regards,
>
> Hans
>
> On 09/05/2016 04:06 PM, Jean-Christophe Trotin wrote:
>> version 6:
>> - "depends on HAS_DMA" added in Kconfig
>> - g/s parm only supported for output
>> - V4L2_CAP_TIMEPERFRAME capability set in g/
Add entry for the HVA driver to the MAINTAINERS file.
Signed-off-by: Jean-Christophe Trotin
---
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 20bb1d0..5939be5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5634,6 +5634,14 @@ M
test USERPTR: OK (Not Supported)
test DMABUF: Cannot test, specify --expbuf-device
Total: 46, Succeeded: 46, Failed: 0, Warnings: 7
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 en
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
Signed-off-by: Jean-Christophe Trotin
Acked-by: Peter Griffin
---
drivers/media/platform/Kc
: 1x1 only
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
Acked-by: Peter Griffin
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1050 +
drivers/media/platform/sti/hva/hva-v4l2.c | 109
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
Acked-by: Peter Griffin
---
.../devicetree/bindings/media/st,st-hva.txt| 24 ++
1 file changed, 24 insertions(+)
create
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 only
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]
: 1x1 only
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1050 +
drivers/media/platform/sti/hva/hva-v4l2.c | 109 ++-
drivers/media/platform/sti
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
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig| 14 +
dr
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
.../devicetree/bindings/media/st,st-hva.txt| 24 ++
1 file changed, 24 insertions(+)
create mode 100644 Documentation
ot Supported)
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 sup
UERY_DV_TIMINGS: OK (Not Supported)
>> test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
>> test VIDIOC_G/S_EDID: OK (Not Supported)
>>
>> Control ioctls:
>> test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
>> test VIDIOC_QUERYCTR
: 1x1 only
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/platform/sti/hva/hva-v4l2.c | 109 ++-
drivers/media/platform/sti
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
.../devicetree/bindings/media/st,st-hva.txt| 24 ++
1 file changed, 24 insertions(+)
create mode 100644 Documentation
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
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig| 14 +
dr
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: 46, Succeeded: 46, Failed: 0, Warnings: 8
Jean-Christ
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 do n
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
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig| 14 +
dr
: 1x1 only
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/platform/sti/hva/hva-v4l2.c | 109 ++-
drivers/media/platform/sti
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
.../devicetree/bindings/media/st,sti-hva.txt | 24 ++
1 file changed, 24 insertions(+)
create mode 100644 Documentation
FS/CREATE_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-devi
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
lly, I've taken into account all 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
>>
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: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
On 07/11/2016 08:57 PM, Javier Martinez Canillas wrote:
> On Mon, Jul 11, 2016 at 1:48 PM, Nicolas Dufresne
> wrote:
>> Le lundi 11 juillet 2016 à 17:14 +0200, Jean-Christophe Trotin a
>> écrit :
>
> [snip]
>
>>>
>>> Below is the v4l2-compl
: 1x1 only
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/sti/hva/Makefile |2 +-
drivers/media/platform/sti/hva/hva-h264.c | 1053 +
drivers/media/platform/sti/hva/hva-v4l2.c | 107 ++-
drivers/media/platform/sti
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
Signed-off-by: Jean-Christophe Trotin
---
drivers/media/platform/Kconfig| 14 +
dr
This patch adds DT binding documentation for STMicroelectronics hva
driver.
Signed-off-by: Yannick Fertre
Signed-off-by: Jean-Christophe Trotin
---
.../devicetree/bindings/media/st,sti-hva.txt | 24 ++
1 file changed, 24 insertions(+)
create mode 100644 Documentation
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
51 matches
Mail list logo
