Re: [PATCH 1/3] Add support for TPM devices over I2C bus

[Ninad Palsule]

On 3/21/23 6:35 PM, Stefan Berger wrote:
> On 3/21/23 01:29, Ninad Palsule wrote:
> > This is a documentation change for I2C TPM device support.
> >
> > Qemu already supports devices attached to ISA and sysbus.
> > This drop adds support for the I2C bus attached TPM devices.
> >
> > Signed-off-by: Ninad Palsule <ninad@linux.ibm.com>
> > ---
> >   docs/specs/tpm.rst | 5 ++++-
> >   1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/docs/specs/tpm.rst b/docs/specs/tpm.rst
> > index 535912a92b..79a79f0640 100644
> > --- a/docs/specs/tpm.rst
> > +++ b/docs/specs/tpm.rst
> > @@ -21,11 +21,14 @@ QEMU files related to TPM TIS interface:
> >    - ``hw/tpm/tpm_tis_common.c``
> >    - ``hw/tpm/tpm_tis_isa.c``
> >    - ``hw/tpm/tpm_tis_sysbus.c``
> > + - ``hw/tpm/tpm_tis_i2c.c``
> >    - ``hw/tpm/tpm_tis.h``
> >     Both an ISA device and a sysbus device are available. The former is
> >   used with pc/q35 machine while the latter can be instantiated in the
> > -Arm virt machine.
> > +Arm virt machine. An I2C device support is also added which can be
> > +instantiated in the arm based emulation machine. I2C model only 
> > supports
> > +TPM2 protocol.
>
>
> An I2C device is also supported for the Arm virt machine. This device 
> only supports the TPM 2 protocol.
>
>    Stefan
Updated the document.

Thank you for the review!


Ninad Palsule


> >     CRB interface
> >   -------------