Author: marius
Date: Wed Jan 13 21:12:31 2010
New Revision: 202253
URL: http://svn.freebsd.org/changeset/base/202253

Log:
  MFC: r200921
  
  - Add quirk handling for ALi M5229, mainly setting the magic "force
    enable IDE I/O" bit which prevents data access traps with revision
    0xc8 in Fire-based machines when pci(4) enables PCIM_CMD_PORTEN.
  - Like for sun4v also don't add the PCI side of host-PCIe bridges to
    the bus on sun4u as they don't have configuration space implement
    there either.

Modified:
  stable/7/sys/sparc64/pci/ofw_pcibus.c
Directory Properties:
  stable/7/sys/   (props changed)
  stable/7/sys/cddl/contrib/opensolaris/   (props changed)
  stable/7/sys/contrib/dev/acpica/   (props changed)
  stable/7/sys/contrib/pf/   (props changed)

Modified: stable/7/sys/sparc64/pci/ofw_pcibus.c
==============================================================================
--- stable/7/sys/sparc64/pci/ofw_pcibus.c       Wed Jan 13 21:12:27 2010        
(r202252)
+++ stable/7/sys/sparc64/pci/ofw_pcibus.c       Wed Jan 13 21:12:31 2010        
(r202253)
@@ -3,6 +3,7 @@
  * Copyright (c) 2000, Michael Smith <msm...@freebsd.org>
  * Copyright (c) 2000, BSDi
  * Copyright (c) 2003, Thomas Moestl <t...@freebsd.org>
+ * Copyright (c) 2005 - 2009 Marius Strobl <mar...@freebsd.org>
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -196,6 +197,22 @@ ofw_pcibus_setup_device(device_t bridge,
 #endif
 
        /*
+        * Ensure that ALi M5229 report the actual content of PCIR_PROGIF
+        * and that IDE I/O is force enabled.  The former is done in order
+        * to have unique behavior across revisions as some default to
+        * hiding bits 4-6 for compliance with PCI 2.3.  The latter is done
+        * as at least revision 0xc8 requires the PCIM_CMD_PORTEN bypass
+        * to be always enabled as otherwise even enabling PCIM_CMD_PORTEN
+        * results in an instant data access trap on Fire-based machines.
+        * Thus these quirks have to be handled before pci(4) adds the maps.
+        * Note that for older revisions bit 0 of register 0x50 enables the
+        * internal IDE function instead of force enabling IDE I/O.
+        */
+       if ((CS_READ(PCIR_VENDOR, 2) == 0x10b9 &&
+           CS_READ(PCIR_DEVICE, 2) == 0x5229))
+               CS_WRITE(0x50, CS_READ(0x50, 1) | 0x3, 1);
+
+       /*
         * The preset in the intline register is usually wrong.  Reset
         * it to 255, so that the PCI code will reroute the interrupt if
         * needed.
@@ -224,9 +241,14 @@ ofw_pcibus_attach(device_t dev)
                    domain, busno);
        node = ofw_bus_get_node(dev);
 
-#ifndef SUN4V
-       /* Add the PCI side of the HOST-PCI bridge itself to the bus. */
+       /*
+        * Add the PCI side of the host-PCI bridge itself to the bus.
+        * Note that we exclude the host-PCIe bridges here as these
+        * have no configuration space implemented themselves.
+        */
        if (strcmp(device_get_name(device_get_parent(pcib)), "nexus") == 0 &&
+           ofw_bus_get_type(pcib) != NULL &&
+           strcmp(ofw_bus_get_type(pcib), OFW_TYPE_PCIE) != 0 &&
            (dinfo = (struct ofw_pcibus_devinfo *)pci_read_device(pcib,
            domain, busno, 0, 0, sizeof(*dinfo))) != NULL) {
                if (ofw_bus_gen_setup_devinfo(&dinfo->opd_obdinfo, node) != 0)
@@ -234,7 +256,6 @@ ofw_pcibus_attach(device_t dev)
                else
                        pci_add_child(dev, (struct pci_devinfo *)dinfo);
        }
-#endif
 
        if (OF_getprop(ofw_bus_get_node(pcib), "clock-frequency", &clock,
            sizeof(clock)) == -1)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to