Am 30. August 2016 00:06:49 GMT-07:00, schrieb Jarkko Sakkinen [off-list ref]:

On Mon, Aug 29, 2016 at 11:41:51PM -0700, Peter Huewe wrote:

quoted

Am 29. August 2016 23:36:31 GMT-07:00, schrieb Jarkko Sakkinen

[off-list ref]:

quoted

quoted

On Tue, Aug 30, 2016 at 12:44:37AM -0400, Nayna Jain wrote:

quoted

This is documenting device tree binding for
I2C based TPM, similar concept which being used
for virtual TPM on POWER7 and POWER8 systems running PowerVM.

Signed-off-by: Nayna Jain <redacted>
---
 Documentation/devicetree/bindings/i2c/i2c-tpm.txt | 29

+++++++++++++++++++++++

quoted

 1 file changed, 29 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/i2c/i2c-tpm.txt

quoted

quoted

quoted

diff --git a/Documentation/devicetree/bindings/i2c/i2c-tpm.txt

b/Documentation/devicetree/bindings/i2c/i2c-tpm.txt

quoted

new file mode 100644
index 0000000..8fdee14

--- /dev/null
+++ b/Documentation/devicetree/bindings/i2c/i2c-tpm.txt

@@ -0,0 +1,29 @@
+Device Tree Bindings for I2C based Trusted Platform Module(TPM)
+---------------------------------------------------------------
+
+This node describes a TPM device connected to Processor on i2c

bus.

quoted

quoted

quoted

+
+Required properties:
+
+- compatible : 'manufacturer,model'
+- label : represents device type
+- linux,sml-base : base address of the Event Log. It is a

physical

quoted

quoted

address.

quoted

+		   sml stands for shared memory log.
+- linux,sml-size : size of the memory allocated for the Event

Log.

quoted

quoted

quoted

+
+Optional properties:
+
+- status: indicates whether the device is enabled or disabled.

"okay" for

quoted

+          enabled and "disabled" for disabled.
+
+Example
+-------
+
+tpm@57 {
+	reg = <0x57>;
+	label = "tpm";
+	compatible = "nuvoton,npct650", "nuvoton,npct601";
+	linux,sml-base = <0x7f 0xfd450000>;
+	linux,sml-size = <0x10000>;
+	status = "okay";
+};

I would rather name the fields event-log-base and event-log-size.

They

quoted

quoted

would be much more readable and obvious names.

I agree - I always get stuck upon the sml thing.

quoted

Also, enabled should be "enabled", not "okay".

No!
okay/ok is a dt keyword! (Or at least used in everything else)

It has nothing to do whether the TPM is enabled/disabled/activated

whatever

quoted

Peter

OK, just to educate myself, where can I find these standard keywords?

See http://www.devicetree.org/specifications-pdf table 2.4

The granularity is wrong (section 8.7 of TPM 2.0 Structures
specification). There are four different things that you can
enabled/disable.

/Jarkko

-- 
Sent from my mobile

------------------------------------------------------------------------------