diff options
| author | Simon Glass | 2021-11-23 11:03:52 -0700 |
|---|---|---|
| committer | Simon Glass | 2021-12-02 09:16:30 -0700 |
| commit | 3fc20fd8055f59137293e487244d8b3d885bbbe5 (patch) | |
| tree | 79ab5f2dcdc6df3286e94b4392276e0b329ffc30 /tools/binman/etype | |
| parent | 080f859cf180e860ec5f3c8b7940820c5a32b02a (diff) | |
binman: cbfs: Refactor the init process
Update the constructor to work in the recommended way, where the node
properties are read in a separate function. This makes it more similar to
entry_Section.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/binman/etype')
| -rw-r--r-- | tools/binman/etype/cbfs.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tools/binman/etype/cbfs.py b/tools/binman/etype/cbfs.py index 873b199a91d..a5120127059 100644 --- a/tools/binman/etype/cbfs.py +++ b/tools/binman/etype/cbfs.py @@ -168,12 +168,16 @@ class Entry_cbfs(Entry): from binman import state super().__init__(section, etype, node) - self._cbfs_arg = fdt_util.GetString(node, 'cbfs-arch', 'x86') self.align_default = None self._entries = OrderedDict() - self.ReadEntries() self.reader = None + def ReadNode(self): + """Read properties from the atf-fip node""" + super().ReadNode() + self._cbfs_arg = fdt_util.GetString(self._node, 'cbfs-arch', 'x86') + self.ReadEntries() + def ReadEntries(self): """Read the subnodes to find out what should go in this CBFS""" for node in self._node.subnodes: |