Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Mathew King, Angel Pons.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51045 )
Change subject: soc/amd/cezanne: Fill out pci devices in chipset.cb
......................................................................
Patch Set 8:
(1 comment)
File src/soc/amd/cezanne/chipset.cb:
https://review.coreboot.org/c/coreboot/+/51045/comment/d7961be9_0a5ba2e0
PS6, Line 20: device pci 8.0 alias dummy_8 off end # Dummy Host Bridge
Done
i'd keep the dummy bridges on and don't give them an alias. leaving them on doesn't really hurt, but it might cause not completely expected behaviour to switch them off. function 0 is a dummy device that should probably always be on. when function 0 of a device isn't present, all other functions are also assumed to be not present. so to avoid having to renumber the functions like it is done in intel systems if function 0 isn't used, but some other function on the same device is, function 0 here is just a dummy that should be kept active and now there's no case where the other functions might need to get renumbered in order for things to work. ok, for bus c that might result in unnecessarily eating up one bus number
--
To view, visit
https://review.coreboot.org/c/coreboot/+/51045
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iecc75afd7a914651ca15b811163d3559bf73ac9b
Gerrit-Change-Number: 51045
Gerrit-PatchSet: 8
Gerrit-Owner: Mathew King
mathewk@chromium.org
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Jason Glenesk
jason.glenesk@gmail.com
Gerrit-Reviewer: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Reviewer: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Attention: Jason Glenesk
jason.glenesk@gmail.com
Gerrit-Attention: Raul Rangel
rrangel@chromium.org
Gerrit-Attention: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Attention: Mathew King
mathewk@chromium.org
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Comment-Date: Thu, 25 Feb 2021 22:31:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Mathew King
mathewk@chromium.org
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Gerrit-MessageType: comment