Hi Tom, On Mon, 6 Jan 2025 at 14:41, Tom Rini <[email protected]> wrote: > > On Mon, Jan 06, 2025 at 07:31:59AM -0700, Simon Glass wrote: > > > We want to encourage people to use an allocated bloblist since it is > > more flexible than a fixed one. Make this the default, being sure not to > > change existing users. > > > > The unit tests require BLOBLIST_FIXED so add a dependency in the > > Makefile to avoid build errors. > > > > Signed-off-by: Simon Glass <[email protected]> > [snip] > > diff --git a/common/Kconfig b/common/Kconfig > > index e8d89bf6eb9..a00387a4609 100644 > > --- a/common/Kconfig > > +++ b/common/Kconfig > > @@ -1039,16 +1039,10 @@ if BLOBLIST > > > > choice > > prompt "Bloblist location" > > + default BLOBLIST_FIXED if SANDBOX > > help > > Select the location of the bloblist, via various means. > > This should just be "default BLOBLIST_ALLOC" and sandbox platforms that > are using the bloblist can be like the other platforms using a fixed > address.
We don't need 'default BLOBLIST_ALLOC' as it is now first, so is the default. I would like the 'fixed' thing to be later on so that people know we want to stop using it. All the sandbox platforms use BLOBLIST_FIXED. I'm not sure what change you are wanting me to make to this patch? Regards, Simon
