Author: oxygene
Date: Sat Sep 25 19:24:10 2010
New Revision: 5846
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5846

Log:
Mark read-only data as read-only, so the global vars test doesn't fail on it.

Signed-off-by: Patrick Georgi <[email protected]>
Acked-by: Stefan Reinauer <[email protected]>

Modified:
   trunk/src/northbridge/amd/amdht/h3finit.c
   trunk/src/northbridge/amd/amdht/h3finit.h
   trunk/src/northbridge/amd/amdht/ht_wrapper.c

Modified: trunk/src/northbridge/amd/amdht/h3finit.c
==============================================================================
--- trunk/src/northbridge/amd/amdht/h3finit.c   Sat Sep 25 19:01:13 2010        
(r5845)
+++ trunk/src/northbridge/amd/amdht/h3finit.c   Sat Sep 25 19:24:10 2010        
(r5846)
@@ -899,7 +899,7 @@
        u32 unitIDcnt;
        SBDFO currentPtr;
        u8 depth;
-       u8 *pSwapPtr;
+       const u8 *pSwapPtr;
 
        SBDFO lastSBDFO = ILLEGAL_SBDFO;
        u8 lastLink = 0;

Modified: trunk/src/northbridge/amd/amdht/h3finit.h
==============================================================================
--- trunk/src/northbridge/amd/amdht/h3finit.h   Sat Sep 25 19:01:13 2010        
(r5845)
+++ trunk/src/northbridge/amd/amdht/h3finit.h   Sat Sep 25 19:24:10 2010        
(r5846)
@@ -195,7 +195,7 @@
         *
         * 
---------------------------------------------------------------------------------------
         */
-       BOOL (*AMD_CB_ManualBUIDSwapList)(u8 Node, u8 Link, u8 **List);
+       BOOL (*AMD_CB_ManualBUIDSwapList)(u8 Node, u8 Link, const u8 **List);
 
        
/**----------------------------------------------------------------------------------------
         *

Modified: trunk/src/northbridge/amd/amdht/ht_wrapper.c
==============================================================================
--- trunk/src/northbridge/amd/amdht/ht_wrapper.c        Sat Sep 25 19:01:13 
2010        (r5845)
+++ trunk/src/northbridge/amd/amdht/ht_wrapper.c        Sat Sep 25 19:24:10 
2010        (r5846)
@@ -117,9 +117,9 @@
  *     @param[out] BOOL result = true to use a manual list
  *                               false to initialize the link automatically
  */
-static BOOL AMD_CB_ManualBUIDSwapList (u8 node, u8 link, u8 **List)
+static BOOL AMD_CB_ManualBUIDSwapList (u8 node, u8 link, const u8 **List)
 {
-       static u8 swaplist[] = { 0xFF, CONFIG_HT_CHAIN_UNITID_BASE, 
CONFIG_HT_CHAIN_END_UNITID_BASE, 0xFF };
+       static const u8 swaplist[] = { 0xFF, CONFIG_HT_CHAIN_UNITID_BASE, 
CONFIG_HT_CHAIN_END_UNITID_BASE, 0xFF };
        /* If the BUID was adjusted in early_ht we need to do the manual 
override */
        if ((CONFIG_HT_CHAIN_UNITID_BASE != 0) && 
(CONFIG_HT_CHAIN_END_UNITID_BASE != 0)) {
                printk(BIOS_DEBUG, "AMD_CB_ManualBUIDSwapList()\n");

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

Reply via email to