Attention is currently required from: Zheng Bao. Hello Zheng Bao,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/54349
to review the following change.
Change subject: amdfwtool: Move "--help" to general option section ......................................................................
amdfwtool: Move "--help" to general option section
Change-Id: I442be6b60ce939c110ffec4f3de8b158f4130212 Signed-off-by: Zheng Bao fishbaozi@gmail.com --- M util/amdfwtool/amdfwtool.c 1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/54349/1
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 46910cb..b0fe106 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -196,7 +196,6 @@ printf("-C | --soc-name <socname> Specify SOC name. Supported names are\n"); printf(" Stoneyridge, Raven, Picasso, Renoir, Cezanne\n"); printf(" or Lucienne\n"); - printf("-h | --help Show this help\n"); printf("\nEmbedded Firmware Structure options used by the PSP:\n"); printf("--spi-speed <HEX_VAL> SPI fast speed to place in EFS Table\n"); printf(" 0x0 66.66Mhz\n"); @@ -219,9 +218,11 @@ printf(" 0x1 Micron parts are always used\n"); printf(" 0x2 Micron parts optional, this option is only\n"); printf(" supported with RN/LCN SOC\n"); + printf("\nGeneral options:\n") printf("-c | --config <config file> Config file\n"); printf("-d | --debug Print debug message\n"); printf("-D | --depend List out the firmware files\n"); + printf("-h | --help Show this help\n"); }
amd_fw_entry amd_psp_fw_table[] = {