diff options
| author | Michael Brown <mcb30@etherboot.org> | 2009-03-12 19:41:40 +0000 |
|---|---|---|
| committer | Michael Brown <mcb30@etherboot.org> | 2009-03-13 02:06:30 +0000 |
| commit | 1266d7902bf7f2534ee279671d48613ef9b2434c (patch) | |
| tree | a1a5b188148d983fa962a887476259768f1751d4 /src/core/device.c | |
| parent | 3ed468e0c58bb0eb94925c18a0461cd5014b5777 (diff) | |
| download | ipxe-1266d7902bf7f2534ee279671d48613ef9b2434c.tar.gz | |
[tables] Redefine methods for accessing linker tables
Intel's C compiler (icc) chokes on the zero-length arrays that we
currently use as part of the mechanism for accessing linker table
entries. Abstract away the zero-length arrays, to make a port to icc
easier.
Introduce macros such as for_each_table_entry() to simplify the common
case of iterating over all entries in a linker table.
Represent table names as #defined string constants rather than
unquoted literals; this avoids visual confusion between table names
and C variable or type names, and also allows us to force a
compilation error in the event of incorrect table names.
Diffstat (limited to 'src/core/device.c')
| -rw-r--r-- | src/core/device.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/core/device.c b/src/core/device.c index 84915c2d..1f57e4c6 100644 --- a/src/core/device.c +++ b/src/core/device.c @@ -29,11 +29,6 @@ * */ -static struct root_device root_devices[0] - __table_start ( struct root_device, root_devices ); -static struct root_device root_devices_end[0] - __table_end ( struct root_device, root_devices ); - /** Registered root devices */ static LIST_HEAD ( devices ); @@ -77,7 +72,7 @@ static void probe_devices ( void ) { struct root_device *rootdev; int rc; - for ( rootdev = root_devices; rootdev < root_devices_end; rootdev++ ) { + for_each_table_entry ( rootdev, ROOT_DEVICES ) { list_add ( &rootdev->dev.siblings, &devices ); INIT_LIST_HEAD ( &rootdev->dev.children ); if ( ( rc = rootdev_probe ( rootdev ) ) != 0 ) |