[coreboot-gerrit] Change in coreboot[master]: pciexp: Support for allocating PCI express hotplug resources

[build bot (Jenkins) (Code Review)]

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35946 )

Change subject: pciexp: Support for allocating PCI express hotplug resources
......................................................................

Patch Set 2:

(3 comments)

https://review.coreboot.org/c/coreboot/+/35946/2/src/device/pci_device.c 
File src/device/pci_device.c:

https://review.coreboot.org/c/coreboot/+/35946/2/src/device/pci_device.c@886 
PS2, Line 886: 			} else
else is not generally useful after a break or return

https://review.coreboot.org/c/coreboot/+/35946/2/src/device/pci_device.c@886 
PS2, Line 886: 			} else
suspect code indent for conditional statements (24, 24)

https://review.coreboot.org/c/coreboot/+/35946/2/src/device/pciexp_device.c 
File src/device/pciexp_device.c:

https://review.coreboot.org/c/coreboot/+/35946/2/src/device/pciexp_device.c@... 
PS2, Line 500: 	printk(BIOS_DEBUG, "%s: add 0x%llx of prefetch memory space\n", __func__, resource->size);
line over 96 characters