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

[Jeremy Soller (Code Review)]

Jeremy Soller has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35946 )

Change subject: pciexp: Add support for allocating PCI express hotplug resources
......................................................................

Patch Set 4:

(3 comments)

I addressed some of the comments in patch set 4. I am working on a variety of GPUs and expect to have further updates as necessary.

https://review.coreboot.org/c/coreboot/+/35946/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35946/3//COMMIT_MSG@27 
PS3, Line 27: assign-buses
> It should be "assign-busses", as Documentation/x86/x86_64/boot-options.rst of Linux kernel states.

Done

https://review.coreboot.org/c/coreboot/+/35946/3/src/device/Kconfig 
File src/device/Kconfig:

https://review.coreboot.org/c/coreboot/+/35946/3/src/device/Kconfig@576 
PS3, Line 576: PCIEXP_HOTPLUG_MEM
> this should probably be PCIEXP_HOTPLUG_PREFETCH_MEM

Done

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 488: 	resource->gran = 22;
> I'm not sure this needs to be anything other than page aligned. […]

Done