Attention is currently required from: Nick Vaccaro, Patrick Rudolph.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61719 )
Change subject: src/soc/intel/common/block/i2c: Use early BAR in ENV_PAYLOAD_LOADER
......................................................................
Patch Set 2:
(1 comment)
File src/soc/intel/common/block/i2c/i2c.c:
https://review.coreboot.org/c/coreboot/+/61719/comment/2942b1e9_357ece07
PS2, Line 135: pci_read_config32
Ack
This is if no resources were found (probe_resource checks the resource_list), in this case the reason is because (with this patch train) I2C communication now will happen in ramstage but before PCI enumeration (other reasons could be because the device was disabled or had requested no resources), so we reuse the BAR that was assigned during early init because resource allocation hasn't happened yet.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/61719
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8ab599199592a72ae96cd9f95accfaa0d84e66b6
Gerrit-Change-Number: 61719
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Reviewer: Subrata Banik
subratabanik@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Nick Vaccaro
nvaccaro@google.com
Gerrit-Attention: Patrick Rudolph
siro@das-labor.org
Gerrit-Comment-Date: Thu, 10 Feb 2022 16:26:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik
subratabanik@google.com
Comment-In-Reply-To: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-MessageType: comment