11 comments:
File src/soc/intel/common/block/cse/cse.c:
Patch Set #14, Line 609: memset(&reply, 0, reply_size);
Not required. Removed the code.
Done
Patch Set #14, Line 650: padding
Done in the patch @ https://review.coreboot. […]
Done
Patch Set #14, Line 662: or temporary disable mode
Please refer patch: https://review.coreboot.org/c/coreboot/+/37283/13 for latest. […]
Done
Patch Set #14, Line 664: hfs1.fields.working_state
Helper function wouldn't be unnecessay function call & additional operations to return the bitfield.
Helper function wouldn't be unnecessay function calls & additional operations to return the bitfield?
Patch Set #14, Line 679: return 1;
keeping global reset out of this command handler will help supporting all flows of CSE FW Update.
Done
Patch Set #14, Line 687: DISABLES
DISABLED
Implemented in patch - https://review.coreboot.org/c/coreboot/+/37283/17
Patch Set #14, Line 698: padding
reserved
Implemented in https://review.coreboot.org/c/coreboot/+/37283/17
This command can be sent only if CWS is normal. […]
Yes, I add 'TODO' statement, and will update HMRFPO_STATUS once Chrome SKU is available.
File src/soc/intel/common/block/include/intelblocks/cse.h:
Patch Set #15, Line 47: #if CONFIG(SOC_INTEL_APOLLOLAKE)
correct.
moved me_hfs1 to SoC specific folders.
File src/soc/intel/common/block/include/intelblocks/cse.h:
Change is done in the patch - https://review.coreboot. […]
Done
File src/soc/intel/common/block/include/intelblocks/cse.h:
Patch Set #13, Line 49: u32 bist_test_result: 1;
The bit fields have been documented in the respective SOC's ME BIOS Writer Guide.
Done
To view, visit change 35546. To unsubscribe, or for help writing mail filters, visit settings.