Hi Simon,
On 2/27/26 12:47 AM, Simon Glass wrote:
From: Simon Glass <[email protected]>
Move about 40 test files for FIT images (signing, external data,
split-elf, encryption, alignment, firmware loadables, templates) into
a fit/ subdirectory. Drop the numeric prefixes and the redundant fit_
filename prefix, and update all references.
Rename the three signature.dts variants to have unique names:
signature.dts, signature_multi_key.dts and signature_no_nodes.dts.
What about aes256.bin, 340_rsa2048.key, dummy-rsa-engine.c,
340_dummy-rsa4096.crt, 340_openssl.conf, 340_softhsm2.conf, and dev.key?
Can be done in a separate patch.
Signed-off-by: Simon Glass <[email protected]>
---
tools/binman/ftest.py | 118 +++++++++---------
.../test/{275_fit_align.dts => fit/align.dts} | 0
.../{227_fit_bad_dir.dts => fit/bad_dir.dts} | 2 +-
.../bad_dir_config.dts} | 2 +-
.../bad_oper.dts} | 0
.../encrypt_data.dts} | 0
.../encrypt_data_no_key.dts} | 0
.../external.dts} | 0
.../extract_replace.dts} | 0
.../test/{170_fit_fdt.dts => fit/fdt.dts} | 0
.../fdt_compat.dts} | 0
.../{333_fit_fdt_dir.dts => fit/fdt_dir.dts} | 0
.../fdt_list.dts} | 0
.../fdt_missing_prop.dts} | 0
.../fdt_name.dts} | 0
.../fdt_oper.dts} | 0
.../fdt_phase.dts} | 0
.../firmware_loadables.dts} | 0
.../binman/test/{161_fit.dts => fit/fit.dts} | 0
.../{219_fit_gennode.dts => fit/gennode.dts} | 0
.../image_subentry_alignment.dts} | 0
.../key_name_hint_dir_signature.dts} | 0
.../missing_blob.dts} | 0
.../replace_sibling.dts} | 0
.../test/{280_fit_sign.dts => fit/sign.dts} | 0
.../signature.dts} | 0
.../signature_engine.dts} | 0
.../signature_engine_encrypt.dts} | 0
.../signature_engine_pkcs11.dts} | 0
.../signature_engine_pkcs11_object.dts} | 0
.../signature_multi_key.dts} | 0
.../signature_no_nodes.dts} | 0
.../split_elf.dts} | 0
.../subentry_bintool.dts} | 0
.../subentry_hash.dts} | 0
.../tee_os_opt.dts} | 0
.../tee_os_opt_bad.dts} | 0
.../template.dts} | 0
38 files changed, 61 insertions(+), 61 deletions(-)
rename tools/binman/test/{275_fit_align.dts => fit/align.dts} (100%)
rename tools/binman/test/{227_fit_bad_dir.dts => fit/bad_dir.dts} (71%)
rename tools/binman/test/{228_fit_bad_dir_config.dts =>
fit/bad_dir_config.dts} (71%)
rename tools/binman/test/{224_fit_bad_oper.dts => fit/bad_oper.dts} (100%)
rename tools/binman/test/{343_fit_encrypt_data.dts => fit/encrypt_data.dts}
(100%)
rename tools/binman/test/{344_fit_encrypt_data_no_key.dts =>
fit/encrypt_data_no_key.dts} (100%)
rename tools/binman/test/{162_fit_external.dts => fit/external.dts} (100%)
rename tools/binman/test/{240_fit_extract_replace.dts =>
fit/extract_replace.dts} (100%)
rename tools/binman/test/{170_fit_fdt.dts => fit/fdt.dts} (100%)
rename tools/binman/test/{334_fit_fdt_compat.dts => fit/fdt_compat.dts} (100%)
rename tools/binman/test/{333_fit_fdt_dir.dts => fit/fdt_dir.dts} (100%)
rename tools/binman/test/{284_fit_fdt_list.dts => fit/fdt_list.dts} (100%)
rename tools/binman/test/{171_fit_fdt_missing_prop.dts =>
fit/fdt_missing_prop.dts} (100%)
rename tools/binman/test/{345_fit_fdt_name.dts => fit/fdt_name.dts} (100%)
rename tools/binman/test/{223_fit_fdt_oper.dts => fit/fdt_oper.dts} (100%)
rename tools/binman/test/{335_fit_fdt_phase.dts => fit/fdt_phase.dts} (100%)
rename tools/binman/test/{276_fit_firmware_loadables.dts =>
fit/firmware_loadables.dts} (100%)
rename tools/binman/test/{161_fit.dts => fit/fit.dts} (100%)
rename tools/binman/test/{219_fit_gennode.dts => fit/gennode.dts} (100%)
rename tools/binman/test/{167_fit_image_subentry_alignment.dts =>
fit/image_subentry_alignment.dts} (100%)
rename tools/binman/test/{355_key_name_hint_dir_fit_signature.dts =>
fit/key_name_hint_dir_signature.dts} (100%)
This file doesn't exist. What is this series based on? I'm assuming
https://lore.kernel.org/u-boot/[email protected]/
but can you please specify series dependencies in the cover-letter so we
don't have to guess. I'm also thinking the parent dependency have two
patches we wouldn't need with this series here (rename and drop the
number, patch 1 and 2 of the series).
Cheers,
Quentin
