Hi Raymond, On Mon, 18 Dec 2023 at 20:20, Raymond Mao <raymond....@linaro.org> wrote: > > From: Simon Glass <s...@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? [...] Thanks /Ilias