[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/gspi: Use GSPI bus id to map to the controller

[Aamir Bohra (Code Review)]

Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34761 )

Change subject: soc/intel/common/gspi: Use GSPI bus id to map to the controller
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34761/3/src/soc/intel/common/block/... 
File src/soc/intel/common/block/gspi/gspi.c:

https://review.coreboot.org/c/coreboot/+/34761/3/src/soc/intel/common/block/... 
PS3, Line 473: 	if (devfn < 0) {
             : 		printk(BIOS_ERR, "%s: No GSPI controller found on SPI bus %u.\n",
             : 			__func__, dev->bus);
             : 		return -1;
             : 	}
nit: Do we need the check here now? as gspi_ctrlr_params_init would already check for valid dev and devfn?