Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak. Maulik V Vaghela 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: […]
Hi Tim, Please disregard my comment above for recovery. After flashing latest EC, I see below error. This is with VBOOT early sync also enabled: VB2:check_ec_hash() Hexp RW(active): b2799ceccdc0f7fdedadfcec4c247c14e049d7d26d0c56dcb9a2ceddffed8919 VB2:check_ec_hash() Hmir: 0000000000000000000000000000000000000000000000000000000000000000 VB2:check_ec_hash() Hmir != Hexp. Update Hmir. ec_hash_image: Hash status not done: 2 VB2:check_ec_hash() vb2ex_ec_hash_image(select, &heff, &heff_len) returned 0x10000001 VB2:vb2api_fail() Need recovery, reason: 0x57 / 0x1 VB2:vb2api_ec_sync() ec_sync_phase1(ctx) returned 0x1004
We see below recovery reason: EC software sync unable to get EC image hash
-
Maulik V Vaghela (Code Review)