On Mon, Dec 1, 2008 at 2:15 PM, Myles Watson <[EMAIL PROTECTED]> wrote:
>
>
> On Mon, Dec 1, 2008 at 1:52 PM, ron minnich <[EMAIL PROTECTED]> wrote:
>
>> did you need to do an svn add for the device code?
>
>
> I don't think so. They were already there.
>
I guess they were only there in my tree :)
Sorry about that. New patch attached.
Signed-off-by: Myles Watson <[EMAIL PROTECTED]>
Thanks,
Myles
Index: svn/include/arch/x86/amd/k8/k8.h
===================================================================
--- svn.orig/include/arch/x86/amd/k8/k8.h
+++ svn/include/arch/x86/amd/k8/k8.h
@@ -580,7 +580,7 @@ enum {
/* note: we'd like to have this sysinfo common to all K8, there's no need to
* have one different kind per different kind of k8 at this point.
*/
-//#include "raminit.h"
+#include "raminit.h"
struct dimm_size {
u8 per_rank; // it is rows + col + bank_lines + data lines */
@@ -608,14 +608,6 @@ struct mem_info { // pernode
u8 rsv[3];
} __attribute__((packed));
-struct mem_controller {
- unsigned node_id;
- /* NOTE: these are in "BDF" format -- i.e. bus in upper 16, devfn in upper 8 of lower 16 */
- u32 f0, f1, f2, f3;
- u32 channel0[DIMM_SOCKETS];
- u32 channel1[DIMM_SOCKETS];
-};
-
struct link_pair_st {
u32 udev;
u32 upos;
@@ -685,8 +677,9 @@ unsigned int get_core_count(void);
void fill_mem_ctrl(int controllers, struct mem_controller *ctrl_a,
const u16 *spd_addr);
int lapic_remote_read(int apicid, int reg, unsigned int *pvalue);
-void print_apicid_nodeid_coreid(unsigned apicid, struct node_core_id id,
+/*void print_apicid_nodeid_coreid(unsigned apicid, struct node_core_id id,
const char *str);
+*/
unsigned int wait_cpu_state(unsigned apicid, unsigned state);
void set_apicid_cpuid_lo(void);
/* fidvid.c */
Index: svn/mainboard/amd/serengeti/mainboard.h
===================================================================
--- svn.orig/mainboard/amd/serengeti/mainboard.h
+++ svn/mainboard/amd/serengeti/mainboard.h
@@ -21,8 +21,6 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
-#define DIMM_SOCKETS 4
-#define NODE_NUMS 16
#define CPU_SOCKET_TYPE SOCKET_AM2
#define MEM_TRAIN_SEQ 0 /* for now */
#define HW_MEM_HOLE_SIZE_AUTO_INC 0
Index: svn/northbridge/amd/k8/Makefile
===================================================================
--- svn.orig/northbridge/amd/k8/Makefile
+++ svn/northbridge/amd/k8/Makefile
@@ -27,6 +27,8 @@ STAGE2_CHIPSET_SRC += $(src)/northbridge
$(src)/northbridge/amd/k8/cpu.c \
$(src)/northbridge/amd/k8/domain.c \
$(src)/northbridge/amd/k8/pci.c \
+ $(src)/northbridge/amd/k8/mcf3.c \
+ $(src)/northbridge/amd/k8/apic.c \
$(src)/northbridge/amd/k8/util.c
endif
Index: svn/northbridge/amd/k8/apic
===================================================================
--- /dev/null
+++ svn/northbridge/amd/k8/apic
@@ -0,0 +1,24 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2008 Ronald G. Minnich <[EMAIL PROTECTED]>
+ *
+ * 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; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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
+ */
+
+{
+ /* Miscellaneous Control Function 3 for Athlon/Opteron. */
+ device_operations = "k8_apic_ops";
+};
Index: svn/northbridge/amd/k8/apic.c
===================================================================
--- /dev/null
+++ svn/northbridge/amd/k8/apic.c
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2008 Ronald G. Minnich <[EMAIL PROTECTED]>
+ *
+ * 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; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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
+ */
+
+#include <device/device.h>
+
+struct device_operations k8_apic_ops = {
+ .id = {.type = DEVICE_ID_APIC},
+ .ops_pci = 0,
+};
Index: svn/northbridge/amd/k8/mcf3
===================================================================
--- /dev/null
+++ svn/northbridge/amd/k8/mcf3
@@ -0,0 +1,24 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2008 Ronald G. Minnich <[EMAIL PROTECTED]>
+ *
+ * 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; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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
+ */
+
+{
+ /* Miscellaneous Control Function 3 for Athlon/Opteron. */
+ device_operations = "mcf3_ops";
+};
Index: svn/northbridge/amd/k8/mcf3.c
===================================================================
--- /dev/null
+++ svn/northbridge/amd/k8/mcf3.c
@@ -0,0 +1,241 @@
+/* Turn off machine check triggers when reading
+ * pci space where there are no devices.
+ * This is necessary when scaning the bus for
+ * devices which is done by the kernel
+ *
+ * written in 2003 by Eric Biederman
+ *
+ * - Athlon64 workarounds by Stefan Reinauer
+ * - "reset once" logic by Yinghai Lu
+ */
+
+#include <console.h>
+#include <lib.h>
+#include <device/device.h>
+#include <device/pci.h>
+#include <device/pci_ids.h>
+#include <device/pci_ops.h>
+#warning Broken hard_reset
+//#include <part/hard_reset.h>
+#include <mc146818rtc.h>
+#include <amd/k8/k8.h>
+
+#warning Make AGP_APERTURE_SIZE a CONFIG variable
+#define AGP_APERTURE_SIZE 0x4000000 //64M
+
+/**
+ * @brief Read resources for AGP aperture
+ *
+ * @param
+ *
+ * There is only one AGP aperture resource needed. The resoruce is added to
+ * the northbridge of BSP.
+ *
+ * The same trick can be used to augment legacy VGA resources which can
+ * be detect by generic pci reousrce allocator for VGA devices.
+ * BAD: it is more tricky than I think, the resource allocation code is
+ * implemented in a way to NOT DOING legacy VGA resource allcation on
+ * purpose :-(.
+ */
+static void mcf3_read_resources(struct device *dev)
+{
+ struct resource *resource;
+ unsigned char iommu;
+ /* Read the generic PCI resources */
+ pci_dev_read_resources(dev);
+
+ /* If we are not the first processor don't allocate the gart apeture */
+ if (dev->path.pci.devfn != PCI_DEVFN(0x18, 3)) {
+ return;
+ }
+
+ iommu = 1;
+ get_option(&iommu, "iommu");
+
+ if (iommu) {
+ /* Add a Gart apeture resource */
+ resource = new_resource(dev, 0x94);
+ resource->size = iommu ? AGP_APERTURE_SIZE : 1;
+ resource->align = log2f(resource->size);
+ resource->gran = log2f(resource->size);
+ resource->limit = 0xffffffff; /* 4G */
+ resource->flags = IORESOURCE_MEM;
+ }
+}
+
+static void set_agp_aperture(struct device *dev)
+{
+ struct resource *resource;
+
+ resource = probe_resource(dev, 0x94);
+ if (resource) {
+ struct device *pdev;
+ struct device_id id;
+ u32 gart_base, gart_acr;
+
+ /* Remember this resource has been stored */
+ resource->flags |= IORESOURCE_STORED;
+
+ /* Find the size of the GART aperture */
+ gart_acr = (0 << 6) | (0 << 5) | (0 << 4) |
+ ((resource->gran - 25) << 1) | (0 << 0);
+
+ /* Get the base address */
+ gart_base = ((resource->base) >> 25) & 0x00007fff;
+
+ /* Update the other northbriges */
+ pdev = NULL;
+ id.type = DEVICE_ID_PCI;
+ id.pci.vendor = PCI_VENDOR_ID_AMD;
+ id.pci.device = 0x1103;
+ while ((pdev = dev_find_device(&id, pdev))) {
+ /* Store the GART size but don't enable it */
+ pci_write_config32(pdev, 0x90, gart_acr);
+
+ /* Store the GART base address */
+ pci_write_config32(pdev, 0x94, gart_base);
+
+ /* Don't set the GART Table base address */
+ pci_write_config32(pdev, 0x98, 0);
+
+ /* Report the resource has been stored... */
+ report_resource_stored(pdev, resource, " <gart>");
+ }
+ }
+}
+
+static void mcf3_set_resources(struct device *dev)
+{
+ printk(BIOS_DEBUG, "%s...\n", __func__);
+ /* Set the gart apeture */
+ set_agp_aperture(dev);
+
+ /* Set the generic PCI resources */
+ pci_set_resources(dev);
+}
+
+static void misc_control_init(struct device *dev)
+{
+ u32 cmd, cmd_ref;
+ int needs_reset;
+ struct device *f0_dev;
+
+ printk(BIOS_DEBUG, "NB: Function 3 Misc Control..\n");
+ needs_reset = 0;
+
+ /* Disable Machine checks from Invalid Locations.
+ * This is needed for PC backwards compatibility.
+ */
+ cmd = pci_read_config32(dev, 0x44);
+ cmd |= (1 << 6) | (1 << 25);
+ pci_write_config32(dev, 0x44, cmd);
+#if 0
+ if (is_cpu_pre_c0()) {
+
+ /* Errata 58
+ * Disable CPU low power states C2, C1 and throttling
+ */
+ cmd = pci_read_config32(dev, 0x80);
+ cmd &= ~(1 << 0);
+ pci_write_config32(dev, 0x80, cmd);
+ cmd = pci_read_config32(dev, 0x84);
+ cmd &= ~(1 << 24);
+ cmd &= ~(1 << 8);
+ pci_write_config32(dev, 0x84, cmd);
+
+ /* Errata 66
+ * Limit the number of downstream posted requests to 1
+ */
+ cmd = pci_read_config32(dev, 0x70);
+ if ((cmd & (3 << 0)) != 2) {
+ cmd &= ~(3 << 0);
+ cmd |= (2 << 0);
+ pci_write_config32(dev, 0x70, cmd);
+ needs_reset = 1;
+ }
+ cmd = pci_read_config32(dev, 0x7c);
+ if ((cmd & (3 << 4)) != 0) {
+ cmd &= ~(3 << 4);
+ cmd |= (0 << 4);
+ pci_write_config32(dev, 0x7c, cmd);
+ needs_reset = 1;
+ }
+ /* Clock Power/Timing Low */
+ cmd = pci_read_config32(dev, 0xd4);
+ if (cmd != 0x000D0001) {
+ cmd = 0x000D0001;
+ pci_write_config32(dev, 0xd4, cmd);
+ needs_reset = 1; /* Needed? */
+ }
+ } else if (is_cpu_pre_d0()) {
+ u32 dcl;
+ f2_dev = dev_find_slot(0, dev->path.pci.devfn - 3 + 2);
+ /* Errata 98
+ * Set Clk Ramp Hystersis to 7
+ * Clock Power/Timing Low
+ */
+ cmd_ref = 0x04e20707; /* Registered */
+ dcl = pci_read_config32(f2_dev, DRAM_CONFIG_LOW);
+ if (dcl & DCL_UnBufDimm) {
+ cmd_ref = 0x000D0701; /* Unbuffered */
+ }
+ cmd = pci_read_config32(dev, 0xd4);
+ if (cmd != cmd_ref) {
+ pci_write_config32(dev, 0xd4, cmd_ref);
+ needs_reset = 1; /* Needed? */
+ }
+ }
+#endif
+ /* Optimize the Link read pointers */
+ f0_dev = dev_find_slot(0, dev->path.pci.devfn - 3);
+ if (f0_dev) {
+ int link;
+ cmd_ref = cmd = pci_read_config32(dev, 0xdc);
+ for (link = 0; link < 3; link++) {
+ u32 link_type;
+ unsigned reg;
+ /* This works on an Athlon64 because unimplemented links return 0 */
+ reg = 0x98 + (link * 0x20);
+ link_type = pci_read_config32(f0_dev, reg);
+ /* Only handle coherent link here please */
+ if ((link_type &
+ (LinkConnected | InitComplete | NonCoherent))
+ == (LinkConnected | InitComplete)) {
+ cmd &= ~(0xff << (link * 8));
+ /* FIXME this assumes the device on the other side is an AMD device */
+ cmd |= 0x25 << (link * 8);
+ }
+ }
+ if (cmd != cmd_ref) {
+ pci_write_config32(dev, 0xdc, cmd);
+ needs_reset = 1;
+ }
+ } else {
+ printk(BIOS_ERR, "Missing f0 device!\n");
+ }
+ if (needs_reset) {
+ printk(BIOS_DEBUG, "resetting cpu\n");
+ //hard_reset();
+ }
+ printk(BIOS_DEBUG, "done.\n");
+}
+
+struct device_operations mcf3_ops = {
+ .id = {.type = DEVICE_ID_PCI,
+ {.pci = {.vendor = PCI_VENDOR_ID_AMD,
+ .device = 0x1103}}},
+ .phase3_scan = 0,
+ .phase4_read_resources = mcf3_read_resources,
+ .phase4_set_resources = mcf3_set_resources,
+ .phase5_enable_resources = pci_dev_enable_resources,
+ .phase6_init = misc_control_init,
+ .ops_pci = 0,
+};
+
+#if 0
+static const struct pci_driver mcf3_driver __pci_driver = {
+ .ops = &mcf3_ops,
+ .vendor = PCI_VENDOR_ID_AMD,
+ .device = 0x1103,
+};
+#endif
--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
