Hi Ilias,
On Wed, 27 Dec 2023 at 04:43, Ilias Apalodimas ilias.apalodimas@linaro.org wrote:
Hi Raymond,
On Mon, 18 Dec 2023 at 20:20, Raymond Mao raymond.mao@linaro.org wrote:
From: Simon Glass sjg@chromium.org
Drop spare value from bloblist record header.
For now it is still present in the header, with an underscore, so that tests continue to pass.
I am not sure I understand the commit message. Doesn't the spec define this as 'reserved' now? On top of that, it's a better idea to follow the spec naming. It's easier to map the spec -> code for new readers. So I would rename
tag -> tag_id size -> data_size and flags -> reserved.
Can we at least do the flags -> reserved in this patch?
The spec only defines a spare field for TL header (bloblist header) but
not for a TE header (bloblist record header), so we just remove the spare field, no need to update this patch I think.
[...]
Regards, Raymond