[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:

(4 comments)

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

https://review.coreboot.org/c/coreboot/+/34449/6/src/soc/intel/common/block/... 
PS6, Line 456: devfn < 0
> Yes. Line 236 also shows check for devfn < 0.

Done

https://review.coreboot.org/c/coreboot/+/34449/6/src/soc/intel/common/block/... 
PS6, Line 460:          }
> code indent should use tabs where possible

Done

https://review.coreboot.org/c/coreboot/+/34449/6/src/soc/intel/common/block/... 
PS6, Line 460:
> tab ?

Done

https://review.coreboot.org/c/coreboot/+/34449/6/src/soc/intel/common/block/... 
PS6, Line 460:          }
> please, no spaces at the start of a line

Done