Attention is currently required from: Felix Singer, Nico Huber, Furquan Shaikh, Martin Roth, Tim Wawrzynczak, Paul Menzel, Angel Pons, Julius Werner, Kyösti Mälkki, Yu-Ping Wu, Jianjun Wang.
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57861 )
Change subject: Documentation/RFC: Generalize PCI support in coreboot
......................................................................
Patch Set 12:
(1 comment)
File Documentation/RFC/pci_config_access.md:
https://review.coreboot.org/c/coreboot/+/57861/comment/bc8dd8de_b47f7ffe
PS12, Line 125: ### Specifying MMIO support for PCI devices that don't support ECAM
:
: We’ll need to be able to distinguish between a platform using the ECAM access
: method and any other MMIO access mechanism. For devices that support ECAM, they
: would have to select the `ECAM_MMONCF_SUPPORT` config. For devices that don't
: support ECAM and would still need MMIO support, they would need to explictly
: select the `NO_ECAM_MMCONF_SUPPORT` config and include the `pci_mmio_cfg.h'
: header file in their platform-specific source.
+1 to what Nico said. […]
Sorry, I'll try to fix up the example so it's not confusing.
Agree with Nico/Angel. This just defines the software support. It seems like your hardware supports both access mechanisms? If you prefer the non-ECAM method due to space constraints, it's fine to reflect this in your pci_map_bus() function implementation.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/57861
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3a56d932f6e047087c38a7687564065cc1562363
Gerrit-Change-Number: 57861
Gerrit-PatchSet: 12
Gerrit-Owner: Shelley Chen
shchen@google.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Furquan Shaikh
furquan.m.shaikh@gmail.com
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Paul Menzel
paulepanter@mailbox.org
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Felix Singer
felixsinger@posteo.net
Gerrit-CC: Jianjun Wang
jianjun.wang@mediatek.corp-partner.google.com
Gerrit-CC: Raul Rangel
rrangel@chromium.org
Gerrit-CC: Yu-Ping Wu
yupingso@google.com
Gerrit-Attention: Felix Singer
felixsinger@posteo.net
Gerrit-Attention: Nico Huber
nico.h@gmx.de
Gerrit-Attention: Furquan Shaikh
furquan.m.shaikh@gmail.com
Gerrit-Attention: Martin Roth
martinroth@google.com
Gerrit-Attention: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Julius Werner
jwerner@chromium.org
Gerrit-Attention: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Attention: Yu-Ping Wu
yupingso@google.com
Gerrit-Attention: Jianjun Wang
jianjun.wang@mediatek.corp-partner.google.com
Gerrit-Comment-Date: Mon, 29 Nov 2021 22:33:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nico Huber
nico.h@gmx.de
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Comment-In-Reply-To: Jianjun Wang
jianjun.wang@mediatek.corp-partner.google.com
Gerrit-MessageType: comment