[flashrom] [PATCH 1/1] Fix programmer-centric probe (patch v2)

[Michael Karcher]

Am Donnerstag, den 22.12.2011, 08:15 +0100 schrieb Michael Karcher:
> Signed-off-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>
self-review :)

> +		while( chipcount < ARRAY_SIZE(flashes) ) {
wrong spacing convention, as Stefan Tauner already pointed out in v1 of
the patch. Fixed locally to
                while (chipcount < ARRAY_SIZE(flashes)) {

>  			startchip = probe_flash(&registered_programmers[j],
> -						startchip, &flashes[i], 0);
> +						startchip, 
> +						&flashes[chipcount], 0);
>  			if (startchip == -1)
> -				break;
> +                        	break;
spaces before tabs. The whitespace change of this line is unintentional.
Fixed locally. The end result is a small and clean patch, replacing only
the bad for loop and using chipcount instead of the local counter i as
index for flashes.

Regards,
  Michael Karcher
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20111222/8bfe494e/attachment.asc>