Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752140Ab3EOFSj (ORCPT ); Wed, 15 May 2013 01:18:39 -0400 Received: from mail-we0-f172.google.com ([74.125.82.172]:59485 "EHLO mail-we0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751213Ab3EOFSh (ORCPT ); Wed, 15 May 2013 01:18:37 -0400 MIME-Version: 1.0 In-Reply-To: <4149647.zqpmlIlQDP@avalon> References: <1368528334-13595-1-git-send-email-prabhakar.csengg@gmail.com> <1368528334-13595-6-git-send-email-prabhakar.csengg@gmail.com> <4149647.zqpmlIlQDP@avalon> From: Prabhakar Lad Date: Wed, 15 May 2013 10:48:05 +0530 Message-ID: Subject: Re: [PATCH 5/5] media: i2c: tvp7002: add OF support To: Laurent Pinchart Cc: LMML , LKML , DLOS , Hans Verkuil , Mauro Carvalho Chehab , Guennadi Liakhovetski , Sylwester Nawrocki , Sakari Ailus , Grant Likely , Rob Herring , Rob Landley , devicetree-discuss@lists.ozlabs.org, linux-doc@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2730 Lines: 75 Hi Laurent, Thanks for the review. On Tue, May 14, 2013 at 9:04 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > Thank you for the patch. > > On Tuesday 14 May 2013 16:15:34 Lad Prabhakar wrote: >> From: Lad, Prabhakar >> >> add OF support for the tvp7002 driver. >> >> Signed-off-by: Lad, Prabhakar >> Cc: Hans Verkuil >> Cc: Laurent Pinchart >> Cc: Mauro Carvalho Chehab >> Cc: Guennadi Liakhovetski >> Cc: Sylwester Nawrocki >> Cc: Sakari Ailus >> Cc: Grant Likely >> Cc: Rob Herring >> Cc: Rob Landley >> Cc: devicetree-discuss@lists.ozlabs.org >> Cc: linux-doc@vger.kernel.org >> Cc: linux-kernel@vger.kernel.org >> Cc: davinci-linux-open-source@linux.davincidsp.com >> --- >> .../devicetree/bindings/media/i2c/tvp7002.txt | 42 +++++++++++++ >> drivers/media/i2c/tvp7002.c | 64 +++++++++++++++-- >> 2 files changed, 99 insertions(+), 7 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/media/i2c/tvp7002.txt >> >> diff --git a/Documentation/devicetree/bindings/media/i2c/tvp7002.txt >> b/Documentation/devicetree/bindings/media/i2c/tvp7002.txt new file mode >> 100644 >> index 0000000..1ebd8b1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/i2c/tvp7002.txt >> @@ -0,0 +1,42 @@ >> +* Texas Instruments TV7002 video decoder >> + >> +The TVP7002 device supports digitizing of video and graphics signal in RGB >> and >> +YPbPr color space. >> + >> +Required Properties : >> +- compatible : Must be "ti,tvp7002" >> + >> +- hsync-active: HSYNC Polarity configuration for endpoint. >> + >> +- vsync-active: VSYNC Polarity configuration for endpoint. >> + >> +- pclk-sample: Clock polarity of the endpoint. >> + >> +- ti,tvp7002-fid-polarity: Active-high Field ID polarity of the endpoint. >> + >> +- ti,tvp7002-sog-polarity: Sync on Green output polarity of the endpoint. > > Would it make sense to define field-active and sog-active properties in the > V4L2 bindings instead of having per-chip properties ? > yes you are right these properties need to be in the device node rather than the port node. I'll send alone this patch of the series as v2 fixing the above. Regards, --Prabhakar Lad -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/