summaryrefslogtreecommitdiff
path: root/src/include
diff options
context:
space:
mode:
authorJacob Garber <jgarber1@ualberta.ca>2019-06-05 16:32:28 -0600
committerMartin Roth <martinroth@google.com>2019-06-28 19:30:09 +0000
commitf77f7cdf891a3ab47f71eb392e4229ece33f27ac (patch)
treeda68aa379496d3d822889a8323ad9c1370477eec /src/include
parentd10680bbbfc045627625ca0ce79e04491978cedf (diff)
downloadcoreboot-f77f7cdf891a3ab47f71eb392e4229ece33f27ac.tar.xz
device,nb/amd: Deduplicate add_more_links()
This function is duplicated in many AMD northbridge files, and all the definitions have started to diverge somewhat. This moves a single copy into device utils and deletes the rest. The function definition from nb/amd/amdfam10 was chosen to be kept, since it contains several fixes from commit 59d609217b (AMD fam10: Fix add_more_links) that the others don't have. For the ease of diffing, the checkpatch lints and other small cleanups will be done in a follow-up patch. Change-Id: I5ea73126092449db3f18e19ac0660439313072de Signed-off-by: Jacob Garber <jgarber1@ualberta.ca> Reviewed-on: https://review.coreboot.org/c/coreboot/+/33237 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/include')
-rw-r--r--src/include/device/device.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/include/device/device.h b/src/include/device/device.h
index 32cf07282b..52635e1bd2 100644
--- a/src/include/device/device.h
+++ b/src/include/device/device.h
@@ -191,6 +191,7 @@ const char *bus_path(struct bus *bus);
void dev_set_enabled(struct device *dev, int enable);
void disable_children(struct bus *bus);
bool dev_is_active_bridge(struct device *dev);
+void add_more_links(struct device *dev, unsigned total_links);
/* Option ROM helper functions */
void run_bios(struct device *dev, unsigned long addr);