hello Usama,

On Tue, Sep 10, 2024 at 04:46:15PM +0100, Usama Arif wrote:
> --- a/drivers/firmware/efi/tpm.c
> +++ b/drivers/firmware/efi/tpm.c
> @@ -60,7 +60,9 @@ int __init efi_tpm_eventlog_init(void)
>       }
>  
>       tbl_size = sizeof(*log_tbl) + log_tbl->size;
> -     memblock_reserve(efi.tpm_log, tbl_size);
> +     if (!memblock_reserve(efi.tpm_log, tbl_size)) {
> +             arch_update_firmware_area(efi.tpm_log, tbl_size);
> +     }

Shouldn't you reserve the region into 8250 independently of
memblock_reserve() return value?

Thanks for the patch,
--breno


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120545): https://edk2.groups.io/g/devel/message/120545
Mute This Topic: https://groups.io/mt/108376671/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to