Attention is currently required from: Shelley Chen, Ravi kumar, Furquan Shaikh, Arthur Heymans. Prasad Malisetty has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57615 )
Change subject: libpayload: Add support for PCI MMIO CONFIG access. ......................................................................
Patch Set 18:
(2 comments)
Patchset:
PS17:
Hi Prasad, I've created a CL with proposed code changes for the libpayload MMIO ops for PCI CONFIG s […]
Hi Shelley,
Thanks for the update.
I have acknowledged your comments. I will update new patch series with addressing all comments.
Thanks -Prasad
File payloads/libpayload/drivers/pci_ops.c:
https://review.coreboot.org/c/coreboot/+/57615/comment/beaad023_278764d3 PS17, Line 34: uintptr_t get_pci_mmio_cfgbase(pcidev_t dev)
This only works if you have one device, correct? Would it be possible to port the coreboot atu mapp […]
ATU configuration needed for every config/mem read/write. ATU configuration function is in coreboot PCIe driver. I am not really how we can call coreboot functions from payloads. Could you please help if we have that infra.
Thanks -Prasad