On Thu, Sep 03, 2020 at 07:28:51PM -0600, Simon Glass wrote: > At present it is painful to put Edison in a hardware lab because it has > two separate recovery modes. When the board has a functioning U-Boot, DFU > can be used. Otherwise an xFSTK image must be used. > > This series converts Andy's script to a binman description so that U-Boot > can produce an xFSTK image directly. > > With this, I can put an Edison in my lab fairly easily. > > The series is available at u-boot-dm/edison-working and is based on the > reset binman series for sunxi.
Thanks for doing this! It will reduce burden when unbricking the board! I have few minor comments (individually per patch) and one ask to Cc next version to Ferry Toth <fnt...@gmail.com>. > [1] https://gist.github.com/andy-shev/2c388310f2773ead647d9c1a3f1c813f > > > Simon Glass (5): > x86: Use multiple images > binman: Show an error when a file is missing > binman: Support adding a U-Boot environment > x86: edison: Generate an image suitable for xFSTK > x86: edison: Add documentation for using am xFSTK image > > arch/x86/cpu/tangier/Kconfig | 1 + > arch/x86/dts/edison.dts | 34 ++++++ > arch/x86/dts/u-boot.dtsi | 7 -- > board/intel/edison/edison-environment.txt | 48 +++++++++ > board/intel/edison/edison-mbr.dat | Bin 0 -> 512 bytes > doc/board/intel/edison.rst | 120 ++++++++++++++++++++++ > tools/binman/etype/blob.py | 7 +- > tools/binman/etype/u_boot_env.py | 42 ++++++++ > tools/binman/ftest.py | 38 +++++++ > tools/binman/test/173_missing_blob.dts | 14 +++ > tools/binman/test/174_env.dts | 20 ++++ > tools/binman/test/175_env_no_size.dts | 19 ++++ > tools/binman/test/176_env_too_small.dts | 20 ++++ > 13 files changed, 360 insertions(+), 10 deletions(-) > create mode 100644 board/intel/edison/edison-environment.txt > create mode 100644 board/intel/edison/edison-mbr.dat > create mode 100644 tools/binman/etype/u_boot_env.py > create mode 100644 tools/binman/test/173_missing_blob.dts > create mode 100644 tools/binman/test/174_env.dts > create mode 100644 tools/binman/test/175_env_no_size.dts > create mode 100644 tools/binman/test/176_env_too_small.dts > > -- > 2.28.0.526.ge36021eeef-goog > -- With Best Regards, Andy Shevchenko