[PATCH] flashrom: Autoverify all writes, unless --noverify is used

[Uwe Hermann]

See patch.

Tested on one board, other tests are welcome.

Uwe.

[PATCH] flashrom: Autoverify all writes, unless --noverify is used

[Urja Rannikko]

On Wed, Jun 24, 2009 at 17:00, Uwe Hermann wrote:
> See patch.
>
> Tested on one board, other tests are welcome.
> Uwe.
>
> http://www.hermann-uwe.de  | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org

I would like the noverify flag to instead disable (eg, report success
without doing anything) verify_range - eg disable all verifies the
code does (incl verify for erase, verify single page write's in
jedec.c, etc). It would be nice if flashrom would give some sort of
"You have been warned"-message when this is used.

[PATCH] flashrom: Autoverify all writes, unless --noverify is used

[Carl-Daniel Hailfinger]

On 24.06.2009 16:00, Uwe Hermann wrote:
> Always verify write operations automatically.
>
> Should this be undesireable because of speed reasons, --noverify can be
> used to suppress an auto-verify.
>
> Signed-off-by: Uwe Hermann
>
> Tested on one board, other tests are welcome.

Thanks for the patch.
Acked-by: Carl-Daniel Hailfinger

I think Urja's comments make sense, but they can be addressed later.

Regards,
Carl-Daniel

[Harald Gutmann]

On Wednesday 24 June 2009 16:00:12 Uwe Hermann wrote:
> See patch.
>
> Tested on one board, other tests are welcome.

Tested on M57SLI v2 (spi). Works fine!

Acked-by: Harald Gutmann
> Uwe.

Harald

[PATCH] flashrom: Autoverify all writes, unless --noverify is used

[Uwe Hermann]

On Wed, Jun 24, 2009 at 05:07:51PM +0200, Harald Gutmann wrote:
> On Wednesday 24 June 2009 16:00:12 Uwe Hermann wrote:
> > See patch.
> >
> > Tested on one board, other tests are welcome.
>
> Tested on M57SLI v2 (spi). Works fine!

Great, thanks!
> Acked-by: Harald Gutmann

r631, with some manpage updates.

Uwe.