Re: [PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Tony Lindgren]

* Rob Herring  [170519 18:49]:
> On Fri, May 19, 2017 at 12:57:07PM -0500, Dave Gerlach wrote:
> > Update the Texas Instruments EMIF binding document to include the device
> > tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
> > the ti-emif-sram driver to provide low-level PM functionality.
> > 
> > Signed-off-by: Dave Gerlach 
> > ---
> >  .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 
> > +-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> I acked v1. Please add acks when posting new versions.

And if I did not do so already:

Acked-by: Tony Lindgren 

Re: [PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Tony Lindgren]

* Rob Herring  [170519 18:49]:
> On Fri, May 19, 2017 at 12:57:07PM -0500, Dave Gerlach wrote:
> > Update the Texas Instruments EMIF binding document to include the device
> > tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
> > the ti-emif-sram driver to provide low-level PM functionality.
> > 
> > Signed-off-by: Dave Gerlach 
> > ---
> >  .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 
> > +-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> I acked v1. Please add acks when posting new versions.

And if I did not do so already:

Acked-by: Tony Lindgren 

Re: [PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Rob Herring]

On Fri, May 19, 2017 at 12:57:07PM -0500, Dave Gerlach wrote:
> Update the Texas Instruments EMIF binding document to include the device
> tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
> the ti-emif-sram driver to provide low-level PM functionality.
> 
> Signed-off-by: Dave Gerlach 
> ---
>  .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 
> +-
>  1 file changed, 17 insertions(+), 1 deletion(-)

I acked v1. Please add acks when posting new versions.

Rob

Re: [PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Rob Herring]

On Fri, May 19, 2017 at 12:57:07PM -0500, Dave Gerlach wrote:
> Update the Texas Instruments EMIF binding document to include the device
> tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
> the ti-emif-sram driver to provide low-level PM functionality.
> 
> Signed-off-by: Dave Gerlach 
> ---
>  .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 
> +-
>  1 file changed, 17 insertions(+), 1 deletion(-)

I acked v1. Please add acks when posting new versions.

Rob

[PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Dave Gerlach]

Update the Texas Instruments EMIF binding document to include the device
tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
the ti-emif-sram driver to provide low-level PM functionality.

Signed-off-by: Dave Gerlach 
---
 .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 +-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt 
b/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
index 0db60470ebb6..8c0214b17c2b 100644
--- a/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
@@ -8,6 +8,7 @@ of the EMIF IP and memory parts attached to it.
 Required properties:
 - compatible   : Should be of the form "ti,emif-" where 
   is the IP revision of the specific EMIF instance.
+ For am335x should be ti,emif-am3352.
  For am437x should be ti,emif-am4372.
 
 - phy-type :  indicating the DDR phy type. Following are the
@@ -21,6 +22,13 @@ Required properties:
   the value shall be "emif" where  is the number of the EMIF
   instance with base 1.
 
+Required only for "ti,emif-am3352" and "ti,emif-am4372":
+- sram : Phandles for generic sram driver nodes,
+  first should be type 'protect-exec' for the driver to use to copy
+  and run PM functions, second should be regular pool to be used for
+  data region for code. See Documentation/devicetree/bindings/sram/sram.txt
+  for more details.
+
 Optional properties:
 - cs1-used : Have this property if CS1 of this EMIF
   instance has a memory part attached to it. If there is a memory
@@ -42,7 +50,7 @@ Optional properties:
 - hw-caps-temp-alert   : Have this property if the controller
   has capability for generating SDRAM temperature alerts
 
-Example:
+-Examples:
 
 emif1: emif@0x4c00 {
compatible  = "ti,emif-4d";
@@ -54,3 +62,11 @@ emif1: emif@0x4c00 {
hw-caps-ll-interface;
hw-caps-temp-alert;
 };
+
+/* From am33xx.dtsi */
+emif: emif@4c00 {
+compatible = "ti,emif-am3352";
+reg =   <0x4C00 0x1000>;
+sram = <_sram_code
+_sram_data>;
+};
-- 
2.13.0

[PATCH v2 1/2] Documentation: dt: Update ti,emif bindings

[Dave Gerlach]

Update the Texas Instruments EMIF binding document to include the device
tree bindings for ti,emif-am3352 and ti,emif-am4372 which are used by
the ti-emif-sram driver to provide low-level PM functionality.

Signed-off-by: Dave Gerlach 
---
 .../devicetree/bindings/memory-controllers/ti/emif.txt | 18 +-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt 
b/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
index 0db60470ebb6..8c0214b17c2b 100644
--- a/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/ti/emif.txt
@@ -8,6 +8,7 @@ of the EMIF IP and memory parts attached to it.
 Required properties:
 - compatible   : Should be of the form "ti,emif-" where 
   is the IP revision of the specific EMIF instance.
+ For am335x should be ti,emif-am3352.
  For am437x should be ti,emif-am4372.
 
 - phy-type :  indicating the DDR phy type. Following are the
@@ -21,6 +22,13 @@ Required properties:
   the value shall be "emif" where  is the number of the EMIF
   instance with base 1.
 
+Required only for "ti,emif-am3352" and "ti,emif-am4372":
+- sram : Phandles for generic sram driver nodes,
+  first should be type 'protect-exec' for the driver to use to copy
+  and run PM functions, second should be regular pool to be used for
+  data region for code. See Documentation/devicetree/bindings/sram/sram.txt
+  for more details.
+
 Optional properties:
 - cs1-used : Have this property if CS1 of this EMIF
   instance has a memory part attached to it. If there is a memory
@@ -42,7 +50,7 @@ Optional properties:
 - hw-caps-temp-alert   : Have this property if the controller
   has capability for generating SDRAM temperature alerts
 
-Example:
+-Examples:
 
 emif1: emif@0x4c00 {
compatible  = "ti,emif-4d";
@@ -54,3 +62,11 @@ emif1: emif@0x4c00 {
hw-caps-ll-interface;
hw-caps-temp-alert;
 };
+
+/* From am33xx.dtsi */
+emif: emif@4c00 {
+compatible = "ti,emif-am3352";
+reg =   <0x4C00 0x1000>;
+sram = <_sram_code
+_sram_data>;
+};
-- 
2.13.0