Provided everyone is okay with the current method -- then we're stuck waiting for the CQ-DEPEND CL to be downstreamed through vboot git submodule.
3 comments:
Patch Set #2, Line 164: REGION(vboot2_work, addr, CONFIG_VBOOT_WORKING_DATA_SIZE, 16)
I mean, you can put a quick /* 4K gap */ comment in there to make it easier to notice if you want, b […]
I'll just leave it without a note.
File src/security/vboot/Kconfig:
Patch Set #2, Line 95: VBOOT_WORKING_DATA_SIZE
Yeah... like I mentioned on the vboot CL I really don't think that's very clean. […]
Yeah, I guess as long as *which files may be included* is well-defined, we can allow more flexibility than just vb2_api.h. Which is probably necessary if we want to turn vboot_reference into the "toolkit" concept.
I also asked a question on the other CL about the other NEED_SHA2_WHATEVER_IT_WAS -- I suppose you've answered it here.
File src/security/vboot/common.c:
Patch Set #3, Line 43: static const size_t vb2_wd_size = VB2_WORKBUF_RECOMMENDED_SIZE;
This constant doesn't seem useful now, why not just use VB2_WORKBUF_RECOMMENDED_SIZE directly in the […]
Right, let's do that instead.
To view, visit change 31474. To unsubscribe, or for help writing mail filters, visit settings.