[coreboot-gerrit] Change in coreboot[master]: trogdor: QCSDI loading depends on VB2_GBB_FLAG_RUNNING_FAFT setting flag

[Ravi Kumar Bokka (Code Review)]

Ravi Kumar Bokka has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36518 )

Change subject: trogdor: QCSDI loading depends on VB2_GBB_FLAG_RUNNING_FAFT setting flag
......................................................................

Patch Set 35:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36518/28/src/soc/qualcomm/common/qc... 
File src/soc/qualcomm/common/qclib.c:

https://review.coreboot.org/c/coreboot/+/36518/28/src/soc/qualcomm/common/qc... 
PS28, Line 130: 	struct vb2_context *ctx = vboot_get_context();
> Where? There are no actual changes in the latest uploads.

Julius, we have tried your suggestion as is and it introduced issues that were picked up by BVT testing related to SDI and IDP.  Until we resolve these issues we cannot adopt your suggestion.  We will keep you updated on our progress.