- all the integration parameters have been captured by the binding.
- the block name really uniquely identifies this hardware.

Some advocate putting SoC names everywhere in case software needs
to work around some chip-specific bug, but more precise SoC
information already exists in SVR, and board information already
exists in the top-level device tree node.

Note that sometimes the SoC name is a worse identifier than the
block version, as the block version can change between revisions
of the same SoC.

As a matter of historical reference, neither SEC versions 2.x
nor 3.x (driven by talitos) ever needed CHIP references.

Signed-off-by: Kim Phillips <kim.phill...@freescale.com>
Cc: Kumar Gala <kumar.g...@freescale.com>
Cc: Scott Wood <scottw...@freescale.com>
---
 .../devicetree/bindings/crypto/fsl-sec4.txt        |   64 ++++++++------------
 arch/powerpc/boot/dts/p4080ds.dts                  |   32 ++++-------
 2 files changed, 37 insertions(+), 59 deletions(-)

diff --git a/Documentation/devicetree/bindings/crypto/fsl-sec4.txt 
b/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
index 568aa3c..bf57ecd 100644
--- a/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
+++ b/Documentation/devicetree/bindings/crypto/fsl-sec4.txt
@@ -38,7 +38,7 @@ in the memory partition devoted to a particular core.  The 
P4080 has 4 JRs, so
 up to 4 JRs can be configured; and all 4 JRs process requests in parallel.
 
 =====================================================================
-P4080 SEC 4 Node
+SEC 4 Node
 
 Description
 
@@ -53,7 +53,7 @@ PROPERTIES
    - compatible
       Usage: required
       Value type: <string>
-      Definition: Must include "fsl,p4080-sec-v4.0","fsl,sec-v4.0"
+      Definition: Must include "fsl,sec-v4.0"
 
    - #address-cells
        Usage: required
@@ -105,7 +105,7 @@ PROPERTIES
 
 EXAMPLE
        crypto@300000 {
-               compatible = "fsl,p4080-sec-v4.0", "fsl,sec-v4.0";
+               compatible = "fsl,sec-v4.0";
                #address-cells = <1>;
                #size-cells = <1>;
                reg = <0x300000 0x10000>;
@@ -115,7 +115,7 @@ EXAMPLE
        };
 
 =====================================================================
-P4080 Job Ring (JR) Node
+Job Ring (JR) Node
 
     Child of the crypto node defines data processing interface to SEC 4
     across the peripheral bus for purposes of processing
@@ -127,7 +127,7 @@ P4080 Job Ring (JR) Node
   - compatible
       Usage: required
       Value type: <string>
-      Definition: Must include 
"fsl,p4080-sec-v4.0-job-ring","fsl,sec-v4.0-job-ring"
+      Definition: Must include "fsl,sec-v4.0-job-ring"
 
   - reg
       Usage: required
