Browse Source

dtoc: Use None to mean stdout

At present dtoc uses '-' internally to mean that output should go to
stdout. This is not necessary and None is more convenient. Update it.

Signed-off-by: Simon Glass <sjg@chromium.org>
Simon Glass 3 years ago
parent
commit
f62cea0e20
3 changed files with 7 additions and 7 deletions
  1. 5 5
      tools/dtoc/dtb_platdata.py
  2. 1 1
      tools/dtoc/main.py
  3. 1 1
      tools/dtoc/test_dtoc.py

+ 5 - 5
tools/dtoc/dtb_platdata.py

@@ -237,12 +237,12 @@ class DtbPlatdata():
         file.
 
         Args:
-            fname (str): Filename to send output to, or '-' for stdout
+            fname (str): Filename to send output to, or None for stdout
         """
-        if fname == '-':
-            self._outfile = sys.stdout
-        else:
+        if fname:
             self._outfile = open(fname, 'w')
+        else:
+            self._outfile = sys.stdout
 
     def out(self, line):
         """Output a string to the output file
@@ -765,7 +765,7 @@ def run_steps(args, dtb_file, include_disabled, output, warning_disabled=False,
         args (list): List of non-option arguments provided to the problem
         dtb_file (str): Filename of dtb file to process
         include_disabled (bool): True to include disabled nodes
-        output (str): Name of output file
+        output (str): Name of output file (None for stdout)
         warning_disabled (bool): True to avoid showing warnings about missing
             drivers
         drivers_additional (list): List of additional drivers to use during

+ 1 - 1
tools/dtoc/main.py

@@ -91,7 +91,7 @@ parser.add_option('-d', '--dtb-file', action='store',
                   help='Specify the .dtb input file')
 parser.add_option('--include-disabled', action='store_true',
                   help='Include disabled nodes')
-parser.add_option('-o', '--output', action='store', default='-',
+parser.add_option('-o', '--output', action='store',
                   help='Select output filename')
 parser.add_option('-P', '--processes', type=int,
                   help='set number of processes to use for running tests')

+ 1 - 1
tools/dtoc/test_dtoc.py

@@ -875,7 +875,7 @@ U_BOOT_DEVICE(spl_test2) = {
         """Test output to stdout"""
         dtb_file = get_dtb_file('dtoc_test_simple.dts')
         with test_util.capture_sys_output() as _:
-            self.run_test(['struct'], dtb_file, '-')
+            self.run_test(['struct'], dtb_file, None)
 
     def test_no_command(self):
         """Test running dtoc without a command"""