Re: [PATCH 00/10] ARM: davinci: add vpif display support

2017-03-09 Thread Bartosz Golaszewski
2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverk...@xs4all.nl>: > On 07/02/17 17:41, Bartosz Golaszewski wrote: >> The following series adds support for v4l2 display on da850-evm with >> a UI board in device tree boot mode. > > As far as I could tell from the comments

Re: [PATCH] media: vpif: use a configurable i2c_adapter_id for vpif display

2017-03-08 Thread Bartosz Golaszewski
2017-03-07 18:12 GMT+01:00 Lad, Prabhakar <prabhakar.cse...@gmail.com>: > Hi Bartosz, > > Thanks for the patch. > > On Thu, Feb 16, 2017 at 6:08 PM, Bartosz Golaszewski > <bgolaszew...@baylibre.com> wrote: >> >> The vpif display driver uses a static i

Re: [PATCH] media: vpif: use a configurable i2c_adapter_id for vpif display

2017-03-07 Thread Bartosz Golaszewski
2017-02-16 19:08 GMT+01:00 Bartosz Golaszewski <bgolaszew...@baylibre.com>: > The vpif display driver uses a static i2c adapter ID of 1 but on the > da850-evm board in DT boot mode the i2c adapter ID is actually 0. > > Make the adapter ID configurable like it already i

Re: [PATCH] media: vpif: request enable-gpios

2017-02-28 Thread Bartosz Golaszewski
2017-02-22 14:27 GMT+01:00 Bartosz Golaszewski <bgolaszew...@baylibre.com>: > This change is needed to make vpif capture work on the da850-evm board > where the capture function must be selected on the UI expander. > > Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.

[PATCH v3 0/2] media: dt-bindings: extend the vpif bindings

2017-02-28 Thread Bartosz Golaszewski
, so I decided to respin the series without it and send it later as a follow-up - added Rob Herring's acks Bartosz Golaszewski (2): media: dt-bindings: vpif: fix whitespace errors media: dt-bindings: vpif: extend the example with an output port .../devicetree/bindings/media/ti,da850-vp

[PATCH v3 1/2] media: dt-bindings: vpif: fix whitespace errors

2017-02-28 Thread Bartosz Golaszewski
The examples have been copied from the DT with whitespace errors. Fix them. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++-- 1 file changed, 6 inse

[PATCH v3 2/2] media: dt-bindings: vpif: extend the example with an output port

2017-02-28 Thread Bartosz Golaszewski
This makes the example more or less correspond with the da850-evm hardware setup. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> Acked-by: Rob Herring <r...@kernel.org> --- .../devicetree/bindings/media/ti,da850-vpif.txt| 40 +- 1 file

[PATCH] media: vpif: request enable-gpios

2017-02-22 Thread Bartosz Golaszewski
This change is needed to make vpif capture work on the da850-evm board where the capture function must be selected on the UI expander. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- drivers/media/platform/davinci/vpif_capture.c | 7 +++ 1 file changed, 7 inse

[PATCH] media: vpif: use a configurable i2c_adapter_id for vpif display

2017-02-16 Thread Bartosz Golaszewski
The vpif display driver uses a static i2c adapter ID of 1 but on the da850-evm board in DT boot mode the i2c adapter ID is actually 0. Make the adapter ID configurable like it already is for vpif capture. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> Acked-by: Kevin Hilman

Re: [PATCH 00/10] ARM: davinci: add vpif display support

2017-02-16 Thread Bartosz Golaszewski
2017-02-13 10:22 GMT+01:00 Sekhar Nori <nsek...@ti.com>: > Hi Bartosz, > > On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote: >> The following series adds support for v4l2 display on da850-evm with >> a UI board in device tree boot mode. >> >> P

Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port

2017-02-16 Thread Bartosz Golaszewski
2017-02-15 23:08 GMT+01:00 Rob Herring <r...@kernel.org>: > On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote: >> This makes the example more or less correspond with the da850-evm >> hardware setup. >> >> Signed-off-by: Bartosz Golas

Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm

