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

[Sean Nelson]

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