[coreboot-gerrit] [M] Change in coreboot[master]: include/cpu/msr.h: transform into an union

[Angel Pons (Code Review)]

Attention is currently required from: Martin Roth, Arthur Heymans.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68572 )

Change subject: include/cpu/msr.h: transform into an union
......................................................................

Patch Set 3: Code-Review+2

(2 comments)

File src/include/cpu/x86/msr_access.h:

https://review.coreboot.org/c/coreboot/+/68572/comment/2d66d491_093917ce 
PS3, Line 10: 	struct {
            : 		unsigned int lo;
            : 		unsigned int hi;
            : 	};
            : 	uint64_t raw;
> lol, touche!

Don't worry, had the same question over here before remembering the portability of MSRs 😄

File src/soc/intel/common/block/sgx/sgx.c:

https://review.coreboot.org/c/coreboot/+/68572/comment/3f23409c_08fd0077 
PS3, Line 81: 	if ((prmrr_base.hi == 0) && (prmrr_base.lo == 0)
A bunch of these could be cleaned up in follow-ups.