linux-kernel - [PATCH] dt-bindings: security: tpm: Convert TPM tis spi to yaml

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <0f59acbb394cd09ae2a7950d8c01f8e40abf9f34.1704272496.git.michal.simek@amd.com>
Date: Wed, 3 Jan 2024 10:01:46 +0100
From: Michal Simek <michal.simek@....com>
To: <linux-kernel@...r.kernel.org>, <monstr@...str.eu>,
	<michal.simek@...inx.com>, <git@...inx.com>
CC: Conor Dooley <conor+dt@...nel.org>, Jarkko Sakkinen <jarkko@...nel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Peter Huewe
	<peterhuewe@....de>, Rob Herring <robh+dt@...nel.org>,
	<devicetree@...r.kernel.org>
Subject: [PATCH] dt-bindings: security: tpm: Convert TPM tis spi to yaml

Convert binding from txt to yaml. Enforce that node name starts with tpm@
which should be generic enough for these devices.
Deprecating tcg,tpm_tis-spi because it is using "_" which shouldn't be used
by in compatible string that's why infineon compatible string is used for
file name too.
Also add current TPM maintainers and maintainers of this dt binding too.

Signed-off-by: Michal Simek <michal.simek@....com>
---

I want to do just conversion but not really take care about this device.
---
 .../security/tpm/infineon,slb9670.yaml        | 54 +++++++++++++++++++
 .../bindings/security/tpm/tpm_tis_spi.txt     | 23 --------
 2 files changed, 54 insertions(+), 23 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/security/tpm/infineon,slb9670.yaml
 delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt

diff --git a/Documentation/devicetree/bindings/security/tpm/infineon,slb9670.yaml b/Documentation/devicetree/bindings/security/tpm/infineon,slb9670.yaml
new file mode 100644
index 000000000000..309b91881774
--- /dev/null
+++ b/Documentation/devicetree/bindings/security/tpm/infineon,slb9670.yaml
@@ -0,0 +1,54 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/security/tpm/infineon,slb9670.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Trusted Platform Module (TPM) with a SPI interface
+
+maintainers:
+  - Peter Huewe <peterhuewe@....de>
+  - Jarkko Sakkinen <jarkko@...nel.org>
+
+properties:
+  $nodename:
+    pattern: "^tpm@[0-9a-f]{1,2}$"
+
+  compatible:
+    oneOf:
+      - enum:
+          - st,st33htpm-spi
+          - infineon,slb9670
+          - google,cr50
+      - const: tcg,tpm_tis-spi
+        deprecated: true
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - spi-max-frequency
+
+allOf:
+  - $ref: /schemas/spi/spi-peripheral-props.yaml#
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    spi {
+      #address-cells = <1>;
+      #size-cells = <0>;
+      tpm@1 {
+        compatible = "infineon,slb9670";
+        reg = <0x1>;
+        pinctrl-names = "default";
+        pinctrl-0 = <&pinctrl_tpm>;
+        spi-max-frequency = <20000000>;
+      };
+    };
diff --git a/Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt b/Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt
deleted file mode 100644
index b800667da92b..000000000000
--- a/Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt
+++ /dev/null
@@ -1,23 +0,0 @@
-Required properties:
-- compatible: should be one of the following
-    "st,st33htpm-spi"
-    "infineon,slb9670"
-    "tcg,tpm_tis-spi"
-- spi-max-frequency: Maximum SPI frequency (depends on TPMs).
-
-Optional SoC Specific Properties:
-- pinctrl-names: Contains only one value - "default".
-- pintctrl-0: Specifies the pin control groups used for this controller.
-
-Example (for ARM-based BeagleBoard xM with TPM_TIS on SPI4):
-
-&mcspi4 {
-
-
-        tpm_tis@0 {
-
-                compatible = "tcg,tpm_tis-spi";
-
-                spi-max-frequency = <10000000>;
-        };
-};
-- 
2.36.1