[coreboot-gerrit] Change in coreboot[master]: src/soc/intel/common/itss: Add support to get interrupt table for PCI...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34089 )

Change subject: src/soc/intel/common/itss: Add support to get interrupt table for PCI devices
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34089/8/src/soc/intel/common/block/... 
File src/soc/intel/common/block/itss/irq.c:

https://review.coreboot.org/c/coreboot/+/34089/8/src/soc/intel/common/block/... 
PS8, Line 150: while (irq_config) {
             : 		memcpy(dev, &(irq_config->irq), sizeof(struct dev_irq));
             : 		tmp = irq_config;
             : 		irq_config = irq_config->next;
             : 		free(tmp);
             : 		dev++;
             : 		int_dev_entry++;
             : 	}
> Since the implementation was to directly memcpy the interrupt config into FSP intconfig, I had done  […]

Aren't you just passing in pointer to the table in FSP UPD? Then why do you need the memcpy?