binman: Plumb expanded entries through fully

Add support for this feature in the control, image and section modules, so
that expanded entries will be selected by default. So far there are no
expanded entry types, so this is a nop.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2021-03-18 20:25:06 +13:00
parent 63aeaeb463
commit 0b6023ee41
3 changed files with 36 additions and 8 deletions

View File

@ -28,7 +28,7 @@ images = OrderedDict()
# value: Text for the help # value: Text for the help
missing_blob_help = {} missing_blob_help = {}
def _ReadImageDesc(binman_node): def _ReadImageDesc(binman_node, use_expanded):
"""Read the image descriptions from the /binman node """Read the image descriptions from the /binman node
This normally produces a single Image object called 'image'. But if This normally produces a single Image object called 'image'. But if
@ -36,15 +36,17 @@ def _ReadImageDesc(binman_node):
Args: Args:
binman_node: Node object of the /binman node binman_node: Node object of the /binman node
use_expanded: True if the FDT will be updated with the entry information
Returns: Returns:
OrderedDict of Image objects, each of which describes an image OrderedDict of Image objects, each of which describes an image
""" """
images = OrderedDict() images = OrderedDict()
if 'multiple-images' in binman_node.props: if 'multiple-images' in binman_node.props:
for node in binman_node.subnodes: for node in binman_node.subnodes:
images[node.name] = Image(node.name, node) images[node.name] = Image(node.name, node,
use_expanded=use_expanded)
else: else:
images['image'] = Image('image', binman_node) images['image'] = Image('image', binman_node, use_expanded=use_expanded)
return images return images
def _FindBinmanNode(dtb): def _FindBinmanNode(dtb):
@ -399,7 +401,7 @@ def ReplaceEntries(image_fname, input_fname, indir, entry_paths,
return image return image
def PrepareImagesAndDtbs(dtb_fname, select_images, update_fdt): def PrepareImagesAndDtbs(dtb_fname, select_images, update_fdt, use_expanded):
"""Prepare the images to be processed and select the device tree """Prepare the images to be processed and select the device tree
This function: This function:
@ -413,6 +415,9 @@ def PrepareImagesAndDtbs(dtb_fname, select_images, update_fdt):
dtb_fname: Filename of the device tree file to use (.dts or .dtb) dtb_fname: Filename of the device tree file to use (.dts or .dtb)
selected_images: List of images to output, or None for all selected_images: List of images to output, or None for all
update_fdt: True to update the FDT wth entry offsets, etc. update_fdt: True to update the FDT wth entry offsets, etc.
use_expanded: True to use expanded versions of entries, if available.
So if 'u-boot' is called for, we use 'u-boot-expanded' instead. This
is needed if update_fdt is True (although tests may disable it)
Returns: Returns:
OrderedDict of images: OrderedDict of images:
@ -438,7 +443,7 @@ def PrepareImagesAndDtbs(dtb_fname, select_images, update_fdt):
raise ValueError("Device tree '%s' does not have a 'binman' " raise ValueError("Device tree '%s' does not have a 'binman' "
"node" % dtb_fname) "node" % dtb_fname)
images = _ReadImageDesc(node) images = _ReadImageDesc(node, use_expanded)
if select_images: if select_images:
skip = [] skip = []
@ -611,6 +616,13 @@ def Binman(args):
elf.debug = args.debug elf.debug = args.debug
cbfs_util.VERBOSE = args.verbosity > 2 cbfs_util.VERBOSE = args.verbosity > 2
state.use_fake_dtb = args.fake_dtb state.use_fake_dtb = args.fake_dtb
# Normally we replace the 'u-boot' etype with 'u-boot-expanded', etc.
# When running tests this can be disabled using this flag. When not
# updating the FDT in image, it is not needed by binman, but we use it
# for consistency, so that the images look the same to U-Boot at
# runtime.
use_expanded = not args.no_expanded
try: try:
tools.SetInputDirs(args.indir) tools.SetInputDirs(args.indir)
tools.PrepareOutputDir(args.outdir, args.preserve) tools.PrepareOutputDir(args.outdir, args.preserve)
@ -618,7 +630,7 @@ def Binman(args):
state.SetEntryArgs(args.entry_arg) state.SetEntryArgs(args.entry_arg)
images = PrepareImagesAndDtbs(dtb_fname, args.image, images = PrepareImagesAndDtbs(dtb_fname, args.image,
args.update_fdt) args.update_fdt, use_expanded)
missing = False missing = False
for image in images.values(): for image in images.values():
missing |= ProcessImage(image, args.update_fdt, args.map, missing |= ProcessImage(image, args.update_fdt, args.map,

View File

@ -84,7 +84,8 @@ class Entry_section(Entry):
for node in self._node.subnodes: for node in self._node.subnodes:
if node.name.startswith('hash') or node.name.startswith('signature'): if node.name.startswith('hash') or node.name.startswith('signature'):
continue continue
entry = Entry.Create(self, node) entry = Entry.Create(self, node,
expanded=self.GetImage().use_expanded)
entry.ReadNode() entry.ReadNode()
entry.SetPrefix(self._name_prefix) entry.SetPrefix(self._name_prefix)
self._entries[node.name] = entry self._entries[node.name] = entry

View File

@ -47,9 +47,23 @@ class Image(section.Entry_section):
exception). This should be used if the Image is being loaded from exception). This should be used if the Image is being loaded from
a file rather than generated. In that case we obviously don't need a file rather than generated. In that case we obviously don't need
the entry arguments since the contents already exists. the entry arguments since the contents already exists.
use_expanded: True if we are updating the FDT wth entry offsets, etc.
and should use the expanded versions of the U-Boot entries.
Any entry type that includes a devicetree must put it in a
separate entry so that it will be updated. For example. 'u-boot'
normally just picks up 'u-boot.bin' which includes the
devicetree, but this is not updateable, since it comes into
binman as one piece and binman doesn't know that it is actually
an executable followed by a devicetree. Of course it could be
taught this, but then when reading an image (e.g. 'binman ls')
it may need to be able to split the devicetree out of the image
in order to determine the location of things. Instead we choose
to ignore 'u-boot-bin' in this case, and build it ourselves in
binman with 'u-boot-dtb.bin' and 'u-boot.dtb'. See
Entry_u_boot_expanded and Entry_blob_phase for details.
""" """
def __init__(self, name, node, copy_to_orig=True, test=False, def __init__(self, name, node, copy_to_orig=True, test=False,
ignore_missing=False): ignore_missing=False, use_expanded=False):
super().__init__(None, 'section', node, test=test) super().__init__(None, 'section', node, test=test)
self.copy_to_orig = copy_to_orig self.copy_to_orig = copy_to_orig
self.name = 'main-section' self.name = 'main-section'
@ -59,6 +73,7 @@ class Image(section.Entry_section):
self.fdtmap_data = None self.fdtmap_data = None
self.allow_repack = False self.allow_repack = False
self._ignore_missing = ignore_missing self._ignore_missing = ignore_missing
self.use_expanded = use_expanded
if not test: if not test:
self.ReadNode() self.ReadNode()