Attention is currently required from: Arthur Heymans, Angel Pons.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59131 )
Change subject: device/pci_device.c: Improve pci_bridge_route() readability
......................................................................
Patch Set 2:
(2 comments)
File src/device/pci_device.c:
https://review.coreboot.org/c/coreboot/+/59131/comment/225b2d73_34cd5407
PS1, Line 1337: u32 raw;
: struct {
: u8 primary;
: u8 secondary;
: u8 subordinate;
: u8 _latency;
: } __packed;
This is not portable.
As PCI is a little endian bus, I think this is more portable than the existing code.
https://review.coreboot.org/c/coreboot/+/59131/comment/e220b9ba_d18410c6
PS1, Line 1356: parent
link
Done
--
To view, visit
https://review.coreboot.org/c/coreboot/+/59131
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3bafd6a2e1d3a0b8d1d43997868a787ce3940ca9
Gerrit-Change-Number: 59131
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans
arthur.heymans@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-CC: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Attention: Arthur Heymans
arthur.heymans@9elements.com
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Comment-Date: Fri, 12 Nov 2021 14:16:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Gerrit-MessageType: comment