[PATCH] jedec timing cleanup

[David Borg]

Fix TIMING_IGNORED with correct timing info.

Regards,

David

Signed-off-by: David Borg

Index: flashchips.c
===================================================================
--- flashchips.c	(revision 1145)
+++ flashchips.c	(working copy)
@@ -155,7 +155,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -181,7 +181,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -233,7 +233,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -256,10 +256,11 @@
    	.model_id	= AM_29LV080B,
    	.total_size	= 1024,
    	.page_size	= 64 * 1024,
+				  /* datasheet specifies address as don't care */
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -890,7 +891,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -921,7 +922,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_OK_PRW,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -952,7 +953,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -3737,7 +3738,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -3769,7 +3770,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -3801,7 +3802,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -3832,7 +3833,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_OK_PRW,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -3863,7 +3864,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_SHORT_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine don't use probe_timing
(mx29f002.c) */
+		.probe_timing	= TIMING_ZERO,
    	.block_erasers	=
    	{
    		{
@@ -6125,7 +6126,7 @@
    	.feature_bits	= FEATURE_ADDR_2AA | FEATURE_EITHER_RESET,
    	.tested		= TEST_UNTESTED,
    	.probe		= probe_jedec,
-		.probe_timing	= TIMING_IGNORED, /* routine doesn't use probe_timing
(am29f040b.c) */
+		.probe_timing	= TIMING_ZERO, /* datasheet specifies no timing */
    	.block_erasers	=
    	{
    		{

[David Borg]

Patches seem to be getting mangled up in gmail.. patch attached.

[David Borg]

Updated against r1163.

Signed-off-by: David Borg

Regards,

David

On 1 September 2010 17:10, David Borg  wrote:
> Patches seem to be getting mangled up in gmail.. patch attached.

[Carl-Daniel Hailfinger]

On 14.09.2010 01:37, David Borg wrote:
> Updated against r1163.
>
> Signed-off-by: David Borg

Thanks for your patch. Sorry it took me so long to apply it.

Acked-by: Carl-Daniel Hailfinger 
and committed in r1242.

Regards,
Carl-Daniel