On 6 July 2018 at 10:27, Simon Glass <s...@chromium.org> wrote: > This method is supposed to return the contents of an entry. However at > present there is no check that it actually does. Also some implementations > do not return 'True' to indicate success, as required. > > Add a check for things working as expected, and correct the > implementations. > > This requires some additional test cases to cover things which were missed > originally. Add these at the same time. > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > Changes in v2: None > > tools/binman/bsection.py | 4 ++ > tools/binman/etype/_testing.py | 5 ++ > tools/binman/etype/section.py | 2 +- > tools/binman/etype/u_boot_spl_bss_pad.py | 1 + > tools/binman/etype/u_boot_ucode.py | 9 ++- > tools/binman/ftest.py | 57 +++++++++++++++---- > tools/binman/test/57_unknown_contents.dts | 14 +++++ > .../test/58_x86_ucode_spl_needs_retry.dts | 36 ++++++++++++ > 8 files changed, 114 insertions(+), 14 deletions(-) > create mode 100644 tools/binman/test/57_unknown_contents.dts > create mode 100644 tools/binman/test/58_x86_ucode_spl_needs_retry.dts
Applied to u-boot-dm _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
