Attention is currently required from: Lance Zhao, Marc Jones, Martin Roth, Stefan Reinauer, Angel Pons, Kyösti Mälkki, Patrick Rudolph.
View Change
1 comment:
Patchset:
Patch Set #1:
Yes, I linked and read the commit this reverts, you are only answering what I was able to read there […]
best I can tell, the PCI-mode support for Baytrail LPSS devices is incomplete, since setting them to PCI mode in devicetree (as we've done with the ChromeOS Braswell boards) did not result in them being usable by the payload (either SeaBIOS or Tianocore). In my own tree, I've ended up doing an ugly hack where coreboot does the ACPI mode switch in the payload (as here, via SMM) so that the payload can detect/use the eMMC in PCI mode but the OS sees it in ACPI mode.
But as for the code removed in this patch, I'm not aware that it was ever actually used, even in shipped ChromeOS firmware. SeaBIOS on RAMBI-based boards required a special config to specify the MMIO base address of the eMMC/SD devices in ACPI mode, since depthcharge definitely did not switch them back to PCI mode.
To view, visit change 49492. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I76736fa72af30003d86dfc2551b673037c1dd522
Gerrit-Change-Number: 49492
Gerrit-PatchSet: 6
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Lance Zhao
Gerrit-Reviewer: Marc Jones <marc@marcjonesconsulting.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier@gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Stefan Reinauer <reinauer@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Attention: Lance Zhao
Gerrit-Attention: Marc Jones <marc@marcjonesconsulting.com>
Gerrit-Attention: Martin Roth <martinroth@google.com>
Gerrit-Attention: Stefan Reinauer <reinauer@chromium.org>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Attention: Patrick Rudolph <siro@das-labor.org>
Gerrit-Comment-Date: Sun, 21 Feb 2021 17:52:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Lance Zhao
Comment-In-Reply-To: Marc Jones <marc@marcjonesconsulting.com>
Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com>
Comment-In-Reply-To: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-MessageType: comment