On 15.02.22 18:06, Jan Kiszka wrote:
On 15.02.22 17:50, Jan Kiszka wrote:
On 15.02.22 13:27, Alper Nebi Yasak wrote:
On 14/02/2022 12:09, Jan Kiszka wrote:
On 07.02.22 23:08, Alper Nebi Yasak wrote:
The binman FIT entry type shares some code with the Section entry type. This shared code is bound to grow, since FIT entries are conceptually a variation of Section entries.
Make FIT entry type a subclass of Section entry type, simplifying it a bit and providing us the features that Section implements. Also fix the subentry alignment test which now attempts to write symbols to a nonexistent SPL ELF test file by creating it first.
Signed-off-by: Alper Nebi Yasak alpernebiyasak@gmail.com Reviewed-by: Simon Glass sjg@chromium.org
Only came to testing this now, and it causes a regression.
Before this commit (I've added fdtmap to arch/arm/dts/k3-am65-iot2050-boot-image.dtsi for this):
$ source/tools/binman/binman -D ls -i flash.bin Name Image-pos Size Entry-type Offset Uncomp-size
main-section 0 8c0000 section 0 blob-ext@0x000000 0 37373 blob-ext@0x000000 0 blob@0x080000 80000 84484 blob@0x080000 80000 fit@0x280000 280000 f445f fit@0x280000 280000 fdtmap 37445f cf9 fdtmap 37445f fill@0x680000 680000 20000 fill@0x680000 680000 fill@0x6a0000 6a0000 20000 fill@0x6a0000 6a0000 blob-ext@0x6c0000 6c0000 415de blob-ext@0x6c0000 6c0000 blob-ext@0x740000 740000 43952 blob-ext@0x740000 740000 blob-ext@0x7c0000 7c0000 415e2 blob-ext@0x7c0000 7c0000 blob-ext@0x840000 840000 4395a blob-ext@0x840000 840000
With this commit:
$ source/tools/binman/binman -D ls -i flash.bin Name Image-pos Size Entry-type Offset Uncomp-size
main-section 0 8c0000 section 0 blob-ext@0x000000 0 37373 blob-ext@0x000000 0 blob@0x080000 80000 84484 blob@0x080000 80000 fit@0x280000 fit@0x280000 280000 u-boot section u-boot-nodtb u-boot-nodtb fdt-iot2050-basic section blob blob fdt-iot2050-basic-pg2 section blob blob fdt-iot2050-advanced section blob blob fdt-iot2050-advanced-pg2 section blob blob k3-rti-wdt-firmware section blob-ext blob-ext fdtmap 37445f cd9 fdtmap 37445f fill@0x680000 680000 20000 fill@0x680000 680000 fill@0x6a0000 6a0000 20000 fill@0x6a0000 6a0000 blob-ext@0x6c0000 6c0000 415de blob-ext@0x6c0000 6c0000 blob-ext@0x740000 740000 43952 blob-ext@0x740000 740000 blob-ext@0x7c0000 7c0000 415e2 blob-ext@0x7c0000 7c0000 blob-ext@0x840000 840000 4395a blob-ext@0x840000 840000
The AddMissingProperties() and SetCalculatedProperties() methods were disabled for FIT as a fixup to this patch, that's why image-pos etc. aren't available. See comments at [1] for some context.
Hopefully my two patches [2][3] fix things, can you test with them?
[1] "binman: Correct the error message for a bad hash algorithm" https://patchwork.ozlabs.org/project/uboot/patch/20220208105941.1.I8f212a115...
[2] "binman: Skip processing "hash" subnodes of FIT subsections" https://patchwork.ozlabs.org/project/uboot/patch/20220209190236.26479-1-alpe...
This one helped, indeed.
...not completely:
$ source/tools/binman/binman replace -i flash.bin -f fit@0x380000.fit fit@0x380000binman: [Errno 13] Permission denied: '/.fit@0x380000.itb'
Ping.
$ source/tools/binman/binman -D replace -i flash.bin -f fit@0x380000.fit fit@0x380000 binman: [Errno 13] Permission denied: '/.fit@0x380000.itb'
Traceback (most recent call last): File "source/tools/binman/binman", line 141, in RunBinman ret_code = control.Binman(args) File "u-boot/tools/binman/control.py", line 644, in Binman allow_resize=not args.fix_size, write_map=args.map) File "u-boot/tools/binman/control.py", line 404, in ReplaceEntries allow_resize=allow_resize, write_map=write_map) File "u-boot/tools/binman/control.py", line 341, in WriteEntryToImage AfterReplace(image, allow_resize=allow_resize, write_map=write_map) File "u-boot/tools/binman/control.py", line 333, in AfterReplace get_contents=False, allow_resize=allow_resize) File "u-boot/tools/binman/control.py", line 560, in ProcessImage image.PackEntries() File "u-boot/tools/binman/image.py", line 155, in PackEntries super().Pack(0) File "u-boot/tools/binman/etype/section.py", line 385, in Pack self._PackEntries() File "u-boot/tools/binman/etype/section.py", line 403, in _PackEntries offset = entry.Pack(offset) File "u-boot/tools/binman/etype/section.py", line 390, in Pack data = self.BuildSectionData(True) File "u-boot/tools/binman/etype/fit.py", line 265, in BuildSectionData tools.write_file(input_fname, data) File "u-boot/tools/patman/tools.py", line 482, in write_file with open(filename(fname), binary and 'wb' or 'w') as fd: PermissionError: [Errno 13] Permission denied: '/.fit@0x380000.itb'
Something seems fairly broken here. That '/.' does not come from the output directory name, it's generated by Entry.GetUniqueName. Looks like this path should not been taken under these conditions.
Jan