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 3:
(1 comment)
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.