diff options
| author | Marshall Dawson <marshalldawson3rd@gmail.com> | 2017-08-16 17:21:41 -0600 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-08-24 11:48:38 +0000 |
| commit | 23e5ba9a733272562cc69b031055c6a7149cbabc (patch) | |
| tree | 48f57200052f885b6be742645d2ec226efacc7e0 /src/mainboard/pcengines | |
| parent | 6dd620bf107990e5cd2b64648925edad5c1cbc3f (diff) | |
| download | coreboot-23e5ba9a733272562cc69b031055c6a7149cbabc.tar.xz | |
binarypi mainboards: Clean up IS_ENABLED fan control
Remove all checks for #if IS_ENABLED(CONFIG_HUDSON_IMC_FWM) around
the imc.h includes. Convert from #if to if() for all fan control
setup code. Where necessary, make functions non-static to match the
prototypes in imc.h.
Change-Id: If88af42d00227285931829441909a982fc292b2b
Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-on: https://review.coreboot.org/21058
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'src/mainboard/pcengines')
| -rw-r--r-- | src/mainboard/pcengines/apu2/BiosCallOuts.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/mainboard/pcengines/apu2/BiosCallOuts.c b/src/mainboard/pcengines/apu2/BiosCallOuts.c index a239b476e4..ec9c71966c 100644 --- a/src/mainboard/pcengines/apu2/BiosCallOuts.c +++ b/src/mainboard/pcengines/apu2/BiosCallOuts.c @@ -23,9 +23,7 @@ #include "FchPlatform.h" #include "cbfs.h" #include "gpio_ftns.h" -#if IS_ENABLED(CONFIG_HUDSON_IMC_FWM) #include "imc.h" -#endif #include "hudson.h" #include <stdlib.h> @@ -55,7 +53,7 @@ const int BiosCalloutsLen = ARRAY_SIZE(BiosCallouts); * software switches the I2C address. AMD recommends using IMC * to control fans, instead of HWM. */ -static void oem_fan_control(FCH_DATA_BLOCK *FchParams) +void oem_fan_control(FCH_DATA_BLOCK *FchParams) { FchParams->Imc.ImcEnable = FALSE; FchParams->Hwm.HwMonitorEnable = FALSE; |