From: Simon Glass Date: Sat, 20 Jul 2019 18:23:54 +0000 (-0600) Subject: binman: Move Image.BuildImage() into a single function X-Git-Tag: v2025.01-rc5-pxa1908~2857^2~22 X-Git-Url: http://git.dujemihanovic.xyz/html/index.html?a=commitdiff_plain;h=7400107e467da52c7e6772b677f69f4464f6d2ce;p=u-boot.git binman: Move Image.BuildImage() into a single function Now that an Image is an Entry_section, there is no need for the separate BuildSection() function. Drop it and add a bit of logging. Signed-off-by: Simon Glass --- diff --git a/tools/binman/image.py b/tools/binman/image.py index 893e8cb4cd..fd4f504492 100644 --- a/tools/binman/image.py +++ b/tools/binman/image.py @@ -142,16 +142,14 @@ class Image(section.Entry_section): """Write symbol values into binary files for access at run time""" section.Entry_section.WriteSymbols(self, self) - def BuildSection(self, fd, base_offset): - """Write the section to a file""" - fd.seek(base_offset) - fd.write(self.GetData()) - def BuildImage(self): """Write the image to a file""" fname = tools.GetOutputFilename(self._filename) + tout.Info("Writing image to '%s'" % fname) with open(fname, 'wb') as fd: - self.BuildSection(fd, 0) + data = self.GetData() + fd.write(data) + tout.Info("Wrote %#x bytes" % len(data)) def WriteMap(self): """Write a map of the image to a .map file