Re: [PATCH v2 3/3] arm/acpi: Add the TPM2.0 device under the DSDT

[Stefan Berger]

On 5/8/20 11:25 AM, Ard Biesheuvel wrote:
> On Fri, 8 May 2020 at 17:24, Shannon Zhao <address@hidden> wrote:
> > Hi,
> >
> > On 2020/5/5 22:44, Eric Auger wrote:
> > > +static void acpi_dsdt_add_tpm(Aml *scope, VirtMachineState *vms)
> > > +{
> > > +    hwaddr pbus_base = vms->memmap[VIRT_PLATFORM_BUS].base;
> > > +    PlatformBusDevice *pbus = PLATFORM_BUS_DEVICE(vms->platform_bus_dev);
> > > +    MemoryRegion *sbdev_mr;
> > > +    SysBusDevice *sbdev;
> > > +    hwaddr tpm_base;
> > > +
> > > +    sbdev = (SysBusDevice *)object_dynamic_cast(OBJECT(tpm_find()),
> > > +                                                TYPE_SYS_BUS_DEVICE);
> > Does it need to check the tpm version like you do in previous patch?
> >
> > tpm_get_version(tpm_find()) == TPM_VERSION_2_0
> I don't think so. The device node could in theory be used to describe
> a TPM 1.2/1.3 as well, even though we never actually do that.

There is no TPM 1.3. There may be a TIS v1.3.