[flashrom] [PATCH] Use large reads for verify

Michael Karcher flashrom at mkarcher.dialup.fu-berlin.de
Tue Nov 16 17:53:51 CET 2010


Am Dienstag, den 16.11.2010, 17:27 +0100 schrieb Carl-Daniel Hailfinger:
> Thanks to Michael Karcher for reviewing the patch and pointing out a
> missing abort on read failure.

> +	ret = flash->read(flash, readbuf, start, len);
> +	if (ret) {
> +		msg_gerr("Verification impossible because read failed "
> +			 "at 0x%x (len 0x%x)\n", start, len);
> +		return 1;

make this "return ret" to not lose the error code.

Otherwise, the patch looks good to me.

Acked-By: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>

Regards,
  Michael Karcher





More information about the flashrom mailing list