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

(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();
> I mean... […]

Hi Julius,
Do you mean to remove this member "struct vb2_context *ctx = vboot_get_context();"

And replace like below condition:
    if (CONFIG(QC_SDI_ENABLE) && CONFIG(VBOOT) && 
(!(vb2api_gbb_get_flags(vboot_get_context()) & VB2_GBB_FLAG_RUNNING_FAFT))))

after doing this changes, not able to work SDI_ENABLE feature. Could you please suggest/correct me,if i understood wrong.