Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35735 )
Change subject: [WIP] intel/skylake: Refactor IRQ assignments ......................................................................
Patch Set 1:
(2 comments)
Is there some quirk with SI_PCH_DEVICE_INTERRUPT_CONFIG that would require entries where arguments device and function would not originate from same PCI BDF?
Also, I do not see fsp2_0/skylake/FspUpdVpd.h anywhere, how does this code work from chip_fsp20.c?
https://review.coreboot.org/c/coreboot/+/35735/1/src/soc/intel/skylake/irq.c File src/soc/intel/skylake/irq.c:
https://review.coreboot.org/c/coreboot/+/35735/1/src/soc/intel/skylake/irq.c... PS1, Line 192: sizeof(SI_PCH_DEVICE_INTERRUPT_CONFIG)); Who allocates target buffer?
https://review.coreboot.org/c/coreboot/+/35735/1/src/soc/intel/skylake/irq.c... PS1, Line 212: memcpy(params->PxRcConfig, irq_config, PCH_MAX_IRQ_CONFIG); Who allocates target buffer? FSP? But PCH_MAX_IRQ_CONFIG is not defined in FSP headers.