[coreboot-gerrit] Change in coreboot[master]: drivers/amd/agesa/s3_mtrr.c: Use a struct over pointer aritmetic

[Arthur Heymans (Code Review)]

Attention is currently required from: Kyösti Mälkki.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44293 )

Change subject: drivers/amd/agesa/s3_mtrr.c: Use a struct over pointer aritmetic
......................................................................

Patch Set 4:

(1 comment)

File src/drivers/amd/agesa/s3_mtrr.c:

https://review.coreboot.org/c/coreboot/+/44293/comment/77f59c52_c6f0959d 
PS4, Line 47: 		mtrr_save->mtrr_phys_mask[i] = rdmsr(MTRR_PHYS_BASE(i));
> MTRR_PHYS_MASK(i)
>
> Maybe you want to read BASE() first, althought it does not matter

You are right. It also looks like the default MTRR_DEF_TYPE_MSR is not backup/set?