[flashrom] [Patch] Remove filename parameter from chip_safety_check()

Stefan Reinauer stepan at coreboot.org
Tue Oct 19 01:10:46 CEST 2010


* David Hendricks <dhendrix at google.com> [101015 03:03]:
> Seems like the filename parameter was left in by accident.
> 
> This patch has only been compile tested.
> 
> Signed-off-by: David Hendricks <dhendrix at google.com>
> 

Nice spotting!

Acked-by: Stefan Reinauer <stepan at coreboot.org>


> --
> David Hendricks (dhendrix)
> Systems Software Engineer, Google Inc.

> Index: flashrom-foo/flashrom.c
> ===================================================================
> --- flashrom-foo.orig/flashrom.c	2010-10-14 17:47:27.000000000 -0700
> +++ flashrom-foo/flashrom.c	2010-10-14 17:47:33.000000000 -0700
> @@ -1543,7 +1543,7 @@
>  /* FIXME: This function signature needs to be improved once doit() has a better
>   * function signature.
>   */
> -int chip_safety_check(struct flashchip *flash, int force, char *filename, int read_it, int write_it, int erase_it, int verify_it)
> +int chip_safety_check(struct flashchip *flash, int force, int read_it, int write_it, int erase_it, int verify_it)
>  {
>  	if (!programmer_may_write && (write_it || erase_it)) {
>  		msg_perr("Write/erase is not working yet on your programmer in "
> @@ -1606,7 +1606,7 @@
>  	int ret = 0;
>  	unsigned long size;
>  
> -	if (chip_safety_check(flash, force, filename, read_it, write_it, erase_it, verify_it)) {
> +	if (chip_safety_check(flash, force, read_it, write_it, erase_it, verify_it)) {
>  		msg_cerr("Aborting.\n");
>  		programmer_shutdown();
>  		return 1;





More information about the flashrom mailing list