2017-02-09 Thread Bartosz Golaszewski
2017-02-09 16:23 GMT+01:00 Sekhar Nori <nsek...@ti.com>: > On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote: >> Bartosz Golaszewski <bgolaszew...@baylibre.com> writes: >> >>> When we enable vpif capture on the da850-evm we hit a BUG_ON() becaus

Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display

2017-02-08 Thread Bartosz Golaszewski
2017-02-07 19:20 GMT+01:00 Kevin Hilman <khil...@baylibre.com>: > Bartosz Golaszewski <bgolaszew...@baylibre.com> writes: > >> Similarly to vpif capture: we need to register the vpif display driver >> and the corresponding adv7343 encoder in pdata-quirks as the

[PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors

2017-02-07 Thread Bartosz Golaszewski
Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devi

[PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port

2017-02-07 Thread Bartosz Golaszewski
This makes the example more or less correspond with the da850-evm hardware setup. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- .../devicetree/bindings/media/ti,da850-vpif.txt| 35 ++ 1 file changed, 29 insertions(+), 6 deletions(-) diff

[PATCH 06/10] ARM: davinci_all_defconfig: enable VPIF display modules

2017-02-07 Thread Bartosz Golaszewski
Enable the VPIF display module and the video encoder present on the da850-evm UI board. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/configs/davinci_all_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/configs/davinci_all_defconfig

[PATCH 05/10] ARM: dts: da850: add vpif video display pins

2017-02-07 Thread Bartosz Golaszewski
Add a new pinctrl sub-node for vpif display pins. Move VP_CLKIN3 and VP_CLKIN2 to the display node where they actually belong (vide section 35.2.2 of the da850 datasheet). Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/boot/dts/da850-evm.dts | 2 +- arch/ar

[PATCH 07/10] ARM: davinci: fix a whitespace error

2017-02-07 Thread Bartosz Golaszewski
There's a stray tab in da850_vpif_legacy_init(). Remove it. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/mach-davinci/pdata-quirks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-d

[PATCH 00/10] ARM: davinci: add vpif display support

2017-02-07 Thread Bartosz Golaszewski
display. Tested with a modified version of yavta[1] as gstreamer support for v4l2 seems to be broken and results in picture artifacts. [1] https://github.com/brgl/yavta davinci/vpif-display Bartosz Golaszewski (10): media: dt-bindings: vpif: fix whitespace errors ARM: dts: da850-evm: fix

[PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display

2017-02-07 Thread Bartosz Golaszewski
Similarly to vpif capture: we need to register the vpif display driver and the corresponding adv7343 encoder in pdata-quirks as the DT support is not complete. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/mach-davinci/pdata-quirks.

[PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display

2017-02-07 Thread Bartosz Golaszewski
The vpif display driver uses a static i2c adapter ID of 1 but on the da850-evm board in DT boot mode the i2c adapter ID is actually 0. Make the adapter ID configurable like it already is for vpif capture. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/mach-d

[PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm

2017-02-07 Thread Bartosz Golaszewski
When we enable vpif capture on the da850-evm we hit a BUG_ON() because the i2c adapter can't be found. The board file boot uses i2c adapter 1 but in the DT mode it's actually adapter 0. Drop the problematic lines. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/ar

[PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors

2017-02-07 Thread Bartosz Golaszewski
Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/boot/dts/da850-evm.dts | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts index c970b6e..94938a3 100644 --- a/arch/arm/bo

[PATCH 04/10] ARM: dts: da850-evm: add the output port to the vpif node

2017-02-07 Thread Bartosz Golaszewski
Extend the vpif node with an output port with a single channel. NOTE: this is still just hardware description - the actual driver is registered using pdata-quirks. Signed-off-by: Bartosz Golaszewski <bgolaszew...@baylibre.com> --- arch/arm/boot/dts/da850-evm.dts | 14 +++--- ar