Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/66698 )
Change subject: soc/intel/xeon_sp/skx: Remove nested check for ACPI support ......................................................................
soc/intel/xeon_sp/skx: Remove nested check for ACPI support
Remove redundant nested check for ACPI support.
Change-Id: Ie4b40382d304028135bcdd7851e2f48333570421 Signed-off-by: Marc Jones marcj303@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/66698 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Elyes Haouas ehaouas@noos.fr --- M src/soc/intel/xeon_sp/skx/chip.c 1 file changed, 15 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Elyes Haouas: Looks good to me, approved
diff --git a/src/soc/intel/xeon_sp/skx/chip.c b/src/soc/intel/xeon_sp/skx/chip.c index efdeef8..35e834a 100644 --- a/src/soc/intel/xeon_sp/skx/chip.c +++ b/src/soc/intel/xeon_sp/skx/chip.c @@ -28,10 +28,8 @@ .scan_bus = &xeonsp_pci_domain_scan_bus, #if CONFIG(HAVE_ACPI_TABLES) .write_acpi_tables = &northbridge_write_acpi_tables, - #if CONFIG(HAVE_ACPI_TABLES) .acpi_name = soc_acpi_name #endif -#endif };
static struct device_operations cpu_bus_ops = {