why does aptina_pll_calculate insist on exact division?

2018-08-13 Thread Helmut Grohne
Hi, I tried using the aptina_pll_calculate for a "new" imager and ran into problems. After filling out aptina_pll_limits from the data sheet, I was having a hard time finding a valid pix_clock. Most of the ones I tried are rejected by aptina_pll_calculate for various reasons. In particular, no pix

cron job: media_tree daily build: WARNINGS

2018-08-13 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: Tue Aug 14 05:00:13 CEST 2018 media-tree git hash:da2048b7348a0be92f706ac019e022139e29495e media_build gi

Re: [PATCH v2 2/7] [media] dt-bindings: tvp5150: Add input port connectors DT bindings

2018-08-13 Thread Rob Herring
On Mon, Aug 13, 2018 at 11:25:03AM +0200, Marco Felsch wrote: > The TVP5150/1 decoders support different video input sources to their > AIP1A/B pins. > > Possible configurations are as follows: > - Analog Composite signal connected to AIP1A. > - Analog Composite signal connected to AIP1B. >

Re: [PATCH v1 2/2] v4l: Document Intel IPU3 meta data uAPI

2018-08-13 Thread Mauro Carvalho Chehab
Em Mon, 13 Aug 2018 15:42:34 +0200 Hans Verkuil escreveu: > On 15/06/18 05:29, Yong Zhi wrote: > > These meta formats are used on Intel IPU3 ImgU video queues > > to carry 3A statistics and ISP pipeline parameters. > > > > V4L2_META_FMT_IPU3_3A > > V4L2_META_FMT_IPU3_PARAMS > > > > Signed-off-b

Re: [PATCH v2 1/6] dt-bindings: Document the Rockchip VPU bindings

2018-08-13 Thread Rob Herring
On Thu, Aug 02, 2018 at 05:00:05PM -0300, Ezequiel Garcia wrote: > Add devicetree binding documentation for Rockchip Video Processing > Unit IP. > > Signed-off-by: Ezequiel Garcia > --- > .../bindings/media/rockchip-vpu.txt | 30 +++ > 1 file changed, 30 insertions(+) >

[BUG, RFC] media: Wrong module gets acquired

2018-08-13 Thread petrcvekcz
From: Petr Cvek When transferring a media sensor driver from the soc_camera I've found the controller module can get removed (which will cause a stack dump because the sensor driver depends on resources from the controller driver). When I've tried to remove the driver module of the sensor it sai

Re: [PATCHv17 34/34] RFC: media-requests: add debugfs node

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:26 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Keep track of the number of requests and request objects of a media > device. Helps to verify that all request-related memory is freed. > > Signed-off-by: Hans Verkuil > --- > drivers/media/media-device.c

Re: [PATCHv17 33/34] vivid: add request support

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:25 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add support for requests to vivid. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- > drivers/media/platform/vivid/vivid-core.c| 8 > drivers/media/platform/vivid/vi

