[coreboot-gerrit] Change in coreboot[master]: sb/intel/bd82x6x: Disable unused bridges

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19818 )

Change subject: sb/intel/bd82x6x: Disable unused bridges
......................................................................


Patch Set 3: Code-Review+2

> > On my x200/ICH9 all SB PCIe functions seem to support hotplug.
 > > How is the situation on bd82x6x?
 > 
 > All ports support hotplug. But you won't use all of them. For that
 > reason you can select which of them are used (for Express card) by
 > using a devicetree array. On my board with current devixetree, no
 > ports would be disabled as two are in use and one is marked hotplug
 > let. It's only useful if you remove on the PCIe devices (like WiFi
 > card).

wups, I missed that.

This is good default behavior, people wanting to change it can modify devicetree.

-- 
To view, visit https://review.coreboot.org/19818
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6ee5e5f33824acdbca0f6ed28e90beab7fe10002
Gerrit-PatchSet: 3
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-HasComments: No