From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lf0-f66.google.com ([209.85.215.66]:45549 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751577AbdKLQTT (ORCPT ); Sun, 12 Nov 2017 11:19:19 -0500 Subject: Re: [PATCH v4 2/5] media: dt: bindings: Add binding for NVIDIA Tegra Video Decoder Engine To: Vladimir Zapolskiy , Thierry Reding , Jonathan Hunter , Greg Kroah-Hartman , Rob Herring , Mauro Carvalho Chehab , Stephen Warren Cc: Dan Carpenter , linux-media@vger.kernel.org, linux-tegra@vger.kernel.org, devel@driverdev.osuosl.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <6492d1af-19fa-253f-2b75-2c37ccd44cbe@mleia.com> From: Dmitry Osipenko Message-ID: <619c789c-decc-ae80-aed6-c93c594847a3@gmail.com> Date: Sun, 12 Nov 2017 19:19:15 +0300 MIME-Version: 1.0 In-Reply-To: <6492d1af-19fa-253f-2b75-2c37ccd44cbe@mleia.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 11.11.2017 17:21, Vladimir Zapolskiy wrote: > Hi Dmitry, > > On 10/20/2017 12:34 AM, Dmitry Osipenko wrote: >> Add binding documentation for the Video Decoder Engine which is found >> on NVIDIA Tegra20/30/114/124/132 SoC's. >> >> Signed-off-by: Dmitry Osipenko >> --- >> .../devicetree/bindings/media/nvidia,tegra-vde.txt | 55 ++++++++++++++++++++++ >> 1 file changed, 55 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt >> >> diff --git a/Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt b/Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt >> new file mode 100644 >> index 000000000000..470237ed6fe5 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt >> @@ -0,0 +1,55 @@ >> +NVIDIA Tegra Video Decoder Engine >> + >> +Required properties: >> +- compatible : Must contain one of the following values: >> + - "nvidia,tegra20-vde" >> + - "nvidia,tegra30-vde" >> + - "nvidia,tegra114-vde" >> + - "nvidia,tegra124-vde" >> + - "nvidia,tegra132-vde" >> +- reg : Must contain an entry for each entry in reg-names. >> +- reg-names : Must include the following entries: >> + - sxe >> + - bsev >> + - mbe >> + - ppe >> + - mce >> + - tfe >> + - ppb >> + - vdma >> + - frameid > > I've already mentioned it in my review of the driver code, but the > version from v3 with a single region is more preferable. > > Also it implies that "reg-names" property will be removed. > Please see my reply to the drivers code review. >> +- iram : Must contain phandle to the mmio-sram device node that represents >> + IRAM region used by VDE. >> +- interrupts : Must contain an entry for each entry in interrupt-names. >> +- interrupt-names : Must include the following entries: >> + - sync-token >> + - bsev >> + - sxe >> +- clocks : Must include the following entries: >> + - vde >> +- resets : Must include the following entries: >> + - vde >> + >> +Example: >> + >> +video-codec@6001a000 { >> + compatible = "nvidia,tegra20-vde"; >> + reg = <0x6001a000 0x1000 /* Syntax Engine */ >> + 0x6001b000 0x1000 /* Video Bitstream Engine */ >> + 0x6001c000 0x100 /* Macroblock Engine */ >> + 0x6001c200 0x100 /* Post-processing Engine */ >> + 0x6001c400 0x100 /* Motion Compensation Engine */ >> + 0x6001c600 0x100 /* Transform Engine */ >> + 0x6001c800 0x100 /* Pixel prediction block */ >> + 0x6001ca00 0x100 /* Video DMA */ >> + 0x6001d800 0x300 /* Video frame controls */>; >> + reg-names = "sxe", "bsev", "mbe", "ppe", "mce", >> + "tfe", "ppb", "vdma", "frameid"; >> + iram = <&vde_pool>; /* IRAM region */ >> + interrupts = , /* Sync token interrupt */ >> + , /* BSE-V interrupt */ >> + ; /* SXE interrupt */ >> + interrupt-names = "sync-token", "bsev", "sxe"; >> + clocks = <&tegra_car TEGRA20_CLK_VDE>; >> + resets = <&tegra_car 61>; >> +}; >>