summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKelsey Skunberg <skunberg.kelsey@gmail.com>2019-06-19 10:48:57 -0600
committerMartin Mares <mj@ucw.cz>2020-01-21 21:49:11 +0100
commit25f2e4db6ecad1868302bc27d5b1ec7d74a59c19 (patch)
tree11f087373ac1e9e0f484d53c2a98b802ba87c450
parentc5751fb0197751e9f691b7a924fb7fe1d9d971d7 (diff)
downloadpciutils-25f2e4db6ecad1868302bc27d5b1ec7d74a59c19.tar.gz
lspci: Remove unnecessary !verbose check in show_range()
Remove 'if (!verbose)' code in show_range() due to not being called. show_range() will only be called when verbose is true. Additional call to check for verbosity within show_range() is dead code. !verbose was used so nothing would print if the range behind a bridge had a base > limit and verbose == false. Since show_range() will not be called when verbose == false, not printing bridge information is still accomplished. Signed-off-by: Kelsey Skunberg <skunberg.kelsey@gmail.com>
-rw-r--r--lspci.c12
1 files changed, 3 insertions, 9 deletions
diff --git a/lspci.c b/lspci.c
index 937c6e4..7418b07 100644
--- a/lspci.c
+++ b/lspci.c
@@ -376,17 +376,11 @@ show_size(u64 x)
static void
show_range(char *prefix, u64 base, u64 limit, int is_64bit)
{
- if (base > limit)
+ if (base > limit && verbose < 3)
{
- if (!verbose)
- return;
- else if (verbose < 3)
- {
- printf("%s: None\n", prefix);
- return;
- }
+ printf("%s: None\n", prefix);
+ return;
}
-
printf("%s: ", prefix);
if (is_64bit)
printf("%016" PCI_U64_FMT_X "-%016" PCI_U64_FMT_X, base, limit);