[flashrom] [PATCH] Confirm EN29LV800 bus, timing, and page size

Sean Nelson audiohacked at gmail.com
Sat Feb 6 03:59:13 CET 2010


On 2/5/10 4:29 PM, TJ wrote:
> Uncertain values now confirmed.
>
> If you're happy with the new EN29LV800 code I'll submit a combined patch
> with a Signed-off-by tag.
> ---
>   flashchips.c |   12 ++++++------
>   1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/flashchips.c b/flashchips.c
> index b5be180..10b8dea 100644
> --- a/flashchips.c
> +++ b/flashchips.c
> @@ -2166,15 +2166,15 @@ struct flashchip flashchips[] = {
>   	{
>   		.vendor		= "EON",
>   		.name		= "EN29LV800(B)(C)B",
> -		.bustype	= CHIP_BUSTYPE_UNKNOWN,
> +		.bustype	= CHIP_BUSTYPE_PARALLEL,
>   		.manufacture_id	= EON_ID,
>   		.model_id	= EN_29LV800B,
>   		.total_size	= 1024,
> -		.page_size	= 1024, // TODO: check this
> +		.page_size	= 1024,
>   		.feature_bits	= FEATURE_ADDR_AAA | FEATURE_EITHER_RESET,
>   		.tested		= TEST_UNTESTED,
>   		.probe		= probe_jedec,
> -		.probe_timing	= TIMING_ZERO,	/* TODO: check Datasheet has no timing info specified */
> +		.probe_timing	= TIMING_ZERO,
>   		.erase		= NULL,
>   		.block_erasers	=
>   		{
> @@ -2198,15 +2198,15 @@ struct flashchip flashchips[] = {
>   	{
>   		.vendor		= "EON",
>   		.name		= "EN29LV800(B)(C)T",
> -		.bustype	= CHIP_BUSTYPE_UNKNOWN,
> +		.bustype	= CHIP_BUSTYPE_PARALLEL,
>   		.manufacture_id	= EON_ID,
>   		.model_id	= EN_29LV800T,
>   		.total_size	= 1024,
> -		.page_size	= 1024, // TODO: check this
> +		.page_size	= 1024,
>   		.feature_bits	= FEATURE_ADDR_AAA | FEATURE_EITHER_RESET,
>   		.tested		= TEST_UNTESTED,
>   		.probe		= probe_jedec,
> -		.probe_timing	= TIMING_ZERO,	/* TODO: check Datasheet has no timing info specified */
> +		.probe_timing	= TIMING_ZERO,
>   		.erase		= NULL,
>   		.block_erasers	=
>   		{
>    
EN29LV800 is not a JEDEC compatible chip, it uses an odd command 
sequence like the M29F400BT




More information about the flashrom mailing list