diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2023-03-24 00:25:39 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-03-27 12:13:43 +0000 |
| commit | 00d496f92fdf600c0fd674b9bc9d5b0b471b1b06 (patch) | |
| tree | dd2a18aa32d391aa5005147e648594f75c6a219d /src/soc/amd | |
| parent | 509321f5a815f67e5d22526fdfbd94519f23bd7d (diff) | |
| download | coreboot-00d496f92fdf600c0fd674b9bc9d5b0b471b1b06.tar.gz coreboot-00d496f92fdf600c0fd674b9bc9d5b0b471b1b06.tar.bz2 coreboot-00d496f92fdf600c0fd674b9bc9d5b0b471b1b06.zip | |
soc/amd/common/block/cpu/svi2: drop unneeded core_vid check
A core voltage ID larger than 0xff shouldn't happen, since SVI2's core
VID is only 8 bit long. In order for making it more difficult to use
this function in a wrong way that results in a very wrong voltage being
returned, also return 0 for those invalid core VID values.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I95417c45db86cd2373879cdad8a07fb9eb8dfdda
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74000
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd')
| -rw-r--r-- | src/soc/amd/common/block/cpu/svi2.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/amd/common/block/cpu/svi2.c b/src/soc/amd/common/block/cpu/svi2.c index 21d459fb2a1e..2e8766cf327b 100644 --- a/src/soc/amd/common/block/cpu/svi2.c +++ b/src/soc/amd/common/block/cpu/svi2.c @@ -10,8 +10,8 @@ uint32_t get_uvolts_from_vid(uint16_t core_vid) { - if ((core_vid >= 0xF8) && (core_vid <= 0xFF)) { - /* Voltage off for VID codes 0xF8 to 0xFF */ + if (core_vid >= 0xF8) { + /* Voltage off for VID codes >= 0xF8 */ return 0; } else { return SERIAL_VID_2_MAX_MICROVOLTS - |