diff mbox series

dt-bindings: tpm: Add st,st33tphf2ei2c

Message ID 20241226171124.83735-1-marex@denx.de (mailing list archive)
State New
Headers show
Series dt-bindings: tpm: Add st,st33tphf2ei2c | expand

Commit Message

Marek Vasut Dec. 26, 2024, 5:10 p.m. UTC
Add the ST chip st33tphf2ei2c to the supported compatible strings of the
TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
Profile specification.

For reference, a databrief is available at:
https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Conor Dooley <conor+dt@kernel.org>
Cc: Jarkko Sakkinen <jarkko@kernel.org>
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
Cc: Lukas Wunner <lukas@wunner.de>
Cc: Peter Huewe <peterhuewe@gmx.de>
Cc: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org
Cc: linux-integrity@vger.kernel.org
---
 Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Jarkko Sakkinen Dec. 26, 2024, 7:54 p.m. UTC | #1
On Thu Dec 26, 2024 at 7:10 PM EET, Marek Vasut wrote:
> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
                          ~~~~
			  chip

> Profile specification.
>
> For reference, a databrief is available at:
> https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf

Or just this would be enough:

Link: https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf

>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: Jarkko Sakkinen <jarkko@kernel.org>
> Cc: Jason Gunthorpe <jgg@ziepe.ca>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Lukas Wunner <lukas@wunner.de>
> Cc: Peter Huewe <peterhuewe@gmx.de>
> Cc: Rob Herring <robh@kernel.org>
> Cc: devicetree@vger.kernel.org
> Cc: linux-integrity@vger.kernel.org
> ---
>  Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml b/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
> index af7720dc4a12c..fdd7fd874e01e 100644
> --- a/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
> +++ b/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
> @@ -33,6 +33,7 @@ properties:
>                - infineon,slb9673
>                - nuvoton,npct75x
>                - st,st33ktpm2xi2c
> +              - st,st33tphf2ei2c
>            - const: tcg,tpm-tis-i2c
>  
>        - description: TPM 1.2 and 2.0 chips with vendor-specific I²C interface


BR, Jarkko
Marek Vasut Dec. 26, 2024, 8:29 p.m. UTC | #2
On 12/26/24 8:54 PM, Jarkko Sakkinen wrote:
> On Thu Dec 26, 2024 at 7:10 PM EET, Marek Vasut wrote:
>> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
>> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
>                            ~~~~
> 			  chip

Fixed, thanks.

>> Profile specification.
>>
>> For reference, a databrief is available at:
>> https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf
> 
> Or just this would be enough:
> 
> Link: https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf
I don't think Link: tag should be used for datasheets and other such 
stuff, my understanding of the Link: tag is that it is used to point to 
the patch at lore.k.o once applied ?
Jarkko Sakkinen Dec. 27, 2024, 5:24 a.m. UTC | #3
On Thu Dec 26, 2024 at 10:29 PM EET, Marek Vasut wrote:
> On 12/26/24 8:54 PM, Jarkko Sakkinen wrote:
> > On Thu Dec 26, 2024 at 7:10 PM EET, Marek Vasut wrote:
> >> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
> >> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
> >                            ~~~~
> > 			  chip
>
> Fixed, thanks.
>
> >> Profile specification.
> >>
> >> For reference, a databrief is available at:
> >> https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf
> > 
> > Or just this would be enough:
> > 
> > Link: https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf
> I don't think Link: tag should be used for datasheets and other such 
> stuff, my understanding of the Link: tag is that it is used to point to 
> the patch at lore.k.o once applied ?

Oh, you are right and I did not know this :-) I've never read this page
before (ever) because I did not know it was necessary:

https://www.kernel.org/doc/html/latest/maintainer/configure-git.html

So I've used up until today link-tag for links... I just thought that
"well, I have Git configured"...

BR, Jarkko
Krzysztof Kozlowski Dec. 27, 2024, 7 a.m. UTC | #4
On 26/12/2024 18:10, Marek Vasut wrote:
> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
> Profile specification.
> 

Where is any user of this?

Best regards,
Krzysztof
Marek Vasut Dec. 27, 2024, 3 p.m. UTC | #5
On 12/27/24 8:00 AM, Krzysztof Kozlowski wrote:
> On 26/12/2024 18:10, Marek Vasut wrote:
>> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
>> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
>> Profile specification.
>>
> 
> Where is any user of this?
Submitted to linux-arm-kernel in

[PATCH] arm64: dts: imx8mp: Update Data Modul i.MX8M Plus eDM SBC DT to 
rev.903
Krzysztof Kozlowski Dec. 28, 2024, 10:32 a.m. UTC | #6
On Thu, Dec 26, 2024 at 06:10:52PM +0100, Marek Vasut wrote:
> Add the ST chip st33tphf2ei2c to the supported compatible strings of the
> TPM TIS I2C schema. The Chip is compliant with the TCG PC Client TPM
> Profile specification.
> 
> For reference, a databrief is available at:
> https://www.st.com/resource/en/data_brief/st33tphf2ei2c.pdf
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml b/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
index af7720dc4a12c..fdd7fd874e01e 100644
--- a/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
+++ b/Documentation/devicetree/bindings/tpm/tcg,tpm-tis-i2c.yaml
@@ -33,6 +33,7 @@  properties:
               - infineon,slb9673
               - nuvoton,npct75x
               - st,st33ktpm2xi2c
+              - st,st33tphf2ei2c
           - const: tcg,tpm-tis-i2c
 
       - description: TPM 1.2 and 2.0 chips with vendor-specific I²C interface