smallsolar/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fit: Use a boolean to simplify type checks

Browse Source 
Add a boolean variable 'loadable' that combines the common type check
for kernel, standalone, and ramdisk images.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
This commit is contained in:
Simon Glass
2025-11-18 12:10:51 -07:00
parent 8f5e311f3b
commit b6674168b9
1 changed files with 9 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
boot/fit_print.c
View File

@@ -340,29 +340,27 @@ void fit_image_print(struct fit_print_ctx *ctx, int image_noffset)
genimg_print_size(size);
/* Remaining, type dependent properties */
if (type == IH_TYPE_KERNEL || type == IH_TYPE_STANDALONE ||
type == IH_TYPE_RAMDISK || type == IH_TYPE_FIRMWARE ||
type == IH_TYPE_FLATDT) {
bool loadable = type == IH_TYPE_KERNEL || type == IH_TYPE_STANDALONE ||
type == IH_TYPE_RAMDISK;
if (loadable || type == IH_TYPE_FIRMWARE || type == IH_TYPE_FLATDT) {
fit_image_get_arch(fit, image_noffset, &arch);
emit_label_val(ctx, "Architecture", genimg_get_arch_name(arch));
}
if (type == IH_TYPE_KERNEL || type == IH_TYPE_RAMDISK ||
type == IH_TYPE_FIRMWARE) {
if (loadable || type == IH_TYPE_FIRMWARE) {
fit_image_get_os(fit, image_noffset, &os);
emit_label_val(ctx, "OS", genimg_get_os_name(os));
}
if (type == IH_TYPE_KERNEL || type == IH_TYPE_STANDALONE ||
type == IH_TYPE_FIRMWARE || type == IH_TYPE_RAMDISK ||
type == IH_TYPE_FPGA || type == IH_TYPE_FLATDT) {
if (loadable || type == IH_TYPE_FIRMWARE || type == IH_TYPE_FPGA ||
type == IH_TYPE_FLATDT) {
ret = fit_image_get_load(fit, image_noffset, &load);
if ((type != IH_TYPE_FLATDT) || !ret)
if (type != IH_TYPE_FLATDT || !ret)
emit_addr(ctx, "Load Address", load, !ret);
}
if (type == IH_TYPE_KERNEL || type == IH_TYPE_STANDALONE ||
type == IH_TYPE_RAMDISK) {
if (loadable) {
ret = fit_image_get_entry(fit, image_noffset, &entry);
emit_addr(ctx, "Entry Point", entry, !ret);
}