Hi Simon,
On Thu, 25 Nov 2021 at 15:47, Ilias Apalodimas <[email protected]> wrote: > +cc Sandrine > > On Thu, 25 Nov 2021 at 11:42, Ilias Apalodimas > <[email protected]> wrote: > > > > Hi Simon, > > > > > > On Wed, 24 Nov 2021 at 06:09, Simon Glass <[email protected]> wrote: > > > > > > > > > This series adds support for the FIP format as used by ARM Trusted > > > Firmware (in particular TF-A). > > > > I will use a question you use often "what problem are you trying to solve?". If FIP format is used it means that TF-A/BL2 is going to parse it and verify the hashes/signatures according to TF-A scheme. The packager will embed in a FIP components like Secure Monitor, Secure hypervisor, Secure partitions code and manifests. All in all, U-Boot will be representing a small percentage of the functionality offered by secure firmware as a whole and it feels odd to make another implementation that is more "accessory" rather than critical for the U-Boot community. It may be a good idea but I wish you could explain it. > > This allows images to be created containing a FIP, which itself contains > > > various binaries. With this, image creation can be handled from an > in-tree > > > image description instead of needing to perform a lot of manual steps > or > > > custom scripts to build the FIP. > > > > That's not my experience of building a fip. Packaging even Linux as a BL33 (instead of U-Boot) is very simple. > > > > > > Simon Glass (2): > > > binman: Add a utility module for ATF FIP > > > binman: Add support for ATF FIP > > > > > > scripts/pylint.base | 9 +- > > > tools/binman/entries.rst | 154 ++++++ > > > tools/binman/etype/atf_fip.py | 273 ++++++++++ > > > tools/binman/fip_util.py | 653 +++++++++++++++++++++++ > > > tools/binman/fip_util_test.py | 405 ++++++++++++++ > > > tools/binman/ftest.py | 217 ++++++++ > > > tools/binman/main.py | 4 +- > > > tools/binman/test/203_fip.dts | 21 + > > > tools/binman/test/204_fip_other.dts | 22 + > > > tools/binman/test/205_fip_no_type.dts | 15 + > > > tools/binman/test/206_fip_uuid.dts | 22 + > > > tools/binman/test/207_fip_ls.dts | 25 + > > > tools/binman/test/208_fip_replace.dts | 33 ++ > > > tools/binman/test/209_fip_missing.dts | 19 + > > > tools/binman/test/210_fip_size.dts | 19 + > > > tools/binman/test/211_fip_bad_align.dts | 18 + > > > tools/binman/test/212_fip_collection.dts | 24 + > > > 17 files changed, 1929 insertions(+), 4 deletions(-) > > > create mode 100644 tools/binman/etype/atf_fip.py > > > create mode 100755 tools/binman/fip_util.py > > > create mode 100755 tools/binman/fip_util_test.py > > > create mode 100644 tools/binman/test/203_fip.dts > > > create mode 100644 tools/binman/test/204_fip_other.dts > > > create mode 100644 tools/binman/test/205_fip_no_type.dts > > > create mode 100644 tools/binman/test/206_fip_uuid.dts > > > create mode 100644 tools/binman/test/207_fip_ls.dts > > > create mode 100644 tools/binman/test/208_fip_replace.dts > > > create mode 100644 tools/binman/test/209_fip_missing.dts > > > create mode 100644 tools/binman/test/210_fip_size.dts > > > create mode 100644 tools/binman/test/211_fip_bad_align.dts > > > create mode 100644 tools/binman/test/212_fip_collection.dts > > > > > > -- > > > 2.34.0.rc2.393.gf8c9666880-goog > > > > > > > My python is mediocre at best. I'll try having a look, but CC'ing > > TF-A developers would be a good idea. > > > > Thanks > > /Ilias > -- François-Frédéric Ozog | *Director Business Development* T: +33.67221.6485 [email protected] | Skype: ffozog

