| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <16001a6f-faed-ee14-5ade-79d5462f6d51@ti.com>
Date: Wed, 13 Feb 2019 20:52:15 -0600
From: Suman Anna <s-anna@...com>
To: Roger Quadros <rogerq@...com>, <tony@...mide.com>,
<ohad@...ery.com>, <bjorn.andersson@...aro.org>
CC: <david@...hnology.com>, <nsekhar@...com>, <t-kristo@...com>,
<nsaulnier@...com>, <jreeder@...com>, <m-karicheri2@...com>,
<woods.technical@...il.com>, <linux-omap@...r.kernel.org>,
<linux-remoteproc@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<devicetree@...r.kernel.org>
Subject: Re: [PATCH v2 01/14] dt-bindings: remoteproc: Add TI PRUSS bindings
Hi Roger,
On 2/4/19 8:22 AM, Roger Quadros wrote:
> From: Suman Anna <s-anna@...com>
>
> This patch adds the bindings for the Programmable Real-Time Unit
> and Industrial Communication Subsystem (PRU-ICSS) present on various
> SoCs such as AM33xx, AM437x, AM57xx, Keystone 66AK2G SoC, etc. It is
> present on the Davinci based OMAPL138 SoCs and K3 architecture
> based AM65x SoCs as well (not covered for now).
>
> Signed-off-by: Suman Anna <s-anna@...com>
> Signed-off-by: Roger Quadros <rogerq@...com>
> ---
> .../devicetree/bindings/soc/ti/ti,pruss.txt | 212 +++++++++++++++++++++
> 1 file changed, 212 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/soc/ti/ti,pruss.txt
>
> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.txt b/Documentation/devicetree/bindings/soc/ti/ti,pruss.txt
> new file mode 100644
> index 0000000..5ac76fd
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.txt
> @@ -0,0 +1,212 @@
> +PRU-ICSS on TI SoCs
> +===================
> +
> +The Programmable Real-Time Unit and Industrial Communication Subsystem
> +(PRU-ICSS) is present on various TI SoCs such as AM335x, AM437x, Keystone
> +66AK2G, etc. A PRUSS consists of dual 32-bit RISC cores (Programmable
> +Real-Time Units, or PRUs) with program memory and data memory.
> +
> +The programmable nature of the PRUs provide flexibility to implement
> +custom peripheral interfaces, fast real-time responses, or specialized
> +data handling. The common peripheral modules include the following,
> +
> + - Enhanced GPIO with async capture and serial support
> + - an Ethernet MII_RT module with two MII ports
> + - an MDIO port to control external Ethernet PHYs
> + - an Industrial Ethernet Peripheral (IEP) to manage/generate Industrial
> + Ethernet functions
> + - an Enhanced Capture Module (eCAP)
> + - a 16550-compatible UART to support PROFIBUS
> + - Interrupt controller with 64 input events and 10 Host interrupts.
> +
> +A shared Data RAM, if present, can be accessed by both the PRU cores. The
> +Interrupt Controller (INTC) and a CFG module are common to both the PRU
> +cores.
> +
> +Various sub-modules within a PRU-ICSS subsystem are represented as individual
> +nodes.
> +
> +PRUSS Node
> +=============
> +
> +This node represents the entire ICSS instance and the various modules are
> +contained as children. The PRUSS driver is responsible for managing the
> +common resources i.e. DRAM0, DRAM1, SHARED_RAM and CFG space.
> +
> +Required Properties:
> +--------------------
> +- compatible : should be one of,
> + "ti,am3356-pruss" for AM335x family of SoCs
> + "ti,am4376-pruss" for AM437x family of SoCs
> + "ti,am5728-pruss" for AM57xx family of SoCs
> + "ti,k2g-pruss" for 66AK2G family of SoCs
> +- reg : base address and size for each of the Data RAMs as
> + mentioned in reg-names, and in the same order as the
> + reg-names
Hmm, not sure what prompted you to deviate from the design we had on TI
SDK kernels. Your revised bindings does not allow us to use this device
for the scalability with either UIO/VFIO.
Let's sort this out before you post the next series.
regards
Suman
> +- reg-names : should contain a string(s) from among the following names,
> + each representing a specific Data RAM region. Some PRU-ICSS
> + instances on certain SoCs might not have Shared DRAM.
> + "dram0" for Data RAM0,
> + "dram1" for Data RAM1,
> + "shrdram2" for Shared Data RAM,
> +- #address-cells : should be 1
> +- #size-cells : should be 1
> +- ranges : no specific range translations required, child nodes have the
> + same address view as the parent, so should be mentioned without
> + any value for the property
> +
> +Optional Properties:
> +--------------------
> +- no-shared-ram : Should be present if the instance doesn't have Shared RAM.
> + e.g. AM4376 ICSS0 instance doesn't have Shared RAM.
> +
> +The PRUSS node will have one or more of the folowing child nodes.
> +
> +PRU CORES
> +=========
> +ICSS typically has 2 PRU cores. These should be represented as remoteproc devices.
> +
> +INTC node
> +=========
> +ICSS has one INTC interrupt controller module. This should be represented as
> +a standard interrupt-controller node.
> +
> +CFG, IEP, MII_RT
> +================
> +The individual sub-modules CFG, IEP and MII_RT are represented as a syscon
> +node each with specific node names as below:
> + "cfg" for CFG sub-module,
> + "iep" for IEP sub-module,
> + "mii_rt" for MII-RT sub-module,
> +
> +See Documentation/devicetree/bindings/mfd/syscon.txt for details.
> +
> +MDIO
> +====
> +Each PRUSS has an MDIO module that can be used to control external PHYs. The
> +MDIO module used within the PRU-ICSS is an instance of the MDIO Controller
> +used in TI Davinci SoCs. Please refer to the corresponding binding document,
> +Documentation/devicetree/bindings/net/davinci-mdio.txt for details.
> +
> +Application/User Nodes
> +=======================
> +A PRU application/user node typically uses one or more PRU device nodes to
> +implement a PRU application/functionality. Each application/client node would
> +need a reference to at least a PRU node, and optionally pass some configuration
> +parameters.
> +
> +Required Properties:
> +--------------------
> +- prus : phandles to the PRU nodes used
> +
> +Optional Properties:
> +--------------------
> +- firmware-name : firmwares for the PRU cores, the default firmware
> + for the core from the PRU node will be used if not
> + provided. The firmware names should correspond to
> + the PRU cores listed in the 'prus' property
> +- ti,pruss-gp-mux-sel : array of values for the GP_MUX_SEL under PRUSS_GPCFG
> + register for a PRU. This selects the internal muxing
> + scheme for the PRU instance. If not provided, the
> + default out-of-reset value (0) for the PRU core is
> + used. Values should correspond to the PRU cores listed
> + in the 'prus' property
> +- ti,pru-interrupt-map : PRU interrupt mappings, containing an array of entries
> + with each entry consisting of 4 cell-values. First one
> + is an index towards the "prus" property to identify the
> + PRU core for the interrupt map, second is the PRU
> + System Event id, third is the PRU interrupt channel id
> + and fourth is the PRU host interrupt id. If provided,
> + this map will supercede any other configuration
> + provided through firmware
> +
> +Example:
> +========
> +1. /* AM33xx PRU-ICSS */
> +
> + pruss: pruss@0 {
> + compatible = "ti,am3356-pruss";
> + reg = <0x0 0x2000>,
> + <0x2000 0x2000>,
> + <0x10000 0x3000>;
> + reg-names = "dram0", "dram1",
> + "shrdram2";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + pruss_cfg: cfg@...00 {
> + compatible = "syscon";
> + reg = <0x26000 0x2000>;
> + };
> +
> + pruss_iep: iep@...00 {
> + compatible = "syscon";
> + reg = <0x2e000 0x31c>;
> + };
> +
> + pruss_mii_rt: mii_rt@...00 {
> + compatible = "syscon";
> + reg = <0x32000 0x58>;
> + };
> +
> + pruss_intc: intc@...00 {
> + compatible = "ti,am3356-pruss-intc";
> + reg = <0x20000 0x2000>;
> + reg-names = "intc";
> + interrupt-controller;
> + #interrupt-cells = <1>;
> + interrupts = <20 21 22 23 24 25 26 27>;
> + interrupt-names = "host2", "host3", "host4",
> + "host5", "host6", "host7",
> + "host8", "host9";
> + };
> +
> + pru0: pru@...00 {
> + compatible = "ti,am3356-pru";
> + reg = <0x34000 0x2000>,
> + <0x22000 0x400>,
> + <0x22400 0x100>;
> + reg-names = "iram", "control", "debug";
> + gpcfg = <&pruss_cfg 0x8>;
> + firmware-name = "am335x-pru0-fw";
> + interrupt-parent = <&pruss_intc>;
> + interrupts = <16>, <17>;
> + interrupt-names = "vring", "kick";
> + };
> +
> + pru1: pru@...00 {
> + compatible = "ti,am3356-pru";
> + reg = <0x38000 0x2000>,
> + <0x24000 0x400>,
> + <0x24400 0x100>;
> + reg-names = "iram", "control", "debug";
> + gpcfg = <&pruss_cfg 0xc>;
> + firmware-name = "am335x-pru1-fw";
> + interrupt-parent = <&pruss_intc>;
> + interrupts = <18>, <19>;
> + interrupt-names = "vring", "kick";
> + };
> +
> + pruss_mdio: mdio@...00 {
> + compatible = "ti,davinci_mdio";
> + reg = <0x32400 0x90>;
> + clocks = <&dpll_core_m4_ck>;
> + clock-names = "fck";
> + bus_freq = <1000000>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + status = "disabled";
> + };
> + };
> +
> +2: /* PRU application node example */
> + app_node: app_node {
> + prus = <&pru0>, <&pru1>;
> + firmware-name = "pruss-app-fw", "pruss-app-fw-2";
> + ti,pruss-gp-mux-sel = <2>, <1>;
> + /* setup interrupts for prus:
> + prus[0] => pru1_0: ev=16, chnl=2, host-irq=7,
> + prus[1] => pru1_1: ev=19, chnl=1, host-irq=3 */
> + ti,pru-interrupt-map = <0 16 2 7 >, <1 19 1 3>;
> + }
>
Powered by blists - more mailing lists