cron job: media_tree daily build: OK

2018-10-10 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Thu Oct 11 05:00:11 CEST 2018 media-tree git hash:8caec72e8cbff65afa38928197bea5a393b67975 media_build

Re: [PATCH v2 0/2] Add SECO Boards CEC device driver

2018-10-10 Thread ektor5
On Wed, Oct 10, 2018 at 03:45:35PM +0200, Hans Verkuil wrote: > On 10/10/18 14:09, ektor5 wrote: > > Hi Hans, > > > > On Sat, Oct 06, 2018 at 11:54:38AM +0200, Hans Verkuil wrote: > >> Hi Ettore, > >> > >> On 10/05/2018 07:33 PM, ektor5 wrote: > >>> This series of patches aims to add CEC

Re: [PATCH v2 0/2] Add SECO Boards CEC device driver

2018-10-10 Thread Hans Verkuil
On 10/10/18 14:09, ektor5 wrote: > Hi Hans, > > On Sat, Oct 06, 2018 at 11:54:38AM +0200, Hans Verkuil wrote: >> Hi Ettore, >> >> On 10/05/2018 07:33 PM, ektor5 wrote: >>> This series of patches aims to add CEC functionalities to SECO >>> devices, in particular UDOO X86. >>> >>> The communication

Re: [PATCH 1/4] media: ov5640: fix resolution update

2018-10-10 Thread Laurent Pinchart
Hi Jacopo, On Wednesday, 10 October 2018 13:58:04 EEST jacopo mondi wrote: > Hi Sam, >thanks for the patch, I see the same issue you reported, but I > think this patch can be improved. > > (expanding the Cc list to all people involved in recent ov5640 > developemts, not just for this patch,

Re: [PATCH v2 0/2] Add SECO Boards CEC device driver

2018-10-10 Thread ektor5
Hi Hans, On Sat, Oct 06, 2018 at 11:54:38AM +0200, Hans Verkuil wrote: > Hi Ettore, > > On 10/05/2018 07:33 PM, ektor5 wrote: > > This series of patches aims to add CEC functionalities to SECO > > devices, in particular UDOO X86. > > > > The communication is achieved via Braswell SMBus

[PATCH] cec: check for non-OK/NACK conditions while claiming a LA

2018-10-10 Thread Hans Verkuil
During the configuration phase of a CEC adapter it is trying to claim a free logical address by polling. However, the code doesn't check if there were errors other than OK or NACK, those are just treated as if the poll was NACKed. Instead check for such errors and retry the poll. And if the

Re: [PATCH 1/4] media: ov5640: fix resolution update

2018-10-10 Thread jacopo mondi
Hi Sam, thanks for the patch, I see the same issue you reported, but I think this patch can be improved. (expanding the Cc list to all people involved in recent ov5640 developemts, not just for this patch, but for the whole series to look at. Copying names from another series cover letter,

Re: [PATCH] vicodec: lower minimum height to 360

2018-10-10 Thread Kieran Bingham
Hi Hans, Thank you for the patch, On 10/10/18 08:03, Hans Verkuil wrote: > Lower the minimum height to 360 to be consistent with the webcam input of > vivid. > > The 480 was rather arbitrary but it made it harder to use vivid as a source > for > encoding since the default resolution when you

Re: [PATCH 2/4] media: ov5640: fix get_light_freq on auto

2018-10-10 Thread jacopo mondi
Hi Sam, On Mon, Oct 08, 2018 at 11:48:00PM -0700, Sam Bobrowicz wrote: > Light frequency was not properly returned when in auto > mode and the detected frequency was 60Hz. Fix this. > > Signed-off-by: Sam Bobrowicz This is indeed a bugfix Acked-by: Jacopo Mondi Thanks j > --- >

Re: [PATCH 3/4] media: ov5640: Don't access ctrl regs when off

2018-10-10 Thread jacopo mondi
Hi Sam, thanks for the patch. On Mon, Oct 08, 2018 at 11:48:01PM -0700, Sam Bobrowicz wrote: > Add a check to g_volatile_ctrl to prevent trying to read > registers when the sensor is not powered. > > Signed-off-by: Sam Bobrowicz I've been carrying a similar patch in my tree. I found it is

[PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-10 Thread Sakari Ailus
While there are issues related to object lifetime management, unregister the media device first, followed immediately by other device nodes when the driver is being unbound. Only then the resources needed by the driver may be released. This is slightly safer. Signed-off-by: Sakari Ailus ---

[PATCH 2/2] ipu3-cio2: Use cio2_queues_exit

2018-10-10 Thread Sakari Ailus
The ipu3-cio2 driver has a function to tear down video devices as well as the associated video buffer queues. Use it. Signed-off-by: Sakari Ailus --- drivers/media/pci/intel/ipu3/ipu3-cio2.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH 0/2] Trivial CIO2 patches

2018-10-10 Thread Sakari Ailus
Hi, Here are two simple cio2 patches. Slightly safer module removal plus a cleanup. Sakari Ailus (2): ipu3-cio2: Unregister device nodes first, then release resources ipu3-cio2: Use cio2_queues_exit drivers/media/pci/intel/ipu3/ipu3-cio2.c | 6 ++ 1 file changed, 2 insertions(+), 4

[PATCH] vicodec: lower minimum height to 360

2018-10-10 Thread Hans Verkuil
Lower the minimum height to 360 to be consistent with the webcam input of vivid. The 480 was rather arbitrary but it made it harder to use vivid as a source for encoding since the default resolution when you load vivid is 640x360. Signed-off-by: Hans Verkuil --- diff --git

Re: [RFC] Informal meeting during ELCE to discuss userspace support for stateless codecs

2018-10-10 Thread Hans Verkuil
On 10/08/2018 01:53 PM, Hans Verkuil wrote: > Hi all, > > I would like to meet up somewhere during the ELCE to discuss userspace support > for stateless (and perhaps stateful as well?) codecs. > > It is also planned as a topic during the summit, but I would prefer to prepare > for that in