[flashrom] [PATCH] Use the register mapping feature bit

Michael Karcher flashrom at mkarcher.dialup.fu-berlin.de
Sat Jan 9 01:07:07 CET 2010


Am Donnerstag, den 07.01.2010, 19:31 +0100 schrieb Carl-Daniel
Hailfinger:
> Use the register mapping feature bit.
> All functions which just call probe_jedec and then map flash registers
> are replaced by probe_jedec. All functions which call probe_jedec, map
> flash registers and do something else can at least eliminate mapping
> flash registers.
> Fix logic inversion in probe_jedec to map flash registers on success
> instead of on failure.
> Change a few TIMING_IGNORED to TIMING_FIXME where probe_jedec is used.
> 
> Total savings: One probe function simplified, three probe functions
> eliminated.
> 
> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
[...]
Acked-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>

> Index: flashrom-registermap_featurebit/flashchips.c
> ===================================================================
> --- flashrom-registermap_featurebit/flashchips.c	(Revision 834)
> +++ flashrom-registermap_featurebit/flashchips.c	(Arbeitskopie)
> @@ -827,7 +827,7 @@
>  		.total_size	= 64,
>  		.page_size	= 128,
>  		.tested		= TEST_OK_PRW,
> -		.probe		= probe_jedec, 
> +		.probe		= probe_jedec,
>  		.probe_timing	= 10000, /* 10mS, Enter=Exec */
>  		.erase		= NULL,
>  		.block_erasers	=
Is this whitespace-only change (removes a trailing space that shouldn't
have been there from the beginning) intentional?

Regards,
  Michael Karcher





More information about the flashrom mailing list