From 6fac8e4883d59e78b7195bf75c174544a4ccfa2d Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Mon, 26 Aug 2024 13:11:36 -0600 Subject: [PATCH] binman: fit: Set the image_pos attributes only once The section etype has its own implementation of SetImagePos(), most of which is not useful since the code is included here. So call Entry.SetImagePos() which has the only piece of this which we actually want. Signed-off-by: Simon Glass --- tools/binman/etype/fit.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/binman/etype/fit.py b/tools/binman/etype/fit.py index 3adc4a959e..96f4fdf333 100644 --- a/tools/binman/etype/fit.py +++ b/tools/binman/etype/fit.py @@ -894,7 +894,10 @@ class Entry_fit(Entry_section): """ if self.build_done: return - super().SetImagePos(image_pos) + + # Skip the section processing, since we do that below. Just call the + # entry method + Entry.SetImagePos(self, image_pos) # If mkimage is missing we'll have empty data, # which will cause a FDT_ERR_BADMAGIC error -- 2.39.5