Change in coreboot[master]: Documentation/RFC: Generalize PCI support in coreboot

[Angel Pons (Code Review)]

Attention is currently required from: Shelley Chen,  Felix Singer, Nico Huber, Furquan Shaikh, Martin Roth, Tim Wawrzynczak, Paul Menzel, Julius Werner, Kyösti Mälkki, Yu-Ping Wu, Jianjun Wang.
Angel Pons 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:

(2 comments)

File Documentation/RFC/pci_config_access.md:

https://review.coreboot.org/c/coreboot/+/57861/comment/ec0ce5b8_e949faaf 
PS12, Line 129: MMONCF
typo: MMCONF

https://review.coreboot.org/c/coreboot/+/57861/comment/89598e77_f3b387ee 
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.
> The "example" linked below around get_pci_mmio_cfgbase() looks […]

+1 to what Nico said.

To me, the `NO_ECAM_MMCONF_SUPPORT` option is about the coreboot code, not the hardware. Only implementing method 1 with `pci_map_bus()` and selecting `NO_ECAM_MMCONF_SUPPORT` is completely fine, even if the hardware supports MMCONF.