Re: [PATCH v3 09/14] bloblist: Handle alignment with a void entry

2023-12-27 Thread Ilias Apalodimas
On Wed, 27 Dec 2023 at 19:49, Simon Glass wrote: > > Hi Ilias, > > On Wed, Dec 27, 2023 at 9:58 AM Ilias Apalodimas > wrote: > > > > On Mon, 18 Dec 2023 at 20:20, Raymond Mao wrote: > > > > > > From: Simon Glass > > > > > > Rather than setting the alignment using the header size, add an entirel

Re: [PATCH v3 09/14] bloblist: Handle alignment with a void entry

2023-12-27 Thread Simon Glass
Hi Ilias, On Wed, Dec 27, 2023 at 9:58 AM Ilias Apalodimas wrote: > > On Mon, 18 Dec 2023 at 20:20, Raymond Mao wrote: > > > > From: Simon Glass > > > > Rather than setting the alignment using the header size, add an entirely > > new entry to cover the gap left by the alignment. > > Hmm, why? D

Re: [PATCH v3 09/14] bloblist: Handle alignment with a void entry

2023-12-27 Thread Raymond Mao
Hi Ilias, On Wed, 27 Dec 2023 at 04:58, Ilias Apalodimas wrote: > On Mon, 18 Dec 2023 at 20:20, Raymond Mao wrote: > > > > From: Simon Glass > > > > Rather than setting the alignment using the header size, add an entirely > > new entry to cover the gap left by the alignment. > > Hmm, why? Does

Re: [PATCH v3 09/14] bloblist: Handle alignment with a void entry

2023-12-27 Thread Ilias Apalodimas
On Mon, 18 Dec 2023 at 20:20, Raymond Mao wrote: > > From: Simon Glass > > Rather than setting the alignment using the header size, add an entirely > new entry to cover the gap left by the alignment. Hmm, why? Does it make out life easier somehow if new entries get added? > > Signed-off-by: Si

[PATCH v3 09/14] bloblist: Handle alignment with a void entry

2023-12-18 Thread Raymond Mao
From: Simon Glass Rather than setting the alignment using the header size, add an entirely new entry to cover the gap left by the alignment. Signed-off-by: Simon Glass Co-developed-by: Raymond Mao Signed-off-by: Raymond Mao Reviewed-by: Simon Glass --- common/bloblist.c | 23 +++