Skip to content
Snippets Groups Projects
  1. Apr 25, 2022
    • Alper Nebi Yasak's avatar
      binman: Collect bintools for images when replacing entries · 8ee4ec9b
      Alper Nebi Yasak authored and Tom Rini's avatar Tom Rini committed
      
      Binman entries can use other executables to compute their data, usually
      in their ObtainContents() methods. Subclasses of Entry_section would use
      bintools in their BuildSectionData() method instead, which is called
      from several places including their Pack().
      
      These binary tools are resolved correctly while building an image from a
      device-tree description so that they can be used from these methods.
      However, this is not being done when replacing entries in an image,
      which can result in an error as the Pack() methods attempt to use them.
      
      Collect and resolve entries' bintools also when replacing entries to fix
      Pack() errors. Add a way to mock bintool usage in the testing entry type
      and tests that check bintools are being resolved for such an entry.
      
      Signed-off-by: default avatarAlper Nebi Yasak <alpernebiyasak@gmail.com>
      Reviewed-by: Simon Glass's avatarSimon Glass <sjg@chromium.org>
      8ee4ec9b
    • Alper Nebi Yasak's avatar
      binman: Fix unique names having '/.' for images read from files · 67bf2c8d
      Alper Nebi Yasak authored and Tom Rini's avatar Tom Rini committed
      
      Binman can embed a copy of the image description into the images it
      builds as a fdtmap entry, but it omits the /binman/<image-name> prefix
      from the node paths while doing so. When reading an already-built image
      file, entries are reconstructed using this fdtmap and their associated
      nodes still lack that prefix.
      
      Some entries like fit and vblock create intermediate files whose names
      are based on an entry unique name. This name is constructed from their
      node's path by concatenating the parents with dots up to the binman
      node, e.g. /binman/image/foo/bar becomes 'image.foo.bar'.
      
      However, we don't have this /binman/image prefix when replacing entries
      in such an image. The /foo/bar entry we read when doing so erroneously
      has the unique name of '/.foo.bar', causing permission errors when the
      entry attempts to create files based on that.
      
      Fix the unique-name generation by stopping at the '/' node like how it
      stops at the binman node. As the unique names are used as filenames, add
      tests that check if they're safe to use as filenames.
      
      Signed-off-by: default avatarAlper Nebi Yasak <alpernebiyasak@gmail.com>
      Reviewed-by: Simon Glass's avatarSimon Glass <sjg@chromium.org>
      67bf2c8d
  2. Apr 23, 2022
    • Tom Rini's avatar
      Merge branch '2022-04-22-assorted-updates' · 9bb99fa9
      Tom Rini authored
      - Add "-q" to fdt addr and use it in distro_bootcmd to make the user
        experience less scary reading in normal try/fail cases.
      - Let the adc update an environment variable like many other commands do
      - Fix TPL SEPARATE_BSS check when locating DTB
      - Allow ":" in PXE file names again
      - Two Apple M1 fixes
      9bb99fa9
  3. Apr 22, 2022
  4. Apr 21, 2022
Loading