Re: [PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Suman Anna]

On 08/10/2017 09:01 AM, Rob Herring wrote:
> On Tue, Aug 01, 2017 at 10:48:43AM -0500, Suman Anna wrote:
>> Add the device tree bindings document for the DSP processor
>> subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.
>>
>> Signed-off-by: Suman Anna 
>> ---
>> v2: 
>>  - Updated the patch header
>>  - Revised the binding status statement slightly
>>  - Updated the example to address Rob's comments
>>  - No changes in properties description themselves
>> v1: https://patchwork.kernel.org/patch/9751269/
>>
>>  .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 
>> ++
>>  1 file changed, 86 insertions(+)
>>  create mode 100644 
>> Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
> 
> Acked-by: Rob Herring 

Thanks Rob.

Bjorn,
Appreciate it if you can pick up the series for v4.14.

Thanks,
Suman

Re: [PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Suman Anna]

On 08/10/2017 09:01 AM, Rob Herring wrote:
> On Tue, Aug 01, 2017 at 10:48:43AM -0500, Suman Anna wrote:
>> Add the device tree bindings document for the DSP processor
>> subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.
>>
>> Signed-off-by: Suman Anna 
>> ---
>> v2: 
>>  - Updated the patch header
>>  - Revised the binding status statement slightly
>>  - Updated the example to address Rob's comments
>>  - No changes in properties description themselves
>> v1: https://patchwork.kernel.org/patch/9751269/
>>
>>  .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 
>> ++
>>  1 file changed, 86 insertions(+)
>>  create mode 100644 
>> Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
> 
> Acked-by: Rob Herring 

Thanks Rob.

Bjorn,
Appreciate it if you can pick up the series for v4.14.

Thanks,
Suman

Re: [PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Rob Herring]

On Tue, Aug 01, 2017 at 10:48:43AM -0500, Suman Anna wrote:
> Add the device tree bindings document for the DSP processor
> subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.
> 
> Signed-off-by: Suman Anna 
> ---
> v2: 
>  - Updated the patch header
>  - Revised the binding status statement slightly
>  - Updated the example to address Rob's comments
>  - No changes in properties description themselves
> v1: https://patchwork.kernel.org/patch/9751269/
> 
>  .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 
> ++
>  1 file changed, 86 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt

Acked-by: Rob Herring 

Re: [PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Rob Herring]

On Tue, Aug 01, 2017 at 10:48:43AM -0500, Suman Anna wrote:
> Add the device tree bindings document for the DSP processor
> subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.
> 
> Signed-off-by: Suman Anna 
> ---
> v2: 
>  - Updated the patch header
>  - Revised the binding status statement slightly
>  - Updated the example to address Rob's comments
>  - No changes in properties description themselves
> v1: https://patchwork.kernel.org/patch/9751269/
> 
>  .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 
> ++
>  1 file changed, 86 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt

Acked-by: Rob Herring 

[PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Suman Anna]

Add the device tree bindings document for the DSP processor
subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.

Signed-off-by: Suman Anna 
---
v2: 
 - Updated the patch header
 - Revised the binding status statement slightly
 - Updated the example to address Rob's comments
 - No changes in properties description themselves
v1: https://patchwork.kernel.org/patch/9751269/

 .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 ++
 1 file changed, 86 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt

diff --git a/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt 
b/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
new file mode 100644
index ..e44a97e21164
--- /dev/null
+++ b/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
@@ -0,0 +1,86 @@
+TI Davinci DSP devices
+===
+
+Binding status: Unstable - Subject to changes for DT representation of clocks
+  and resets
+
+The TI Davinci family of SoCs usually contains a TI DSP Core sub-system that
+is used to offload some of the processor-intensive tasks or algorithms, for
+achieving various system level goals.
+
+The processor cores in the sub-system usually contain additional sub-modules
+like L1 and/or L2 caches/SRAMs, an Interrupt Controller, an external memory
+controller, a dedicated local power/sleep controller etc. The DSP processor
+core used in Davinci SoCs is usually a C674x DSP CPU.
+
+DSP Device Node:
+
+Each DSP Core sub-system is represented as a single DT node.
+
+Required properties:
+
+The following are the mandatory properties:
+
+- compatible:  Should be one of the following,
+   "ti,da850-dsp" for DSPs on OMAP-L138 SoCs
+
+- reg: Should contain an entry for each value in 'reg-names'.
+   Each entry should have the memory region's start address
+   and the size of the region, the representation matching
+   the parent node's '#address-cells' and '#size-cells' 
values.
+
+- reg-names:   Should contain strings with the following names, each
+   representing a specific internal memory region or a
+   specific register space,
+"l2sram", "l1pram", "l1dram", "host1cfg", 
"chipsig_base"
+
+- interrupts:  Should contain the interrupt number used to receive the
+   interrupts from the DSP. The value should follow the
+   interrupt-specifier format as dictated by the
+   'interrupt-parent' node.
+
+- memory-region:   phandle to the reserved memory node to be associated
+   with the remoteproc device. The reserved memory node
+   can be a CMA memory node, and should be defined as
+   per the bindings in
+   
Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
+
+Optional properties:
+
+- interrupt-parent:phandle to the interrupt controller node. This property
+   is needed if the device node hierarchy doesn't have an
+   interrupt controller.
+
+
+Example:
+
+
+   /* DSP Reserved Memory node */
+   reserved-memory {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges;
+
+   dsp_memory_region: dsp-memory@c300 {
+   compatible = "shared-dma-pool";
+   reg = <0xc300 0x100>;
+   reusable;
+   };
+   };
+
+   /* DSP node */
+   {
+   dsp: dsp@1180 {
+   compatible = "ti,da850-dsp";
+   reg = <0x1180 0x4>,
+ <0x11e0 0x8000>,
+ <0x11f0 0x8000>,
+ <0x01c14044 0x4>,
+ <0x01c14174 0x8>;
+   reg-names = "l2sram", "l1pram", "l1dram", "host1cfg",
+   "chipsig";
+   interrupt-parent = <>;
+   interrupts = <28>;
+   memory-region = <_memory_region>;
+   };
+   };
-- 
2.13.1

