I checked the code and found a stupid bug.
I wonder if it can fix your problem.

Signed-off-by: Zheng Bao <[email protected]>


Index: src/southbridge/amd/rs780/rs780_cmn.c
===================================================================
--- src/southbridge/amd/rs780/rs780_cmn.c (revision 5926)
+++ src/southbridge/amd/rs780/rs780_cmn.c (working copy)
@@ -70,7 +70,7 @@
      reg &= ~mask;
      reg |= val;
      if (reg != reg_old) {
-           *((u32 *) addr) = val;
+           *((u32 *) addr) = reg;
      }
 }
 
Index: src/southbridge/amd/rs690/rs690_cmn.c
===================================================================
--- src/southbridge/amd/rs690/rs690_cmn.c (revision 5926)
+++ src/southbridge/amd/rs690/rs690_cmn.c (working copy)
@@ -72,7 +72,7 @@
      reg &= ~mask;
      reg |= val;
      if (reg != reg_old) {
-           *((u32 *) addr) = val;
+           *((u32 *) addr) = reg;
      }
 }
 
Zheng

Attachment: amd_RSxx0_pci_extension_access.patch
Description: amd_RSxx0_pci_extension_access.patch

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to