[coreboot] New patch to review for coreboot: 300422e AMD/Persimmon: LVDS assignment was made to wrong DPx

Dave Frodin (dave.frodin@se-eng.com) gerrit at coreboot.org
Fri Feb 1 16:22:59 CET 2013


Dave Frodin (dave.frodin at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2259

-gerrit

commit 300422e3da810eb4f6fc4d87a0e627a29a47d928
Author: Dave Frodin <dave.frodin at se-eng.com>
Date:   Fri Feb 1 08:21:50 2013 -0700

    AMD/Persimmon: LVDS assignment was made to wrong DPx
    
    The LVDS is on DP0, not DP1.
    
    Change-Id: I724764d0f013e7a10d974a8716e075139982ded2
    Signed-off-by: Dave Frodin <dave.frodin at se-eng.com>
---
 src/mainboard/amd/persimmon/PlatformGnbPcie.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/mainboard/amd/persimmon/PlatformGnbPcie.c b/src/mainboard/amd/persimmon/PlatformGnbPcie.c
index 5697d60..e3cba85 100644
--- a/src/mainboard/amd/persimmon/PlatformGnbPcie.c
+++ b/src/mainboard/amd/persimmon/PlatformGnbPcie.c
@@ -96,14 +96,14 @@ PCIe_DDI_DESCRIPTOR DdiList [] = {
 			0,	 //Descriptor flags
 			PCIE_ENGINE_DATA_INITIALIZER (PcieDdiEngine, 8, 11),
 			//PCIE_DDI_DATA_INITIALIZER (ConnectorTypeDP, Aux1, Hdp1)
-			{ConnectorTypeDP, Aux1, Hdp1}
+			{ConnectorTypeLvds, Aux1, Hdp1}
 		},
 		// Initialize Ddi descriptor (DDI interface Lanes 12:15, DdB, ...)
 		{
 			DESCRIPTOR_TERMINATE_LIST, //Descriptor flags	!!!IMPORTANT!!! Terminate last element of array
 			PCIE_ENGINE_DATA_INITIALIZER (PcieDdiEngine, 12, 15),
 			//PCIE_DDI_DATA_INITIALIZER (ConnectorTypeDP, Aux2, Hdp2)
-			{ConnectorTypeLvds, Aux2, Hdp2}
+			{ConnectorTypeDP, Aux2, Hdp2}
 		}
 };
 



More information about the coreboot mailing list