Hi Ilias, On Wed, 27 Dec 2023 at 04:43, Ilias Apalodimas <ilias.apalodi...@linaro.org> wrote:
> 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? > > 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
