| Message ID | 8886271d52025065eddf5915bb7778ab14362255.1705140898.git.lukas@wunner.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add missing TPM compatible strings | expand |
On Sat Jan 13, 2024 at 7:10 PM EET, Lukas Wunner wrote: > Commit 4f2a348aa365 ("arm64: dts: imx8mm-venice-gw73xx: add TPM device") > added a devicetree node for the Trusted Platform Module on certain > Gateworks boards. > > The commit only used the generic "tcg,tpm_tis-spi" compatible string, > but public documentation shows that the chip is an ATTPM20P from Atmel > (nowadays Microchip): > https://trac.gateworks.com/wiki/tpm > > Add the chip to the supported compatible strings of the TPM TIS SPI > schema. > > For reference, a datasheet is available at: > https://ww1.microchip.com/downloads/en/DeviceDoc/ATTPM20P-Trusted-Platform-Module-TPM-2.0-SPI-Interface-Summary-Data-Sheet-DS40002082A.pdf > > Signed-off-by: Lukas Wunner <lukas@wunner.de> > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > Cc: Tim Harvey <tharvey@gateworks.com> > --- > Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > index c3413b4..6cb2de7 100644 > --- a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > +++ b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > @@ -20,6 +20,7 @@ properties: > compatible: > items: > - enum: > + - atmel,attpm20p > - infineon,slb9670 > - st,st33htpm-spi > - st,st33zp24-spi You should to send the patch set with Rob Herring as CC for syncing up. Please do it for future versions, if there is need for additional versions. Rob, 3 out of 4 patches are TPM patches. Do you mind if I take all four patches once the patch set is ready or do you want to pick this patch (assuming that you think it is correctly implemented ofc)? BR, Jarkko
On Sat, Jan 13, 2024 at 11:17:39PM +0200, Jarkko Sakkinen wrote: > On Sat Jan 13, 2024 at 7:10 PM EET, Lukas Wunner wrote: > > Commit 4f2a348aa365 ("arm64: dts: imx8mm-venice-gw73xx: add TPM device") > > added a devicetree node for the Trusted Platform Module on certain > > Gateworks boards. > > > > The commit only used the generic "tcg,tpm_tis-spi" compatible string, > > but public documentation shows that the chip is an ATTPM20P from Atmel > > (nowadays Microchip): > > https://trac.gateworks.com/wiki/tpm > > > > Add the chip to the supported compatible strings of the TPM TIS SPI > > schema. > > > > For reference, a datasheet is available at: > > https://ww1.microchip.com/downloads/en/DeviceDoc/ATTPM20P-Trusted-Platform-Module-TPM-2.0-SPI-Interface-Summary-Data-Sheet-DS40002082A.pdf > > > > Signed-off-by: Lukas Wunner <lukas@wunner.de> > > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > > Cc: Tim Harvey <tharvey@gateworks.com> > > --- > > Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > > index c3413b4..6cb2de7 100644 > > --- a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > > +++ b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml > > @@ -20,6 +20,7 @@ properties: > > compatible: > > items: > > - enum: > > + - atmel,attpm20p > > - infineon,slb9670 > > - st,st33htpm-spi > > - st,st33zp24-spi > > You should to send the patch set with Rob Herring as CC for syncing > up. Please do it for future versions, if there is need for additional > versions. More specifically, use what get_maintainers.pl outputs. I don't actually care if I'm Cc'ed because I'm just Cc'ed on everything to the DT list. > Rob, 3 out of 4 patches are TPM patches. Do you mind if I take all > four patches once the patch set is ready or do you want to pick this > patch (assuming that you think it is correctly implemented ofc)? You can take it. Rob
On Sat, 13 Jan 2024 18:10:51 +0100, Lukas Wunner wrote: > Commit 4f2a348aa365 ("arm64: dts: imx8mm-venice-gw73xx: add TPM device") > added a devicetree node for the Trusted Platform Module on certain > Gateworks boards. > > The commit only used the generic "tcg,tpm_tis-spi" compatible string, > but public documentation shows that the chip is an ATTPM20P from Atmel > (nowadays Microchip): > https://trac.gateworks.com/wiki/tpm > > Add the chip to the supported compatible strings of the TPM TIS SPI > schema. > > For reference, a datasheet is available at: > https://ww1.microchip.com/downloads/en/DeviceDoc/ATTPM20P-Trusted-Platform-Module-TPM-2.0-SPI-Interface-Summary-Data-Sheet-DS40002082A.pdf > > Signed-off-by: Lukas Wunner <lukas@wunner.de> > Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > Cc: Tim Harvey <tharvey@gateworks.com> > --- > Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml | 1 + > 1 file changed, 1 insertion(+) > Acked-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml index c3413b4..6cb2de7 100644 --- a/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml +++ b/Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml @@ -20,6 +20,7 @@ properties: compatible: items: - enum: + - atmel,attpm20p - infineon,slb9670 - st,st33htpm-spi - st,st33zp24-spi
Commit 4f2a348aa365 ("arm64: dts: imx8mm-venice-gw73xx: add TPM device") added a devicetree node for the Trusted Platform Module on certain Gateworks boards. The commit only used the generic "tcg,tpm_tis-spi" compatible string, but public documentation shows that the chip is an ATTPM20P from Atmel (nowadays Microchip): https://trac.gateworks.com/wiki/tpm Add the chip to the supported compatible strings of the TPM TIS SPI schema. For reference, a datasheet is available at: https://ww1.microchip.com/downloads/en/DeviceDoc/ATTPM20P-Trusted-Platform-Module-TPM-2.0-SPI-Interface-Summary-Data-Sheet-DS40002082A.pdf Signed-off-by: Lukas Wunner <lukas@wunner.de> Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> Cc: Tim Harvey <tharvey@gateworks.com> --- Documentation/devicetree/bindings/tpm/tcg,tpm_tis-spi.yaml | 1 + 1 file changed, 1 insertion(+)