diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2017-11-10 19:52:30 +0200 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2017-12-18 23:05:52 -0600 |
commit | 9d987eba8ed794b9fcf0d08cf1b312d2ece598de (patch) | |
tree | 4ca8f71fa0dd8293cabaac39d378d0a91c558cc1 /tools/perf/scripts/python/check-perf-trace.py | |
parent | dd1ea5763e65c69983697dc97ea9d6e3cb9df922 (diff) |
powerpc/pci: Unroll two pass loop when scanning bridges
The current scanning code is really hard to understand because it calls the
same function in a loop where pass value is changed without any comments
explaining it:
for (pass = 0; pass < 2; pass++)
for_each_pci_bridge(dev, bus)
max = pci_scan_bridge(bus, dev, max, pass);
Unfamiliar reader cannot tell easily what is the purpose of this loop
without looking at internals of pci_scan_bridge().
In order to make this bit easier to understand, open-code the loop in
pci_scan_child_bus() and pci_hp_add_bridge() with added comments.
No functional changes intended.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Diffstat (limited to 'tools/perf/scripts/python/check-perf-trace.py')
0 files changed, 0 insertions, 0 deletions