diff options
author | Angel Pons <th3fanbus@gmail.com> | 2020-05-21 16:30:00 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2020-05-26 11:47:19 +0000 |
commit | ec6e03e4d8b75af84bbc5698912202edf43ba20a (patch) | |
tree | e24fc485418c95301ef6225ba248323be509eb4a /src/superio | |
parent | 64829161111ea833453df8479d1bada1f91ee511 (diff) | |
download | coreboot-ec6e03e4d8b75af84bbc5698912202edf43ba20a.tar.xz |
AGESA f14/f15tn/f16kb: Deduplicate RAM settings
On AGESA f14/f15tn, various RAM-related options were defined in an enum.
However, the preprocessor mess can't compare enum values. To make AGESA
build, each board redefined them as macros, shadowing the enum elements.
Clean this up by replacing the enums with macros in AGESA headers, and
delete the now-redundant redefinitions from all the mainboards.
Note that AGESA f16kb already uses macros, but each mainboard still had
commented-out definitions. Remove them as well, as they are unnecessary.
TEST=Use abuild --timeless to check that all AGESA f14/f15tn/f16kb
mainboards result in identical coreboot binaries.
Change-Id: Ie1085539013d3ae0363b1596fa48555300e45172
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41666
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/superio')
0 files changed, 0 insertions, 0 deletions