diff mbox series

[Concept,28/30] fit: Use a boolean to simplify type checks

Message ID	20251120025614.2215587-29-sjg@u-boot.org
State	New
Headers	From: Simon Glass <sjg@u-boot.org> To: U-Boot Concept <concept@u-boot.org> Date: Wed, 19 Nov 2025 19:56:03 -0700 Message-ID: <20251120025614.2215587-29-sjg@u-boot.org> In-Reply-To: <20251120025614.2215587-1-sjg@u-boot.org> References: <20251120025614.2215587-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: PRLXAKNCMVPCUXOF7ZKT4TOYP2TM6UHY CC: Heinrich Schuchardt <xypron.glpk@gmx.de>, Simon Glass <simon.glass@canonical.com>, Claude <noreply@anthropic.com> Precedence: list Subject: [Concept] [PATCH 28/30] fit: Use a boolean to simplify type checks Archived-At: <https://lists.u-boot.org/archives/list/concept@u-boot.org/message/PRLXAKNCMVPCUXOF7ZKT4TOYP2TM6UHY/> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit
Series	fit: Improve and test the code to print FIT info \| [Concept,00/30] fit: Improve and test the code to print FIT info [Concept,01/30] docs: Merge .claude/README.md into CLAUDE.md [Concept,02/30] tools: Fix debug() to avoid unused-variable warnings [Concept,03/30] test: Increase expect_str/actual_str buffer size [Concept,04/30] fit: Remove unused len parameter from fit_get_name() [Concept,05/30] test: Add a test for FIT image printing [Concept,06/30] test: Add signature-testing to the FIT-printing test [Concept,07/30] test: fit: Test printing a FIT with multiple FDTs [Concept,08/30] test: fit: Test the remaining features and edge cases [Concept,09/30] test: fit: Add test for missing FIT description [Concept,10/30] test: fit: Add pytest for mkimage output [Concept,11/30] fit: Refactor fit_print_contents() to use new fit_print() [Concept,12/30] fit: Move printing code to its own file [Concept,13/30] fit: Drop showing an unused 'required' property [Concept,14/30] fit: Add a context struct for FIT-printing [Concept,15/30] fit: Put the indent string into print context [Concept,16/30] fit: Create some helpers for printing [Concept,17/30] fit: Use emit_label() helper in fit_image_print() [Concept,18/30] fit: Use emit_label() helper in fit_conf_print() [Concept,19/30] fit: Add a a function to print labels with values [Concept,20/30] fit: Change indent from string to int [Concept,21/30] fit: Move values one column to the right [Concept,22/30] fit: Use emit_label_val() where possible [Concept,23/30] fit: Add a helper to output optional properties [Concept,24/30] fit: Add a helper for timestamp printing [Concept,25/30] fit: Add a helper for stringlist properties [Concept,26/30] fit: Add a helper for printing descriptions [Concept,27/30] fit: Add a helper for address printing [Concept,28/30] fit: Use a boolean to simplify type checks [Concept,29/30] fit: Add a helper to iterate through hash/signature nodes [Concept,30/30] fit: Use the libfdt subnode iterator

Commit Message

Simon Glass Nov. 20, 2025, 2:56 a.m. UTC

  From: Simon Glass <simon.glass@canonical.com>

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>
---

 boot/fit_print.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff mbox series

Patch

diff --git a/boot/fit_print.c b/boot/fit_print.c
index 70199a34c18..6772460a151 100644
--- a/boot/fit_print.c
+++ b/boot/fit_print.c
@@ -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);
 	}