diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2015-11-13 09:39:22 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-11-14 18:43:06 +0100 |
| commit | 92bcaa2226ff8fe16f2144358b3ba310da555a2c (patch) | |
| tree | a2fb3cb401a4594dd7e7d8accf2437c6a80d339d | |
| parent | 2035d9aa94187040cfd73b12cc94c34ac1326f16 (diff) | |
| download | coreboot-92bcaa2226ff8fe16f2144358b3ba310da555a2c.tar.xz | |
sconfig: remove warning about root_complex
It's not deprecated if it's still in active use. The code layout is just
"funny" (and could warrant a chipset-side cleanup, but not today)
Change-Id: I5f7776ceba0134f20364a0c4a1ca51382e9877e2
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: http://review.coreboot.org/12429
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
| -rw-r--r-- | util/sconfig/main.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/util/sconfig/main.c b/util/sconfig/main.c index 5eb74dace2..27027f567c 100644 --- a/util/sconfig/main.c +++ b/util/sconfig/main.c @@ -164,10 +164,10 @@ struct device *new_chip(struct device *parent, struct device *bus, char *path) { char *chip_h = malloc(strlen(path)+18); sprintf(chip_h, "src/%s", path); if ((stat(chip_h, &st) == -1) && (errno == ENOENT)) { - if (strstr(path, "/root_complex")) { - fprintf(stderr, "WARNING: Use of deprecated chip component %s\n", - path); - } else { + /* root_complex gets away without a separate directory, but + * exists on on pretty much all AMD chipsets. + */ + if (!strstr(path, "/root_complex")) { fprintf(stderr, "ERROR: Chip component %s does not exist.\n", path); exit(1); |