soc/intel/common/block: fix storage size of HEST log address
This patch fixes the storage size to reflect the proper bits instead
of bytes. It was a bug in the initial HEST patch
(https://review.coreboot.org/c/coreboot/+/52090), and commit ID d4db36e672
. Also fixed the
comments to properly reflect the range being used.
Change-Id: I9e968bb09f1c9cd805ff1d0849551b9c2ce2e2b6
Signed-off-by: Rocky Phagura <rphagura@fb.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55393
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
36941bd672
commit
ff82accadb
|
@ -24,5 +24,5 @@ Field (GNVS, ByteAcc, NoLock, Preserve)
|
||||||
UIOR, 8, // 0x2f - UART debug controller init on S3 resume
|
UIOR, 8, // 0x2f - UART debug controller init on S3 resume
|
||||||
A4GB, 64, // 0x30 - 0x37 Base of above 4GB MMIO Resource
|
A4GB, 64, // 0x30 - 0x37 Base of above 4GB MMIO Resource
|
||||||
A4GS, 64, // 0x38 - 0x3f Length of above 4GB MMIO Resource
|
A4GS, 64, // 0x38 - 0x3f Length of above 4GB MMIO Resource
|
||||||
, 8, // 0x40 - 0x48 Hest log buffer (used in SMM, not ASL code)
|
, 64, // 0x40 - 0x47 Hest log buffer (used in SMM, not ASL code)
|
||||||
}
|
}
|
||||||
|
|
|
@ -26,7 +26,7 @@ struct __packed global_nvs {
|
||||||
u8 uior; /* 0x2f - UART debug controller init on S3 resume */
|
u8 uior; /* 0x2f - UART debug controller init on S3 resume */
|
||||||
u64 a4gb; /* 0x30 - 0x37 Base of above 4GB MMIO Resource */
|
u64 a4gb; /* 0x30 - 0x37 Base of above 4GB MMIO Resource */
|
||||||
u64 a4gs; /* 0x38 - 0x3f Length of above 4GB MMIO Resource */
|
u64 a4gs; /* 0x38 - 0x3f Length of above 4GB MMIO Resource */
|
||||||
u64 hest_log_addr; /* 0x40 - 48 err log addr (used in SMM, not ASL code) */
|
u64 hest_log_addr; /* 0x40 - 47 err log addr (used in SMM, not ASL code) */
|
||||||
};
|
};
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
Loading…
Reference in New Issue