[PATCH v2 3/4] dt-bindings: remoteproc: Add bindings for Davinci DSP processors

[Suman Anna]

Add the device tree bindings document for the DSP processor
subsystem devices on TI Davinci DA8xx/OMAP-L13x SoCs.

Signed-off-by: Suman Anna 
---
v2: 
 - Updated the patch header
 - Revised the binding status statement slightly
 - Updated the example to address Rob's comments
 - No changes in properties description themselves
v1: https://patchwork.kernel.org/patch/9751269/

 .../bindings/remoteproc/ti,davinci-rproc.txt   | 86 ++
 1 file changed, 86 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt

diff --git a/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt 
b/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
new file mode 100644
index ..e44a97e21164
--- /dev/null
+++ b/Documentation/devicetree/bindings/remoteproc/ti,davinci-rproc.txt
@@ -0,0 +1,86 @@
+TI Davinci DSP devices
+===
+
+Binding status: Unstable - Subject to changes for DT representation of clocks
+  and resets
+
+The TI Davinci family of SoCs usually contains a TI DSP Core sub-system that
+is used to offload some of the processor-intensive tasks or algorithms, for
+achieving various system level goals.
+
+The processor cores in the sub-system usually contain additional sub-modules
+like L1 and/or L2 caches/SRAMs, an Interrupt Controller, an external memory
+controller, a dedicated local power/sleep controller etc. The DSP processor
+core used in Davinci SoCs is usually a C674x DSP CPU.
+
+DSP Device Node:
+
+Each DSP Core sub-system is represented as a single DT node.
+
+Required properties:
+
+The following are the mandatory properties:
+
+- compatible:  Should be one of the following,
+   "ti,da850-dsp" for DSPs on OMAP-L138 SoCs
+
+- reg: Should contain an entry for each value in 'reg-names'.
+   Each entry should have the memory region's start address
+   and the size of the region, the representation matching
+   the parent node's '#address-cells' and '#size-cells' 
values.
+
+- reg-names:   Should contain strings with the following names, each
+   representing a specific internal memory region or a
+   specific register space,
+"l2sram", "l1pram", "l1dram", "host1cfg", 
"chipsig_base"
+
+- interrupts:  Should contain the interrupt number used to receive the
+   interrupts from the DSP. The value should follow the
+   interrupt-specifier format as dictated by the
+   'interrupt-parent' node.
+
+- memory-region:   phandle to the reserved memory node to be associated
+   with the remoteproc device. The reserved memory node
+   can be a CMA memory node, and should be defined as
+   per the bindings in
+   
Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
+
+Optional properties:
+
+- interrupt-parent:phandle to the interrupt controller node. This property
+   is needed if the device node hierarchy doesn't have an
+   interrupt controller.
+
+
+Example:
+
+
+   /* DSP Reserved Memory node */
+   reserved-memory {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges;
+
+   dsp_memory_region: dsp-memory@c300 {
+   compatible = "shared-dma-pool";
+   reg = <0xc300 0x100>;
+   reusable;
+   };
+   };
+
+   /* DSP node */
+   {
+   dsp: dsp@1180 {
+   compatible = "ti,da850-dsp";
+   reg = <0x1180 0x4>,
+ <0x11e0 0x8000>,
+ <0x11f0 0x8000>,
+ <0x01c14044 0x4>,
+ <0x01c14174 0x8>;
+   reg-names = "l2sram", "l1pram", "l1dram", "host1cfg",
+   "chipsig";
+   interrupt-parent = <>;
+   interrupts = <28>;
+   memory-region = <_memory_region>;
+   };
+   };
-- 
2.13.1