[LinuxBIOS] r2846 - trunk/util/flashrom
Stefan Reinauer
stepan at coresystems.de
Thu Oct 11 18:55:05 CEST 2007
* Uwe Hermann <uwe at hermann-uwe.de> [071010 20:47]:
> OK, second try, this time adding instead of removing full stops.
>
>
> Uwe.
> --
> http://www.hermann-uwe.de | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
> Some cosmetic cleanups in the flashrom code and output.
>
> Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
> if (ret)
> - printf("Failed!\n");
> + printf("failed!\n");
Since we use a capital "OK" some times, should we also use a capital
FAILED ? This is usually a pretty nasty situation.
> - fprintf(stdout, "Found LinuxBIOS table at: %08lx\n", addr);
> + fprintf(stdout, "Found LinuxBIOS table at 0x%08lx.\n", addr);
Maybe even %p?
> - printf_debug("lb_table found at address %p\n", lb_table);
> + printf_debug("LinuxBIOS table found at 0x%p.\n", lb_table);
%p already prints 0x on my system... so this would be 0x0x00000500 or so
> if (board) {
> - printf("Found board \"%s\": Enabling flash write... ",
> + printf("Found board \"%s\", enabling flash write... ",
> board->name);
>
> ret = board->enable(board->name);
> if (ret)
> - printf("Failed!\n");
> + printf("failed!\n");
I'd rather leave words after periods capital. Maybe we should even
CAPITALIZE it completely since it's a failure condition?
Stefan
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
More information about the coreboot
mailing list