Edward O'Callaghan submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Edward O'Callaghan: Looks good to me, approved
Mark ENE_LPC and MEC1308 as NEED_LIBPCI

Both depend on functions from chipset_enable.c and board_enable.c, which
require libpci.

Signed-off-by: Victor Ding <victording@google.com>
Change-Id: I9505d7f18f4781a264e29e9667b717c061ba33b8
Reviewed-on: https://review.coreboot.org/c/flashrom/+/46812
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M Makefile
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/Makefile b/Makefile
index 3a0748c..241814a 100644
--- a/Makefile
+++ b/Makefile
@@ -823,8 +823,10 @@
override CONFIG_ATAHPT = no
override CONFIG_ATAVIA = no
override CONFIG_ATAPROMISE = no
+override CONFIG_ENE_LPC = no
override CONFIG_IT8212 = no
override CONFIG_DRKAISER = no
+override CONFIG_MEC1308 = no
override CONFIG_NICREALTEK = no
override CONFIG_NICNATSEMI = no
override CONFIG_NICINTEL = no
@@ -899,12 +901,14 @@
FEATURE_CFLAGS += -D'CONFIG_ENE_LPC=1'
PROGRAMMER_OBJS += ene_lpc.o
NEED_RAW_ACCESS += CONFIG_ENE_LPC
+NEED_LIBPCI += CONFIG_ENE_LPC
endif

ifeq ($(CONFIG_MEC1308), yes)
FEATURE_CFLAGS += -D'CONFIG_MEC1308=1'
PROGRAMMER_OBJS += mec1308.o
NEED_RAW_ACCESS += CONFIG_MEC1308
+NEED_LIBPCI += CONFIG_MEC1308
endif

ifeq ($(CONFIG_SERPROG), yes)

To view, visit change 46812. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I9505d7f18f4781a264e29e9667b717c061ba33b8
Gerrit-Change-Number: 46812
Gerrit-PatchSet: 3
Gerrit-Owner: Victor Ding <victording@google.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Septs <github@septs.pw>
Gerrit-MessageType: merged