[PATCH V5 3/9] tools: binman: section: replace @ with -

Peng Fan (OSS) peng.fan at oss.nxp.com
Tue May 24 10:00:33 CEST 2022


From: Peng Fan <peng.fan at nxp.com>

In arch/arm/dts/imx8mp-u-boot.dtsi, there are blob-ext at 1, blob-ext at 2 and
etc which is for packing ddr phy firmware. However we could not declare
symbol name such as 'binman_sym_declare(ulong, blob_ext at 1, image_pos)',
because '@' is not allowed, so we choose to declare the symbol
'binman_sym_declare(ulong, blob_ext_1, image_pos);' with '@' replaced with
'_'. It does not impact if there is no '@' in section name.

Tested-by: Tim Harvey <tharvey at gateworks.com> #imx8m[m,n,p]-venice
Reviewed-by: Tom Rini <trini at konsulko.com>
Signed-off-by: Peng Fan <peng.fan at nxp.com>
---
 tools/binman/etype/section.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/binman/etype/section.py b/tools/binman/etype/section.py
index bd67238b919..e3f362b442b 100644
--- a/tools/binman/etype/section.py
+++ b/tools/binman/etype/section.py
@@ -875,7 +875,7 @@ class Entry_section(Entry):
                 entries[entry.GetPath()] = entry
             for entry in to_add.values():
                 self._CollectEntries(entries, entries_by_name, entry)
-        entries_by_name[add_entry.name] = add_entry
+        entries_by_name[add_entry.name.replace('@', '-')] = add_entry
 
     def MissingArgs(self, entry, missing):
         """Report a missing argument, if enabled
-- 
2.36.0



More information about the U-Boot mailing list