]> git.proxmox.com Git - mirror_edk2.git/blobdiff - BaseTools/Scripts/BinToPcd.py
BaseTools/BinToPcd: Update for Python 3 compatibility
[mirror_edk2.git] / BaseTools / Scripts / BinToPcd.py
index 68a7ac652d70598502b4fa4e41910f5a0bea9721..52c231615f21a20a105e5e5ab88373efee2de709 100644 (file)
@@ -1,7 +1,7 @@
 ## @file\r
 # Convert a binary file to a VOID* PCD value or DSC file VOID* PCD statement.\r
 #\r
-# Copyright (c) 2016, Intel Corporation. All rights reserved.<BR>\r
+# Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR>\r
 # This program and the accompanying materials\r
 # are licensed and made available under the terms and conditions of the BSD License\r
 # which accompanies this distribution.  The full text of the license may be found at\r
@@ -18,53 +18,67 @@ BinToPcd
 import sys\r
 import argparse\r
 import re\r
+import xdrlib\r
 \r
 #\r
 # Globals for help information\r
 #\r
 __prog__        = 'BinToPcd'\r
-__version__     = '%s Version %s' % (__prog__, '0.9 ')\r
-__copyright__   = 'Copyright (c) 2016, Intel Corporation. All rights reserved.'\r
-__description__ = 'Convert a binary file to a VOID* PCD value or DSC file VOID* PCD statement.\n'\r
+__copyright__   = 'Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.'\r
+__description__ = 'Convert one or more binary files to a VOID* PCD value or DSC file VOID* PCD statement.\n'\r
 \r
 if __name__ == '__main__':\r
   def ValidateUnsignedInteger (Argument):\r
     try:\r
       Value = int (Argument, 0)\r
     except:\r
-      Message = '%s is not a valid integer value.' % (Argument)\r
+      Message = '{Argument} is not a valid integer value.'.format (Argument = Argument)\r
       raise argparse.ArgumentTypeError(Message)\r
     if Value < 0:\r
-      Message = '%s is a negative value.' % (Argument)\r
+      Message = '{Argument} is a negative value.'.format (Argument = Argument)\r
       raise argparse.ArgumentTypeError(Message)\r
     return Value\r
 \r
   def ValidatePcdName (Argument):\r
-    if re.split('[a-zA-Z\_][a-zA-Z0-9\_]*\.[a-zA-Z\_][a-zA-Z0-9\_]*', Argument) <> ['','']:\r
-      Message = '%s is not in the form <PcdTokenSpaceGuidCName>.<PcdCName>' % (Argument)\r
+    if re.split('[a-zA-Z\_][a-zA-Z0-9\_]*\.[a-zA-Z\_][a-zA-Z0-9\_]*', Argument) != ['','']:\r
+      Message = '{Argument} is not in the form <PcdTokenSpaceGuidCName>.<PcdCName>'.format (Argument = Argument)\r
       raise argparse.ArgumentTypeError(Message)\r
     return Argument\r
 \r
   def ValidateGuidName (Argument):\r
-    if re.split('[a-zA-Z\_][a-zA-Z0-9\_]*', Argument) <> ['','']:\r
-      Message = '%s is not a valid GUID C name' % (Argument)\r
+    if re.split('[a-zA-Z\_][a-zA-Z0-9\_]*', Argument) != ['','']:\r
+      Message = '{Argument} is not a valid GUID C name'.format (Argument = Argument)\r
       raise argparse.ArgumentTypeError(Message)\r
     return Argument\r
-    \r
-  def ByteArray (Buffer):\r
+\r
+  def ByteArray (Buffer, Xdr = False):\r
+    if Xdr:\r
+      #\r
+      # If Xdr flag is set then encode data using the Variable-Length Opaque\r
+      # Data format of RFC 4506 External Data Representation Standard (XDR).\r
+      #\r
+      XdrEncoder = xdrlib.Packer()\r
+      for Item in Buffer:\r
+        XdrEncoder.pack_bytes(Item)\r
+      Buffer = bytearray(XdrEncoder.get_buffer())\r
+    else:\r
+      #\r
+      # If Xdr flag is not set, then concatenate all the data\r
+      #\r
+      Buffer = b''.join(Buffer)\r
     #\r
-    # Append byte array of values of the form '{0x01, 0x02, ...}'\r
+    # Return a PCD value of the form '{0x01, 0x02, ...}' along with the PCD length in bytes\r
     #\r
-    return '{%s}' % (', '.join(['0x%02x' % (ord(Item)) for Item in Buffer]))\r
-    \r
+    return '{' + (', '.join(['0x{Byte:02X}'.format(Byte = Item) for Item in Buffer])) + '}', len (Buffer)\r
+\r
   #\r
   # Create command line argument parser object\r
   #\r