Re: [PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset

2018-08-13 Thread Dmitry Osipenko
On Monday, 13 August 2018 17:50:14 MSK Thierry Reding wrote: > From: Thierry Reding > > The BSEV clock has a separate gate bit and can not be assumed to be > always enabled. Add explicit handling for the BSEV clock and reset. > > This fixes an issue on Tegra124 where the BSEV clock is not enable

Re: [PATCHv17 32/34] vivid: add mc

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:24 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add support for the media_device to vivid. This is a prerequisite > for request support. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- > drivers/media/platform/vivid/vivid-core.c

Re: [PATCHv17 31/34] vim2m: support requests

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:23 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add support for requests to vim2m. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- > drivers/media/platform/vim2m.c | 26 ++ > 1 file changed, 26 insertions(

Re: [PATCHv17 30/34] vim2m: use workqueue

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:22 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > v4l2_ctrl uses mutexes, so we can't setup a ctrl_handler in > interrupt context. Switch to a workqueue instead and drop the timer. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab Shouldn't

Re: [PATCHv17 29/34] v4l2-mem2mem: add vb2_m2m_request_queue

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:21 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > For mem2mem devices we have to make sure that v4l2_m2m_try_schedule() > is called whenever a request is queued. > > We do that by creating a vb2_m2m_request_queue() helper that should > be used instead of the 'n

Re: [PATCHv17 28/34] videobuf2-v4l2: refuse qbuf if queue uses requests or vv.

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:20 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Check if the vb2 queue uses requests, and if so refuse to > add buffers that are not part of a request. Also check for > the reverse: a vb2 queue did not use requests, and an attempt > was made to queue a buffer

Re: [PATCHv17 27/34] videobuf2-core: add uses_requests/qbuf flags

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:19 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Set the first time a buffer from a request is queued to vb2 > (uses_requests) or directly queued (uses_qbuf). > Cleared when the queue is canceled. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Ch

Re: [PATCHv17 26/34] videobuf2-v4l2: add vb2_request_queue/validate helpers

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:18 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > The generic vb2_request_validate helper function checks if > there are buffers in the request and if so, prepares (validates) > all objects in the request. > > The generic vb2_request_queue helper function queue

[PATCH 14/14] ARM: tegra: Enable SMMU for VDE on Tegra124

2018-08-13 Thread Thierry Reding
From: Thierry Reding The video decode engine can use the SMMU to use buffers that are not physically contiguous in memory. This allows better memory usage for video decoding, since fragmentation may cause contiguous allocations to fail. Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra

Re: [PATCHv17 25/34] videobuf2-core: add request helper functions

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:17 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add a new helper function to tell if a request object is a buffer. > > Add a new helper function that returns true if a media_request > contains at least one buffer. > > Signed-off-by: Hans Verkuil > --- > ..

[PATCH 12/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra20

2018-08-13 Thread Thierry Reding
From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra20.dtsi | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi index 15b73bd377f0..abb5738a0705 100644 --- a/arch/arm/boot/dts/te

[PATCH 13/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra30

2018-08-13 Thread Thierry Reding
From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra30.dtsi | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi index a6781f653310..492917d61bab 100644 --- a/arch/arm/boot/dts/te

[PATCH 08/14] staging: media: tegra-vde: Track struct device *

2018-08-13 Thread Thierry Reding
From: Thierry Reding The pointer to the struct device is frequently used, so store it in struct tegra_vde. Also, pass around a pointer to a struct tegra_vde instead of struct device in some cases to prepare for subsequent patches referencing additional data from that structure. Signed-off-by: Th

[PATCH 09/14] staging: media: tegra-vde: Add IOMMU support

2018-08-13 Thread Thierry Reding
From: Thierry Reding Implement support for using an IOMMU to map physically discontiguous buffers into contiguous I/O virtual mappings that the VDE can use. This allows importing arbitrary DMA-BUFs for use by the VDE. While at it, make sure that the device is detached from any DMA/IOMMU mapping

[PATCH 10/14] staging: media: tegra-vde: Keep VDE in reset when unused

2018-08-13 Thread Thierry Reding
From: Thierry Reding There is no point in keeping the VDE module out of reset when it is not in use. Reset it on runtime suspend. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/media/tegra-vde/te

[PATCH 11/14] ARM: tegra: Enable VDE on Tegra124

2018-08-13 Thread Thierry Reding
From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra124.dtsi | 40 + 1 file changed, 40 insertions(+) diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi index b113e47b2b2a..8fdca4723205 100644 --- a/arch/arm/

[PATCH 07/14] staging: media: tegra-vde: Add some clarifying comments

2018-08-13 Thread Thierry Reding
From: Thierry Reding Add some comments specifying what tables are being set up in VRAM. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/staging/media/tegra-vde/tegra-vde.c b/drivers/

[PATCH 05/14] staging: media: tegra-vde: Properly mark invalid entries

2018-08-13 Thread Thierry Reding
From: Thierry Reding Entries in the reference picture list are marked as invalid by setting the frame ID to 0x3f. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/tegra-vde/te

[PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset

2018-08-13 Thread Thierry Reding
From: Thierry Reding The BSEV clock has a separate gate bit and can not be assumed to be always enabled. Add explicit handling for the BSEV clock and reset. This fixes an issue on Tegra124 where the BSEV clock is not enabled by default and therefore accessing the BSEV registers will hang the CPU

[PATCH 06/14] staging: media: tegra-vde: Print out invalid FD

2018-08-13 Thread Thierry Reding
From: Thierry Reding Include the invalid file descriptor when reporting an error message to help diagnosing why importing the buffer failed. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH 02/14] staging: media: tegra-vde: Support reference picture marking

2018-08-13 Thread Thierry Reding
From: Thierry Reding Tegra114 and Tegra124 support reference picture marking, which will cause BSEV to write picture marking data to SDRAM. Make sure there is a valid destination address for that data to avoid error messages from the memory controller. Signed-off-by: Thierry Reding --- drivers

[PATCH 03/14] staging: media: tegra-vde: Prepare for interlacing support

2018-08-13 Thread Thierry Reding
From: Thierry Reding The number of frames doubles when decoding interlaced content and the structures describing the frames double in size. Take that into account to prepare for interlacing support. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 73

[PATCH 00/14] staging: media: tegra-vdea: Add Tegra124 support

2018-08-13 Thread Thierry Reding
From: Thierry Reding Hi, this set of patches perform a bit of cleanup and extend support to the VDE implementation found on Tegra114 and Tegra124. This requires adding handling for a clock and a reset for the BSEV block that is separate from the main VDE block. The new VDE revision also supports

[PATCH 04/14] staging: media: tegra-vde: Use DRM/KMS framebuffer modifiers

2018-08-13 Thread Thierry Reding
From: Thierry Reding VDE on Tegra20 through Tegra114 supports reading and writing frames in 16x16 tiled layout. Similarily, the various block-linear layouts that are supported by the GPU on Tegra124 can also be read from and written to by the Tegra124 VDE. Enable userspace to specify the desired

Re: [PATCHv17 24/34] videobuf2-v4l2: integrate with media requests

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:16 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > This implements the V4L2 part of the request support. The main > change is that vb2_qbuf and vb2_prepare_buf now have a new > media_device pointer. This required changes to several drivers > that did not use the

Re: [PATCH v1 4/5] [media] i2c: drop soc_camera code from ov9640 and switch to v4l2_async

2018-08-13 Thread jacopo mondi
Hi Petr, On Sun, Aug 12, 2018 at 03:13:39PM +0200, Petr Cvek wrote: > Dne 10.8.2018 v 09:51 jacopo mondi napsal(a): > > Hi Petr, > > > > On Thu, Aug 09, 2018 at 03:39:48AM +0200, petrcve...@gmail.com wrote: > >> From: Petr Cvek > >> > >> This patch removes the dependency on an obsoleted soc_came

Re: [PATCH v1 2/2] v4l: Document Intel IPU3 meta data uAPI

2018-08-13 Thread Hans Verkuil
On 15/06/18 05:29, Yong Zhi wrote: > These meta formats are used on Intel IPU3 ImgU video queues > to carry 3A statistics and ISP pipeline parameters. > > V4L2_META_FMT_IPU3_3A > V4L2_META_FMT_IPU3_PARAMS > > Signed-off-by: Yong Zhi > Signed-off-by: Chao C Li > Signed-off-by: Rajmohan Mani > -

Re: [PATCHv17 23/34] videobuf2-core: integrate with media requests

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:15 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Buffers can now be prepared or queued for a request. > > A buffer is unbound from the request at vb2_buffer_done time or > when the queue is cancelled. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carval

Re: [PATCHv17 22/34] videobuf2-core: embed media_request_object

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:14 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Make vb2_buffer a request object. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- > include/media/videobuf2-core.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/inclu

Re: [PATCHv17 21/34] vb2: add init_buffer buffer op

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:13 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > We need to initialize the request_fd field in struct vb2_v4l2_buffer > to -1 instead of the default of 0. So we need to add a new op that > is called when struct vb2_v4l2_buffer is allocated. > > Signed-off-by:

Re: [PATCHv17 20/34] videodev2.h: Add request_fd field to v4l2_buffer

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:12 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > When queuing buffers allow for passing the request that should > be associated with this buffer. > > If V4L2_BUF_FLAG_REQUEST_FD is set, then request_fd is used as > the file descriptor. > > If a buffer is stor

Re: [PATCHv17 19/34] vb2: drop VB2_BUF_STATE_PREPARED, use bool prepared/synced instead

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:11 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > The PREPARED state becomes a problem with the request API: a buffer > could be PREPARED but dequeued, or PREPARED and in state IN_REQUEST. > > PREPARED is really not a state as such, but more a property of the >

Re: [PATCHv17 18/34] davinci_vpfe: remove bogus vb2->state check

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:10 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > There is no need to check the vb2 state in the buf_prepare > callback: it can never be wrong. > > Since VB2_BUF_STATE_PREPARED will be removed in the next patch > we'll remove this unnecessary check (and use of

Re: [PATCHv17 17/34] vb2: store userspace data in vb2_v4l2_buffer

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:09 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > The userspace-provided plane data needs to be stored in > vb2_v4l2_buffer. Currently this information is applied by > __fill_vb2_buffer() which is called by the core prepare_buf > and qbuf functions, but when usi

Re: [PATCHv17 16/34] v4l2-ctrls: add v4l2_ctrl_request_hdl_find/put/ctrl_find functions

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:08 +0200 Hans Verkuil escreveu: > If a driver needs to find/inspect the controls set in a request then > it can use these functions. > > E.g. to check if a required control is set in a request use this in the > req_validate() implementation: > > int res = -EINVAL

Re: [PATCHv17 15/34] v4l2-ctrls: support g/s_ext_ctrls for requests

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:07 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > The v4l2_g/s_ext_ctrls functions now support control handlers that > represent requests. > > The v4l2_ctrls_find_req_obj() function is responsible for finding the > request from the fd. > > Signed-off-by: Hans

Re: [PATCHv17 14/34] v4l2-ctrls: add core request support

2018-08-13 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:06 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Integrate the request support. This adds the v4l2_ctrl_request_complete > and v4l2_ctrl_request_setup functions to complete a request and (as a > helper function) to apply a request to the hardware. > > It takes

mobile apps for you

2018-08-13 Thread Jack Dike
Do you need mobile apps development? We can do it for you. We are an India base company. Here are the details about us: Years in business: 8 Staffs: 125 App developed: 350 We work on Android, iOS, Ionic, and PhoneGap platforms, we have clients across different kind of industries. Such like retai

[PATCH v2 1/5] media: ov5640: fix exposure regression

2018-08-13 Thread Hugues Fruchet
fixes: bf4a4b518c20 ("media: ov5640: Don't force the auto exposure state at start time"). Symptom was black image when capturing HD or 5Mp picture due to manual exposure set to 1 while it was intended to set autoexposure to "manual", fix this. Signed-off-by: Hugues Fruchet --- drivers/media/i2

[PATCH v2 0/5] Fix OV5640 exposure & gain

2018-08-13 Thread Hugues Fruchet
This patch serie fixes some problems around exposure & gain in OV5640 driver. The 4th patch about autocontrols requires also a fix in v4l2-ctrls.c: https://www.mail-archive.com/linux-media@vger.kernel.org/msg133164.html Here is the test procedure used for exposure & gain controls check: 1) Previe

[PATCH v2 3/5] media: ov5640: fix wrong binning value in exposure calculation

2018-08-13 Thread Hugues Fruchet
ov5640_set_mode_exposure_calc() is checking binning value but binning value read is buggy, fix this. Rename ov5640_binning_on() to ov5640_get_binning() as per other similar functions. Signed-off-by: Hugues Fruchet --- drivers/media/i2c/ov5640.c | 8 1 file changed, 4 insertions(+), 4 de

[PATCH v2 4/5] media: ov5640: fix auto controls values when switching to manual mode

2018-08-13 Thread Hugues Fruchet
When switching from auto to manual mode, V4L2 core is calling g_volatile_ctrl() in manual mode in order to get the manual initial value. Remove the manual mode check/return to not break this behaviour. Signed-off-by: Hugues Fruchet --- drivers/media/i2c/ov5640.c | 4 1 file changed, 4 delet

[PATCH v2 2/5] media: ov5640: fix auto gain & exposure when changing mode

2018-08-13 Thread Hugues Fruchet
Ensure that auto gain and auto exposure are well restored when changing mode. Signed-off-by: Hugues Fruchet --- drivers/media/i2c/ov5640.c | 96 ++ 1 file changed, 54 insertions(+), 42 deletions(-) diff --git a/drivers/media/i2c/ov5640.c b/drivers/med

[PATCH v2 5/5] media: ov5640: fix restore of last mode set

2018-08-13 Thread Hugues Fruchet
Mode setting depends on last mode set, in particular because of exposure calculation when downscale mode change between subsampling and scaling. At stream on the last mode was wrongly set to current mode, so no change was detected and exposure calculation was not made, fix this. Signed-off-by: Hug

mobile apps design

2018-08-13 Thread Jack Dike
Do you need mobile apps development? We can do it for you. We are an India base company. Here are the details about us: Years in business: 8 Staffs: 125 App developed: 350 We work on Android, iOS, Ionic, and PhoneGap platforms, we have clients across different kind of industries. Such like retai

[PATCH v2 6/7] [media] tvp5150: initialize subdev before parsing device tree

2018-08-13 Thread Marco Felsch
From: Michael Tretter There are several debug prints in the tvp5150_parse_dt() function, which do not print the prefix, because the v4l2_subdev is not initialized, yet. Initialize the v4l2_subdev before parsing the device tree to fix the debug messages. Signed-off-by: Michael Tretter Signed-of

we do mobile apps

2018-08-13 Thread Jack Dike
Do you need mobile apps development? We can do it for you. We are an India base company. Here are the details about us: Years in business: 8 Staffs: 125 App developed: 350 We work on Android, iOS, Ionic, and PhoneGap platforms, we have clients across different kind of industries. Such like retai

[PATCH v2 7/7] [media] tvp5150: add s_power callback

2018-08-13 Thread Marco Felsch
Don't en-/disable the interrupts during s_stream because someone can disable the stream but wants to get informed if the stream is locked again. So keep the interrupts enabled the whole time the pipeline is opened. Signed-off-by: Marco Felsch --- drivers/media/i2c/tvp5150.c | 23

[PATCH v2 4/7] [media] v4l2-subdev: fix v4l2_subdev_get_try_* dependency

2018-08-13 Thread Marco Felsch
These helpers make us of the media-controller entity which is only available if the CONFIG_MEDIA_CONTROLLER is enabled. Signed-off-by: Marco Felsch --- include/media/v4l2-subdev.h | 100 ++-- 1 file changed, 50 insertions(+), 50 deletions(-) diff --git a/include/

[PATCH v2 0/7] TVP5150 fixes and new features

2018-08-13 Thread Marco Felsch
Hi, this is my v2 with the integrated reviews from my v1 [1]. Since Mauro applied the most patches from my v1 to his experimental/tvp5150-3 branch [2], this series only contains those which aren't applied. Patches I changed contain a changelog, so I will omit these here. Patch ('[media] tvp5150:

[PATCH v2 3/7] [media] v4l2-subdev: add stubs for v4l2_subdev_get_try_*

2018-08-13 Thread Marco Felsch
In case of missing CONFIG_VIDEO_V4L2_SUBDEV_API those helpers aren't available. So each driver have to add ifdefs around those helpers or add the CONFIG_VIDEO_V4L2_SUBDEV_API as dependcy. Make these helpers available in case of CONFIG_VIDEO_V4L2_SUBDEV_API isn't set to avoid ifdefs. This approach

[PATCH v2 5/7] [media] tvp5150: add FORMAT_TRY support for get/set selection handlers

2018-08-13 Thread Marco Felsch
Since commit 10d5509c8d50 ("[media] v4l2: remove g/s_crop from video ops") the 'which' field for set/get_selection must be FORMAT_ACTIVE. There is no way to try different selections. The patch adds a helper function to select the correct selection memory space (sub-device file handle or driver stat

[PATCH v2 1/7] [media] tvp5150: add input source selection of_graph support

2018-08-13 Thread Marco Felsch
This patch adds the of_graph support to describe the tvp connections. Physical the TVP5150 has three ports: AIP1A, AIP1B and YOUT. As result of discussion [1],[2] the device-tree maps these ports 1:1. The svideo connector must be conneted to port@0/endpoint@1, look at the Documentation for more inf

[PATCH v2 2/7] [media] dt-bindings: tvp5150: Add input port connectors DT bindings

2018-08-13 Thread Marco Felsch
The TVP5150/1 decoders support different video input sources to their AIP1A/B pins. Possible configurations are as follows: - Analog Composite signal connected to AIP1A. - Analog Composite signal connected to AIP1B. - Analog S-Video Y (luminance) and C (chrominance) signals connected to

[PATCH v2] media: ov5640: do not change mode if format or frame interval is unchanged

2018-08-13 Thread Hugues Fruchet
Save load of mode registers array when V4L2 client sets a format or a frame interval which selects the same mode than the current one. Signed-off-by: Hugues Fruchet --- Version 2: - Fix fuzzy image because of JPEG default format not being changed according to new format selected, fix this.

Re: [PATCH v3 2/2] media: rc: remove ir-rx51 in favour of generic pwm-ir-tx

2018-08-13 Thread Sean Young
On Fri, Jul 13, 2018 at 05:38:25PM +0300, Ivaylo Dimitrov wrote: > Hi, > > On 13.07.2018 15:22, Sean Young wrote: > > The ir-rx51 is a pwm-based TX driver specific to the N900. This can be > > handled entirely by the generic pwm-ir-tx driver. > > > > Note that the suspend code in the ir-rx51 driv

Re: [PATCH 00/21] V4L2 fwnode rework; support for default configuration

2018-08-13 Thread Sakari Ailus
Hi Jacopo, On Fri, Aug 10, 2018 at 12:38:57PM +0200, jacopo mondi wrote: > Hi Sakari, >thanks for this nice rework > > On Mon, Jul 23, 2018 at 04:46:45PM +0300, Sakari Ailus wrote: > > Hello everyone, > > > > I've long thought the V4L2 fwnode framework requires some work (it's buggy > > and i