From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH v3 2/2] ARM: dts: tegra20: Add video decoder node Date: Thu, 12 Oct 2017 13:51:00 +0300 Message-ID: <6fe375ea-4d8d-5dc2-2d17-c13aa1f46d71@gmail.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jon Hunter , Thierry Reding , Greg Kroah-Hartman , Rob Herring , Mauro Carvalho Chehab , Stephen Warren Cc: Dan Carpenter , linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 12.10.2017 11:49, Jon Hunter wrote: > > On 11/10/17 21:08, Dmitry Osipenko wrote: >> Add a device node for the video decoder engine found on Tegra20. >> >> Signed-off-by: Dmitry Osipenko >> --- >> arch/arm/boot/dts/tegra20.dtsi | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi >> index 7c85f97f72ea..1b5d54b6c0cb 100644 >> --- a/arch/arm/boot/dts/tegra20.dtsi >> +++ b/arch/arm/boot/dts/tegra20.dtsi >> @@ -249,6 +249,23 @@ >> */ >> }; >> >> + vde@6001a000 { >> + compatible = "nvidia,tegra20-vde"; >> + reg = <0x6001a000 0x3D00 /* VDE registers */ >> + 0x40000400 0x3FC00>; /* IRAM region */ >> + reg-names = "regs", "iram"; >> + interrupts = , /* UCQ error interrupt */ >> + , /* Sync token interrupt */ >> + , /* BSE-V interrupt */ >> + , /* BSE-A interrupt */ >> + ; /* SXE interrupt */ >> + interrupt-names = "ucq-error", "sync-token", "bsev", "bsea", "sxe"; >> + clocks = <&tegra_car TEGRA20_CLK_VDE>; >> + clock-names = "vde"; >> + resets = <&tegra_car 61>; >> + reset-names = "vde"; >> + }; >> + > > I don't see any binding documentation for this node. We need to make > sure we add this. > It's in the first patch. +++ b/Documentation/devicetree/bindings/arm/tegra/nvidia,tegra20-vde.txt