From d380ca64d04c39055988c0dac2ea00e2b67a794a Mon Sep 17 00:00:00 2001 From: Nicholas Chin Date: Mon, 26 Aug 2024 19:36:30 -0600 Subject: mb/dell/snb_ivb_latitude: Move early_init.c out of variants Now that the USB configs are in the devicetree, only the bootblock_mainboard_early_init function remains in early_init.c. It is identical between every variant except the E6230, which enabled fewer decode ranges in the LPC_EN register. Enabling the additional decode ranges probably shouldn't cause issues, so go with the majority. TEST=Timeless builds do not change with the exception of the E6230. Change-Id: Ic43915888f5893652991b7402ebab3bd3a2cf278 Signed-off-by: Nicholas Chin Reviewed-on: https://review.coreboot.org/c/coreboot/+/84097 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Nico Huber --- src/mainboard/dell/snb_ivb_latitude/Makefile.mk | 4 ++-- src/mainboard/dell/snb_ivb_latitude/early_init.c | 14 ++++++++++++++ .../dell/snb_ivb_latitude/variants/e5420/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e5520/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e5530/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6220/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6230/early_init.c | 12 ------------ .../dell/snb_ivb_latitude/variants/e6320/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6330/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6420/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6430/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6520/early_init.c | 14 -------------- .../dell/snb_ivb_latitude/variants/e6530/early_init.c | 14 -------------- 13 files changed, 16 insertions(+), 154 deletions(-) create mode 100644 src/mainboard/dell/snb_ivb_latitude/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e5420/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e5520/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e5530/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6220/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6230/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6320/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6330/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6420/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6430/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6520/early_init.c delete mode 100644 src/mainboard/dell/snb_ivb_latitude/variants/e6530/early_init.c (limited to 'src') diff --git a/src/mainboard/dell/snb_ivb_latitude/Makefile.mk b/src/mainboard/dell/snb_ivb_latitude/Makefile.mk index 299fbb9bad..bd50ae7717 100644 --- a/src/mainboard/dell/snb_ivb_latitude/Makefile.mk +++ b/src/mainboard/dell/snb_ivb_latitude/Makefile.mk @@ -1,9 +1,9 @@ # SPDX-License-Identifier: GPL-2.0-only -bootblock-y += variants/$(VARIANT_DIR)/early_init.c +bootblock-y += early_init.c bootblock-y += variants/$(VARIANT_DIR)/gpio.c -romstage-y += variants/$(VARIANT_DIR)/early_init.c +romstage-y += early_init.c romstage-y += variants/$(VARIANT_DIR)/gpio.c ramstage-$(CONFIG_MAINBOARD_USE_LIBGFXINIT) += gma-mainboard.ads diff --git a/src/mainboard/dell/snb_ivb_latitude/early_init.c b/src/mainboard/dell/snb_ivb_latitude/early_init.c new file mode 100644 index 0000000000..ff83db095b --- /dev/null +++ b/src/mainboard/dell/snb_ivb_latitude/early_init.c @@ -0,0 +1,14 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#include +#include +#include +#include + +void bootblock_mainboard_early_init(void) +{ + pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN + | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN + | COMB_LPC_EN | COMA_LPC_EN); + mec5035_early_init(); +} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e5420/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e5420/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e5420/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e5520/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e5520/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e5520/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e5530/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e5530/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e5530/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6220/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6220/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6220/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6230/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6230/early_init.c deleted file mode 100644 index 24c1b32467..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6230/early_init.c +++ /dev/null @@ -1,12 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN | KBC_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6320/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6320/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6320/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6330/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6330/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6330/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6420/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6420/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6420/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6430/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6430/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6430/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6520/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6520/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6520/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} diff --git a/src/mainboard/dell/snb_ivb_latitude/variants/e6530/early_init.c b/src/mainboard/dell/snb_ivb_latitude/variants/e6530/early_init.c deleted file mode 100644 index ff83db095b..0000000000 --- a/src/mainboard/dell/snb_ivb_latitude/variants/e6530/early_init.c +++ /dev/null @@ -1,14 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include - -void bootblock_mainboard_early_init(void) -{ - pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN - | KBC_LPC_EN | FDD_LPC_EN | LPT_LPC_EN - | COMB_LPC_EN | COMA_LPC_EN); - mec5035_early_init(); -} -- cgit v1.2.3