From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from galahad.ideasonboard.com ([185.26.127.97]:56546 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750826AbdBTXlw (ORCPT ); Mon, 20 Feb 2017 18:41:52 -0500 From: Laurent Pinchart To: Ulrich Hecht Cc: Simon Horman , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org, magnus.damm@gmail.com, hans.verkuil@cisco.com, niklas.soderlund@ragnatech.se, geert@linux-m68k.org, sergei.shtylyov@cogentembedded.com Subject: Re: [PATCH v2 0/3] r8a7793 Gose video input support Date: Tue, 21 Feb 2017 01:42:15 +0200 Message-ID: <3952799.qV1GSftN8A@avalon> In-Reply-To: <20161020084911.GH4612@verge.net.au> References: <1476802943-5189-1-git-send-email-ulrich.hecht+renesas@gmail.com> <20161020084911.GH4612@verge.net.au> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-media-owner@vger.kernel.org List-ID: Hi Ulrich, On Thursday 20 Oct 2016 10:49:11 Simon Horman wrote: > On Tue, Oct 18, 2016 at 05:02:20PM +0200, Ulrich Hecht wrote: > > Hi! > > > > This is a by-the-datasheet implementation of analog and digital video > > input on the Gose board. > > > > I have tried to address all concerns raised by reviewers, with the > > exception of the composite input patch, which has been left as is for > > now. > > > > CU > > Uli > > > > > > Changes since v1: > > - r8a7793.dtsi: added VIN2 > > - modeled HDMI decoder input/output and connector > > - added "renesas,rcar-gen2-vin" compat strings > > - removed unnecessary "remote" node and aliases > > - set ADV7612 interrupt to GP4_2 > > > > Ulrich Hecht (3): > > ARM: dts: r8a7793: Enable VIN0-VIN2 > > I have queued up the above patch with Laurent and Geert's tags. > > > ARM: dts: gose: add HDMI input > > ARM: dts: gose: add composite video input > > Please address the review of the above two patches and repost. Could you please do so ? Feedback on 2/3 should be easy to handle. For 3/3, you might need to ping the DT maintainers. > > arch/arm/boot/dts/r8a7793-gose.dts | 100 +++++++++++++++++++++++++++++++ > > arch/arm/boot/dts/r8a7793.dtsi | 27 ++++++++++ > > 2 files changed, 127 insertions(+) -- Regards, Laurent Pinchart