Patch Set 1: Code-Review-2

blocking this for now. no need to merge this if it's overrun tomorrow

Of course I had the best intentions to look deeper into getting
rid of .scan_bus() but didn't take the time ._.

Now I don't even recall what could have overrun this. The first
three patches look sane to me. The first two simply rename and
consolidate things. The third allows bridge devices behind LPC
(actually I guess now we should just always scan bridges every-
where? I don't see why not).

I'll look into cases that could rely on predictable numbers for
I2C (SMBus) buses now, hoping to get rid of scan_generic_bus()
and scan_smbus(), too.

Patch set 2:-Code-Review

View Change

To view, visit change 31900. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9c9dc437b6117112bb28550855a2c38044dfbfa5
Gerrit-Change-Number: 31900
Gerrit-PatchSet: 2
Gerrit-Owner: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Wed, 18 Sep 2019 10:42:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment