[coreboot] [commit] r5397 - in trunk/src: arch/i386/include/arch mainboard/asus/a8v-e_se mainboard/asus/m2v-mx_se mainboard/broadcom/blast northbridge/amd/amdk8 southbridge/broadcom/bcm5785 southbridge/intel/...

repository service svn at coreboot.org
Fri Apr 9 17:29:13 CEST 2010


Author: stepan
Date: Fri Apr  9 17:29:13 2010
New Revision: 5397
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5397

Log:
fix the broken nvidia chipset boards,
remove more warnings.
Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>

Modified:
   trunk/src/arch/i386/include/arch/stages.h
   trunk/src/mainboard/asus/a8v-e_se/acpi_tables.c
   trunk/src/mainboard/asus/a8v-e_se/mptable.c
   trunk/src/mainboard/asus/m2v-mx_se/mainboard.c
   trunk/src/mainboard/broadcom/blast/mptable.c
   trunk/src/mainboard/broadcom/blast/romstage.c
   trunk/src/northbridge/amd/amdk8/setup_resource_map.c
   trunk/src/southbridge/broadcom/bcm5785/bcm5785_early_setup.c
   trunk/src/southbridge/intel/i82801ax/i82801ax_smbus.h

Modified: trunk/src/arch/i386/include/arch/stages.h
==============================================================================
--- trunk/src/arch/i386/include/arch/stages.h	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/arch/i386/include/arch/stages.h	Fri Apr  9 17:29:13 2010	(r5397)
@@ -1,5 +1,25 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2010 coresystems GmbH
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#ifndef __ARCH_STAGES_H
+#define __ARCH_STAGES_H
 void cbfs_and_run_core(const char *filename, unsigned int ebp);
 void __attribute__((regparm(0))) copy_and_run(unsigned cpu_reset);
 void __attribute__((regparm(0))) copy_and_run_ap_code_in_car(unsigned ret_addr);
-
-
+#endif

Modified: trunk/src/mainboard/asus/a8v-e_se/acpi_tables.c
==============================================================================
--- trunk/src/mainboard/asus/a8v-e_se/acpi_tables.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/mainboard/asus/a8v-e_se/acpi_tables.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -87,8 +87,6 @@
 	acpi_srat_t *srat;
 	acpi_rsdt_t *rsdt;
 	acpi_mcfg_t *mcfg;
-	acpi_hpet_t *hpet;
-	acpi_madt_t *madt;
 	acpi_fadt_t *fadt;
 	acpi_facs_t *facs;
 	acpi_header_t *dsdt;

Modified: trunk/src/mainboard/asus/a8v-e_se/mptable.c
==============================================================================
--- trunk/src/mainboard/asus/a8v-e_se/mptable.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/mainboard/asus/a8v-e_se/mptable.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -29,7 +29,6 @@
 	static const char oem[8] = "LNXB    ";
 	static const char productid[12] = "A8V-E SE    ";
 	struct mp_config_table *mc;
-	unsigned int conforms = 0;
 	int bus_isa = 42;
 
 	mc = (void *)(((char *)v) + SMP_FLOATING_TABLE_LEN);

Modified: trunk/src/mainboard/asus/m2v-mx_se/mainboard.c
==============================================================================
--- trunk/src/mainboard/asus/m2v-mx_se/mainboard.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/mainboard/asus/m2v-mx_se/mainboard.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -21,8 +21,9 @@
 #include <device/pci.h>
 #include <device/pci_ids.h>
 #include <boot/tables.h>
+#include <arch/coreboot_tables.h>
 #include "chip.h"
