Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74610 )
Change subject: include/cpu/amd/mtrr: return uint32_t from get_top_of_mem_below_4gb ......................................................................
include/cpu/amd/mtrr: return uint32_t from get_top_of_mem_below_4gb
The top of memory below 4GB will always fit into 32 bits, so change the return type accordingly.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I6b463a17f2db3b7a99ff3572f318c9c22aac7431 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74610 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin Roth martin.roth@amd.corp-partner.google.com --- M src/include/cpu/amd/mtrr.h 1 file changed, 17 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/src/include/cpu/amd/mtrr.h b/src/include/cpu/amd/mtrr.h index b8b820c..e96dc90 100644 --- a/src/include/cpu/amd/mtrr.h +++ b/src/include/cpu/amd/mtrr.h @@ -65,7 +65,7 @@ ); }
-static inline uint64_t get_top_of_mem_below_4gb(void) +static inline uint32_t get_top_of_mem_below_4gb(void) { return rdmsr(TOP_MEM).lo; }