[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Set controller state to active in GSPI init

[Aamir Bohra (Code Review)]

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

Change subject: soc/intel/common: Set controller state to active in GSPI init
......................................................................

Patch Set 5:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34449/5/src/soc/intel/common/block/... 
PS5, Line 450: uint32_t
> Yes, right. […]

Yes, please, I was wrong, the function does return signed int. The reason I asked for change was the devfn would be always unsigned for a valid gspi bus. which would be the case for all ctrl setup call.

May be now also add a  check for a error in gspi_soc_to_devfn(if it returns -1) and exit the setup call.