diff options
| author | Lubomir Rintel <lkundrak@v3.sk> | 2017-04-03 15:35:09 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-04-14 17:20:26 +0200 |
| commit | b0161fd2d8679882a50d1f3ce55cdf69278aa32f (patch) | |
| tree | 4e1afe4ceae7eec8b832a61fa1de9da079b856ef /src/mainboard/via | |
| parent | 38d1eb44031eb023f00d52481fc77286d28e5157 (diff) | |
southbridge/via/vt8237r: Get rid of #include early_smbus.c
Use linker instead of '#include *.c'.
The smbus_fixup() was changed not to use a structure that's defined by a
northbridge since multiple different northbridges can be used. Instead
the caller now directly passed the memory slot details.
Change-Id: Ia369ece6365accbc531736fc463c713bbc134807
Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
Reviewed-on: https://review.coreboot.org/19082
Tested-by: build bot (Jenkins)
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/mainboard/via')
| -rw-r--r-- | src/mainboard/via/epia-cn/romstage.c | 4 | ||||
| -rw-r--r-- | src/mainboard/via/epia-m700/romstage.c | 2 | ||||
| -rw-r--r-- | src/mainboard/via/pc2500e/romstage.c | 4 |
3 files changed, 5 insertions, 5 deletions
diff --git a/src/mainboard/via/epia-cn/romstage.c b/src/mainboard/via/epia-cn/romstage.c index 5293aa6387..27c1a46c6c 100644 --- a/src/mainboard/via/epia-cn/romstage.c +++ b/src/mainboard/via/epia-cn/romstage.c @@ -26,7 +26,7 @@ #include <cpu/x86/bist.h> #include <cpu/amd/car.h> #include <delay.h> -#include "southbridge/via/vt8237r/early_smbus.c" +#include <southbridge/via/vt8237r/vt8237r.h> #include "southbridge/via/vt8237r/early_serial.c" #include <spd.h> @@ -80,7 +80,7 @@ void main(unsigned long bist) enable_vt8237r_serial(); console_init(); enable_smbus(); - smbus_fixup(&ctrl); + smbus_fixup(ctrl.channel0, ARRAY_SIZE(ctrl.channel0)); report_bist_failure(bist); enable_mainboard_devices(); ddr_ram_setup(&ctrl); diff --git a/src/mainboard/via/epia-m700/romstage.c b/src/mainboard/via/epia-m700/romstage.c index 1c10115492..57b59495c0 100644 --- a/src/mainboard/via/epia-m700/romstage.c +++ b/src/mainboard/via/epia-m700/romstage.c @@ -439,7 +439,7 @@ void main(unsigned long bist) enable_smbus(); /* This fix does help vx800!, but vx855 doesn't need this. */ - /* smbus_fixup(&ctrl); */ + /* smbus_fixup(ctrl.channel0, ARRAY_SIZE(ctrl.channel0)); */ /* Halt if there was a built-in self test failure. */ report_bist_failure(bist); diff --git a/src/mainboard/via/pc2500e/romstage.c b/src/mainboard/via/pc2500e/romstage.c index 2f002be40e..7a2edbfd43 100644 --- a/src/mainboard/via/pc2500e/romstage.c +++ b/src/mainboard/via/pc2500e/romstage.c @@ -26,7 +26,7 @@ #include <cpu/x86/bist.h> #include <cpu/amd/car.h> #include <delay.h> -#include "southbridge/via/vt8237r/early_smbus.c" +#include <southbridge/via/vt8237r/vt8237r.h> #include <superio/ite/common/ite.h> #include <superio/ite/it8716f/it8716f.h> #include <spd.h> @@ -56,7 +56,7 @@ void main(unsigned long bist) ite_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE); console_init(); enable_smbus(); - smbus_fixup(&ctrl); + smbus_fixup(ctrl.channel0, ARRAY_SIZE(ctrl.channel0)); report_bist_failure(bist); ddr_ram_setup(&ctrl); } |