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