From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E0D9CC742D7 for ; Fri, 12 Jul 2019 21:24:21 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B22A9217D4 for ; Fri, 12 Jul 2019 21:24:21 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="aBpMNU8k"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ti.com header.i=@ti.com header.b="AZWMUvjK" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B22A9217D4 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description :Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=c47MbZ/fl7wv70S5Mfv/RbX/cql1eVopf7GCLF4Q0Pc=; b=aBpMNU8kJg8Vpv f8FSP9NgDYmsuB/Tn/Y4tkamwBUWERKvaAIJIswK9FcsnMNp6KALUq+27MEivs3gf4PmIFKRmderD u5P/k74zxaBAuR6nUAoPRZNywenVp1vIyagufK5J9SjCm8Z2ukfNjyT8CMgOh7JDBwHaqnXhmayZ4 xM6zCOMmy/boSnUZfZimYSyGZBb9We/kKs9aWfaB4mAnBLFNU7pQIVgiECtrssfU+z571pldFvoH7 O52/2wVaav2LAzxGmAV9guwmSw9m6n0Dax4nubZqJqbygg2jjS2U6dszncN6Bx/VtbjZlABUpqTay Xt5f/YIBAioy8JK0J9xQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hm31e-0001vw-Em; Fri, 12 Jul 2019 21:24:18 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hm31b-0001vY-8n for linux-arm-kernel@lists.infradead.org; Fri, 12 Jul 2019 21:24:17 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id x6CLO3cs002593; Fri, 12 Jul 2019 16:24:03 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1562966643; bh=GfAJsBGH2xX/GO8Wv9K0wODdgZXrpSTs4Vm46bwoB+0=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=AZWMUvjK8oppEegu5FFwVAs2fPLmSxA+TvqMbmgBG+dzDX0Qm6Ok8tfpKstTN1Qf5 ye4dNKI/7+24yxMbv2Xvq1wK60S1KSFDbKiH8GjNN4OYrcPZjhhU5DLlgZfNgm3rPQ ktVU1nd40I/gh2dIouvJU1nUJNAf8Y8xu05IwZ1k= Received: from DLEE105.ent.ti.com (dlee105.ent.ti.com [157.170.170.35]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id x6CLO37j075610 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 12 Jul 2019 16:24:03 -0500 Received: from DLEE107.ent.ti.com (157.170.170.37) by DLEE105.ent.ti.com (157.170.170.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1713.5; Fri, 12 Jul 2019 16:24:02 -0500 Received: from lelv0327.itg.ti.com (10.180.67.183) by DLEE107.ent.ti.com (157.170.170.37) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1713.5 via Frontend Transport; Fri, 12 Jul 2019 16:24:02 -0500 Received: from [10.250.145.87] (ileax41-snat.itg.ti.com [10.172.224.153]) by lelv0327.itg.ti.com (8.15.2/8.15.2) with ESMTP id x6CLNxjA001463; Fri, 12 Jul 2019 16:23:59 -0500 Subject: Re: [PATCH 1/3] dt-bindings: dma: Add YAML schemas for the generic DMA bindings To: Rob Herring , Maxime Ripard , Jon Hunter References: <20190711092158.14678-1-maxime.ripard@bootlin.com> From: Peter Ujfalusi Message-ID: <28a776e2-52fa-60e9-a7d9-8caeec78f1d1@ti.com> Date: Sat, 13 Jul 2019 00:27:42 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.2 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190712_142415_406935_657005B8 X-CRM114-Status: GOOD ( 33.23 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree@vger.kernel.org, Chen-Yu Tsai , Vinod Koul , "open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM" , Frank Rowand , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 11.7.2019 20.33, Rob Herring wrote: > On Thu, Jul 11, 2019 at 3:34 AM Maxime Ripard wrote: >> >> The DMA controllers and consumers have a bunch of generic properties that >> are needed in a device tree. Add a YAML schemas for those. >> >> Signed-off-by: Maxime Ripard >> --- >> .../devicetree/bindings/dma/dma-consumer.yaml | 60 +++++++++ > > This already exists in the dt-schema/schemas/dma/dma.yaml though not > the descriptions because that needs relicensing. > > Looks like we need NVidia's (Jon H) and TI's (Peter U) permission. If I'm not mistaken the new license is GPL-2.0, if so I don't see any issue, but I'll ask our legal to be sure. And one comment for the change. > >> .../bindings/dma/dma-controller.yaml | 79 ++++++++++++ >> Documentation/devicetree/bindings/dma/dma.txt | 114 +----------------- >> 3 files changed, 140 insertions(+), 113 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/dma/dma-consumer.yaml >> create mode 100644 Documentation/devicetree/bindings/dma/dma-controller.yaml >> >> diff --git a/Documentation/devicetree/bindings/dma/dma-consumer.yaml b/Documentation/devicetree/bindings/dma/dma-consumer.yaml >> new file mode 100644 >> index 000000000000..2f6315863ad1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/dma/dma-consumer.yaml >> @@ -0,0 +1,60 @@ >> +# SPDX-License-Identifier: GPL-2.0 >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/dma/dma-consumer.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: DMA Consumer Generic Binding >> + >> +maintainers: >> + - Vinod Koul >> + >> +select: true >> + >> +properties: >> + dmas: >> + description: >> + List of one or more DMA specifiers, each consisting of >> + - A phandle pointing to DMA controller node >> + - A number of integer cells, as determined by the >> + \#dma-cells property in the node referenced by phandle >> + containing DMA controller specific information. This >> + typically contains a DMA request line number or a >> + channel number, but can contain any data that is >> + required for configuring a channel. >> + >> + dma-names: >> + description: >> + Contains one identifier string for each DMA specifier in the >> + dmas property. The specific strings that can be used are defined >> + in the binding of the DMA client device. Multiple DMA >> + specifiers can be used to represent alternatives and in this >> + case the dma-names for those DMA specifiers must be identical >> + (see examples). >> + >> +dependencies: >> + dma-names: [ dmas ] >> + >> +examples: >> + - | >> + /* A device with one DMA read channel, one DMA write channel */ >> + i2c1: i2c@1 { >> + /* ... */ >> + dmas = <&dma 2>, /* read channel */ >> + <&dma 3>; /* write channel */ >> + dma-names = "rx", "tx"; >> + /* ... */ >> + }; >> + >> + - | >> + /* A single read-write channel with three alternative DMA controllers */ >> + dmas = <&dma1 5>, <&dma2 7>, <&dma3 2>; >> + dma-names = "rx-tx", "rx-tx", "rx-tx"; >> + >> + - | >> + /* A device with three channels, one of which has two alternatives */ >> + dmas = <&dma1 2>, /* read channel */ >> + <&dma1 3>, /* write channel */ >> + <&dma2 0>, /* error read */ >> + <&dma3 0>; /* alternative error read */ >> + dma-names = "rx", "tx", "error", "error"; >> diff --git a/Documentation/devicetree/bindings/dma/dma-controller.yaml b/Documentation/devicetree/bindings/dma/dma-controller.yaml >> new file mode 100644 >> index 000000000000..17c650131b78 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/dma/dma-controller.yaml >> @@ -0,0 +1,79 @@ >> +# SPDX-License-Identifier: GPL-2.0 >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/dma/dma-controller.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: DMA Controller Generic Binding >> + >> +maintainers: >> + - Vinod Koul >> + >> +description: >> + Generic binding to provide a way for a driver using DMA Engine to >> + retrieve the DMA request or channel information that goes from a >> + hardware device to a DMA controller. >> + >> +properties: >> + $nodename: >> + pattern: "^dma-controller(@.*)?$" >> + >> + "#dma-cells": >> + # minimum: 1 >> + description: >> + Used to provide DMA controller specific information. >> + >> + dma-channel-masks: >> + $ref: /schemas/types.yaml#definitions/uint32 >> + description: >> + Bitmask of available DMA channels in ascending order that are >> + not reserved by firmware and are available to the >> + kernel. i.e. first channel corresponds to LSB. >> + >> + dma-channels: >> + $ref: /schemas/types.yaml#definitions/uint32 >> + description: >> + Number of DMA channels supported by the controller. >> + >> + dma-masters: >> + $ref: /schemas/types.yaml#definitions/phandle-array >> + description: >> + DMA routers are transparent IP blocks used to route DMA request >> + lines from devices to the DMA controller. Some SoCs (like TI >> + DRA7x) have more peripherals integrated with DMA requests than >> + what the DMA controller can handle directly. >> + >> + In such a case, dma-masters is an array of phandle to the DMA >> + controllers the router can direct the signal to. It is no longer clear what is needed and what is optional anymore and if I would need to look up how a DMA router node should look like I will be in trouble, iow if I need to figure out how to describe an SoC with DMA controller and DMA event router. >> + >> + dma-requests: >> + $ref: /schemas/types.yaml#definitions/uint32 >> + description: >> + Number of DMA request signals supported by the controller. >> + >> +examples: >> + - | >> + dma: dma@48000000 { > > dma-controller@... > > This is a case where I'd like some check that the schema is actually > applied to the schema's example. > >> + compatible = "ti,omap-sdma"; >> + reg = <0x48000000 0x1000>; >> + interrupts = <0 12 0x4 >> + 0 13 0x4 >> + 0 14 0x4 >> + 0 15 0x4>; >> + #dma-cells = <1>; >> + dma-channels = <32>; >> + dma-requests = <127>; >> + dma-channel-mask = <0xfffe>; >> + }; >> + >> + - | >> + sdma_xbar: dma-router@4a002b78 { >> + compatible = "ti,dra7-dma-crossbar"; >> + reg = <0x4a002b78 0xfc>; >> + #dma-cells = <1>; >> + dma-requests = <205>; >> + ti,dma-safe-map = <0>; >> + dma-masters = <&sdma>; >> + }; >> + >> +... >> diff --git a/Documentation/devicetree/bindings/dma/dma.txt b/Documentation/devicetree/bindings/dma/dma.txt >> index eeb4e4d1771e..90a67a016a48 100644 >> --- a/Documentation/devicetree/bindings/dma/dma.txt >> +++ b/Documentation/devicetree/bindings/dma/dma.txt >> @@ -1,113 +1 @@ >> -* Generic DMA Controller and DMA request bindings >> - >> -Generic binding to provide a way for a driver using DMA Engine to retrieve the >> -DMA request or channel information that goes from a hardware device to a DMA >> -controller. >> - >> - >> -* DMA controller >> - >> -Required property: >> -- #dma-cells: Must be at least 1. Used to provide DMA controller >> - specific information. See DMA client binding below for >> - more details. >> - >> -Optional properties: >> -- dma-channels: Number of DMA channels supported by the controller. >> -- dma-requests: Number of DMA request signals supported by the >> - controller. >> -- dma-channel-mask: Bitmask of available DMA channels in ascending order >> - that are not reserved by firmware and are available to >> - the kernel. i.e. first channel corresponds to LSB. >> - >> -Example: >> - >> - dma: dma@48000000 { >> - compatible = "ti,omap-sdma"; >> - reg = <0x48000000 0x1000>; >> - interrupts = <0 12 0x4 >> - 0 13 0x4 >> - 0 14 0x4 >> - 0 15 0x4>; >> - #dma-cells = <1>; >> - dma-channels = <32>; >> - dma-requests = <127>; >> - dma-channel-mask = <0xfffe> >> - }; >> - >> -* DMA router >> - >> -DMA routers are transparent IP blocks used to route DMA request lines from >> -devices to the DMA controller. Some SoCs (like TI DRA7x) have more peripherals >> -integrated with DMA requests than what the DMA controller can handle directly. >> - >> -Required property: >> -- dma-masters: phandle of the DMA controller or list of phandles for >> - the DMA controllers the router can direct the signal to. >> -- #dma-cells: Must be at least 1. Used to provide DMA router specific >> - information. See DMA client binding below for more >> - details. >> - >> -Optional properties: >> -- dma-requests: Number of incoming request lines the router can handle. >> -- In the node pointed by the dma-masters: >> - - dma-requests: The router driver might need to look for this in order >> - to configure the routing. >> - >> -Example: >> - sdma_xbar: dma-router@4a002b78 { >> - compatible = "ti,dra7-dma-crossbar"; >> - reg = <0x4a002b78 0xfc>; >> - #dma-cells = <1>; >> - dma-requests = <205>; >> - ti,dma-safe-map = <0>; >> - dma-masters = <&sdma>; >> - }; >> - >> -* DMA client >> - >> -Client drivers should specify the DMA property using a phandle to the controller >> -followed by DMA controller specific data. >> - >> -Required property: >> -- dmas: List of one or more DMA specifiers, each consisting of >> - - A phandle pointing to DMA controller node >> - - A number of integer cells, as determined by the >> - #dma-cells property in the node referenced by phandle >> - containing DMA controller specific information. This >> - typically contains a DMA request line number or a >> - channel number, but can contain any data that is >> - required for configuring a channel. >> -- dma-names: Contains one identifier string for each DMA specifier in >> - the dmas property. The specific strings that can be used >> - are defined in the binding of the DMA client device. >> - Multiple DMA specifiers can be used to represent >> - alternatives and in this case the dma-names for those >> - DMA specifiers must be identical (see examples). >> - >> -Examples: >> - >> -1. A device with one DMA read channel, one DMA write channel: >> - >> - i2c1: i2c@1 { >> - ... >> - dmas = <&dma 2 /* read channel */ >> - &dma 3>; /* write channel */ >> - dma-names = "rx", "tx"; >> - ... >> - }; >> - >> -2. A single read-write channel with three alternative DMA controllers: >> - >> - dmas = <&dma1 5 >> - &dma2 7 >> - &dma3 2>; >> - dma-names = "rx-tx", "rx-tx", "rx-tx"; >> - >> -3. A device with three channels, one of which has two alternatives: >> - >> - dmas = <&dma1 2 /* read channel */ >> - &dma1 3 /* write channel */ >> - &dma2 0 /* error read */ >> - &dma3 0>; /* alternative error read */ >> - dma-names = "rx", "tx", "error", "error"; >> +This file has been moved to dma-controller.yaml. >> -- >> 2.21.0 >> -- Peter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel