[PATCH v2 2/7] binman: Update missing optional external blob warning text
    Jonas Karlman 
    jonas at kwiboo.se
       
    Tue Jul 18 22:34:34 CEST 2023
    
    
  
Make it more clear that the missing external blob is optional in the
printed warning message.
Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
---
v2:
- New patch
 tools/binman/control.py | 2 +-
 tools/binman/ftest.py   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/binman/control.py b/tools/binman/control.py
index 25e66814837d..3560cadba4c2 100644
--- a/tools/binman/control.py
+++ b/tools/binman/control.py
@@ -674,7 +674,7 @@ def ProcessImage(image, update_fdt, write_map, get_contents=True,
     image.CheckOptional(optional_list)
     if optional_list:
         tout.warning(
-            "Image '%s' is missing external blobs but is still functional: %s" %
+            "Image '%s' is missing optional external blobs but is still functional: %s" %
             (image.name, ' '.join([e.name for e in optional_list])))
         _ShowHelpForMissingBlobs(optional_list)
 
diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py
index e53181afb78a..152f800c676c 100644
--- a/tools/binman/ftest.py
+++ b/tools/binman/ftest.py
@@ -6330,7 +6330,7 @@ fdt         fdtmap                Extract the devicetree blob from the fdtmap
         err = stderr.getvalue()
         self.assertRegex(
             err,
-            "Image '.*' is missing external blobs but is still functional: missing")
+            "Image '.*' is missing optional external blobs but is still functional: missing")
 
     def testSectionInner(self):
         """Test an inner section with a size"""
-- 
2.41.0
    
    
More information about the U-Boot
mailing list