[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/5] acpi: Enable TPM IRQ
From: |
Auger Eric |
Subject: |
Re: [PATCH v2 4/5] acpi: Enable TPM IRQ |
Date: |
Tue, 16 Jun 2020 15:01:32 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi Stefan,
On 6/15/20 7:11 PM, Stefan Berger wrote:
> On 6/15/20 11:13 AM, Marc-André Lureau wrote:
>>
>>> diff --git a/include/hw/acpi/tpm.h b/include/hw/acpi/tpm.h
>>> index 1a2a57a21f..063a9eb42a 100644
>>> --- a/include/hw/acpi/tpm.h
>>> +++ b/include/hw/acpi/tpm.h
>>> @@ -24,7 +24,7 @@
>>> #define TPM_TIS_ADDR_BASE 0xFED40000
>>> #define TPM_TIS_ADDR_SIZE 0x5000
>>>
>>> -#define TPM_TIS_IRQ 5
>>> +#define TPM_TIS_IRQ 13
>
>
> Eric,
>
> does this change have any negative side effects on ARM? If you prefer,
> we can split this part here up into TPM_TIS_ISA_IRQ and TPM_TIS_SYSBUS
> IRQ and leave the latter at '5' because we know that this is working.
The IRQ is not advertised in dt nor ACPI on ARM. However it is
advertised in the capability reg and in the vector. reg So I think this
should be fixed? I guess on ARM we will pick up a completely different
IRQ num, allocated from the platform bus slot.
Thanks
Eric
>
> Stefan
>
>
>>>
>>> #define TPM_TIS_NUM_LOCALITIES 5 /* per spec */
>>> #define TPM_TIS_LOCALITY_SHIFT 12
>>> --
>>> 2.24.1
>>>
>
[PATCH v2 5/5] tests: Add updated DSDT, Stefan Berger, 2020/06/15
[PATCH v2 4/5] acpi: Enable TPM IRQ, Stefan Berger, 2020/06/15
[PATCH v2 3/5] tests: Temporarily ignore DSDT table differences, Stefan Berger, 2020/06/15