@@ -163,8 +163,7 @@ P4080 Job Ring (JR) Node
 
 EXAMPLE
        jr@1000 {
-               compatible = "fsl,p4080-sec-v4.0-job-ring",
-                            "fsl,sec-v4.0-job-ring";
+               compatible = "fsl,sec-v4.0-job-ring";
                reg = <0x1000 0x1000>;
                fsl,liodn = <0x081>;
                interrupt-parent = <&mpic>;
@@ -173,7 +172,7 @@ EXAMPLE
 
 
 =====================================================================
-P4080 Run Time Integrity Check (RTIC) Node
+Run Time Integrity Check (RTIC) Node
 
   Child node of the crypto node.  Defines a register space that
   contains up to 5 sets of addresses and their lengths (sizes) that
@@ -186,7 +185,7 @@ P4080 Run Time Integrity Check (RTIC) Node
   - compatible
       Usage: required
       Value type: <string>
-      Definition: Must include "fsl,p4080-sec-v4.0-rtic","fsl,sec-v4.0-rtic".
+      Definition: Must include "fsl,sec-v4.0-rtic".
 
    - #address-cells
        Usage: required
@@ -219,8 +218,7 @@ P4080 Run Time Integrity Check (RTIC) Node
 
 EXAMPLE
        rtic@6000 {
-               compatible = "fsl,p4080-sec-v4.0-rtic",
-                            "fsl,sec-v4.0-rtic";
+               compatible = "fsl,sec-v4.0-rtic";
                #address-cells = <1>;
                #size-cells = <1>;
                reg = <0x6000 0x100>;
@@ -228,7 +226,7 @@ EXAMPLE
        };
 
 =====================================================================
-P4080 Run Time Integrity Check (RTIC) Memory Node
+Run Time Integrity Check (RTIC) Memory Node
   A child node that defines individual RTIC memory regions that are used to
   perform run-time integrity check of memory areas that should not modified.
   The node defines a register that contains the memory address &
@@ -238,7 +236,7 @@ P4080 Run Time Integrity Check (RTIC) Memory Node
   - compatible
       Usage: required
       Value type: <string>
-      Definition: Must include 
"fsl,p4080-sec-v4.0-rtic-memory","fsl,sec-v4.0-rtic-memory".
+      Definition: Must include "fsl,sec-v4.0-rtic-memory".
 
   - reg
       Usage: required
@@ -270,15 +268,14 @@ P4080 Run Time Integrity Check (RTIC) Memory Node
 
 EXAMPLE
        rtic-a@0 {
-               compatible = "fsl,p4080-sec-v4.0-rtic-memory",
-                            "fsl,sec-v4.0-rtic-memory";
+               compatible = "fsl,sec-v4.0-rtic-memory";
                reg = <0x00 0x20 0x100 0x80>;
                fsl,liodn   = <0x03c>;
                fsl,rtic-region  = <0x12345678 0x12345678 0x12345678>;
        };
 
 =====================================================================
-P4080 Secure Non-Volatile Storage (SNVS) Node
+Secure Non-Volatile Storage (SNVS) Node
 
     Node defines address range and the associated
     interrupt for the SNVS function.  This function
@@ -288,7 +285,7 @@ P4080 Secure Non-Volatile Storage (SNVS) Node
   - compatible
       Usage: required
       Value type: <string>
-      Definition: Must include "fsl,p4080-sec-v4.0-mon", "fsl,sec-v4.0-mon".
+      Definition: Must include "fsl,sec-v4.0-mon".
 
   - reg
       Usage: required
@@ -315,7 +312,7 @@ P4080 Secure Non-Volatile Storage (SNVS) Node
 
 EXAMPLE
        sec_mon@314000 {
-               compatible = "fsl,p4080-sec-v4.0-mon", "fsl,sec-v4.0-mon";
+               compatible = "fsl,sec-v4.0-mon";
                reg = <0x314000 0x1000>;
                interrupt-parent = <&mpic>;
                interrupts = <93 2>;
@@ -325,7 +322,7 @@ EXAMPLE
 FULL EXAMPLE
 
        crypto: crypto@300000 {
-               compatible = "fsl,p4080-sec-v4.0", "fsl,sec-v4.0";
+               compatible = "fsl,sec-v4.0";
                #address-cells = <1>;
                #size-cells = <1>;
                reg = <0x300000 0x10000>;
@@ -334,73 +331,64 @@ FULL EXAMPLE
                interrupts = <92 2>;
 
                sec_jr0: jr@1000 {
-                       compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                    "fsl,sec-v4.0-job-ring";
+                       compatible = "fsl,sec-v4.0-job-ring";
                        reg = <0x1000 0x1000>;
                        interrupt-parent = <&mpic>;
                        interrupts = <88 2>;
                };
 
                sec_jr1: jr@2000 {
-                       compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                    "fsl,sec-v4.0-job-ring";
+                       compatible = "fsl,sec-v4.0-job-ring";
                        reg = <0x2000 0x1000>;
                        interrupt-parent = <&mpic>;
                        interrupts = <89 2>;
                };
 
                sec_jr2: jr@3000 {
-                       compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                    "fsl,sec-v4.0-job-ring";
+                       compatible = "fsl,sec-v4.0-job-ring";
                        reg = <0x3000 0x1000>;
                        interrupt-parent = <&mpic>;
                        interrupts = <90 2>;
                };
 
                sec_jr3: jr@4000 {
-                       compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                    "fsl,sec-v4.0-job-ring";
+                       compatible = "fsl,sec-v4.0-job-ring";
                        reg = <0x4000 0x1000>;
                        interrupt-parent = <&mpic>;
                        interrupts = <91 2>;
                };
 
                rtic@6000 {
-                       compatible = "fsl,p4080-sec-v4.0-rtic",
-                                    "fsl,sec-v4.0-rtic";
+                       compatible = "fsl,sec-v4.0-rtic";
                        #address-cells = <1>;
                        #size-cells = <1>;
                        reg = <0x6000 0x100>;
                        ranges = <0x0 0x6100 0xe00>;
 
                        rtic_a: rtic-a@0 {
-                               compatible = "fsl,p4080-sec-v4.0-rtic-memory",
-                                            "fsl,sec-v4.0-rtic-memory";
+                               compatible = "fsl,sec-v4.0-rtic-memory";
                                reg = <0x00 0x20 0x100 0x80>;
                        };
 
                        rtic_b: rtic-b@20 {
-                               compatible = "fsl,p4080-sec-v4.0-rtic-memory",
-                                            "fsl,sec-v4.0-rtic-memory";
+                               compatible = "fsl,sec-v4.0-rtic-memory";
                                reg = <0x20 0x20 0x200 0x80>;
                        };
 
                        rtic_c: rtic-c@40 {
-                               compatible = "fsl,p4080-sec-v4.0-rtic-memory",
-                                            "fsl,sec-v4.0-rtic-memory";
+                               compatible = "fsl,sec-v4.0-rtic-memory";
                                reg = <0x40 0x20 0x300 0x80>;
                        };
 
                        rtic_d: rtic-d@60 {
-                               compatible = "fsl,p4080-sec-v4.0-rtic-memory",
-                                            "fsl,sec-v4.0-rtic-memory";
+                               compatible = "fsl,sec-v4.0-rtic-memory";
                                reg = <0x60 0x20 0x500 0x80>;
                        };
                };
        };
 
        sec_mon: sec_mon@314000 {
-               compatible = "fsl,p4080-sec-v4.0-mon", "fsl,sec-v4.0-mon";
+               compatible = "fsl,sec-v4.0-mon";
                reg = <0x314000 0x1000>;
                interrupt-parent = <&mpic>;
                interrupts = <93 2>;
diff --git a/arch/powerpc/boot/dts/p4080ds.dts 
b/arch/powerpc/boot/dts/p4080ds.dts
index 0430d24..927f94d 100644
--- a/arch/powerpc/boot/dts/p4080ds.dts
+++ b/arch/powerpc/boot/dts/p4080ds.dts
@@ -423,7 +423,7 @@
                };
 
                crypto: crypto@300000 {
-                       compatible = "fsl,p4080-sec-v4.0", "fsl,sec-v4.0";
+                       compatible = "fsl,sec-v4.0";
                        #address-cells = <1>;
                        #size-cells = <1>;
                        reg = <0x300000 0x10000>;
@@ -432,74 +432,64 @@
                        interrupts = <92 2>;
 
                        sec_jr0: jr@1000 {
-                               compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                            "fsl,sec-v4.0-job-ring";
+                               compatible = "fsl,sec-v4.0-job-ring";
                                reg = <0x1000 0x1000>;
                                interrupt-parent = <&mpic>;
                                interrupts = <88 2>;
                        };
 
                        sec_jr1: jr@2000 {
-                               compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                            "fsl,sec-v4.0-job-ring";
+                               compatible = "fsl,sec-v4.0-job-ring";
                                reg = <0x2000 0x1000>;
                                interrupt-parent = <&mpic>;
                                interrupts = <89 2>;
                        };
 
                        sec_jr2: jr@3000 {
-                               compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                            "fsl,sec-v4.0-job-ring";
+                               compatible = "fsl,sec-v4.0-job-ring";
                                reg = <0x3000 0x1000>;
                                interrupt-parent = <&mpic>;
                                interrupts = <90 2>;
                        };
 
                        sec_jr3: jr@4000 {
-                               compatible = "fsl,p4080-sec-v4.0-job-ring",
-                                            "fsl,sec-v4.0-job-ring";
+                               compatible = "fsl,sec-v4.0-job-ring";
                                reg = <0x4000 0x1000>;
                                interrupt-parent = <&mpic>;
                                interrupts = <91 2>;
                        };
 
                        rtic@6000 {
-                               compatible = "fsl,p4080-sec-v4.0-rtic",
-                                            "fsl,sec-v4.0-rtic";
+                               compatible = "fsl,sec-v4.0-rtic";
                                #address-cells = <1>;
                                #size-cells = <1>;
                                reg = <0x6000 0x100>;
                                ranges = <0x0 0x6100 0xe00>;
 
                                rtic_a: rtic-a@0 {
-                                       compatible = 
"fsl,p4080-sec-v4.0-rtic-memory",
-                                                    "fsl,sec-v4.0-rtic-memory";
+                                       compatible = "fsl,sec-v4.0-rtic-memory";
                                        reg = <0x00 0x20 0x100 0x80>;
                                };
 
                                rtic_b: rtic-b@20 {
-                                       compatible = 
"fsl,p4080-sec-v4.0-rtic-memory",
-                                                    "fsl,sec-v4.0-rtic-memory";
+                                       compatible = "fsl,sec-v4.0-rtic-memory";
                                        reg = <0x20 0x20 0x200 0x80>;
                                };
 
                                rtic_c: rtic-c@40 {
-                                       compatible = 
"fsl,p4080-sec-v4.0-rtic-memory",
-                                                    "fsl,sec-v4.0-rtic-memory";
+                                       compatible = "fsl,sec-v4.0-rtic-memory";
                                        reg = <0x40 0x20 0x300 0x80>;
                                };
 
                                rtic_d: rtic-d@60 {
-                                       compatible = 
"fsl,p4080-sec-v4.0-rtic-memory",
-                                                    "fsl,sec-v4.0-rtic-memory";
+                                       compatible = "fsl,sec-v4.0-rtic-memory";
                                        reg = <0x60 0x20 0x500 0x80>;
                                };
                        };
                };
 
                sec_mon: sec_mon@314000 {
-                       compatible = "fsl,p4080-sec-v4.0-mon",
-                                    "fsl,sec-v4.0-mon";
+                       compatible = "fsl,sec-v4.0-mon";
                        reg = <0x314000 0x1000>;
                        interrupt-parent = <&mpic>;
                        interrupts = <93 2>;
-- 
1.7.4.1


_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to