Hi Simon, On Tue, Oct 22, 2024 at 2:16 PM Simon Glass sjg@chromium.org wrote:
Hi Peter,
On Mon, 21 Oct 2024 at 19:57, Peter Robinson pbrobinson@gmail.com wrote:
Allow to use BLOBLIST_TABLES on arm to store ACPI or other tables.
Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com Cc: Tom Rini trini@konsulko.com
Changelog v9:
- default to BLOBLIST_ALLOC on arm
- Move default for BLOBLIST_SIZE_RELOC up
common/Kconfig | 2 ++ lib/Kconfig | 15 +++++++++------ 2 files changed, 11 insertions(+), 6 deletions(-)
This is fine, but please disable it for snow since it needs the FIXED option for now.
I cannot follow. What needs the FIXED option and what to disable? I run this patch on the CI and test_py_sandbox tests are still working.
I mean that snow cannot use BLOBLIST_ALLOC and needs BLOBLIST_FIXED so if you make ALLOC the default you need to change the default for snow.
Simon by snow do you mean the device (configs/snow_defconfig) snow, I think Patrick doesn't know you're referring to what I believe to be a device config.
Oh OK, yes that is what I mean. If it is too confusing I can send a fix-up patch after this series is applied.
Oh OK, got it. I wasn't aware that BLOBLIST is already used on some ARM devices. I'll send an updated version. Is it possible to migrate those to BLOBLIST_ALLOC? Any reason they would use a fixed address?
Regards, SImon