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

[Subrata Banik (Code Review)]

Subrata Banik 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 7:

(3 comments)

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

https://review.coreboot.org/c/coreboot/+/34449/3/src/soc/intel/common/block/... 
PS3, Line 450: int
> unsigned int or better uint32_t to be consistent with rest of the definition in the file.

Done

https://review.coreboot.org/c/coreboot/+/34449/3/src/soc/intel/common/block/... 
PS3, Line 479: lpss_set_power_state(device, STATE_D0);
> Done

Done

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, please, I was wrong, the function does return signed int. […]

Done