Andrey Petrov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35321 )
Change subject: soc/fsp_broadwell_de: Add devhide functionality ......................................................................
Patch Set 10:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35321/8/src/soc/intel/fsp_broadwell... File src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h:
https://review.coreboot.org/c/coreboot/+/35321/8/src/soc/intel/fsp_broadwell... PS8, Line 54:
Possibly change all the spaces to tabs in a follow on commit? Up to you.
I added tabs in this patch
https://review.coreboot.org/c/coreboot/+/35321/8/src/soc/intel/fsp_broadwell... PS8, Line 113: Coming of from
Reword this?
Done
https://review.coreboot.org/c/coreboot/+/35321/8/src/soc/intel/fsp_broadwell... File src/soc/intel/fsp_broadwell_de/include/soc/ubox.h:
https://review.coreboot.org/c/coreboot/+/35321/8/src/soc/intel/fsp_broadwell... PS8, Line 42: : static inline void iio_hide(const uint8_t devno, const uint8_t funcno)
Why make this an inline function in a . […]
I orignally wanted to avoid overhead of creating another file for just a tiny function. But I decided to do it anyway.