[flashrom] [PATCH] Improve error messages, protect the user

Stefan Reinauer stepan at coresystems.de
Wed Aug 12 11:39:51 CEST 2009


On 8/11/09 5:25 PM, Carl-Daniel Hailfinger wrote:

> +		if (flash->tested & TEST_BAD_ERASE) {
> +			fprintf(stderr, "Erase is not working on this chip "
> +				"and erase is needed for write. Aborting.\n");
> +			return 1;
> +		}
> +		if (flash->tested & TEST_BAD_WRITE) {
> +			fprintf(stderr, "Write is not working on this chip. "
> +				"Aborting.\n");
> +			return 1;
> +		}
>   

I would agree to this as long as there's a --force option

I think the IRC note should go to the documentation, not the code.


-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866





More information about the flashrom mailing list