diff options
| author | Furquan Shaikh <furquan@google.com> | 2018-06-07 23:36:45 -0700 |
|---|---|---|
| committer | Furquan Shaikh <furquan@google.com> | 2018-06-08 23:36:47 +0000 |
| commit | 4ca3a8abfa6456142520f3d53241167730b5fadc (patch) | |
| tree | 2e7a8a6cc67fb41ca1e24c2eea455973ddeb2879 /util/sconfig | |
| parent | 931982600d3ac3dfb9d66e013604ecc0573a1744 (diff) | |
| download | coreboot-4ca3a8abfa6456142520f3d53241167730b5fadc.tar.xz | |
util/sconfig: Get rid of rescnt in struct device
This change gets rid of rescnt member in struct device since it is
redundant. "res" member can be used to determine if resource list is
present or not.
BUG=b:80081934
TEST=Verified that static.c generated with and without this CL is
exactly the same for all boards built using abuild.
Change-Id: I73a2361686ad1130716a7d29576f2d02b9ed33c1
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/26806
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'util/sconfig')
| -rw-r--r-- | util/sconfig/main.c | 7 | ||||
| -rw-r--r-- | util/sconfig/sconfig.h | 3 |
2 files changed, 3 insertions, 7 deletions
diff --git a/util/sconfig/main.c b/util/sconfig/main.c index a3c77eb135..2a7fa15faf 100644 --- a/util/sconfig/main.c +++ b/util/sconfig/main.c @@ -493,7 +493,6 @@ void add_resource(struct bus *bus, int type, int index, int base) } else { dev->res = r; } - dev->rescnt++; } void add_register(struct chip_instance *chip_instance, char *name, char *val) @@ -589,7 +588,7 @@ static void pass0(FILE *fil, struct device *ptr, struct device *next) } fprintf(fil, "DEVTREE_CONST static struct device %s;\n", ptr->name); - if (ptr->rescnt > 0) + if (ptr->res) fprintf(fil, "DEVTREE_CONST struct resource %s_res[];\n", ptr->name); if (dev_has_children(ptr)) @@ -606,7 +605,7 @@ static void pass0(FILE *fil, struct device *ptr, struct device *next) static void emit_resources(FILE *fil, struct device *ptr) { - if (ptr->rescnt == 0) + if (ptr->res == NULL) return; int i = 1; @@ -704,7 +703,7 @@ static void pass1(FILE *fil, struct device *ptr, struct device *next) fprintf(fil, "\t.subsystem_device = 0x%04x,\n", ptr->subsystem_device); - if (ptr->rescnt > 0) { + if (ptr->res) { fprintf(fil, "\t.resource_list = &%s_res[0],\n", ptr->name); } diff --git a/util/sconfig/sconfig.h b/util/sconfig/sconfig.h index f9137fa7ac..63b65fd372 100644 --- a/util/sconfig/sconfig.h +++ b/util/sconfig/sconfig.h @@ -99,9 +99,6 @@ struct device { /* Indicates whether this device is enabled. */ int enabled; - /* Indicates number of resources for the device. */ - int rescnt; - /* Subsystem IDs for the device. */ int subsystem_vendor; int subsystem_device; |