From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754628AbdCILNl (ORCPT ); Thu, 9 Mar 2017 06:13:41 -0500 Received: from lb1-smtp-cloud2.xs4all.net ([194.109.24.21]:33436 "EHLO lb1-smtp-cloud2.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754371AbdCIKz2 (ORCPT ); Thu, 9 Mar 2017 05:55:28 -0500 Subject: Re: [PATCH 00/10] ARM: davinci: add vpif display support To: Bartosz Golaszewski , Kevin Hilman , Sekhar Nori , Patrick Titiano , Michael Turquette , Rob Herring , Mark Rutland , Russell King , Alexandre Bailon , David Lechner , Laurent Pinchart , Mauro Carvalho Chehab , Lad Prabhakar References: <1486485683-11427-1-git-send-email-bgolaszewski@baylibre.com> Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org From: Hans Verkuil Message-ID: <77ced65d-e350-02ba-6dae-b20dcb3faa32@xs4all.nl> Date: Thu, 9 Mar 2017 11:53:49 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <1486485683-11427-1-git-send-email-bgolaszewski@baylibre.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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(-) > From mboxrd@z Thu Jan 1 00:00:00 1970 From: hverkuil@xs4all.nl (Hans Verkuil) Date: Thu, 9 Mar 2017 11:53:49 +0100 Subject: [PATCH 00/10] ARM: davinci: add vpif display support In-Reply-To: <1486485683-11427-1-git-send-email-bgolaszewski@baylibre.com> References: <1486485683-11427-1-git-send-email-bgolaszewski@baylibre.com> Message-ID: <77ced65d-e350-02ba-6dae-b20dcb3faa32@xs4all.nl> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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(-) >