build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34620 )
Change subject: pci: Add support for assigning resources to SR-IOV VF BARs ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c File src/device/pciexp_device.c:
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c@... PS1, Line 456: if (!sriovpos) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c@... PS1, Line 461: if (!sriovpos) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c@... PS1, Line 469: if (numvfs == 0) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c@... PS1, Line 480: if (resource->size == 0) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/34620/1/src/device/pciexp_device.c@... PS1, Line 488: if (resource->flags & IORESOURCE_PCI64) { braces {} are not necessary for single statement blocks