[coreboot-gerrit] [XS] Change in coreboot[main]: mb/dell/optiplex_9020: Fix 0-division and unintended truncation in pa...

[Máté Kukri (Code Review)]

Attention is currently required from: Angel Pons, Nico Huber.

Máté Kukri has posted comments on this change by Máté Kukri. ( https://review.coreboot.org/c/coreboot/+/83736?usp=email )

Change subject: mb/dell/optiplex_9020: Fix 0-division and unintended truncation in package power calculation
......................................................................

Patch Set 2:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83736/comment/9460423b_130ac5c8?usp... :
PS1, Line 7: UB challenged
> What does "UB challenged" mean?

Poor choice of words, fixed now.

File src/mainboard/dell/optiplex_9020/mainboard.c:

https://review.coreboot.org/c/coreboot/+/83736/comment/455d27dc_e9987bcf?usp... :
PS1, Line 306: unsigned int pkg_power, power_unit;
             : 	pkg_power = rdmsr(0x614).lo & 0x7fff;
             : 	power_unit = 1 << (rdmsr(0x606).lo & 0xf);
>

Done