elbpcom: --space now defaults to 0 (the hm2 register file)

Author: SebKuzminsky

docs/man/man1/elbpcom.1

@@ -9,13 +9,13 @@ Common options:
 .BI [\-\-ip= IP ]
 .BI [\-\-port= PORT ]
 .BI [\-\-timeout= TIMEOUT ]
+.BI [\-\-space= MEMSPACE ]
 .RE
 
 Reading data:
 .RS
 .SY elbpcom
 .BI [ common\ options ]
-.BI \-\-space= SPACE
 .B [\-\-info]
 .BI \-\-address= ADDRESS
 .BI \-\-read= LENGTH
@@ -25,7 +25,6 @@ Writing data:
 .RS
 .SY elbpcom
 .BI [ common\ options ]
-.BI \-\-space= SPACE
 .BI \-\-address= ADDRESS
 .BI \-\-write= HEXDATA
 .RE
@@ -54,13 +53,14 @@ If not specified, the default values are
 .BI \-\-ip= 192.168.1.121
 .BI \-\-port= 27181
 .BI \-\-timeout= .2
+.BI \-\-space= 0
 .RE 
 
 This example demonstrates reading the HOSTMOT2 identifying string from the
 IDROM in space 0:
 .RS
 .nf
-$ elbpcom \-\-space 0 \-\-address 0x104 \-\-read 8
+$ elbpcom \-\-address 0x104 \-\-read 8
 > 82420401
 < 484f53544d4f5432
       HOSTMOT2

src/hal/utils/elbpcom.py

@@ -32,9 +32,9 @@
     help="UDP port of board (default: 27181)", type="int", default=27181)
 parser.add_option("-t", "--timeout", dest="timeout",
     help="Response timeout in seconds (default: 0.2)", type="float", default=.2)
-parser.add_option("-s", "--space", dest="space", default=None,
+parser.add_option("-s", "--space", dest="space", default=0,
     choices = ["0", "1", "2", "3", "4", "5", "6", "7"],
-    help="Address space to read or write")
+    help="Address space to read or write (default: 0, the hm2 register file)")
 parser.add_option("--info", dest="info", action="store_true",
     default=False,
     help="Select info area for read operation (default: memory space)")
@@ -109,7 +109,6 @@ def optimal_size(space, info, address, nbytes):
     raise ValueError("Access size incompatible with address or length (address=%d nbytes=%d memsizes=%d)" % (address, nbytes, memsizes))
 
 if options.read:
-    if options.space is None: raise SystemExit("--read must specify --space")
     if options.address is None: raise SystemExit("--read must specify --address")
     size = optimal_size(options.space, options.info, options.address, options.read if options.increment else 0)
     command = make_read_request(options.space, options.info, size, options.increment, options.address, options.read)
@@ -118,7 +117,6 @@ def optimal_size(space, info, address, nbytes):
     transact(command)
 
 elif options.write:
-    if options.space is None: raise SystemExit("--write must specify --space")
     if options.address is None: raise SystemExit("--write must specify --address")
     write = options.write.decode("hex")
     size = optimal_size(options.space, options.info, options.address, len(write) if options.increment else 0)