- Added explanation of device tree enable.
authorJason Schildt <jschildt@gmail.com>
Thu, 13 Oct 2005 00:44:34 +0000 (00:44 +0000)
committerJason Schildt <jschildt@gmail.com>
Thu, 13 Oct 2005 00:44:34 +0000 (00:44 +0000)
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2052 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/southbridge/amd/amd8111/amd8111.c
src/southbridge/amd/amd8111/amd8111_acpi.c
src/southbridge/amd/amd8111/amd8111_pci.c

index 6f5f1beada878a072e1dea22c210e88f157e43da..32797041f37de0736d13ed613f95f27e40eb2a72 100644 (file)
@@ -66,5 +66,8 @@ void amd8111_enable(device_t dev)
 
 struct chip_operations southbridge_amd_amd8111_ops = {
        CHIP_NAME("AMD 8111")
+       /* This only called when this device is listed in the 
+       * static device tree.
+       */
        .enable_dev = amd8111_enable,
 };
index 46a8e69510f51c5874619021cb3722818deffe15..f661ab478e89c4ca985f608849450dc340823324 100644 (file)
@@ -97,6 +97,7 @@ static void acpi_init(struct device *dev)
 
        
 #endif
+       /* power on after power fail */
        on = MAINBOARD_POWER_ON_AFTER_POWER_FAIL;
        get_option(&on, "power_on_after_fail");
        byte = pci_read_config8(dev, PREVIOUS_POWER_STATE);
@@ -177,6 +178,10 @@ static struct device_operations acpi_ops  = {
        .enable_resources = acpi_enable_resources,
        .init             = acpi_init,
        .scan_bus         = scan_static_bus,
+       /*  We don't need amd8111_enable, chip ops takes care of it.
+        *  It could be useful if these devices were not 
+        *  enabled by default.
+        */
 //     .enable           = amd8111_enable,
        .ops_pci          = &lops_pci,
        .ops_smbus_bus    = &lops_smbus_bus,
index 9fe1f98d325f65e3b7953a5b8e3856f97275a7a6..b9a9b3578dd82de953e990911f2a2f21a2703ab7 100644 (file)
@@ -55,6 +55,7 @@ static struct device_operations pci_ops  = {
        .enable_resources = pci_bus_enable_resources,
        .init             = pci_init,
        .scan_bus         = pci_scan_bridge,
+       /* PCI Subordinate bus reset is not implemented */
        .ops_pci          = &lops_pci,
 };