[coreboot] [commit] r5846 - trunk/src/northbridge/amd/amdht

[repository service]

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 <patrick.georgi at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>

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");