[coreboot] r577 - coreboot-v3/southbridge/amd/cs5536
svn at coreboot.org
svn at coreboot.org
Thu Feb 7 17:17:21 CET 2008
Author: rminnich
Date: 2008-02-07 17:17:21 +0100 (Thu, 07 Feb 2008)
New Revision: 577
Modified:
coreboot-v3/southbridge/amd/cs5536/cs5536.c
Log:
This is a subtle error. An operations struct, to work as a constructor, must initialize
the .constructor struct member.
This error is so subtle that we might consider changing the device code; or at least
working on the documentation to make it more clear. The key question: do we ever want
an operations struct that can NOT operate as a device constructor? When this code was
designed over a year ago, it seemed that we might; now, it is not so certain.
Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Modified: coreboot-v3/southbridge/amd/cs5536/cs5536.c
===================================================================
--- coreboot-v3/southbridge/amd/cs5536/cs5536.c 2008-02-07 16:09:24 UTC (rev 576)
+++ coreboot-v3/southbridge/amd/cs5536/cs5536.c 2008-02-07 16:17:21 UTC (rev 577)
@@ -618,6 +618,7 @@
outl(0xDEADBEEF, 0xCFC);
}
#endif
+ printk(BIOS_SPEW, "cs5536: %s() Exit\n", __FUNCTION__);
}
/**
@@ -631,9 +632,11 @@
printk(BIOS_SPEW, "cs5536: %s()\n", __FUNCTION__);
pci_dev_enable_resources(dev);
enable_childrens_resources(dev);
+ printk(BIOS_SPEW, "cs5536: %s() Exit\n", __FUNCTION__);
}
static struct device_operations southbridge_ops = {
+ .constructor = default_device_constructor,
.phase3_scan = scan_static_bus,
.phase4_read_resources = pci_dev_read_resources,
.phase4_set_resources = pci_dev_set_resources,
More information about the coreboot
mailing list