Hi Ilias,
On Wed, 27 Dec 2023 at 09:58, Ilias Apalodimas <ilias.apalodi...@linaro.org>
wrote:
> On Wed, 27 Dec 2023 at 16:50, Raymond Mao <raymond....@linaro.org> wrote:
> >
> > Hi Ilias,
> >
> > On Wed, 27 Dec 2023 at 05:17, Ilias Apalodimas <
> ilias.apalodi...@linaro.org> wrote:
> >>
> >>
> >> [...]
> >>
> >> > - * @chksum: checksum for the entire bloblist allocated area. Since
> any of the
> >> > - * blobs can be altered after being created, this checksum is only
> valid
> >> > - * when the bloblist is finalised before jumping to the next stage
> of boot.
> >> > - * This is the value needed to make all checksummed bytes sum to 0
> >> > */
> >> > struct bloblist_hdr {
> >> > u32 magic;
> >> > - u32 version;
> >> > - u32 hdr_size;
> >> > - u32 flags;
> >> > -
> >> > - u32 size;
> >> > + u8 chksum;
> >> > + u8 version;
> >> > + u8 hdr_size;
> >> > + u8 align_log2;
> >> > u32 alloced;
> >> > + u32 size;
> >> > + u32 flags;
> >> > u32 spare;
> >> > - u32 chksum;
> >> > };
> >> >
> >>
> >> Aren't fields still missing from the current version?
> >> e.g max_size and reserved?
> >>
> > They are all in. Please see:
> > [PATCH v3 14/14] bloblist: Align bloblist used_size and total_size to
> spec
>
> Ok. But why aren't we squashing this to #14 then?
>
> I can squash other changes with #14 except the ones for `align_log2`.
And `align_log2` with patch #12 due to the dependency.
Regards,
Raymond
> Thanks
> /Ilias
> >
> > [...]
> >
> > Regards,
> > Raymond
>
