From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from devils.ext.ti.com ([198.47.26.153]:43669 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751496Ab3LCKIp (ORCPT ); Tue, 3 Dec 2013 05:08:45 -0500 Message-ID: <529DADAA.1060501@ti.com> Date: Tue, 3 Dec 2013 15:38:42 +0530 From: Archit Taneja MIME-Version: 1.0 To: Laurent Pinchart CC: , "linux-omap@vger.kernel.org" Subject: Re: [PATCH 6/6] experimental: arm: dts: dra7xx: Add a DT node for VPE References: <1375452223-30524-1-git-send-email-archit@ti.com> <1375452223-30524-7-git-send-email-archit@ti.com> <16994189.3ZRYT6i8Y5@avalon> In-Reply-To: <16994189.3ZRYT6i8Y5@avalon> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Laurent, On Friday 09 August 2013 03:41 AM, Laurent Pinchart wrote: > Hi Archit, > > Thank you for the patch. > > On Friday 02 August 2013 19:33:43 Archit Taneja wrote: >> Add a DT node for VPE in dra7.dtsi. This is experimental because we might >> need to split the VPE address space a bit more, and also because the IRQ >> line described is accessible the IRQ crossbar driver is added for DRA7XX. >> >> Cc: Rajendra Nayak >> Cc: Sricharan R >> Signed-off-by: Archit Taneja >> --- >> arch/arm/boot/dts/dra7.dtsi | 11 +++++++++++ > > Documentation is missing :-) As this is an experimental patch you can probably > document the bindings later. Sorry for the late reply, I missed out on reading this message somehow. I'm blocked on adding the DT nodes because of some dependencies on dra7x clocks, and the crossbar framework. I'll make sure to add documentation :) > >> 1 file changed, 11 insertions(+) >> >> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi >> index ce9a0f0..3237972 100644 >> --- a/arch/arm/boot/dts/dra7.dtsi >> +++ b/arch/arm/boot/dts/dra7.dtsi >> @@ -484,6 +484,17 @@ >> dmas = <&sdma 70>, <&sdma 71>; >> dma-names = "tx0", "rx0"; >> }; >> + >> + vpe { >> + compatible = "ti,vpe"; >> + ti,hwmods = "vpe"; >> + reg = <0x489d0000 0xd000>, <0x489dd000 0x400>; >> + reg-names = "vpe", "vpdma"; >> + interrupts = <0 159 0x4>; >> + #address-cells = <1>; >> + #size-cells = <0>; > > Are #address-cells and #size-cells really needed ? They aren't needed. The vpe node inherits these params from the parent "ocp" node. Thanks, Archit