Re: [STLinux Kernel] [PATCH v4 0/3] support of v4l2 encoder for STMicroelectronics SOC

2016-08-25 Thread Peter Griffin
Hi Jean-Christophe, On Mon, 25 Jul 2016, Jean-Christophe Trotin wrote: > version 4: > - Module renamed "st-hva" as suggested by Hans > - resource_size() inline function used to calculate the esram size > - V4L2 compliance successfully passed with this version (see report below) For the series:

Re: [PATCH v4 0/3] support of v4l2 encoder for STMicroelectronics SOC

2016-08-23 Thread Jean Christophe TROTIN
Hi Hans, I apologize for the delay in answering, but I was on vacation. Thank you to have pointed out the compilation problem below. I've corrected it (unsigned long is now used for dma_attrs). I've still have few points to fix, and I will propose a new set of patches beginning of next week.

Re: [PATCH v4 0/3] support of v4l2 encoder for STMicroelectronics SOC

2016-08-12 Thread Hans Verkuil
When I compile this driver using COMPILE_TEST I get: In file included from drivers/media/platform/sti/hva/hva-hw.h:11:0, from drivers/media/platform/sti/hva/hva-h264.c:9: drivers/media/platform/sti/hva/hva-mem.h:22:19: error: field ‘attrs’ has incomplete type struct dma_attrs

[PATCH v4 0/3] support of v4l2 encoder for STMicroelectronics SOC

2016-07-25 Thread Jean-Christophe Trotin
version 4: - Module renamed "st-hva" as suggested by Hans - resource_size() inline function used to calculate the esram size - V4L2 compliance successfully passed with this version (see report below) version 3: - Comments from Nicolas, Javier and Hans about version 2 taken into account: -