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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <77ced65d-e350-02ba-6dae-b20dcb3faa32@xs4all.nl>
Date:   Thu, 9 Mar 2017 11:53:49 +0100
From:   Hans Verkuil <hverkuil@...all.nl>
To:     Bartosz Golaszewski <bgolaszewski@...libre.com>,
        Kevin Hilman <khilman@...nel.org>,
        Sekhar Nori <nsekhar@...com>,
        Patrick Titiano <ptitiano@...libre.com>,
        Michael Turquette <mturquette@...libre.com>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Russell King <linux@...linux.org.uk>,
        Alexandre Bailon <abailon@...libre.com>,
        David Lechner <david@...hnology.com>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Lad Prabhakar <prabhakar.csengg@...il.com>
Cc:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-media@...r.kernel.org
Subject: Re: [PATCH 00/10] ARM: davinci: add vpif display support

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(-)
>