[coreboot-gerrit] Patch set updated for coreboot: southbridge/intel/bd82x6x: Use correct EHCI address for USBDEBUG
Tobias Diedrich (ranma+coreboot@tdiedrich.de)
gerrit at coreboot.org
Mon Feb 13 22:18:12 CET 2017
Tobias Diedrich (ranma+coreboot at tdiedrich.de) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18353
-gerrit
commit 8b5decaa195a87fad4c329b70ee073b1e854fec1
Author: Tobias Diedrich <ranma+openocd at tdiedrich.de>
Date: Sun Feb 12 22:58:37 2017 +0100
southbridge/intel/bd82x6x: Use correct EHCI address for USBDEBUG
The bd82x6x hardcoded EHCI bar adresses (0xe8000[04]00) don't match the
CONFIG_EHCI_BAR default of 0xfef00000.
This copies the defines from southbridge/intel/lynxpoint/early_usb.c
so that the early EHCI bar setup is using CONFIG_EHCI_BAR and
CONFIG_USBDEBUG_HCD_INDEX to determine the correct bar addresses.
Change-Id: I6ee6089c9e5f3bc5f457eae47c18c2473fa42571
Signed-off-by: Tobias Diedrich <ranma+coreboot at tdiedrich.de>
---
src/southbridge/intel/bd82x6x/early_usb_mrc.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/southbridge/intel/bd82x6x/early_usb_mrc.c b/src/southbridge/intel/bd82x6x/early_usb_mrc.c
index 8fac3c7..01d7692 100644
--- a/src/southbridge/intel/bd82x6x/early_usb_mrc.c
+++ b/src/southbridge/intel/bd82x6x/early_usb_mrc.c
@@ -20,8 +20,17 @@
#include <device/pci_def.h>
#include "pch.h"
-#define PCH_EHCI1_TEMP_BAR0 0xe8000000
-#define PCH_EHCI2_TEMP_BAR0 0xe8000400
+/*
+ * HCD_INDEX==2 selects 0:1a.0 (PCH_EHCI2), any other index
+ * selects 0:1d.0 (PCH_EHCI1) for usbdebug use.
+ */
+#if CONFIG_USBDEBUG_HCD_INDEX != 2
+#define PCH_EHCI1_TEMP_BAR0 CONFIG_EHCI_BAR
+#define PCH_EHCI2_TEMP_BAR0 (PCH_EHCI1_TEMP_BAR0 + 0x400)
+#else
+#define PCH_EHCI2_TEMP_BAR0 CONFIG_EHCI_BAR
+#define PCH_EHCI1_TEMP_BAR0 (PCH_EHCI2_TEMP_BAR0 + 0x400)
+#endif
/*
* Setup USB controller MMIO BAR to prevent the
More information about the coreboot-gerrit
mailing list