[coreboot] [PATCH] Kconfig HT

Patrick Georgi patrick at georgi-clan.de
Thu Sep 24 16:32:45 CEST 2009


Am Donnerstag, den 24.09.2009, 08:19 -0600 schrieb mylesgw:
> There's no reason for a board without HyperTransport to compile it in.
> This patch removes it (saving 1K after compression.)
> 
> I would have liked to set HYPERTRANSPORT_PLUGIN_SUPPORT the same way,
which way?
would adding "select HYPERTRANSPORT_PLUGIN_SUPPORT" to all HT aware
northbridges work for you?

--- svn.orig/src/devices/Makefile.inc
+++ svn/src/devices/Makefile.inc
@@ -2,7 +2,9 @@ obj-y += device.o
 obj-y += root_device.o
 obj-y += device_util.o
 obj-y += pci_device.o
-obj-y += hypertransport.o
+ifdef CONFIG_HT_CHAIN_UNITID_BASE
+       obj-y += hypertransport.o
+endif

maybe replace that with
obj-$(CONFIG_HT_CHAIN_UNITID_BASE) += hypertransport.o?
or rather use HYPERTRANSPORT_PLUGIN_SUPPORT, once it's defined, but the
pattern still applies..?

> Signed-off-by: Myles Watson <mylesgw at gmail.com>
Acked-by: Patrick Georgi <patrick.georgi at coresystems.de>






More information about the coreboot mailing list