Hi Raymond, On Mon, 4 Dec 2023 at 13:21, Raymond Mao <raymond....@linaro.org> wrote: > > Hi Simon, > > On Sat, 2 Dec 2023 at 13:32, Simon Glass <s...@chromium.org> wrote: >> >> Hi Raymond, >> >> On Mon, 27 Nov 2023 at 12:53, Raymond Mao <raymond....@linaro.org> wrote: >> > >> > During bloblist initialization, when CONFIG_OF_BOARD is defined, >> > invoke the platform custom function to load the bloblist via boot >> > arguments from the previous loader. >> > If the bloblist exists, copy it into the fixed bloblist memory region. >> > >> > Signed-off-by: Raymond Mao <raymond....@linaro.org> >> > --- >> > Changes in v2 >> > - New patch file created for v2. >> > >> > common/bloblist.c | 29 +++++++++++++---------------- >> > include/bloblist.h | 14 ++++++++++++++ >> > 2 files changed, 27 insertions(+), 16 deletions(-) >> > >> >> This should already work so I am hoping that this patch is not >> needed...these changes seem to break sandbox_spl > > By "already work", do you mean any existing patches already done for loading > the > bloblist from the boot args?
No it doesn't have that. But we need to figure out [1]. I have marked in 'new' in patchwork. Regards, Simon [1] https://patchwork.ozlabs.org/project/uboot/patch/20230926141514.2101787-40-...@chromium.org/