Re: [PATCH] efi: Fix TPM code build failure on ARM
On Wed, Jun 05, 2019 at 11:11:40AM -0700, Matthew Garrett wrote: > asm/early_ioremap.h needs to be #included before tpm_eventlog.h in order > to ensure that early_memremap is available. > > Signed-off-by: Matthew Garrett Thanks, squashed to "tpm: Reserve the TPM final events table". /Jarkko
Re: [PATCH] efi: Fix TPM code build failure on ARM
On Thu, 6 Jun 2019 at 18:04, Matthew Garrett wrote: > > On Thu, Jun 6, 2019 at 4:39 AM Ard Biesheuvel > wrote: > > > > On Wed, 5 Jun 2019 at 20:11, Matthew Garrett > > wrote: > > > > > > asm/early_ioremap.h needs to be #included before tpm_eventlog.h in order > > > to ensure that early_memremap is available. > > > > > > > Doesn't that make it tpm_eventlog.h's job to #include it? > > tpm_eventlog.h doesn't use early_memremap directly, it's expanded from > the macros declared in tpm.c . Fair enough Acked-by: Ard Biesheuvel
Re: [PATCH] efi: Fix TPM code build failure on ARM
On Thu, Jun 6, 2019 at 4:39 AM Ard Biesheuvel wrote: > > On Wed, 5 Jun 2019 at 20:11, Matthew Garrett > wrote: > > > > asm/early_ioremap.h needs to be #included before tpm_eventlog.h in order > > to ensure that early_memremap is available. > > > > Doesn't that make it tpm_eventlog.h's job to #include it? tpm_eventlog.h doesn't use early_memremap directly, it's expanded from the macros declared in tpm.c .
Re: [PATCH] efi: Fix TPM code build failure on ARM
On Wed, 5 Jun 2019 at 20:11, Matthew Garrett wrote: > > asm/early_ioremap.h needs to be #included before tpm_eventlog.h in order > to ensure that early_memremap is available. > Doesn't that make it tpm_eventlog.h's job to #include it? > Signed-off-by: Matthew Garrett > --- > drivers/firmware/efi/tpm.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/firmware/efi/tpm.c b/drivers/firmware/efi/tpm.c > index 0bdceb5913aa..1d3f5ca3eaaf 100644 > --- a/drivers/firmware/efi/tpm.c > +++ b/drivers/firmware/efi/tpm.c > @@ -7,13 +7,12 @@ > #define TPM_MEMREMAP(start, size) early_memremap(start, size) > #define TPM_MEMUNMAP(start, size) early_memunmap(start, size) > > +#include > #include > #include > #include > #include > > -#include > - > int efi_tpm_final_log_size; > EXPORT_SYMBOL(efi_tpm_final_log_size); > > -- > 2.22.0.rc1.311.g5d7573a151-goog >
[PATCH] efi: Fix TPM code build failure on ARM
asm/early_ioremap.h needs to be #included before tpm_eventlog.h in order to ensure that early_memremap is available. Signed-off-by: Matthew Garrett --- drivers/firmware/efi/tpm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/firmware/efi/tpm.c b/drivers/firmware/efi/tpm.c index 0bdceb5913aa..1d3f5ca3eaaf 100644 --- a/drivers/firmware/efi/tpm.c +++ b/drivers/firmware/efi/tpm.c @@ -7,13 +7,12 @@ #define TPM_MEMREMAP(start, size) early_memremap(start, size) #define TPM_MEMUNMAP(start, size) early_memunmap(start, size) +#include #include #include #include #include -#include - int efi_tpm_final_log_size; EXPORT_SYMBOL(efi_tpm_final_log_size); -- 2.22.0.rc1.311.g5d7573a151-goog