Edward O'Callaghan submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Sam McNally: Looks good to me, approved
Makefile: Remove 'NEED_LIBUSB1' from lspcon requirements

Since lspcon talks over i2c I doubt libusb is a build
requirement, remove it.

Change-Id: Ic4d71c10d2d8c0c881aa5732daed35c20d905a5e
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/41020
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sam McNally <sammc@google.com>
---
M Makefile
1 file changed, 0 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 31e987c..803529f 100644
--- a/Makefile
+++ b/Makefile
@@ -955,7 +955,6 @@
ifeq ($(CONFIG_LSPCON_I2C_SPI), yes)
FEATURE_CFLAGS += -D'CONFIG_LSPCON_I2C_SPI=1'
PROGRAMMER_OBJS += lspcon_i2c_spi.o
-NEED_LIBUSB1 += CONFIG_LSPCON_I2C_SPI
endif

ifeq ($(CONFIG_REALTEK_MST_I2C_SPI), yes)

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic4d71c10d2d8c0c881aa5732daed35c20d905a5e
Gerrit-Change-Number: 41020
Gerrit-PatchSet: 3
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Edward Hill <ecgh@chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Sam McNally <sammc@google.com>
Gerrit-Reviewer: Shiyu Sun <sshiyu@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged