On Sun, Feb 09, 2025 at 02:14:54PM -0700, Simon Glass wrote: > The logic of this has become too confusing. > > The primary issue with the patch is that U-Boot needs to set up a > bloblist in the first phase where BLOBLIST is enabled. Subsequent > phases can then use that bloblist. > > But the first phase of U-Boot cannot assume that one exists. > > Reverting this commit seems like a better starting point for getting > things working for all use-cases. > > Note: The work to tidy this up is apparently underway. For this series, > a revert is the easiest path. > > This reverts commit 66131310d8ff1ba228f989b41bd8812f43be41c3. > > https://lore.kernel.org/u-boot/CAPnjgZ3hMHtiH=f5zkxnniofv_-vfryq1gn7qz5hku8wjo8...@mail.gmail.com/ > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > (no changes since v1) > > common/bloblist.c | 64 ++++++++++++++-------------------------------- > include/bloblist.h | 10 -------- > 2 files changed, 19 insertions(+), 55 deletions(-)
We aren't reverting this change so if you plan to move this series out of your downstream fork you should start from that and stop posting it, that just leads to confusion. -- Tom
signature.asc
Description: PGP signature
