From: Simon Glass <sjg@chromium.org>
Date: Mon, 26 Oct 2020 23:40:18 +0000 (-0600)
Subject: binman: Drop Entry.CheckOffset()
X-Git-Tag: v2021.10~451^2~8
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6ddd61131f3206c5e385dc7e763aea0dc5caa24c;p=platform%2Fkernel%2Fu-boot.git

binman: Drop Entry.CheckOffset()

This function just calls CheckEntries() in the only non-trivial
implementation. Drop it and use CheckEntries() directly.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

diff --git a/tools/binman/entry.py b/tools/binman/entry.py
index 8fa1dce..8946d2b 100644
--- a/tools/binman/entry.py
+++ b/tools/binman/entry.py
@@ -520,7 +520,7 @@ class Entry(object):
         """
         pass
 
-    def CheckOffset(self):
+    def CheckEntries(self):
         """Check that the entry offsets are correct
 
         This is used for entries which have extra offset requirements (other
diff --git a/tools/binman/etype/section.py b/tools/binman/etype/section.py
index 2812989..fb4bf64 100644
--- a/tools/binman/etype/section.py
+++ b/tools/binman/etype/section.py
@@ -296,7 +296,7 @@ class Entry_section(Entry):
         offset = 0
         prev_name = 'None'
         for entry in self._entries.values():
-            entry.CheckOffset()
+            entry.CheckEntries()
             if (entry.offset < self._skip_at_start or
                     entry.offset + entry.size > self._skip_at_start +
                     self.size):
@@ -337,9 +337,6 @@ class Entry_section(Entry):
                 sizes_ok = False
         return sizes_ok and sizes_ok_base
 
-    def CheckOffset(self):
-        self.CheckEntries()
-
     def WriteMap(self, fd, indent):
         """Write a map of the section to a .map file