Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 07:53:06PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 03:57:54PM +, Winkler, Tomas wrote: > > > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > > > > > In order to provide access to locality registers, this commits > > > >

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 07:53:06PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 03:57:54PM +, Winkler, Tomas wrote: > > > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > > > > > In order to provide access to locality registers, this commits > > > >

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 03:57:54PM +, Winkler, Tomas wrote: > > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > > > In order to provide access to locality registers, this commits > > > > > adds mapping of the head of the CRB registers, which are located right

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 03:57:54PM +, Winkler, Tomas wrote: > > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > > > In order to provide access to locality registers, this commits > > > > > adds mapping of the head of the CRB registers, which are located right

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 03:28:01PM +, 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

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 03:28:01PM +, 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

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > In order to provide access to locality registers, this commits > > > > adds mapping of the head of the CRB registers, which are located right > before the control area. > > > > > > > > Signed-off-by: Jarkko

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > In order to provide access to locality registers, this commits > > > > adds mapping of the head of the CRB registers, which are located right > before the control area. > > > > > > > > Signed-off-by: Jarkko

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 05:31:08PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 05:20:01PM +0200, Jarkko Sakkinen wrote: > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > In order to provide access to locality registers, this commits adds > > > >

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 05:31:08PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 05:20:01PM +0200, Jarkko Sakkinen wrote: > > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > > > In order to provide access to locality registers, this commits adds > > > >

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 05:20:01PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > In order to provide access to locality registers, this commits adds > > > mapping of > > > the head of the CRB registers, which are located right before

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 05:20:01PM +0200, Jarkko Sakkinen wrote: > On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > > > In order to provide access to locality registers, this commits adds > > > mapping of > > > the head of the CRB registers, which are located right before

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > 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")) //

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > 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")) //

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > In order to provide access to locality registers, this commits adds mapping > > of > > the head of the CRB registers, which are located right before the control > > area. > > > > Signed-off-by: Jarkko Sakkinen

Re: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

On Mon, Dec 12, 2016 at 02:25:32PM +, Winkler, Tomas wrote: > > > > In order to provide access to locality registers, this commits adds mapping > > of > > the head of the CRB registers, which are located right before the control > > area. > > > > Signed-off-by: Jarkko Sakkinen > > --- > >

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > In order to provide access to locality registers, this commits adds mapping of > the head of the CRB registers, which are located right before the control > area. > > Signed-off-by: Jarkko Sakkinen > --- > drivers/char/tpm/tpm_crb.c | 89

RE: [tpmdd-devel] [PATCH v3 1/3] tpm_crb: map locality registers

> > In order to provide access to locality registers, this commits adds mapping of > the head of the CRB registers, which are located right before the control > area. > > Signed-off-by: Jarkko Sakkinen > --- > drivers/char/tpm/tpm_crb.c | 89 + > - >