-  parser = argparse.ArgumentParser(prog = __prog__, version = __version__,\r
+  parser = argparse.ArgumentParser(prog = __prog__,\r
                                    description = __description__ + __copyright__,\r
                                    conflict_handler = 'resolve')\r
-  parser.add_argument("-i", "--input", dest = 'InputFile', type = argparse.FileType('rb'),\r
-                      help = "Input binary filename", required = True)\r
+  parser.add_argument("-i", "--input", dest = 'InputFile', type = argparse.FileType('rb'), action='append', required = True,\r
+                      help = "Input binary filename.  Multiple input files are combined into a single PCD.")\r
   parser.add_argument("-o", "--output", dest = 'OutputFile', type = argparse.FileType('wb'),\r
                       help = "Output filename for PCD value or PCD statement")\r
   parser.add_argument("-p", "--pcd", dest = 'PcdName', type = ValidatePcdName,\r
@@ -74,11 +88,13 @@ if __name__ == '__main__':
   parser.add_argument("-m", "--max-size", dest = 'MaxSize', type = ValidateUnsignedInteger,\r
                       help = "Maximum size of the PCD.  Ignored with --type HII.")\r
   parser.add_argument("-f", "--offset", dest = 'Offset', type = ValidateUnsignedInteger,\r
-                      help = "VPD offset if --type is VPD.  UEFI Variable offset if --type is HII.")\r
+                      help = "VPD offset if --type is VPD.  UEFI Variable offset if --type is HII.  Must be 8-byte aligned.")\r
   parser.add_argument("-n", "--variable-name", dest = 'VariableName',\r
                       help = "UEFI variable name.  Only used with --type HII.")\r
   parser.add_argument("-g", "--variable-guid", type = ValidateGuidName, dest = 'VariableGuid',\r
                       help = "UEFI variable GUID C name.  Only used with --type HII.")\r
+  parser.add_argument("-x", "--xdr", dest = 'Xdr', action = "store_true",\r
+                      help = "Encode PCD using the Variable-Length Opaque Data format of RFC 4506 External Data Representation Standard (XDR)")\r
   parser.add_argument("-v", "--verbose", dest = 'Verbose', action = "store_true",\r
                       help = "Increase output messages")\r
   parser.add_argument("-q", "--quiet", dest = 'Quiet', action = "store_true",\r
@@ -92,14 +108,22 @@ if __name__ == '__main__':
   args = parser.parse_args()\r
 \r
   #\r
-  # Read binary input file\r
+  # Read all binary input files\r
   #\r
-  try:\r
-    Buffer = args.InputFile.read()\r
-    args.InputFile.close()\r
-  except:\r
-    print 'BinToPcd: error: can not read binary input file'\r
-    sys.exit()\r
+  Buffer = []\r
+  for File in args.InputFile:\r
+    try:\r
+      Buffer.append(File.read())\r
+      File.close()\r
+    except:\r
+      print ('BinToPcd: error: can not read binary input file {File}'.format (File = File))\r
+      sys.exit(1)\r
+\r
+  #\r
+  # Convert PCD to an encoded string of hex values and determine the size of\r
+  # the encoded PCD in bytes.\r
+  #\r
+  PcdValue, PcdSize = ByteArray (Buffer, args.Xdr)\r
 \r
   #\r
   # Convert binary buffer to a DSC file PCD statement\r
@@ -107,9 +131,10 @@ if __name__ == '__main__':
   if args.PcdName is None:\r
     #\r
     # If PcdName is None, then only a PCD value is being requested.\r
-    Pcd = ByteArray (Buffer)\r
+    #\r
+    Pcd = PcdValue\r
     if args.Verbose:\r
-      print 'PcdToBin: Convert binary file to PCD Value'\r
+      print ('BinToPcd: Convert binary file to PCD Value')\r
   elif args.PcdType is None:\r
     #\r
     # If --type is neither VPD nor HII, then use PCD statement syntax that is\r
@@ -121,62 +146,70 @@ if __name__ == '__main__':
       # If --max-size is not provided, then do not generate the syntax that\r
       # includes the maximum size.\r
       #\r
-      Pcd = '  %s|%s' % (args.PcdName, ByteArray (Buffer))\r
-    elif args.MaxSize < len(Buffer):\r
-      print 'BinToPcd: error: argument --max-size is smaller than input file.'\r
-      sys.exit()\r
+      Pcd = '  {Name}|{Value}'.format (Name = args.PcdName, Value = PcdValue)\r
+    elif args.MaxSize < PcdSize:\r
+      print ('BinToPcd: error: argument --max-size is smaller than input file.')\r
+      sys.exit(1)\r
     else:\r
-      Pcd = '  %s|%s|VOID*|%d' % (args.PcdName, ByteArray (Buffer), args.MaxSize)\r
-      args.MaxSize = len(Buffer)\r
-    \r
+      Pcd = '  {Name}|{Value}|VOID*|{Size}'.format (Name = args.PcdName, Value = PcdValue, Size = args.MaxSize)\r
+\r
     if args.Verbose:\r
-      print 'PcdToBin: Convert binary file to PCD statement compatible with PCD sections:'\r
-      print '    [PcdsFixedAtBuild]'\r
-      print '    [PcdsPatchableInModule]'\r
-      print '    [PcdsDynamicDefault]'\r
-      print '    [PcdsDynamicExDefault]'\r
+      print ('BinToPcd: Convert binary file to PCD statement compatible with PCD sections:')\r
+      print ('    [PcdsFixedAtBuild]')\r
+      print ('    [PcdsPatchableInModule]')\r
+      print ('    [PcdsDynamicDefault]')\r
+      print ('    [PcdsDynamicExDefault]')\r
   elif args.PcdType == 'VPD':\r
     if args.MaxSize is None:\r
       #\r
       # If --max-size is not provided, then set maximum size to the size of the\r
       # binary input file\r
       #\r
-      args.MaxSize = len(Buffer)\r
-    if args.MaxSize < len(Buffer):\r
-      print 'BinToPcd: error: argument --max-size is smaller than input file.'\r
-      sys.exit()\r
+      args.MaxSize = PcdSize\r
+    if args.MaxSize < PcdSize:\r
+      print ('BinToPcd: error: argument --max-size is smaller than input file.')\r
+      sys.exit(1)\r
     if args.Offset is None:\r
       #\r
       # if --offset is not provided, then set offset field to '*' so build\r
       # tools will compute offset of PCD in VPD region.\r
       #\r
-      Pcd = '  %s|*|%d|%s' % (args.PcdName, args.MaxSize, ByteArray (Buffer))\r
+      Pcd = '  {Name}|*|{Size}|{Value}'.format (Name = args.PcdName, Size = args.MaxSize, Value = PcdValue)\r
     else:\r
       #\r
+      # --offset value must be 8-byte aligned\r
+      #\r
+      if (args.Offset % 8) != 0:\r
+        print ('BinToPcd: error: argument --offset must be 8-byte aligned.')\r
+        sys.exit(1)\r
+      #\r
       # Use the --offset value provided.\r
       #\r
-      Pcd = '  %s|%d|%d|%s' % (args.PcdName, args.Offset, args.MaxSize, ByteArray (Buffer))\r
+      Pcd = '  {Name}|{Offset}|{Size}|{Value}'.format (Name = args.PcdName, Offset = args.Offset, Size = args.MaxSize, Value = PcdValue)\r
     if args.Verbose:\r
-      print 'PcdToBin: Convert binary file to PCD statement compatible with PCD sections'\r
-      print '    [PcdsDynamicVpd]'\r
-      print '    [PcdsDynamicExVpd]'\r
+      print ('BinToPcd: Convert binary file to PCD statement compatible with PCD sections')\r
+      print ('    [PcdsDynamicVpd]')\r
+      print ('    [PcdsDynamicExVpd]')\r
   elif args.PcdType == 'HII':\r
-    if args.VariableGuid is None:\r
-      print 'BinToPcd: error: argument --variable-guid is required for --type HII.'\r
-      sys.exit()\r
-    if args.VariableName is None:\r
-      print 'BinToPcd: error: argument --variable-name is required for --type HII.'\r
-      sys.exit()\r
+    if args.VariableGuid is None or args.VariableName is None:\r
+      print ('BinToPcd: error: arguments --variable-guid and --variable-name are required for --type HII.')\r
+      sys.exit(1)\r
     if args.Offset is None:\r
       #\r
       # Use UEFI Variable offset of 0 if --offset is not provided\r
       #\r
       args.Offset = 0\r
-    Pcd = '  %s|L"%s"|%s|%d|%s' % (args.PcdName, args.VariableName, args.VariableGuid, args.Offset, ByteArray (Buffer))\r
+    #\r
+    # --offset value must be 8-byte aligned\r
+    #\r
+    if (args.Offset % 8) != 0:\r
+      print ('BinToPcd: error: argument --offset must be 8-byte aligned.')\r
+      sys.exit(1)\r
+    Pcd = '  {Name}|L"{VarName}"|{VarGuid}|{Offset}|{Value}'.format (Name = args.PcdName, VarName = args.VariableName, VarGuid = args.VariableGuid, Offset = args.Offset, Value = PcdValue)\r
     if args.Verbose:\r
-      print 'PcdToBin: Convert binary file to PCD statement compatible with PCD sections'\r
-      print '    [PcdsDynamicHii]'\r
-      print '    [PcdsDynamicExHii]'\r
+      print ('BinToPcd: Convert binary file to PCD statement compatible with PCD sections')\r
+      print ('    [PcdsDynamicHii]')\r
+      print ('    [PcdsDynamicExHii]')\r
 \r
   #\r
   # Write PCD value or PCD statement to the output file\r
@@ -189,4 +222,4 @@ if __name__ == '__main__':
     # If output file is not specified or it can not be written, then write the\r
     # PCD value or PCD statement to the console\r
     #\r
-    print Pcd\r
+    print (Pcd)\r