-#include <../../../southbridge/via/k8t890/k8t890.h>
+#include <southbridge/via/k8t890/k8t890.h>
 
 int add_mainboard_resources(struct lb_memory *mem)
 {

Modified: trunk/src/mainboard/broadcom/blast/mptable.c
==============================================================================
--- trunk/src/mainboard/broadcom/blast/mptable.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/mainboard/broadcom/blast/mptable.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -62,7 +62,6 @@
 /*I/O APICs:	APIC ID	Version	State		Address*/
         {
                 device_t dev = 0;
-		int i;
 		struct resource *res;
 		for(i=0; i<3; i++) {
 	                dev = dev_find_device(0x1166, 0x0235, dev);

Modified: trunk/src/mainboard/broadcom/blast/romstage.c
==============================================================================
--- trunk/src/mainboard/broadcom/blast/romstage.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/mainboard/broadcom/blast/romstage.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -50,10 +50,10 @@
 static inline void activate_spd_rom(const struct mem_controller *ctrl)
 {
 #define SMBUS_HUB 0x71
-        int ret,i;
         unsigned device=(ctrl->channel0[0])>>8;
         smbus_send_byte(SMBUS_HUB, device);
 }
+
 #if 0
 static inline void change_i2c_mux(unsigned device)
 {
@@ -99,10 +99,8 @@
 	static const uint16_t spd_addr[] = {
         	RC0|DIMM0, RC0|DIMM2, 0, 0,
                 RC0|DIMM1, RC0|DIMM3, 0, 0,
-#if CONFIG_MAX_PHYSICAL_CPUS > 1
                 RC1|DIMM0, RC1|DIMM2, 0, 0,
                 RC1|DIMM1, RC1|DIMM3, 0, 0,
-#endif
 	};
 
         int needs_reset;

Modified: trunk/src/northbridge/amd/amdk8/setup_resource_map.c
==============================================================================
--- trunk/src/northbridge/amd/amdk8/setup_resource_map.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/northbridge/amd/amdk8/setup_resource_map.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -112,7 +112,7 @@
 #endif
 }
 
-#if defined(SOUTHBRIDGE_NVIDIA_MCP55) || defined(SOUTHBRIDGE_NVIDIA_CK804)
+#if defined(CONFIG_SOUTHBRIDGE_NVIDIA_MCP55) || defined(CONFIG_SOUTHBRIDGE_NVIDIA_CK804)
 static void setup_resource_map_x(const unsigned int *register_values, int max)
 {
 	int i;

Modified: trunk/src/southbridge/broadcom/bcm5785/bcm5785_early_setup.c
==============================================================================
--- trunk/src/southbridge/broadcom/bcm5785/bcm5785_early_setup.c	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/southbridge/broadcom/bcm5785/bcm5785_early_setup.c	Fri Apr  9 17:29:13 2010	(r5397)
@@ -3,6 +3,7 @@
  *  by yinghai.lu at amd.com
  */
 
+#include <reset.h>
 static void bcm5785_enable_rom(void)
 {
         unsigned char byte;

Modified: trunk/src/southbridge/intel/i82801ax/i82801ax_smbus.h
==============================================================================
--- trunk/src/southbridge/intel/i82801ax/i82801ax_smbus.h	Fri Apr  9 16:46:51 2010	(r5396)
+++ trunk/src/southbridge/intel/i82801ax/i82801ax_smbus.h	Fri Apr  9 17:29:13 2010	(r5397)
@@ -110,24 +110,21 @@
 	return byte;
 }
 
-/* This function is neither used nor tested by me (Corey Osgood), the author 
-(Yinghai) probably tested/used it on i82801er */
 static int do_smbus_write_block(unsigned device, unsigned length, unsigned cmd,
 				unsigned data1, unsigned data2)
 {
-#warning "do_smbus_write_block is commented out"
-	print_err("Untested smbus_write_block called\n");
-#if 0
 	unsigned char global_control_register;
 	unsigned char global_status_register;
 	unsigned char byte;
 	unsigned char stat;
 	int i;
 
+	print_err("Untested smbus_write_block called\n");
+
 	/* Clear the PM timeout flags, SECOND_TO_STS */
-	outw(inw(0x0400 + 0x66), 0x0400 + 0x66);
+	outw(inw(PMBASE_ADDR + 0x66), PMBASE_ADDR + 0x66);
 
-	if (smbus_wait_until_ready(SMBUS_IO_BASE) < 0) {
+	if (smbus_wait_until_ready() < 0) {
 		return -2;
 	}
 
@@ -159,9 +156,8 @@
 	     SMBUS_IO_BASE + SMBHSTCTL);
 
 	for (i = 0; i < length; i++) {
-
 		/* Poll for transaction completion */
-		if (smbus_wait_until_blk_done(SMBUS_IO_BASE) < 0) {
+		if (smbus_wait_until_blk_done() < 0) {
 			return -3;
 		}
 
@@ -179,5 +175,4 @@
 
 	print_debug("SMBUS Block complete\n");
 	return 0;
-#endif
 }




More information about the coreboot mailing list