Change in coreboot[master]: src/soc/intel/commmon/itss: Add support to get interrupt table for PC...
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34089 )
Change subject: src/soc/intel/commmon/itss: Add support to get interrupt table for PCI devices ......................................................................
Patch Set 6:
(5 comments)
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... File src/soc/intel/common/block/include/intelblocks/irq.h:
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... PS6, Line 22: 1 space will work rather tab
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... PS6, Line 25: use space
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... PS6, Line 37: const int * get_soc_irq_devices(size_t *size); remove whitespace
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... File src/soc/intel/common/block/itss/irq.c:
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... PS6, Line 95: if(!devlist) one space
https://review.coreboot.org/c/coreboot/+/34089/6/src/soc/intel/common/block/... PS6, Line 97: for(int i=0; i < size; i++) {
space required before the open parenthesis '('
1 space
-
Subrata Banik (Code Review)