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

Hans Verkuil
2017-03-09T10:55:28Z

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 this patch series will see a
second version, so I am marking it as 'Changes Requested' in patchwork.

If I'm wrong, then please let me know.

Regards,

Hans

>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 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 whitespace errors
> media: dt-bindings: vpif: extend the example with an output port
> ARM: dts: da850-evm: add the output port to the vpif node
> ARM: dts: da850: add vpif video display pins
> ARM: davinci_all_defconfig: enable VPIF display modules
> ARM: davinci: fix a whitespace error
> ARM: davinci: fix the DT boot on da850-evm
> media: vpif: use a configurable i2c_adapter_id for vpif display
> ARM: davinci: add pdata-quirks for da850-evm vpif display
>
> .../devicetree/bindings/media/ti,da850-vpif.txt | 45 ++++++++---
> arch/arm/boot/dts/da850-evm.dts | 26 +++---
> arch/arm/boot/dts/da850.dtsi | 25 +++++-
> arch/arm/configs/davinci_all_defconfig | 2 +
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> arch/arm/mach-davinci/pdata-quirks.c | 92 ++++++++++++++++++++--
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 8 files changed, 164 insertions(+), 30 deletions(-)
>