diff options
| author | Patrick Georgi <patrick.georgi@coresystems.de> | 2010-08-09 12:58:16 +0000 |
|---|---|---|
| committer | Patrick Georgi <patrick.georgi@coresystems.de> | 2010-08-09 12:58:16 +0000 |
| commit | 35d346fe2d98b3ec5ed0451b87cc31be06177d68 (patch) | |
| tree | 2e93d076906a6dc53fd8b4ab7c624e4d1dad8a29 /util/sconfig/main.c | |
| parent | df61dd28f635bf57a1dc81c75b09b069041ab2f3 (diff) | |
Fix build error introduced in r5868.
aliased_name was a compatibility hack to match the output of the C rewrite
with the python version's results. It seems that we carried these
useless symbols with us for years, just without any impact good or bad.
By declaring devices static and tightening the screws (-Werror), the
compiler now knows that these declarations are useless - and stops.
Signed-off-by: Patrick Georgi <patrick.georgi@coresystems.de>
Acked-by: Stefan Reinauer <stefan.reinauer@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5687 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'util/sconfig/main.c')
| -rwxr-xr-x | util/sconfig/main.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/util/sconfig/main.c b/util/sconfig/main.c index be42381340..40a220aa44 100755 --- a/util/sconfig/main.c +++ b/util/sconfig/main.c @@ -223,8 +223,6 @@ void alias_siblings(struct device *d) { if (device_match(d, cmp)) { d->multidev = 1; - cmp->aliased_name = malloc(12); - sprintf(cmp->aliased_name, "_dev%d", cmp->id); cmp->id = d->id; cmp->name = d->name; cmp->used = 1; @@ -289,8 +287,6 @@ static void pass0(FILE *fil, struct device *ptr) { if (ptr->children || ptr->multidev) fprintf(fil, "struct bus %s_links[];\n", ptr->name); } - if ((ptr->type == device) && (ptr->id != 0) && ptr->used) - fprintf(fil, "static struct device %s;\n", ptr->aliased_name); } static void pass1(FILE *fil, struct device *ptr) { |