Change in flashrom[master]: flashrom.8.tmpl: Clarify man entries for -w/-v/-x

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has submitted this change. ( https://review.coreboot.org/c/flashrom/+/62794 )

Change subject: flashrom.8.tmpl: Clarify man entries for -w/-v/-x
......................................................................

flashrom.8.tmpl: Clarify man entries for -w/-v/-x

This change adds follow up changes to the man page:
 - Explain (-) argument for -w/-v operations
 - Expand on region name handling of -x operation

Also updates cli_classic.c to match with --help output.

BUG=b:224364316

Change-Id: I0cba593da3926c8587027789f4e1e89a2329ca7f
Signed-off-by: Daniel Campello 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62794
Tested-by: build bot (Jenkins) 
Reviewed-by: Edward O'Callaghan 
---
M cli_classic.c
M flashrom.8.tmpl
2 files changed, 13 insertions(+), 7 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Edward O'Callaghan: Looks good to me, approved

diff --git a/cli_classic.c b/cli_classic.c
index b3930d2..0b6e79c 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -47,9 +47,9 @@
    printf(" -h | --help                        print this help text\n"
           " -R | --version                     print version (release)\n"
           " -r | --read <file>                 read flash and save to <file>\n"
-	       " -w | --write <file|->              write <file> or the content provided\n"
+	       " -w | --write (<file>|-)            write <file> or the content provided\n"
           "                                    on the standard input to flash\n"
-	       " -v | --verify <file|->             verify flash against <file>\n"
+	       " -v | --verify (<file>|-)           verify flash against <file>\n"
           "                                    or the content provided on the standard input\n"
           " -E | --erase                       erase flash memory\n"
           " -V | --verbose                     more verbose output\n"
diff --git a/flashrom.8.tmpl b/flashrom.8.tmpl
index fe2d9d6..3e4bc5e 100644
--- a/flashrom.8.tmpl
+++ b/flashrom.8.tmpl
@@ -82,11 +82,13 @@
 .BR <file> .
 If the file already exists, it will be overwritten.
 .TP
-.B "-w, --write <file>"
+.B "-w, --write (<file>|-)"
 Write
 .B <file>
-into flash ROM. This will first automatically
-.B erase
+into flash ROM. If
+.B -
+is provided instead, contents will be read from stdin. This will first automatically
+ B erase
 the chip, then write to it.
 .sp
 In the process the chip is also read several times. First an in-memory backup
@@ -124,16 +126,20 @@
 .BR internal
 programmer. It may be enabled by default in this case in the future.
 .TP
-.B "-v, --verify <file>"
+.B "-v, --verify (<file>|-)"
 Verify the flash ROM contents against the given
 .BR <file> .
+If
+.BR -
+is provided instead, contents will be written to the stdout.
 .TP
 .B "-E, --erase"
 Erase the flash ROM chip.
 .TP
 .B "-x, --extract"
 Extract every region defined on the layout from flash ROM chip to a
-file with the same name as the extracted region.
+file with the same name as the extracted region (replacing spaces with
+underscores).
 .TP
 .B "-V, --verbose"
 More verbose output. This option can be supplied multiple times