[coreboot-gerrit] Change in coreboot[master]: mb/intel/adlrvp: Disable EC sync for adlrvp_ext_ec

[Furquan Shaikh (Code Review)]

Attention is currently required from: Maulik V Vaghela, Tim Wawrzynczak.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54016 )

Change subject: mb/intel/adlrvp: Disable EC sync for adlrvp_ext_ec
......................................................................

Patch Set 3:

(1 comment)

Patchset:

PS3:
> Surprisingly, platform doesn't go to recovery but I see below error: […]

Yeah, this is an attribute of the broken mocktpm behavior and the assumptions we have made in different places (in this case EFS).

The problem here is that `vb2_secdata_kernel_get_ec_hash()` in vboot_reference assumes that it is safe to store/read mirrored hash from TPM and compare it with the expected hash irrespective of whether EFS is being used or not. But, this doesn't really work in case of mocktpm.

Aseda recently ran into this on keeby as well.