From 6e1c471f701b8ba1e9c07a87ecc73c8d855478dd Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Fri, 3 Jul 2020 13:05:10 +0200 Subject: haswell: Turn RCBA configuration into a function Instead of passing around a pointer to an array, just write the relevant registers directly. Note that intel/baskingridge used spaces to indent line continuations and had to be replaced with tabs to quell Jenkins. Change-Id: Ifa06a2ab24da9b8c6aac6480542fa32d04f6d6fe Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/43097 Tested-by: build bot (Jenkins) Reviewed-by: Tristan Corrick Reviewed-by: Arthur Heymans --- src/southbridge/intel/lynxpoint/early_pch.c | 5 ++--- src/southbridge/intel/lynxpoint/pch.h | 4 ++-- 2 files changed, 4 insertions(+), 5 deletions(-) (limited to 'src/southbridge/intel/lynxpoint') diff --git a/src/southbridge/intel/lynxpoint/early_pch.c b/src/southbridge/intel/lynxpoint/early_pch.c index c6999a4fe7..4d29564f3d 100644 --- a/src/southbridge/intel/lynxpoint/early_pch.c +++ b/src/southbridge/intel/lynxpoint/early_pch.c @@ -77,8 +77,7 @@ void __weak mainboard_config_superio(void) { } -int early_pch_init(const void *gpio_map, - const struct rcba_config_instruction *rcba_config) +int early_pch_init(const void *gpio_map) { int wake_from_s3; @@ -101,7 +100,7 @@ int early_pch_init(const void *gpio_map, (void) RCBA16(OIC); /* Mainboard RCBA settings */ - pch_config_rcba(rcba_config); + mainboard_config_rcba(); RCBA32_OR(FD, PCH_DISABLE_ALWAYS); diff --git a/src/southbridge/intel/lynxpoint/pch.h b/src/southbridge/intel/lynxpoint/pch.h index 093ebfaf7e..77377c43c7 100644 --- a/src/southbridge/intel/lynxpoint/pch.h +++ b/src/southbridge/intel/lynxpoint/pch.h @@ -161,10 +161,10 @@ void acpi_create_intel_hpet(acpi_hpet_t * hpet); void acpi_create_serialio_ssdt(acpi_header_t *ssdt); void enable_usb_bar(void); -int early_pch_init(const void *gpio_map, - const struct rcba_config_instruction *rcba_config); +int early_pch_init(const void *gpio_map); void pch_enable_lpc(void); void mainboard_config_superio(void); +void mainboard_config_rcba(void); #define MAINBOARD_POWER_OFF 0 #define MAINBOARD_POWER_ON 1 -- cgit v1.2.3