[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 12 Dec 2016 19:46:15 +0200
From: Jarkko Sakkinen <jarkko.sakkinen@...ux.intel.com>
To: "Winkler, Tomas" <tomas.winkler@...el.com>
Cc: Peter Huewe <peterhuewe@....de>,
open list <linux-kernel@...r.kernel.org>,
"linux-security-module@...r.kernel.org"
<linux-security-module@...r.kernel.org>,
"moderated list:TPM DEVICE DRIVER"
<tpmdd-devel@...ts.sourceforge.net>
Subject: Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers
On Mon, Dec 12, 2016 at 03:28:01PM +0000, Winkler, Tomas wrote:
> >
> > I've tested is completley disjoint from the region pointed by the ACPI device
> > tree (SSDT in PTT's case).
> >
> > Here's example device from my x250:
> >
> > Device (TPM)
> > {
> > Name (_HID, "MSFT0101" /* TPM 2.0 Security Device */) // _HID: Hardware ID
> > Name (_STR, Unicode ("TPM 2.0 Device")) // _STR: Description String Name
> > (CRS, ResourceTemplate () { Memory32Fixed (ReadOnly,
> > 0x00000000, // Address Base
> > 0x00001000, // Address Length
> > _Y00)
>
>
> I'm not sure what is that?
>
> > Memory32Fixed (ReadOnly,
> > 0xFED70000, // Address Base
> > 0x00001000, // Address Length
> > )
>
> But 0xFED70000 should point to CCA
When I implemented originally the driver I followed the MS specification
and used the value from TPM2 ACPI table.
The driver ioremaps 00000000CCDFF000, not the address from SSDT.
> Tomas
> Thanks
/Jarkko
Powered by blists - more mailing lists